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

Ali Alnubani alialnu at nvidia.com
Thu Oct 24 09:57:37 CEST 2024


> -----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.
> 
> 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


More information about the dev mailing list