Ticket UUID: | 1335006 | |||
Title: | obj leak in TclGlob + mem dump enhancement req | |||
Type: | Bug | Version: | obsolete: 8.5a4 | |
Submitter: | melbardis | Created on: | 2005-10-22 17:48:24 | |
Subsystem: | 36. Pathname Management | Assigned To: | vincentdarley | |
Priority: | 9 Immediate | Severity: | ||
Status: | Closed | Last Modified: | 2005-10-24 01:46:52 | |
Resolution: | Fixed | Closed By: | vincentdarley | |
Closed on: | 2005-10-23 18:46:52 | |||
Description: |
using glob, would lose a number of objects for every invocation. problem was that an argument to the glob function would be incremened (pathptr) and never decrement it, i added decr logic to all paths, i may have missed some! i only tested the case were glob would work, and return some values i am including simple test program that shows the problem, note need to run on a debug version since it uses the 'memory' command also it would be nice if the memry dump function 'Tcl_DumpActiveMemory' was modified in two areas 1) display addressed with leading 0's (%p) - makes searching between the output file and other debug output files a little easier, 2) add threadid to the memory header so one knows which thread allocated the memeory, very usefull in multi-threading work | |||
User Comments: |
vincentdarley added on 2005-10-24 01:46:52:
Logged In: YES user_id=32170 Thanks - memory leak fixed in cvs head. Please file a separate feature request for Tcl_DumpActiveMemory if you'd like to pursue that further. melbardis added on 2005-10-23 00:49:25: File Added - 153447: glob.tcl melbardis added on 2005-10-23 00:49:24: Logged In: YES user_id=788816 adding test code to show problem melbardis added on 2005-10-23 00:48:25: File Added - 153446: tclFileName.c.diff |