Changeset 99c8f13 for protocols/yahoo/libyahoo2.c

Timestamp:

2009-10-17T14:48:21Z (15 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

57d8421

Parents:

e71cfbc

Message:

Valgrind pointed me at some memory leaks in the Yahoo! codek, including one
that existed for a while already. Fixed.

File:

• protocols/yahoo/libyahoo2.c (modified) (2 diffs)

protocols/yahoo/libyahoo2.c

@@ -2486 +2486 @@
         yahoo_packet_free(pack);
         
-        return;
-        
 fail:
+        g_free(crumb);
         g_free(had->token);
         g_free(had->chal);
@@ -4135 +4134 @@
         LOG(("yahoo_logoff: current status: %d", yd->current_status));
 
-        if(yd->current_status != -1) {
+        if(yd->current_status != -1 && 0) {
+                /* Meh. Don't send this. The event handlers are not going to
+                   get to do this so it'll just leak memory. And the TCP
+                   connection reset will hopefully be clear enough. */
                 pkt = yahoo_packet_new(YAHOO_SERVICE_LOGOFF, YAHOO_STATUS_AVAILABLE, yd->session_id);
                 yd->current_status = -1;