[Tcl-bugs] [ tcl-Bugs-988703 ] tsdPtr->objThreadMap and data never freed in tclObj.c

[SourceForge.net <no...@so...>]

Bugs item #988703, was opened at 2004-07-10 17:07
Message generated for change (Settings changed) made by hobbs
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=110894&aid=988703&group_id=10894

Category: 10. Objects
Group: current: 8.5a2
Status: Open
Resolution: None
>Priority: 7
Submitted By: Joe Mistachkin (mistachkin)
Assigned to: Jeffrey Hobbs (hobbs)
Summary: tsdPtr->objThreadMap and data never freed in tclObj.c

Initial Comment:

When compiled with threads and debugging, the
tsdPtr->objThreadMap hash table and it's associated
data is never freed.  This leak grows for each object
allocated and for each thread used.

A patch designed to fix this issue is attached.

Please review and commit.

Also, I believe this bug may exist in the
core-8-4-branch as well.  Perhaps this bugfix should be
backported?


----------------------------------------------------------------------

Comment By: Joe Mistachkin (mistachkin)
Date: 2004-07-13 19:09

Message:
Logged In: YES 
user_id=113501


Patch updated. Although, now that I think about it, I'm not
sure what the "right thing" is in this case.  Can people
create a new Tcl_Obj without hitting the code that
initializes the thread storage for "tclEvent.c"?


----------------------------------------------------------------------

Comment By: Joe Mistachkin (mistachkin)
Date: 2004-07-13 16:41

Message:
Logged In: YES 
user_id=113501


Patch needs to be update, the call:

  TclFinalizeThreadObjDebugging();

needs to be outside of the "if" block it is currently in.


----------------------------------------------------------------------

Comment By: miguel sofer (msofer)
Date: 2004-07-13 14:41

Message:
Logged In: YES 
user_id=148712

Dropping the ball: I don't know about these issues.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=110894&aid=988703&group_id=10894