]> git.ipfire.org Git - thirdparty/knot-resolver.git/commitdiff
daemon/session nitpick: avoid a warning
authorVladimír Čunát <vladimir.cunat@nic.cz>
Thu, 4 Oct 2018 12:43:54 +0000 (14:43 +0200)
committerVladimír Čunát <vladimir.cunat@nic.cz>
Fri, 12 Oct 2018 15:36:46 +0000 (17:36 +0200)
lint:clang-scan-build reported:
> warning: The code calls sizeof() on a pointer type.
> This can produce an unexpected result
but in our case it's intentional.
(Yes, using pointers as keys in trie isn't very pretty.)

daemon/session.c

index 0ae93c62035070876a349d6b62654c077a7624ba..8ea88b9051a51f068d4189a943c3b8019d0a948e 100644 (file)
@@ -148,7 +148,7 @@ int session_tasklist_add(struct session *session, struct qr_task *task)
                key_len = sizeof(task_msg_id);
        } else {
                key = (const char *)&task;
-               key_len = sizeof(task);
+               key_len = sizeof(char *);
        }
        trie_val_t *v = trie_get_ins(t, key, key_len);
        if (unlikely(!v)) {
@@ -179,7 +179,7 @@ int session_tasklist_del(struct session *session, struct qr_task *task)
                key_len = sizeof(task_msg_id);
        } else {
                key = (const char *)&task;
-               key_len = sizeof(task);
+               key_len = sizeof(char *);
        }
        int ret = trie_del(t, key, key_len, &val);
        if (ret == kr_ok()) {