From 2b283535544aab26840dae6eaa2a1df07f6b60c9 Mon Sep 17 00:00:00 2001 From: stal Date: Wed, 23 Apr 2014 17:29:50 -0700 Subject: [PATCH] Properly deallocate friends' status messages in kill_messenger Lightly tested. I'm not sure whether this creates a double-free bug --- toxcore/Messenger.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/toxcore/Messenger.c b/toxcore/Messenger.c index 752b164d3..9cc3b7440 100644 --- a/toxcore/Messenger.c +++ b/toxcore/Messenger.c @@ -1816,6 +1816,10 @@ void kill_messenger(Messenger *m) kill_DHT(m->dht); kill_net_crypto(m->net_crypto); kill_networking(m->net); + for (i = 0; i < m->numfriends; ++i) { + if (m->friendlist[i].statusmessage) + free(m->friendlist[i].statusmessage); + } free(m->friendlist); free(m); }