u_bit_scan may return -1 that then may be interpreted as (unsigned)-1 in the following comparison, since num_names is unsigned. Convert the latter to be int as well.
Signed-off-by: Gert Wollny <gw.foss...@gmail.com> Reviewed-by: Brian Paul <bri...@vmware.com> --- src/gallium/auxiliary/util/u_dump_defines.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/util/u_dump_defines.c b/src/gallium/auxiliary/util/u_dump_defines.c index e431cd969b..50dfa37bab 100644 --- a/src/gallium/auxiliary/util/u_dump_defines.c +++ b/src/gallium/auxiliary/util/u_dump_defines.c @@ -99,7 +99,7 @@ util_dump_flags_continuous(FILE *stream, unsigned value, unsigned num_names, while (value) { int i = u_bit_scan(&value); - if (i >= num_names || !names[i]) + if (i >= (int)num_names || !names[i]) unknown |= 1u << i; if (!first) fputs("|", stream); -- 2.13.6 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev