pidgin: 5f8528ff: I have no idea if this check is needed, ...

nosnilmot at pidgin.im nosnilmot at pidgin.im
Thu Jan 8 09:21:08 EST 2009


-----------------------------------------------------------------
Revision: 5f8528ffc18355719bd406e1c579e0e369fb2585
Ancestor: 5b81e4f377729224dc9d281820432e7bd6c637b7
Author: nosnilmot at pidgin.im
Date: 2009-01-08T14:15:16
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/5f8528ffc18355719bd406e1c579e0e369fb2585

Modified files:
        libpurple/protocols/msn/oim.c

ChangeLog: 

I have no idea if this check is needed, but it looked a bit risky without it

-------------- next part --------------
============================================================
--- libpurple/protocols/msn/oim.c	697f272ea83326405b8292f32c65df5b65dcb801
+++ libpurple/protocols/msn/oim.c	5671d21f5fd46800410767e4be9176ac756afdfd
@@ -651,7 +651,7 @@ msn_oim_report_to_user(MsnOimRecvData *r
 
 	/* Match number to user's mobile number, FROM is a phone number
 	   if the other side pages you using your phone number */
-	if (!strncmp(from, "tel:+", 5)) {
+	if (from && !strncmp(from, "tel:+", 5)) {
 		MsnUser *user =	msn_userlist_find_user_with_mobile_phone(
 				rdata->oim->session->userlist, from + 4);
 


More information about the Commits mailing list