Opened 17 years ago

Closed 17 years ago

#2077 closed defect (fixed)

small leak in msGMLGetNamespaces

Reported by: hobu Owned by: dmorissette
Priority: normal Milestone: 5.0 release
Component: WFS Server Version:
Severity: normal Keywords:
Cc:

Description

==12803== 8 bytes in 1 blocks are definitely lost in loss record 1 of 5
==12803==    at 0x4004405: malloc (vg_replace_malloc.c:149)
==12803==    by 0x80E1126: msGMLGetNamespaces (mapgml.c:1133)
==12803==    by 0x80A0551: msWFSGetFeature (mapwfs.c:1348)
==12803==    by 0x80A1B76: msWFSDispatch (mapwfs.c:1596)
==12803==    by 0x809C08E: msOWSDispatch (mapows.c:315)
==12803==    by 0x8053E14: main (mapserv.c:1261)

Change History (6)

comment:1 by hobu, 17 years ago

here's another small leak related to DescribeFeatureType

==12804== 10 (4 direct, 6 indirect) bytes in 1 blocks are definitely lost in loss record 1 of 7
==12804==    at 0x4004405: malloc (vg_replace_malloc.c:149)
==12804==    by 0x805FF41: split (mapstring.c:547)
==12804==    by 0x809F86C: msWFSDescribeFeatureType (mapwfs.c:741)
==12804==    by 0x80A1B17: msWFSDispatch (mapwfs.c:1593)
==12804==    by 0x809C08E: msOWSDispatch (mapows.c:315)
==12804==    by 0x8053E14: main (mapserv.c:1261)

comment:2 by hobu, 17 years ago

and another related to GetCapabilities

==12805== 6 bytes in 1 blocks are definitely lost in loss record 1 of 5
==12805==    at 0x4004405: malloc (vg_replace_malloc.c:149)
==12805==    by 0x7F902F: strdup (in /lib/tls/libc-2.3.4.so)
==12805==    by 0x80A10A8: msWFSParseRequest (mapwfs.c:1690)
==12805==    by 0x80A18F8: msWFSDispatch (mapwfs.c:1487)
==12805==    by 0x809C08E: msOWSDispatch (mapows.c:315)
==12805==    by 0x8053E14: main (mapserv.c:1261)

comment:3 by dmorissette, 17 years ago

Owner: changed from mapserverbugs to dmorissette

Taking this ticket.

comment:4 by dmorissette, 17 years ago

Status: newassigned

GetCapabilities leak of wfsparams->pszVersion from comment:2 above fixed in r6443.

comment:5 by dmorissette, 17 years ago

Fixed the following leak of msStringSplit() output from comment:1 in r6444:

==13513== 9 (4 direct, 5 indirect) bytes in 1 blocks are definitely lost in loss record 1 of 4
==13513==    at 0x401C422: malloc (vg_replace_malloc.c:149)
==13513==    by 0x80615A2: msStringSplit (mapstring.c:521)
==13513==    by 0x80AB958: msWFSDescribeFeatureType (mapwfs.c:657)
==13513==    by 0x80AEB8C: msWFSDispatch (mapwfs.c:1641)
==13513==    by 0x80A7C5D: msOWSDispatch (mapows.c:70)
==13513==    by 0x80541E4: main (mapserv.c:1178)

Note that even though the leak is gone there seemed to be a few problems with this block of code, I have created ticket #2197 about fixing it.

comment:6 by dmorissette, 17 years ago

Resolution: fixed
Status: assignedclosed

Fixed the leak of gmlNamespaceListObj in GetFeature and DescribeFeatureType in r6446 (the first msGMLGetNamespaces leak listed in this ticket).

Closing ticket.

Note: See TracTickets for help on using tickets.