Add suppressions

Signed-off-by: Björn Svensson <bjorn.a.svensson@est.tech>

Author: bjosv

src/cluster.c

@@ -973,6 +973,7 @@ static dict *parse_cluster_nodes(valkeyClusterContext *cc, valkeyContext *c, val
                 }
             }
             /* Retain parsed replica nodes until all primaries are parsed. */
+            // codechecker_false_positive [core.CallAndMessage] primary_id is always initialized via parsed_primary_id when role is replica
             if (retain_replica_node(replicas, primary_id, node) != VALKEY_OK) {
                 freeValkeyClusterNode(node);
                 goto oom;

src/command.c

@@ -260,6 +260,7 @@ void valkey_parse_cmd(struct cmd *r) {
                 continue; /* Keyword can't appear in a position before 'startfrom' */
             if (!strncasecmp(keyword, arg, arglen)) {
                 /* Keyword found. Now the first key is the next arg. */
+                // codechecker_false_positive [deadcode.DeadStores] p is used in the NULL check
                 if ((p = valkey_parse_bulk(p, end, &arg, &arglen)) == NULL)
                     goto error;
                 /* Keep found key. */
@@ -289,6 +290,7 @@ void valkey_parse_cmd(struct cmd *r) {
         if (!strncmp("0", arg, arglen))
             goto done; /* No args. */
         /* One or more args. The first key is the arg after the 'numkeys' arg. */
+        // codechecker_false_positive [deadcode.DeadStores] p is used in the NULL check
         if ((p = valkey_parse_bulk(p, end, &arg, &arglen)) == NULL)
             goto error;
         argidx++;

src/valkey_private.h

@@ -103,6 +103,7 @@ static inline int valkeyContextUpdateConnectTimeout(valkeyContext *c,
             return VALKEY_ERR;
     }
 
+    // codechecker_false_positive [core.NonNullParamChecker] NULL is handled by pointer equality check above
     memcpy(c->connect_timeout, timeout, sizeof(*c->connect_timeout));
     return VALKEY_OK;
 }
@@ -120,6 +121,7 @@ static inline int valkeyContextUpdateCommandTimeout(valkeyContext *c,
             return VALKEY_ERR;
     }
 
+    // codechecker_false_positive [core.NonNullParamChecker] NULL is handled by pointer equality check above
     memcpy(c->command_timeout, timeout, sizeof(*c->command_timeout));
     return VALKEY_OK;
 }

tests/client_test.c

@@ -455,6 +455,7 @@ static void test_format_commands(void) {
     sds_cmd = NULL;
     test("Format command into sds by passing argc/argv without lengths: ");
     len = valkeyFormatSdsCommandArgv(&sds_cmd, argc, argv, NULL);
+    // codechecker_false_positive [core.NonNullParamChecker] sds_cmd is set by valkeyFormatSdsCommandArgv
     test_cond(strncmp(sds_cmd, "*3\r\n$3\r\nSET\r\n$3\r\nfoo\r\n$3\r\nbar\r\n", len) == 0 &&
               len == 4 + 4 + (3 + 2) + 4 + (3 + 2) + 4 + (3 + 2));
     sdsfree(sds_cmd);