diff options
author | dgp <dgp@users.sourceforge.net> | 2016-11-04 14:40:54 (GMT) |
---|---|---|
committer | dgp <dgp@users.sourceforge.net> | 2016-11-04 14:40:54 (GMT) |
commit | a188ceee2158cb9446f926f4d950a3a98377b988 (patch) | |
tree | 3f4fd384aa651b0b58da2de05866a89565aca1d1 /generic | |
parent | eb87ed60f674e52f1e29227bce5422a860b6bd28 (diff) | |
parent | bc94a394c9804482091ed3a2a5d83652397138c8 (diff) | |
download | tcl-a188ceee2158cb9446f926f4d950a3a98377b988.zip tcl-a188ceee2158cb9446f926f4d950a3a98377b988.tar.gz tcl-a188ceee2158cb9446f926f4d950a3a98377b988.tar.bz2 |
[824752f10e] More robust, portable check for integer overflow.
Diffstat (limited to 'generic')
-rw-r--r-- | generic/tclListObj.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/generic/tclListObj.c b/generic/tclListObj.c index 2929423..344d0fd 100644 --- a/generic/tclListObj.c +++ b/generic/tclListObj.c @@ -897,11 +897,8 @@ Tcl_ListObjReplace( } if (count < 0) { count = 0; - } else if (numElems < first+count || first+count < 0) { - /* - * The 'first+count < 0' condition here guards agains integer - * overflow in determining 'first+count'. - */ + } else if (first > INT_MAX - count /* Handle integer overflow */ + || numElems < first+count) { count = numElems - first; } |