Issue #12973: Fix itertools bug caused by signed integer overflow. Thanks Stefan Krah.
This commit is contained in:
parent
adde86d0e3
commit
b2f6bc72a2
|
@ -17,9 +17,9 @@ Core and Builtins
|
|||
- Issue #13021: Missing decref on an error path. Thanks to Suman Saha for
|
||||
finding the bug and providing a patch.
|
||||
|
||||
- Issue #12973: Fix overflow check that relied on undefined behaviour in
|
||||
list_repeat. This bug caused test_list to fail with recent versions
|
||||
of Clang.
|
||||
- Issue #12973: Fix overflow checks that relied on undefined behaviour in
|
||||
list_repeat (listobject.c) and islice_next (itertoolsmodule.c). These bugs
|
||||
caused test failures with recent versions of Clang.
|
||||
|
||||
- Issue #12802: the Windows error ERROR_DIRECTORY (numbered 267) is now
|
||||
mapped to POSIX errno ENOTDIR (previously EINVAL).
|
||||
|
|
|
@ -1234,7 +1234,9 @@ islice_next(isliceobject *lz)
|
|||
return NULL;
|
||||
lz->cnt++;
|
||||
oldnext = lz->next;
|
||||
lz->next += lz->step;
|
||||
/* The (size_t) cast below avoids the danger of undefined
|
||||
behaviour from signed integer overflow. */
|
||||
lz->next += (size_t)lz->step;
|
||||
if (lz->next < oldnext || (stop != -1 && lz->next > stop))
|
||||
lz->next = stop;
|
||||
return item;
|
||||
|
|
Loading…
Reference in New Issue