ouroboros/src/ipcpd/unicast/dir, branch 0.22.0

build: Include tests in 'all' target

2025-11-07T07:36:20+00:00

When BUILD_TESTING=ON, then the default 'all' target will now build also the tests. This behaviour could be controlled by an extra variable (see https://stackoverflow.com/a/42235335), but this increases complexity without many benefits. Signed-off-by: Thijs Paelman Signed-off-by: Sander Vrijders

ipcpd: Fix missing return in DHT

2025-08-24T08:23:47+00:00

The dht_kv_respond_req had a missing return statement. I'm not sure why the double mutex_unlock() never caused havoc in testing. Signed-off-by: Dimitri Staessens

build: Add some hints for scan-build

2025-08-23T12:11:39+00:00

The scan-build tool was pointing out some potential issues that were understood or false positives. Added some asserts so scan-build can complete successfully without these warnings. Signed-off-by: Dimitri Staessens

ipcpd: Fix printf formatting in tests on raspbian

2025-08-23T08:13:36+00:00

Fixes some printf formatting, which was failing the build on raspbian. Signed-off-by: Dimitri Staessens

ipcpd: Add cleanup of DHT replication lists

2025-08-23T08:13:36+00:00

The replication/republication lists could leak on shutdown. Added cleanup handlers for them. Signed-off-by: Dimitri Staessens

ipcpd: Fix request handling at shutdown

2025-08-23T08:13:33+00:00

The IPCP states were not entirely correct causing some operations to be serviced during shutdown. This caused some use-after-free in the pff. States in the IPCP are now correctly set. IRMd states updated to the same strategy. The IRMd registry tracks if the IPCP was ENROLLED or BOOTSTRAPPED, the IPCP just goes to OPERATIONAL. IPCP state diagram:: NULL -> init() -> INIT -> start() -> BOOT -> bootstrap/enroll() -> OPERATIONAL -> shutdown() -> SHUTDOWN -> stop_components() -> BOOT -> stop() -> INIT -> fini() -> NULL Signed-off-by: Dimitri Staessens

ipcpd: Fix readdir() not initializing ptr

2025-08-11T08:17:14+00:00

Some IPCP (sub)components returned successfully with 0 entries but did not initialize the buf ptr when the RIB has no entries, causing a SEGV on free() in the RIB. Signed-off-by: Dimitri Staessens Signed-off-by: Sander Vrijders

ipcpd: Remove old IPCP_CONN_WAIT_DIR build option

2025-08-11T08:15:32+00:00

The IPCP_CONN_WAIT_DIR build option is not needed anymore with the recent update to the DHT. Also cleans up some logging in the IPCP. Signed-off-by: Dimitri Staessens Signed-off-by: Sander Vrijders

ipcpd: Fix DHT test initializer compilation error

2025-08-11T08:00:44+00:00

dht_test.c:167:48: error: initializer element is not a compile-time constant static struct dir_dht_config test_dht_config = default_dht_config; Signed-off-by: Dimitri Staessens Signed-off-by: Sander Vrijders

ipcpd: Add protocol debug option for link-state

2025-08-06T10:34:19+00:00

The link-state routing component now has a protocol debugging option, activated by enabling the DEBUG_PROTO_LS flag at build time in a debug build. Example output (sender and receiver): LSU [81:8a:1f:9b -- 50:c6:2d:03 seq: 000000000] --> 50:c6:2d:03 LSU [81:8a:1f:9b -- 50:c6:2d:03 seq: 000000000] <-- 50:c6:2d:03 In larger networks, forwarded LSUs are marked as such: LSU [ee:53:ae:f8 -- e5:33:e4:8d seq: 000000006] --> e5:33:e4:8d [forwarded] This also aligns the address printing using a similar ethernet-like formatting such as the DHT component. Small code cleanup in the graph component. Note: eventually the link state dissemination should move to a broadcast Layer instead of the link state component doing the forwarding. Signed-off-by: Dimitri Staessens Signed-off-by: Sander Vrijders