On Wednesday 2023-12-20 17:06, Phil Sutter wrote: > { > const struct xt_option_entry *entry = cb->entry; >+ int i = cb->nvals; > >- if (cb->nvals >= ARRAY_SIZE(cb->val.u32_range)) >+ if (i >= ARRAY_SIZE(cb->val.u32_range)) > return; `i` should be unsigned (size_t) because ARRAY_SIZE is, else you get -Wsigned warnings at some point.