[3.13] gh-126341: add release check to __iter__ method of memoryview (GH-126759) (#126778)

miss-islington · ritvikpasham · blurb-it[bot] · web-flow · commit 3fe062c98d5b · 2024-11-13T19:05:59.000Z
gh-126341: add release check to `__iter__` method of `memoryview` (GH-126759) (cherry picked from commit a12690e) Co-authored-by: Ritvik Pasham <ritvikpasham@gmail.com> Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com> Co-authored-by: Peter Bierma <zintensitydev@gmail.com> Co-authored-by: Victor Stinner <vstinner@python.org> Co-authored-by: sobolevn <mail@sobolevn.me>
diff --git a/Lib/test/test_buffer.py b/Lib/test/test_buffer.py
@@ -3910,6 +3910,8 @@ def test_memoryview_check_released(self):
         self.assertRaises(ValueError, memoryview, m)
         # memoryview.cast()
         self.assertRaises(ValueError, m.cast, 'c')
+        # memoryview.__iter__()
+        self.assertRaises(ValueError, m.__iter__)
         # getbuffer()
         self.assertRaises(ValueError, ndarray, m)
         # memoryview.tolist()
diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2024-11-12-19-24-00.gh-issue-126341.5SdAe1.rst b/Misc/NEWS.d/next/Core_and_Builtins/2024-11-12-19-24-00.gh-issue-126341.5SdAe1.rst
@@ -0,0 +1 @@
+Now :exc:`ValueError` is raised instead of :exc:`SystemError` when trying to iterate over a released :class:`memoryview` object.
diff --git a/Objects/memoryobject.c b/Objects/memoryobject.c
@@ -3355,6 +3355,7 @@ memory_iter(PyObject *seq)
         PyErr_BadInternalCall();
         return NULL;
     }
+    CHECK_RELEASED(seq);
     PyMemoryViewObject *obj = (PyMemoryViewObject *)seq;
     int ndims = obj->view.ndim;
     if (ndims == 0) {

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+Now :exc:`ValueError` is raised instead of :exc:`SystemError` when trying to iterate over a released :class:`memoryview` object.
Original file line number	Diff line number	Diff line change
`@@ -3355,6 +3355,7 @@ memory_iter(PyObject *seq)`
`3355`	`3355`	`PyErr_BadInternalCall();`
`3356`	`3356`	`return NULL;`
`3357`	`3357`	`}`
	`3358`	`+ CHECK_RELEASED(seq);`
`3358`	`3359`	`PyMemoryViewObject obj = (PyMemoryViewObject )seq;`
`3359`	`3360`	`int ndims = obj->view.ndim;`
`3360`	`3361`	`if (ndims == 0) {`