From: SourceForge.net <no...@so...> - 2003-08-23 12:27:11
|
Bugs item #753315, was opened at 2003-06-12 06:30 Message generated for change (Comment added) made by vasiljevic You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=753315&group_id=10894 Category: 35. Pathname Management Group: = 8.4.3 >Status: Closed Resolution: Remind Priority: 7 Submitted By: Zoran Vasiljevic (vasiljevic) Assigned to: Zoran Vasiljevic (vasiljevic) Summary: mt-safety issue in filesystem record list Initial Comment: There is a new MT-safety bug appearing in the tclIOUtil.c code. The problem is the shared list of filesystem records. Pointers from this list are being referenced in each path internal rep in every Tcl path object. Now, when objects are getting destroyed, they decrement refcount of the *shared* record. This happens w/o mutex protection, leading to memory corruptions. Especially on MP machines, this comes out pretty soon, i.e. the process cores. ---------------------------------------------------------------------- >Comment By: Zoran Vasiljevic (vasiljevic) Date: 2003-08-23 03:51 Message: Logged In: YES user_id=95086 I have merged the changes into tclIOUtil.c, tclPathObj.c and tclFileSystem.h in current CVS. I'm closing this now. ---------------------------------------------------------------------- Comment By: Jeffrey Hobbs (hobbs) Date: 2003-07-18 12:29 Message: Logged In: YES user_id=72656 Commited for 8.4.4, but not 8.5 (requires some translation to patch tclPathObj.c as well). Zoran - please update for 8.5 and commit there. Also, any chance of core tests to exercise this? ---------------------------------------------------------------------- Comment By: Zoran Vasiljevic (vasiljevic) Date: 2003-06-23 01:09 Message: Logged In: YES user_id=95086 Good. I will do that. I will, however, first wait for people at AOL to double-check this in their MP test environment just to be 100% sure. ---------------------------------------------------------------------- Comment By: Vince Darley (vincentdarley) Date: 2003-06-22 23:46 Message: Logged In: YES user_id=32170 The patch looks good to me; please do apply against both cvs head and 8.4 branch (looks like minor changes will be needed for cvs head, because of the splitting into multiple files of the fs code). Thanks! ---------------------------------------------------------------------- Comment By: Zoran Vasiljevic (vasiljevic) Date: 2003-06-22 07:35 Message: Logged In: YES user_id=95086 Here is the patch which corrects the problem. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=753315&group_id=10894 |