An enum can be either signed or unsigned (implementation defined). datatype_lookup() checks for invalid type arguments. Also check, whether the argument is not negative (which, depending on the compiler it may never be). Signed-off-by: Thomas Haller <thaller@xxxxxxxxxx> --- src/datatype.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/datatype.c b/src/datatype.c index ba1192c83595..91735ff8b360 100644 --- a/src/datatype.c +++ b/src/datatype.c @@ -87,7 +87,7 @@ const struct datatype *datatype_lookup(enum datatypes type) { BUILD_BUG_ON(TYPE_MAX & ~TYPE_MASK); - if (type > TYPE_MAX) + if ((uintmax_t) type > TYPE_MAX) return NULL; return datatypes[type]; } -- 2.41.0