[Pidgin] #4875: MSNP15 does not update tokens correctly.

Pidgin trac at pidgin.im
Sat May 10 01:10:16 EDT 2008


#4875: MSNP15 does not update tokens correctly.
----------------------+-----------------------------------------------------
  Reporter:  QuLogic  |       Owner:  khc             
      Type:  patch    |      Status:  new             
  Priority:  minor    |   Milestone:  Activate MSNPV14
 Component:  MSN      |     Version:                  
Resolution:           |    Keywords:                  
   Pending:  0        |  
----------------------+-----------------------------------------------------
Comment (by QuLogic):

 I noticed while merging things that datallah added a few calls to
 `msn_soap_message_destroy` in soap2.c. It seems that these cause a double
 free (due to how I've had to rework SOAP calls) and I've had to remove
 them. I'm not yet certain if it still leaks without these calls, though;
 the SOAP code is a complicated thing...

 {{{
 --- libpurple/protocols/msn/soap2.c
 a002af0e9217c841a6ba623442584447e5f888d7
 +++ libpurple/protocols/msn/soap2.c
 9920b0d5c39c780723682813be41cb93fe605b87
 @@ -399,11 +399,8 @@ msn_soap_read_cb(gpointer data, gint fd,
                                 conn->message = NULL;
                                 message->xml = node;

 -                               if (!msn_soap_handle_body(conn, message))
 {
 -                                       msn_soap_message_destroy(message);
 +                               if (!msn_soap_handle_body(conn, message))
                                         return;
 -                               }
 -                               msn_soap_message_destroy(message);
                         }

                         msn_soap_connection_handle_next(conn);
 }}}

-- 
Ticket URL: <http://developer.pidgin.im/ticket/4875#comment:9>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list