Ticket UUID: | 19a8c9399d025831a15d5bd0047a005c799e844e | |||
Title: | var-22.1 failing on trunk with TCL_MEM_DEBUG | |||
Type: | Bug | Version: | trunk | |
Submitter: | aspect | Created on: | 2017-05-21 15:22:58 | |
Subsystem: | 47. Bytecode Compiler | Assigned To: | dgp | |
Priority: | 5 Medium | Severity: | Minor | |
Status: | Closed | Last Modified: | 2017-05-24 08:43:02 | |
Resolution: | Fixed | Closed By: | jan.nijtmans | |
Closed on: | 2017-05-24 08:43:02 | |||
Description: |
With --enable-symbols=all (I presume TCL_MEM_DEBUG is the trigger), checkin [abcf4db5930f] introduced some test failures, of which var-22.1 seems to point at the root. | |||
User Comments: |
jan.nijtmans added on 2017-05-24 08:43:02:
Thanks Don, for fixing this!!! dgp added on 2017-05-23 16:00:32: Looks like a simple lack of a required ckfree(). Fixed in trunk. Thank you for reporting the problem! dgp added on 2017-05-23 15:37:22: It appears that the TIP 459 implementation created a memory leak in a cycle of interp creation and deletion. jan.nijtmans added on 2017-05-22 09:51:36: Thanks for the report! Testcase var-22.1 is a test for a memory leak. Commit [abcf4db5930f] starts keeping track of files loaded during package initialization in a Tcl_Obj* list. I suspect that the test-case sees this as a leak, even though in reality it isn't. For sure, the test-case needs to be adapted some way to correct this, or maybe I'm wrong and there is a leak somehow. But anyway, I don't think this can be considered critical. I will investigate further. |