# HG changeset patch # User Richard Laager # Date 1228713644 0 # Node ID efe41c4e0df0d1fa4f85cbe80a550dc6a357d53a # Parent 1f9700b825a44407b945b5ecc4aecff09440a554 Fix Bonjour to better deal with buddy values set to the empty string. diff -r 1f9700b825a4 -r efe41c4e0df0 libpurple/protocols/bonjour/bonjour.c --- a/libpurple/protocols/bonjour/bonjour.c Sun Dec 07 02:36:57 2008 +0000 +++ b/libpurple/protocols/bonjour/bonjour.c Mon Dec 08 05:20:44 2008 +0000 @@ -376,20 +376,20 @@ } /* Only show first/last name if there is a nickname set (to avoid duplication) */ - if (bb->nick != NULL) { - if (bb->first != NULL) + if (bb->nick != NULL && *bb->nick != '\0') { + if (bb->first != NULL && *bb->first != '\0') purple_notify_user_info_add_pair(user_info, _("First name"), bb->first); - if (bb->first != NULL) + if (bb->last != NULL && *bb->last != '\0') purple_notify_user_info_add_pair(user_info, _("Last name"), bb->last); } - if (bb->email != NULL) + if (bb->email != NULL && *bb->email != '\0') purple_notify_user_info_add_pair(user_info, _("Email"), bb->email); - if (bb->AIM != NULL) + if (bb->AIM != NULL && *bb->AIM != '\0') purple_notify_user_info_add_pair(user_info, _("AIM Account"), bb->AIM); - if (bb->jid!= NULL) + if (bb->jid != NULL && *bb->jid != '\0') purple_notify_user_info_add_pair(user_info, _("XMPP Account"), bb->jid); } diff -r 1f9700b825a4 -r efe41c4e0df0 libpurple/protocols/bonjour/buddy.c --- a/libpurple/protocols/bonjour/buddy.c Sun Dec 07 02:36:57 2008 +0000 +++ b/libpurple/protocols/bonjour/buddy.c Mon Dec 08 05:20:44 2008 +0000 @@ -161,7 +161,7 @@ name = purple_buddy_get_name(buddy); /* Create the alias for the buddy using the first and the last name */ - if (bonjour_buddy->nick) + if (bonjour_buddy->nick && *bonjour_buddy->nick) serv_got_alias(purple_account_get_connection(account), name, bonjour_buddy->nick); else { gchar *alias = NULL;