| Ticket UUID: | 788780 | |||
| Title: | Debug version crashes on startup | |||
| Type: | Bug | Version: | obsolete: 8.4.4 | |
| Submitter: | rstahl | Created on: | 2003-08-14 15:49:12 | |
| Subsystem: | 37. File System | Assigned To: | vasiljevic | |
| Priority: | 5 Medium | Severity: | ||
| Status: | Closed | Last Modified: | 2003-12-17 16:16:32 | |
| Resolution: | None | Closed By: | vasiljevic | |
| Closed on: | 2003-12-17 09:16:32 | |||
| Description: |
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; | |||
| User Comments: |
vasiljevic added on 2003-12-17 16:16:32:
Logged In: YES user_id=95086 Close this one since already fixed in 8.4.5. dgp added on 2003-10-03 05:17:50: Logged In: YES user_id=80530 Is this a duplicate report? Can it be closed? mistachkin added on 2003-09-05 04:06:49: Logged In: YES user_id=113501 Apparently a duplicate of 789949. vasiljevic added on 2003-09-01 19:33:30: Logged In: YES user_id=95086 It is now backported to core-8-4-branch. hobbs added on 2003-08-25 23:39:02: Logged In: YES user_id=72656 no, you want core-8-4-branch to commit against. vasiljevic added on 2003-08-24 19:31:34: Logged In: YES user_id=95086 So, I assume if I go, checkout the core-8-4-4, make the changes and commit, the CVS won't freak-out, correct? dkf added on 2003-08-24 19:22:27: 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 vasiljevic added on 2003-08-23 23:11:46: 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 hobbs added on 2003-08-23 22:14:01: Logged In: YES user_id=72656 If the patch is correct, it should be backported. vasiljevic added on 2003-08-23 18:36:05: 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. vasiljevic added on 2003-08-14 23:10:58: 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 :) vincentdarley added on 2003-08-14 23:02:54: 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). rstahl added on 2003-08-14 22:49:14: File Added - 58742: tclIOUtil.c | |||
Attachments:
- tclIOUtil.c [download] added by rstahl on 2003-08-14 22:49:13. [details]