[Pidgin] #3303: msimppl leaks memory in statstring, username, key_context, storing user info, user, MsimMessage dictionary parsing

Pidgin trac at pidgin.im
Mon Sep 24 23:27:02 EDT 2007


#3303: msimppl leaks memory in statstring, username, key_context, storing user
info, user, MsimMessage dictionary parsing
----------------------+-----------------------------------------------------
  Reporter:  jeff     |       Owner:  jeff    
      Type:  defect   |      Status:  new     
  Priority:  major    |   Milestone:  2.2.1   
 Component:  MySpace  |     Version:  2.2.0   
Resolution:           |    Keywords:  msimprpl
   Pending:  0        |  
----------------------+-----------------------------------------------------
Comment (by jeff):

 Oliver wrote:
 >I ran the latest version (revision
 31d62e1a78974a0239e86bfeb42b29ab75216114) and it looks much better. Only
 the msim_msg_dictionary_parse (message.c:1197) leak remains but it's a
 rather big leak.

 >Not sure what to do about it as I'm not familiar with that part but
 judging from your comment it sounds like there might be hope. What about
 g_strdup'ing inside msim_msg_append() and then free'ing it whenever you're
 done with the msg ? In that way the string literals wont cause problems.

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


More information about the Tracker mailing list