From: SourceForge.net <no...@so...> - 2010-12-23 18:41:06
|
Bugs item #3142026, was opened at 2010-12-22 13:28 Message generated for change (Comment added) made by hobbs You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=3142026&group_id=10894 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: 17. Commands I-L Group: current: 8.6b1 Status: Open Resolution: None Priority: 8 Private: No Submitted By: brad harder (bharder) Assigned to: Donal K. Fellows (dkf) Summary: lsort -integer violates memory Initial Comment: lsort -integer violates memory allocation when Tcl is compiled w/ TCL_MEM_DEBUG. This violation was introduced between 20Aug2009 and 21Aug2009 in CVS repo. ---------------------------------------------------------------------- >Comment By: Jeffrey Hobbs (hobbs) Date: 2010-12-23 10:41 Message: Donal - you have to compile with --enable-symbols=all (IOW -DTCL_MEM_DEBUG) to ensure the crash. Brad used NetBSD amd64 and I used SuSE Linux-64 to confirm. ---------------------------------------------------------------------- Comment By: Donal K. Fellows (dkf) Date: 2010-12-23 04:57 Message: BTW, your patch is definitely not minimal. It touches unrelated code (the bits relating to lines 1312 to 1346 are nothing to do with [lsort]). ---------------------------------------------------------------------- Comment By: Donal K. Fellows (dkf) Date: 2010-12-23 04:38 Message: I don't see any problem when building on OSX. Do you have a stack trace? ---------------------------------------------------------------------- Comment By: brad harder (bharder) Date: 2010-12-23 00:22 Message: Can confirm patch applies cleanly against CVS co of 23Dec2010 as well, and that Tcl 8.6 build passes test. ---------------------------------------------------------------------- Comment By: brad harder (bharder) Date: 2010-12-23 00:05 Message: see attached patch-aa for a minimal (absolute minimal?) patch against 14 Dec code that appears to solve this specific issue (as hinted by jeff hobbs). ---------------------------------------------------------------------- Comment By: Jeffrey Hobbs (hobbs) Date: 2010-12-22 13:29 Message: Possibly related to the Tcl_LsortObjCmd change from ckalloc to stackalloc? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=110894&aid=3142026&group_id=10894 |