pidgin: e3e068c1: It seems like aMSN can get a bit stuck w...

qulogic at pidgin.im qulogic at pidgin.im
Fri May 21 18:05:48 EDT 2010


-----------------------------------------------------------------
Revision: e3e068c14157c92fee76edb303d01dece47fee1d
Ancestor: 4acf23b5da3b1678f8103b564bc8d6cecfbec17e
Author: qulogic at pidgin.im
Date: 2010-05-18T06:36:35
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/e3e068c14157c92fee76edb303d01dece47fee1d

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

ChangeLog: 

It seems like aMSN can get a bit stuck when using multiple file transfers,
and the user could accept a file transfer a second time (it seems to fix
itself eventually.) We don't want to try and start an xfer again or bad
things might happen.

-------------- next part --------------
============================================================
--- libpurple/protocols/msn/slplink.c	b6dae4725301c86f5bf647ef74b036d86e52d895
+++ libpurple/protocols/msn/slplink.c	08a0b85cd3f65dc5c068af65d94e1ee4094e3f41
@@ -508,6 +508,9 @@ send_file_cb(MsnSlpCall *slpcall)
 	PurpleXfer *xfer;
 
 	xfer = (PurpleXfer *)slpcall->xfer;
+	if (purple_xfer_get_status(xfer) >= PURPLE_XFER_STATUS_STARTED)
+		return;
+
 	purple_xfer_ref(xfer);
 	purple_xfer_start(xfer, -1, NULL, 0);
 	if (purple_xfer_get_status(xfer) != PURPLE_XFER_STATUS_STARTED) {


More information about the Commits mailing list