summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2016-07-03 11:41:36 (GMT)
committerSerhiy Storchaka <storchaka@gmail.com>2016-07-03 11:41:36 (GMT)
commitaf65872da2b679e4b1876412aab6a731f82a469f (patch)
tree24ae9a673e5203c75b0ac4fc5dd4bfb7acb8fb01
parent8faca61fec8ef004feb534dee3da052f3bed82ce (diff)
downloadcpython-af65872da2b679e4b1876412aab6a731f82a469f.zip
cpython-af65872da2b679e4b1876412aab6a731f82a469f.tar.gz
cpython-af65872da2b679e4b1876412aab6a731f82a469f.tar.bz2
Issue #27443: __length_hint__() of bytearray itearator no longer return
negative integer for resized bytearray.
-rw-r--r--Lib/test/test_bytes.py10
-rw-r--r--Misc/NEWS3
-rw-r--r--Objects/bytearrayobject.c6
3 files changed, 18 insertions, 1 deletions
diff --git a/Lib/test/test_bytes.py b/Lib/test/test_bytes.py
index 65c00d7..cc312b1 100644
--- a/Lib/test/test_bytes.py
+++ b/Lib/test/test_bytes.py
@@ -1233,6 +1233,16 @@ class ByteArrayTest(BaseBytesTest, unittest.TestCase):
test_exhausted_iterator = test.list_tests.CommonTest.test_exhausted_iterator
+ def test_iterator_length_hint(self):
+ # Issue 27443: __length_hint__ can return negative integer
+ ba = bytearray(b'ab')
+ it = iter(ba)
+ next(it)
+ ba.clear()
+ # Shouldn't raise an error
+ self.assertEqual(list(it), [])
+
+
class AssortedBytesTest(unittest.TestCase):
#
# Test various combinations of bytes and bytearray
diff --git a/Misc/NEWS b/Misc/NEWS
index 935f81e..a04212c 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -10,6 +10,9 @@ Release date: TBA
Core and Builtins
-----------------
+- Issue #27443: __length_hint__() of bytearray itearator no longer return
+ negative integer for resized bytearray.
+
Library
-------
diff --git a/Objects/bytearrayobject.c b/Objects/bytearrayobject.c
index 277be59..388e990 100644
--- a/Objects/bytearrayobject.c
+++ b/Objects/bytearrayobject.c
@@ -3192,8 +3192,12 @@ static PyObject *
bytearrayiter_length_hint(bytesiterobject *it)
{
Py_ssize_t len = 0;
- if (it->it_seq)
+ if (it->it_seq) {
len = PyByteArray_GET_SIZE(it->it_seq) - it->it_index;
+ if (len < 0) {
+ len = 0;
+ }
+ }
return PyLong_FromSsize_t(len);
}