pidgin: d78cffbe: Let's not fool anyone here. The slplink ...

qulogic at pidgin.im qulogic at pidgin.im
Tue May 10 00:16:25 EDT 2011


----------------------------------------------------------------------
Revision: d78cffbef133c407226e5feaf504707ea451a3af
Parent:   50dc3a9db6ac0cd2d6aef34eca503cdf06da3645
Author:   qulogic at pidgin.im
Date:     05/09/11 18:50:05
Branch:   im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/d78cffbef133c407226e5feaf504707ea451a3af

Changelog: 

Let's not fool anyone here. The slplink is never NULL.

Changes against parent 50dc3a9db6ac0cd2d6aef34eca503cdf06da3645

  patched  libpurple/protocols/msn/slpmsg.c

-------------- next part --------------
============================================================
--- libpurple/protocols/msn/slpmsg.c	205d2f6942e54428b4285fbb19d28f1e8566d932
+++ libpurple/protocols/msn/slpmsg.c	6878c0746f1b0ceb186c1bc1635bda3a68f79c17
@@ -39,15 +39,14 @@ msn_slpmsg_new(MsnSlpLink *slplink, MsnS
 	MsnSlpMessage *slpmsg;
 	MsnP2PVersion p2p;
 
+	g_return_val_if_fail(slplink != NULL, NULL);
+
 	slpmsg = g_new0(MsnSlpMessage, 1);
 
 	if (purple_debug_is_verbose())
 		purple_debug_info("msn", "slpmsg new (%p)\n", slpmsg);
 
-	if (slplink)
-		msn_slpmsg_set_slplink(slpmsg, slplink);
-	else
-		slpmsg->slplink = NULL;
+	msn_slpmsg_set_slplink(slpmsg, slplink);
 	slpmsg->slpcall = slpcall;
 
 	p2p = msn_p2p_get_user_support(slplink->remote_user);


More information about the Commits mailing list