> -----Original Message-----
> From: David Marchand <david.march...@redhat.com>
> Sent: Monday 23 June 2025 14:53
> To: dev@dpdk.org
> Cc: sta...@dpdk.org; Bruce Richardson <bruce.richard...@intel.com>
> Subject: [PATCH v2 06/10] cmdline: fix highest bit port list parsing
>
> pl->map is a uint32_t.
>
> Caught by UBSan:
>
> ../lib/cmdline/cmdline_parse_portlist.c:27:17: runtime error:
> left shift of 1 by 31 places cannot be represented in type 'int'
> SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior
> ../lib/cmdline/cmdline_parse_portlist.c:27:17 in
>
> Fixes: af75078fece3 ("first public release")
> Cc: sta...@dpdk.org
>
> Signed-off-by: David Marchand <david.march...@redhat.com>
> Acked-by: Bruce Richardson <bruce.richard...@intel.com>
> ---
> Changes since v1:
> - moved variable increment out of the macro call,
>
> ---
> lib/cmdline/cmdline_parse_portlist.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/lib/cmdline/cmdline_parse_portlist.c
> b/lib/cmdline/cmdline_parse_portlist.c
> index 549f6d9671..3efe4143e3 100644
> --- a/lib/cmdline/cmdline_parse_portlist.c
> +++ b/lib/cmdline/cmdline_parse_portlist.c
> @@ -10,7 +10,9 @@
> #include <errno.h>
>
> #include <eal_export.h>
> +#include <rte_bitops.h>
> #include <rte_string_fns.h>
> +
> #include "cmdline_parse.h"
> #include "cmdline_parse_portlist.h"
>
> @@ -26,7 +28,8 @@ static void
> parse_set_list(cmdline_portlist_t *pl, size_t low, size_t high)
> {
> do {
> - pl->map |= (1 << low++);
> + pl->map |= RTE_BIT32(low);
> + low++;
> } while (low <= high);
> }
Reviewed-by: Marat Khalili <marat.khal...@huawei.com>
