# HG changeset patch
# User Nathan Walp <nwalp@pidgin.im>
# Date 1064942054 0
# Node ID 4e56549314016e166c295e57ee1a9a0b0ed51d48
# Parent  c6807437dcd71873b21ce04ce062904ba5f124fb
[gaim-migrate @ 7640]
this is how this function is supposed to behave

committer: Tailor Script <tailor@pidgin.im>

diff -r c6807437dcd7 -r 4e5654931401 src/protocols/jabber/xmlnode.c
--- a/src/protocols/jabber/xmlnode.c	Tue Sep 30 16:36:00 2003 +0000
+++ b/src/protocols/jabber/xmlnode.c	Tue Sep 30 17:14:14 2003 +0000
@@ -210,21 +210,25 @@
 char *
 xmlnode_get_data(xmlnode *node)
 {
-	GString *str;
-	char *ret;
+	GString *str = NULL;
+	char *ret = NULL;
 	xmlnode *c;
 
 	g_return_val_if_fail(node != NULL, NULL);
 
-	str = g_string_new("");
 
 	for(c = node->child; c; c = c->next) {
-		if(c->type == NODE_TYPE_DATA)
+		if(c->type == NODE_TYPE_DATA) {
+			if(!str)
+				str = g_string_new("");
 			str = g_string_append_len(str, c->data, c->data_sz);
+		}
 	}
 
-	ret = str->str;
-	g_string_free(str, FALSE);
+	if(str) {
+		ret = str->str;
+		g_string_free(str, FALSE);
+	}
 
 	return ret;
 }