From: SourceForge.net <no...@so...> - 2003-08-23 16:48:46
|
Bugs item #788780, was opened at 2003-08-14 08:49 Message generated for change (Comment added) made by hobbs You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=788780&group_id=10894 Category: 36. File System Group: 8.4.4 Status: Open Resolution: None Priority: 5 Submitted By: Richard Stahl (rstahl) Assigned to: Zoran Vasiljevic (vasiljevic) Summary: Debug version crashes on startup Initial Comment: The function, "FsRecacheFilesystemList(void)," in tclIOUtil.c references freed memory during list traversal. tmpFsRecPtr = fsRecPtr; should be: tmpFsRecPtr = fsRecPtr->nextPtr; --------------------------------------- fsRecPtr = tmpFsRecPtr->nextPtr; should be: fsRecPtr = tmpFsRecPtr; ---------------------------------------------------------------------- >Comment By: Jeffrey Hobbs (hobbs) Date: 2003-08-23 08:14 Message: Logged In: YES user_id=72656 If the patch is correct, it should be backported. ---------------------------------------------------------------------- Comment By: Zoran Vasiljevic (vasiljevic) Date: 2003-08-23 04:36 Message: Logged In: YES user_id=95086 Fixed in CVS head. I will keep this open until we clarify if this is to be patched in 8.4.4 for some future 8.4.x release. ---------------------------------------------------------------------- Comment By: Zoran Vasiljevic (vasiljevic) Date: 2003-08-14 09:10 Message: Logged In: YES user_id=95086 It is. Strange how I did not get this during testing. I did run the entire test suite and it reveiled no problems. The patched 8.4.3 and 8.4.4 are in some heavy-duty MT environments at AOL and they're also working fine. I will check this out again.... Yes, the patch is not applied to the current CVS head. This is still on my todo list. Maybe this is a good point to speed-up the integration :) ---------------------------------------------------------------------- Comment By: Vince Darley (vincentdarley) Date: 2003-08-14 09:02 Message: Logged In: YES user_id=32170 I believe this is from your recent patch? We should also ensure this thread-safety fix is applied to cvs head (currently I don't believe it is). ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=788780&group_id=10894 |