[Tcl-bugs] [ tcl-Bugs-788780 ] Debug version crashes on startup

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

Bugs item #788780, was opened at 2003-08-14 16:49
Message generated for change (Comment added) made by dkf
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: Donal K. Fellows (dkf)
Date: 2003-08-24 13:22

Message:
Logged In: YES 
user_id=79902

As documented in TIP#31 http://purl.org/tcl/tip/31 the CVS
branch tag for 8.4.* is core-8-4-branch with the last
release being core-8-4-4


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

Comment By: Zoran Vasiljevic (vasiljevic)
Date: 2003-08-23 17:11

Message:
Logged In: YES 
user_id=95086

Did you make a branch for 8.4.4 development? 
If yes, can you tell me, so I can apply the correction 
in 8.4.x 
 
 

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

Comment By: Jeffrey Hobbs (hobbs)
Date: 2003-08-23 16: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 12: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 17: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 17: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