summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjan.nijtmans <nijtmans@users.sourceforge.net>2012-12-27 14:41:15 (GMT)
committerjan.nijtmans <nijtmans@users.sourceforge.net>2012-12-27 14:41:15 (GMT)
commit20253b8c7d3f3c59314380703be48df859cdf9e6 (patch)
tree291afd92fb75aa7117ad7fb895bc76a93c0c978c
parentb59e26b4ebf4d75131241be768955f8ae29e498f (diff)
downloadtcl-20253b8c7d3f3c59314380703be48df859cdf9e6.zip
tcl-20253b8c7d3f3c59314380703be48df859cdf9e6.tar.gz
tcl-20253b8c7d3f3c59314380703be48df859cdf9e6.tar.bz2
[Bug 3598580]: Tcl_ListObjReplace may release deleted elements too early
Tests!? Where are the tests!?! They are in test listobj-11.1
-rw-r--r--ChangeLog5
-rw-r--r--generic/tclListObj.c6
2 files changed, 9 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index 204275f..728b677 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2012-12-27 Jan Nijtmans <nijtmans@users.sf.net>
+
+ * generic/tclListObj.c: [Bug 3598580]: Tcl_ListObjReplace may release
+ deleted elements too early
+
2012-12-21 Jan Nijtmans <nijtmans@users.sf.net>
* unix/dltest/pkgb.c: Turn pkgb.so into a Tcl9 interoperability test
diff --git a/generic/tclListObj.c b/generic/tclListObj.c
index fffe6a2..b4af98a 100644
--- a/generic/tclListObj.c
+++ b/generic/tclListObj.c
@@ -655,6 +655,10 @@ Tcl_ListObjReplace(interp, listPtr, first, count, objc, objv)
count = 0;
}
+ for (i = 0; i < objc; i++) {
+ Tcl_IncrRefCount(objv[i]);
+ }
+
numRequired = (numElems - count + objc);
if (numRequired <= listRepPtr->maxElemCount) {
/*
@@ -689,7 +693,6 @@ Tcl_ListObjReplace(interp, listPtr, first, count, objc, objv)
for (i = 0, j = first; i < objc; i++, j++) {
elemPtrs[j] = objv[i];
- Tcl_IncrRefCount(objv[i]);
}
/*
@@ -745,7 +748,6 @@ Tcl_ListObjReplace(interp, listPtr, first, count, objc, objv)
for (i = 0, j = first; i < objc; i++, j++) {
newPtrs[j] = objv[i];
- Tcl_IncrRefCount(objv[i]);
}
listRepPtr->elemCount = numRequired;