diff options
| author | Aaron LI <aly@aaronly.me> | 2023-11-11 13:13:08 +0000 |
|---|---|---|
| committer | Kristof Provost <kp@FreeBSD.org> | 2023-11-18 09:46:21 +0000 |
| commit | 9ed67f1307f40b3173ea3b3889c6323f9ef4a58f (patch) | |
| tree | 6badb9fb2357a05283920f25cecfae1881469b4d | |
| parent | 51a024c42c4698da1616f1126fb0af4660a3155d (diff) | |
| download | src-9ed67f1307f40b3173ea3b3889c6323f9ef4a58f.tar.gz src-9ed67f1307f40b3173ea3b3889c6323f9ef4a58f.zip | |
if_wg: Missing radix unlock can cause deadlock
In function 'wg_aip_add()', the error path of returning ENOMEM when
(node == NULL) is forgetting to unlock the radix tree, and thus may lead
to a deadlock.
PR: 275001
Reviewed by: kp
MFC after: 1 week
(cherry picked from commit dcc4d2939f789a6d1f272ffeab2068ba2b7525ea)
| -rw-r--r-- | sys/dev/wg/if_wg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/dev/wg/if_wg.c b/sys/dev/wg/if_wg.c index 4a21afe82eb5..0ea40b763416 100644 --- a/sys/dev/wg/if_wg.c +++ b/sys/dev/wg/if_wg.c @@ -562,7 +562,7 @@ wg_aip_add(struct wg_softc *sc, struct wg_peer *peer, sa_family_t af, const void node = root->rnh_lookup(&aip->a_addr, &aip->a_mask, &root->rh); if (!node) { free(aip, M_WG); - return (ENOMEM); + ret = ENOMEM; } else if (node != aip->a_nodes) { free(aip, M_WG); aip = (struct wg_aip *)node; |