[PATCH v3 1/1] event/octeontx: resolve possible integer overflow

[Jerin Jacob]

On Thu, Oct 24, 2024 at 1:27 PM Ali Alnubani <alialnu at nvidia.com> wrote:
>
> > -----Original Message-----
> > From: Hanumanth Pothula <hpothula at marvell.com>
> > Sent: Thursday, October 24, 2024 6:55 AM
> > To: Jerin Jacob <jerinj at marvell.com>
> > Cc: dev at dpdk.org; hkalra at marvell.com; stephen at networkplumber.org;
> > hpothula at marvell.com
> > Subject: [PATCH v3 1/1] event/octeontx: resolve possible integer overflow
> >
> > The last argument passed to ssovf_parsekv() is an
> > unsigned char*, but it is accessed as an integer.
> > This can lead to an integer overflow.
> >
> > Hence, make ensure the argument is accessed as a char
> > and for better error handling use strtol instead of atoi.
> >

Add
Bugzilla ID: 1512

And add Fixes: tag

please start the git commit subject with event/octeontx: fix ...

Good to merge next version

> > Signed-off-by: Hanumanth Pothula <hpothula at marvell.com>
> > ---
> >
> > v2: use strtoul instead of strtol
> > v3: Add value boundry check. Here, value can be either 0 or 1.
> > ---
>
> Confirmed that it resolves https://bugs.dpdk.org/show_bug.cgi?id=1512.
>
> Tested-by: Ali Alnubani <alialnu at nvidia.com>



>
> Thanks,
> Ali