[Pidgin] #7712: Nullprpl cleanup/fixes

Pidgin trac at pidgin.im
Sat Dec 6 00:24:13 EST 2008


#7712: Nullprpl cleanup/fixes
---------------------------+------------------------------------------------
 Reporter:  darkrain42     |        Owner:           
     Type:  patch          |       Status:  closed   
Milestone:  2.5.3          |    Component:  libpurple
  Version:  2.5.2          |   Resolution:  fixed    
 Keywords:  null nullprpl  |  
---------------------------+------------------------------------------------
Changes (by paul at aurich.com):

  * status:  new => closed
  * resolution:  => fixed
  * milestone:  => 2.5.3


Comment:

 (In [4e0cf1c28c6c278bc23f23553b06a6980e2a1413]):[[BR]]
 Various nullprpl fixes. This stemmed from a complaint from a user in
 #pidgin that it crashed if the user didn't change the room from "default"
 to something else when joining.

  Nothing big here, but here's the list:
   * Make a bunch of strings const for good measure
   * g_strdup strings that the caller is supposed to be freeing (one of
     these almost certainly fixes the aforementioned crash)
   * Use g_list_prepend/g_list_reverse
   * Don't leak some strings
   * g_* needs g_free, not free (this might also be the one fixing the
     crash)

 Fixes #7712.

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


More information about the Tracker mailing list