diff options
| author | jan.nijtmans <nijtmans@users.sourceforge.net> | 2012-12-27 15:04:11 (GMT) |
|---|---|---|
| committer | jan.nijtmans <nijtmans@users.sourceforge.net> | 2012-12-27 15:04:11 (GMT) |
| commit | 3179079214bd7ec9eecf26f71b91ea8d5c7308d1 (patch) | |
| tree | f4ed0cc041f03fed34a2c8d4a5d5e5d78d461095 /generic/tclListObj.c | |
| parent | 4abde457b0e71977a198245a76cc4bd3fd9801ca (diff) | |
| parent | 74ba8ceda7f5765eb6d06f6bac99f565b9a71154 (diff) | |
| download | tcl-3179079214bd7ec9eecf26f71b91ea8d5c7308d1.zip tcl-3179079214bd7ec9eecf26f71b91ea8d5c7308d1.tar.gz tcl-3179079214bd7ec9eecf26f71b91ea8d5c7308d1.tar.bz2 | |
[Bug 3598580]: Tcl_ListObjReplace may release deleted elements too early
Diffstat (limited to 'generic/tclListObj.c')
| -rw-r--r-- | generic/tclListObj.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/generic/tclListObj.c b/generic/tclListObj.c index 3668b45..53c3f66 100644 --- a/generic/tclListObj.c +++ b/generic/tclListObj.c @@ -909,6 +909,10 @@ Tcl_ListObjReplace( isShared = (listRepPtr->refCount > 1); numRequired = numElems - count + objc; + for (i = 0; i < objc; i++) { + Tcl_IncrRefCount(objv[i]); + } + if ((numRequired <= listRepPtr->maxElemCount) && !isShared) { int shift; @@ -1027,14 +1031,11 @@ Tcl_ListObjReplace( } /* - * Insert the new elements into elemPtrs before "first". We don't do a - * memcpy here because we must increment the reference counts for the - * added elements, so we must explicitly loop anyway. + * Insert the new elements into elemPtrs before "first". */ for (i=0,j=first ; i<objc ; i++,j++) { elemPtrs[j] = objv[i]; - Tcl_IncrRefCount(objv[i]); } /* |