# HG changeset patch
# User Mark Doliner <mark@kingant.net>
# Date 1145318637 0
# Node ID 562ad7cc2d2987a5356a344f1c831945564075c0
# Parent  b10030f6eab7a99385f10f7df8462741d511bd3a
[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>

diff -r b10030f6eab7 -r 562ad7cc2d29 src/protocols/oscar/family_admin.c
--- 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