diff options
| author | Dimitri Staessens <dimitri@ouroboros.rocks> | 2026-02-21 12:05:11 +0100 |
|---|---|---|
| committer | Sander Vrijders <sander@ouroboros.rocks> | 2026-02-22 16:04:16 +0100 |
| commit | c9662b47be711c00f013ac4606dc86c9a2ad8201 (patch) | |
| tree | 9a9d88cfd300dc27d6429b7341ceb5985c21f246 /src/ipcpd/unicast/dir | |
| parent | c3636005831064e71b03a5f8796a21e89b2a714f (diff) | |
| download | ouroboros-c9662b47be711c00f013ac4606dc86c9a2ad8201.tar.gz ouroboros-c9662b47be711c00f013ac4606dc86c9a2ad8201.zip | |
ipcpd: Fix unused values in unicast IPCP
Use the key in the store operation in the DHT, to avoid static
analyzer seeing it as being set without being used in release builds.
The length of the packet buffer is only used when tracking DT stats,
so should be set within the IPCP_FLOW_STATS guard.
Signed-off-by: Dimitri Staessens <dimitri@ouroboros.rocks>
Signed-off-by: Sander Vrijders <sander@ouroboros.rocks>
Diffstat (limited to 'src/ipcpd/unicast/dir')
| -rw-r--r-- | src/ipcpd/unicast/dir/dht.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/ipcpd/unicast/dir/dht.c b/src/ipcpd/unicast/dir/dht.c index bc8fb820..8eeea800 100644 --- a/src/ipcpd/unicast/dir/dht.c +++ b/src/ipcpd/unicast/dir/dht.c @@ -2798,8 +2798,6 @@ static void do_dht_kv_store(const dht_store_msg_t * store) uint8_t * key; time_t exp; - (void) key; /* Only in logs, not used with DISABLE_TEST_LOGGING */ - assert(store != NULL); val.data = store->val.data; @@ -2807,7 +2805,7 @@ static void do_dht_kv_store(const dht_store_msg_t * store) key = store->key.data; exp = store->exp; - if (dht_kv_store(store->key.data, val, store->exp) < 0) { + if (dht_kv_store(key, val, store->exp) < 0) { log_err(KV_FMT " Failed to store.", KV_VAL(key, val)); return; } |
