diff options
author | stal <stal@zodiaclabs.org> | 2014-04-23 17:29:50 -0700 |
---|---|---|
committer | stal <stal@zodiaclabs.org> | 2014-04-23 17:29:50 -0700 |
commit | 2b283535544aab26840dae6eaa2a1df07f6b60c9 (patch) | |
tree | 33c4610c6604bde9a511284e97d7022855c75692 /toxcore | |
parent | 38e3b3516cbe2ac4208803ccd6c39fbadc052e6f (diff) |
Properly deallocate friends' status messages in kill_messenger
Lightly tested. I'm not sure whether this creates a double-free
bug
Diffstat (limited to 'toxcore')
-rw-r--r-- | toxcore/Messenger.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/toxcore/Messenger.c b/toxcore/Messenger.c index 752b164d..9cc3b744 100644 --- a/toxcore/Messenger.c +++ b/toxcore/Messenger.c | |||
@@ -1816,6 +1816,10 @@ void kill_messenger(Messenger *m) | |||
1816 | kill_DHT(m->dht); | 1816 | kill_DHT(m->dht); |
1817 | kill_net_crypto(m->net_crypto); | 1817 | kill_net_crypto(m->net_crypto); |
1818 | kill_networking(m->net); | 1818 | kill_networking(m->net); |
1819 | for (i = 0; i < m->numfriends; ++i) { | ||
1820 | if (m->friendlist[i].statusmessage) | ||
1821 | free(m->friendlist[i].statusmessage); | ||
1822 | } | ||
1819 | free(m->friendlist); | 1823 | free(m->friendlist); |
1820 | free(m); | 1824 | free(m); |
1821 | } | 1825 | } |