On Thu, Oct 24, 2024 at 1:27 PM Ali Alnubani <alia...@nvidia.com> wrote: > > > -----Original Message----- > > From: Hanumanth Pothula <hpoth...@marvell.com> > > Sent: Thursday, October 24, 2024 6:55 AM > > To: Jerin Jacob <jer...@marvell.com> > > Cc: dev@dpdk.org; hka...@marvell.com; step...@networkplumber.org; > > hpoth...@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 <hpoth...@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 <alia...@nvidia.com> > > Thanks, > Ali
