commit b7824b66b668bbda1f0bde49e25f8538ea9d2fa6
author Benjamin Gwin <bgwin@google.com> Tue Aug 08 18:44:53 2023 +0000
committer Andrew Jeffery <andrew@aj.id.au> Wed Aug 23 14:38:14 2023 +0930
tree 2f1418e566966cac2238d9f61f3526c4cb040667
parent 8f58dca75a989941c67e18b9328b65ebff080785

core: Fix use of wrong 'free' function

This memory was allocated using `__mctp_alloc` but mistakenly being free
with plain `free`, causing a potential heap corruption or crash if users
define their own malloc hooks.

Signed-off-by: Benjamin Gwin <bgwin@google.com>
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I1d78f62098e5d9b57e58e4dd90c4841b2923300d

core.c

diff --git a/core.c b/core.c
index 741082d..4acf075 100644
--- a/core.c
+++ b/core.c
@@ -389,7 +389,7 @@
 	mctp->n_busses = 0;
 	binding->mctp = NULL;
 	binding->bus = NULL;
-	free(mctp->busses);
+	__mctp_free(mctp->busses);
 }
 
 int mctp_bridge_busses(struct mctp *mctp, struct mctp_binding *b1,