On Sat, Oct 16, 2021 at 06:24:18PM +0200, Tim Duesterhus wrote:
> see 6a0dd733906611dea958cf74b9f51bb16028ae20
> 
> Found using GitHub's CodeQL scan.
> ---
>  include/haproxy/stick_table-t.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/haproxy/stick_table-t.h b/include/haproxy/stick_table-t.h
> index 3b1f2b3ef..133f992b5 100644
> --- a/include/haproxy/stick_table-t.h
> +++ b/include/haproxy/stick_table-t.h
> @@ -125,8 +125,8 @@ struct stktable_data_type {
>       const char *name; /* name of the data type */
>       int std_type;     /* standard type we can use for this data, STD_T_* */
>       int arg_type;     /* type of optional argument, ARG_T_* */
> -     int is_array:1;   /* this is an array of gpc/gpt */
> -     int is_local:1;   /* this is local only and never learned */
> +     unsigned int is_array:1;   /* this is an array of gpc/gpt */
> +     unsigned int is_local:1;   /* this is local only and never learned */

Please note, since last patch we've added shorter type names like "uint"
for the unsigned int, that help preserve alignment. Do you mind if I adapt
your patch ?

Willy

Reply via email to