From 5dd29ecd2cc794144de059014a5cbc9c92c3d8bb Mon Sep 17 00:00:00 2001 From: Romain F Date: Mon, 21 Mar 2022 17:01:01 +0100 Subject: [PATCH] KCM: Memory leak in `handle_read` The `inmsg` field of the client structure is malloc/realloc'ed in `handle_read` but never free'ed in `maybe_close`. Seems like Apple already fixed that with this. --- lib/ipc/server.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/ipc/server.c b/lib/ipc/server.c index 40601b974..8df94a057 100644 --- a/lib/ipc/server.c +++ b/lib/ipc/server.c @@ -700,6 +700,7 @@ maybe_close(struct client *c) dispatch_release(c->out); #endif close(c->fd); /* ref count fd close */ + free(c->inmsg); free(c); return 1; }