[Pidgin] #14473: Change PurpleStoredImg refcount type

Pidgin trac at pidgin.im
Mon Aug 1 04:51:49 EDT 2011


#14473: Change PurpleStoredImg refcount type
--------------------------+-------------------------------------------------
 Reporter:  hanzz         |     Owner:  rekkanoryo
     Type:  defect        |    Status:  new       
Component:  unclassified  |   Version:  2.9.0     
 Keywords:                |  
--------------------------+-------------------------------------------------
 Hi,

 In my libpurple usecase, I connect lot of users and set the same avatar
 for each of them. prpl-msn creates 3-5 refs to it (lets presume 4).

 Current type of refcount variable is guint8, so there can be just 255
 refs, otherwise guint8 overflows and libpurple crashes or frees random
 memory later.

 In my case it means I can have just something about 63 users online. I
 would suggest to change guint8 to to guint16, which will be enough for all
 usecase.

 Since PurpleStoredImg is declared in .c, it can be done without breaking
 API.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/14473>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list