Opened 18 years ago

Last modified 14 years ago

#1545 new defect

msShapeGetAnnotation may result memory leak.

Reported by: szekerest Owned by: sdlime
Priority: high Milestone:
Component: MapServer C Library Version: 4.6
Severity: normal Keywords:
Cc:

Description 

msShapeGetAnnotation (in maputil.c) may result memory leak. If the class 
contains .text value the locally allocated string will be replaced without 
being freed.

Tamas

Attachments (2)

msShapeGetAnnotation.diff​ (982 bytes ) - added by bretlambert 14 years ago.
diff to fix memleak
msShapeGetAnnotation.2.diff​ (982 bytes ) - added by bretlambert 14 years ago.
diff to fix memleak

Download all attachments as: .zip

Change History (5)

comment:1 by szekerest, 18 years ago 

Steve,

Do you consider this problem as crucial?


Tamas

comment:2 by sdlime, 18 years ago 

I hadn't thought about it- this one slipped by me unfortunately. It's crucial 
for those java and C# folks that tend to use long running processes. Have you 
peeked at the code to scope out a fix?

Steve

by bretlambert, 14 years ago

Attachment: msShapeGetAnnotation.diff​ added

diff to fix memleak

by bretlambert, 14 years ago

Attachment: msShapeGetAnnotation.2.diff​ added

diff to fix memleak

comment:3 by bretlambert, 14 years ago

diff accidentally added twice

Note: See TracTickets for help on using tickets.