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