From 560a166613504f4e725c1dc280bc6d92d568332d Mon Sep 17 00:00:00 2001 From: Martin Stein Date: Tue, 24 May 2022 11:55:48 +0200 Subject: [PATCH] wireguard: flags value via enum when adding a peer I used to set the peer flags to 2 when adding a peer but merely because I observed that value in my reference Linux. I now found that the flag 1<<1 means 'WGPEER_F_REPLACE_ALLOWEDIPS' and is ignored whenever the peer is new. Anyway, I keep setting the flag but with the clearer enum name because the wg tool is doing it as well when adding new peers and we want to stay close to how the original code interacts. Ref #4520 --- repos/dde_linux/src/app/wireguard/genode_c_api/wireguard.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/repos/dde_linux/src/app/wireguard/genode_c_api/wireguard.c b/repos/dde_linux/src/app/wireguard/genode_c_api/wireguard.c index 538250259b..ae32e94eba 100644 --- a/repos/dde_linux/src/app/wireguard/genode_c_api/wireguard.c +++ b/repos/dde_linux/src/app/wireguard/genode_c_api/wireguard.c @@ -317,7 +317,7 @@ _genode_wg_config_add_peer(genode_wg_u16_t listen_port, peer->flags.header.nla_type = WGPEER_A_FLAGS; peer->flags.header.nla_len = sizeof(peer->flags); - peer->flags.data = 2; /* I don't know what this value means */ + peer->flags.data = WGPEER_F_REPLACE_ALLOWEDIPS; peer->allowedips.header.nla_len = sizeof(peer->allowedips); peer->allowedips.header.nla_type = WGPEER_A_ALLOWEDIPS | NLA_F_NESTED;