Mercurial > pidgin.yaz
changeset 13650:562ad7cc2d29
[gaim-migrate @ 16051]
Fix Coverity CIDs 53 and 54: A possible memleak. I don't think
this ever actually happens--it would require that the server
send the same piece of information twice in a given packet.
But this is better coding.
committer: Tailor Script <tailor@pidgin.im>
author | Mark Doliner <mark@kingant.net> |
---|---|
date | Tue, 18 Apr 2006 00:03:57 +0000 |
parents | b10030f6eab7 |
children | 6776eeaa17f2 |
files | src/protocols/oscar/family_admin.c |
diffstat | 1 files changed, 3 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- a/src/protocols/oscar/family_admin.c Mon Apr 17 23:53:40 2006 +0000 +++ b/src/protocols/oscar/family_admin.c Tue Apr 18 00:03:57 2006 +0000 @@ -81,10 +81,12 @@ switch (type) { case 0x0001: { + free(sn); sn = byte_stream_getstr(bs, length); } break; case 0x0004: { + free(url); url = byte_stream_getstr(bs, length); } break; @@ -93,6 +95,7 @@ } break; case 0x0011: { + free(email); if (length == 0) email = g_strdup("*suppressed"); else