aboutsummaryrefslogtreecommitdiff
path: root/server.c
diff options
context:
space:
mode:
authorXavier Del Campo <xavier.delcampo@midokura.com>2023-11-20 12:20:51 +0100
committerXavier Del Campo <xavier.delcampo@midokura.com>2023-11-20 16:25:58 +0100
commit59a67a7541ee1559b140494c21a25b195246a4fa (patch)
treed2bed79936276954b1d712f33993035d1082062c /server.c
parent65031ca3502e0c27780be847fd97c112546741a9 (diff)
Fix double-free on failed server_client_close
Even if server_client_close fails, it is needed for client_free to remove the dangling reference from h->clients.
Diffstat (limited to 'server.c')
-rw-r--r--server.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/server.c b/server.c
index 8cea044..3d6f370 100644
--- a/server.c
+++ b/server.c
@@ -59,7 +59,6 @@ int server_client_close(struct server *const s, struct server_client *const c)
{
fprintf(stderr, "%s: close(2): %s\n",
__func__, strerror(errno));
- return -1;
}
else if (ref->prev)
ref->prev->next = next;