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