[Pidgin] #5054: Stop a cancelled transfer crashes Pidgin

Pidgin trac at pidgin.im
Sat Mar 22 14:22:41 EDT 2008


#5054: Stop a cancelled transfer crashes Pidgin
---------------------+------------------------------------------------------
  Reporter:  CapnJB  |       Owner:  MarkDoliner  
      Type:  defect  |      Status:  new          
  Priority:  minor   |   Milestone:               
 Component:  AIM     |     Version:  2.4.0        
Resolution:          |    Keywords:  file transfer
   Pending:  0       |  
---------------------+------------------------------------------------------
Changes (by rekkanoryo):

  * owner:  lschiere => MarkDoliner
  * component:  unclassified => AIM

Comment:

 `g_free()` has guards against NULL.  It is perfectly safe to `g_free()` a
 NULL pointer, so the `g_free(info);` can be outside the `if` statement in
 your patch.  Members of `info`, however, cannot be freed if `info` is
 NULL, so that part of the patch is likely correct, but I will leave it to
 Mark to decide.

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


More information about the Tracker mailing list