# HG changeset patch # User Evan Schoenberg # Date 1184437011 0 # Node ID 925ba3e6831ccffbccc8d48ecd998077362ae851 # Parent 70509e165225e6eac70080e8f95afe17b3c5e015 cmd->params[2] could be NULL, or purple_url_decode() could return NULL. Don't pass a potential NULL to strcmp(). diff -r 70509e165225 -r 925ba3e6831c libpurple/protocols/msn/notification.c --- a/libpurple/protocols/msn/notification.c Sat Jul 14 10:52:31 2007 +0000 +++ b/libpurple/protocols/msn/notification.c Sat Jul 14 18:16:51 2007 +0000 @@ -705,7 +705,7 @@ user = msn_userlist_find_user(session->userlist, passport); old_friendly = msn_user_get_friendly_name(user); - if (!old_friendly || (old_friendly && strcmp(old_friendly, friendly))) + if (!old_friendly || (old_friendly && (!friendly || strcmp(old_friendly, friendly)))) { serv_got_alias(gc, passport, friendly); msn_user_set_friendly_name(user, friendly);