Opened 3 years ago

Closed 7 months ago

#22 closed defect (fixed)

Apparent memory leak in WM_DRAWITEM handler in mmfolder.c

Reported by: Lewis Rosenthal Owned by:
Priority: major Milestone: 0.3.0 first from Netlabs
Component: Classes Version: 0.2.9
Keywords: Cc:

Description

It seems that HBITMAP is not freed.

More detail reported in Arca Noae ticket 2756.

Attachments (1)

mmfolder.diff (29.3 KB) - added by erdmann 7 months ago.
Changes in mmfolder.c taken against SVN rev 67

Download all attachments as: .zip

Change History (4)

comment:1 Changed 7 months ago by erdmann

added a diff file for mmfolder.c that should fix the issue.

Unfortunately, nepmd strips all space characters at end of lines, leading to many white-space only changes ...

Last edited 7 months ago by erdmann (previous) (diff)

Changed 7 months ago by erdmann

Attachment: mmfolder.diff added

Changes in mmfolder.c taken against SVN rev 67

comment:2 Changed 7 months ago by erdmann

SVN rev 69 is fixing the problem, however at this location:
http://trac.netlabs.org/cwmmclasses/browser/trunk/classes/c/c_mm/mmfolder.c#L586

needs to spell:
hbm=MMBitmap_cwmmQuerySmallBitmapHandle( (MMBitmap*) wpObject, ulIconSize);

comment:3 Changed 7 months ago by erdmann

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.