diff options
| author | Paul Aurich <darkrain42@pidgin.im> | 2011-06-27 05:24:53 +0000 |
|---|---|---|
| committer | Paul Aurich <darkrain42@pidgin.im> | 2011-06-27 05:24:53 +0000 |
| commit | 23318d88707e288361380494e4468614cf05fa31 (patch) | |
| tree | 34e1445f3c6834c52bfd013757bb61e3f981a6b4 /libpurple/xmlnode.c | |
| parent | f21062cb7a6e76a651946304c76a8f8f7908f6f9 (diff) | |
| download | pidgin-23318d88707e288361380494e4468614cf05fa31.tar.gz | |
jabber: Check for empty string when setting mood.
The xmlnode API didn't check for the empty string ("") as
node name, which leads to...broken XML. I added those
checks to xmlnode, and fixed this occurrence in usermood.
Fixes #14342
Diffstat (limited to 'libpurple/xmlnode.c')
| -rw-r--r-- | libpurple/xmlnode.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libpurple/xmlnode.c b/libpurple/xmlnode.c index 74ec874f41..6fc7d3dac9 100644 --- a/libpurple/xmlnode.c +++ b/libpurple/xmlnode.c @@ -62,7 +62,7 @@ new_node(const char *name, XMLNodeType type) xmlnode* xmlnode_new(const char *name) { - g_return_val_if_fail(name != NULL, NULL); + g_return_val_if_fail(name != NULL && *name != '\0', NULL); return new_node(name, XMLNODE_TYPE_TAG); } @@ -73,7 +73,7 @@ xmlnode_new_child(xmlnode *parent, const char *name) xmlnode *node; g_return_val_if_fail(parent != NULL, NULL); - g_return_val_if_fail(name != NULL, NULL); + g_return_val_if_fail(name != NULL && *name != '\0', NULL); node = new_node(name, XMLNODE_TYPE_TAG); |