]> git.ipfire.org Git - thirdparty/knot-resolver.git/commitdiff
trie_del: use KNOT_EOK instead of kr_ok()
authorTomas Krizek <tomas.krizek@nic.cz>
Thu, 19 Aug 2021 14:59:05 +0000 (16:59 +0200)
committerTomas Krizek <tomas.krizek@nic.cz>
Mon, 11 Oct 2021 13:45:27 +0000 (15:45 +0200)
daemon/http.c
daemon/session.c
daemon/tls.c
lib/generic/test_trie.c

index e2c1783f4510fe8cea38eee035a85bf2a06e18a5..149bdfef8360994b73a5e5cc196193b43cb6f7ad 100644 (file)
@@ -543,7 +543,7 @@ static int on_stream_close_callback(nghttp2_session *h2, int32_t stream_id,
                http_cleanup_stream(ctx);
 
        ret = trie_del(ctx->stream_write_data, (char *)&stream_id, sizeof(stream_id), (trie_val_t*)&data);
-       if (ret == kr_ok() && data)
+       if (ret == KNOT_EOK && data)
                on_pkt_write(data, error_code == 0 ? 0 : kr_error(EIO));
 
        return 0;
index c1e23b929aadeec99862cb32b54a675b61c0da3f..3e530b705ae63f69d8c6a6bde28c685eb3582bdc 100644 (file)
@@ -195,7 +195,7 @@ int session_tasklist_del(struct session *session, struct qr_task *task)
                key_len = sizeof(char *);
        }
        int ret = trie_del(t, key, key_len, &val);
-       if (ret == kr_ok()) {
+       if (ret == KNOT_EOK) {
                kr_require(val == task);
                worker_task_unref(val);
        }
@@ -212,7 +212,7 @@ struct qr_task *session_tasklist_del_first(struct session *session, bool deref)
 {
        trie_val_t val = NULL;
        int res = trie_del_first(session->tasks, NULL, NULL, &val);
-       if (res != kr_ok()) {
+       if (res != KNOT_EOK) {
                val = NULL;
        } else if (deref) {
                worker_task_unref(val);
@@ -229,7 +229,7 @@ struct qr_task* session_tasklist_del_msgid(const struct session *session, uint16
        size_t key_len = sizeof(msg_id);
        trie_val_t val;
        int res = trie_del(t, key, key_len, &val);
-       if (res == kr_ok()) {
+       if (res == KNOT_EOK) {
                if (worker_task_numrefs(val) > 1) {
                        ret = val;
                }
index 935870dfd02e31f79161ba62befceeba642793b0..317a008ffb0e68a8223958f74a3bd2d8fd87b5fe 100644 (file)
@@ -894,7 +894,7 @@ int tls_client_param_remove(tls_client_params_t *params, const struct sockaddr *
                return kr_error(EINVAL);
        trie_val_t param_ptr;
        int ret = trie_del(params, key, len, &param_ptr);
-       if (ret)
+       if (ret != KNOT_EOK)
                return kr_error(ret);
        tls_client_param_unref(param_ptr);
        return kr_ok();
index 6740948e9b4065f427f891e30492fdff58ae506e..c71005aedc0fa3a7dca6d1771584e18bd281d6dc 100644 (file)
@@ -107,7 +107,7 @@ static void test_queue(void **state)
                char key_buf[len];
                ptrdiff_t key_i_new;
                int ret = trie_del_first(t, key_buf, &len, (trie_val_t *)&key_i_new);
-               assert_int_equal(ret, kr_ok());
+               assert_int_equal(ret, KNOT_EOK);
                assert_int_equal(KEY_LEN(key_buf), len);
                assert_int_equal(key_i, key_i_new);
                assert_string_equal(dict[key_i], key_buf);