From d4aa4cc5b1e04d942cfa67738cabc1baa77f598b Mon Sep 17 00:00:00 2001 From: Your Name Date: Sat, 4 Apr 2026 14:06:26 +0200 Subject: [PATCH] code cleanup based on cppcheck nfq2/nfqws.c:1114:19: warning: Either the condition 'p' is redundant or there is possible null pointer dereference: p. [nullPointerRedundantCheck] --- nfq2/nfqws.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/nfq2/nfqws.c b/nfq2/nfqws.c index 8c7137a..bf65ec1 100644 --- a/nfq2/nfqws.c +++ b/nfq2/nfqws.c @@ -1116,12 +1116,9 @@ static bool parse_uid(char *opt, uid_t *uid, gid_t *gid, int *gid_count, int max c = *e; *e = 0; } - if (p) - { - if (sscanf(p, "%u", &u) != 1) return false; - if (*gid_count >= max_gids) return false; - gid[(*gid_count)++] = (gid_t)u; - } + if (sscanf(p, "%u", &u) != 1) return false; + if (*gid_count >= max_gids) return false; + gid[(*gid_count)++] = (gid_t)u; if (e) *e++ = c; p = e; }