Opened 17 years ago

Closed 14 years ago

#2285 closed defect (invalid)

WXS leaks

Reported by: hobu Owned by: assefa
Priority: normal Milestone: 5.6 release
Component: MapServer C Library Version: unspecified
Severity: normal Keywords:
Cc:

Description

http://smoke.hobu.net/msautotest/wxs/result/wms_time_pattern13d.png.vgrind.txt.txt contains a fairly large leak. There are a number of other leaks in the WXS code that should be addressed as well. http://smoke.hobu.net/msautotest/wxs/result/

==8507== Memcheck, a memory error detector.
==8507== Copyright (C) 2002-2007, and GNU GPL'd, by Julian Seward et al.
==8507== Using LibVEX rev 1732, a library for dynamic binary translation.
==8507== Copyright (C) 2004-2007, and GNU GPL'd, by OpenWorks LLP.
==8507== Using valgrind-3.2.3, a dynamic binary instrumentation framework.
==8507== Copyright (C) 2000-2007, and GNU GPL'd, by Julian Seward et al.
==8507== For more details, rerun with: -v
==8507== 
==8507== Invalid free() / delete / delete[]
==8507==    at 0x4005250: free (vg_replace_malloc.c:233)
==8507==    by 0xC73B6C: free_mem (in /lib/libc-2.3.6.so)
==8507==    by 0xC737D1: __libc_freeres (in /lib/libc-2.3.6.so)
==8507==    by 0x4001328: _vgnU_freeres (vg_preloaded.c:60)
==8507==    by 0xBFCAB3: _Exit (in /lib/libc-2.3.6.so)
==8507==    by 0x80542A4: main (mapserv.c:1629)
==8507==  Address 0x4B84A70 is not stack'd, malloc'd or (recently) free'd
==8507== 
==8507== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 65 from 1)
==8507== malloc/free: in use at exit: 595,102 bytes in 870 blocks.
==8507== malloc/free: 5,447 allocs, 4,578 frees, 2,046,602 bytes allocated.
==8507== For counts of detected errors, rerun with: -v
==8507== searching for pointers to 870 not-freed blocks.
==8507== checked 2,053,460 bytes.
==8507== 
==8507== 
==8507== 16,434 (48 direct, 16,386 indirect) bytes in 1 blocks are definitely lost in loss record 6 of 37
==8507==    at 0x40046C1: malloc (vg_replace_malloc.c:149)
==8507==    by 0x80CA67A: msyyalloc (maplexer.c:4202)
==8507==    by 0x80CA7E2: msyy_create_buffer (maplexer.c:3773)
==8507==    by 0x80CAD38: msyyrestart (maplexer.c:3712)
==8507==    by 0x806BEBA: msLoadMap (mapfile.c:4551)
==8507==    by 0x80516C5: loadMap (mapserv.c:201)
==8507==    by 0x8053E1C: main (mapserv.c:1160)
==8507== 
==8507== 
==8507== 96 (16 direct, 80 indirect) bytes in 3 blocks are definitely lost in loss record 7 of 37
==8507==    at 0x40046C1: malloc (vg_replace_malloc.c:149)
==8507==    by 0x80602A6: msStringSplit (mapstring.c:522)
==8507==    by 0x80ECD32: msValidateTimeValue (maptime.c:445)
==8507==    by 0x80D7B5B: msWMSApplyTime (mapwms.c:292)
==8507==    by 0x80D98D0: msWMSLoadGetMapParams (mapwms.c:614)
==8507==    by 0x80DD414: msWMSDispatch (mapwms.c:2912)
==8507==    by 0x809C41E: msOWSDispatch (mapows.c:60)
==8507==    by 0x8053E35: main (mapserv.c:1178)
==8507== 
==8507== LEAK SUMMARY:
==8507==    definitely lost: 64 bytes in 4 blocks.
==8507==    indirectly lost: 16,466 bytes in 5 blocks.
==8507==      possibly lost: 0 bytes in 0 blocks.
==8507==    still reachable: 578,572 bytes in 861 blocks.
==8507==         suppressed: 0 bytes in 0 blocks.
==8507== Reachable blocks (those to which a pointer was found) are not shown.
==8507== To see them, rerun with: --leak-check=full --show-reachable=yes

Change History (4)

comment:1 by dmorissette, 17 years ago

Owner: changed from sdlime to assefa

comment:2 by dmorissette, 17 years ago

Milestone: 5.2 release

comment:3 by assefa, 17 years ago

The last part of the leak report (in maptime.c) should be fixed in r6756. (related bug #2283)

comment:4 by hobu, 14 years ago

Resolution: invalid
Status: newclosed

This bug is too old to be relevant now. If there are leaks, they are probably new. ;) Someone can reopen a bug for new leaks

Note: See TracTickets for help on using tickets.