Issue #18408: Fix listpop(), handle list_ass_slice() failure

diff --git a/Objects/listobject.c b/Objects/listobject.c
index 0ec70e587af815b330fe4c3fc4ca0a9b7902df0c..ce6b70889e8cf2c6813c4f968845c5ed994ecb2d 100644 (file)
--- a/Objects/listobject.c
+++ b/Objects/listobject.c
@@ -934,12 +934,10 @@ listpop(PyListObject *self, PyObject *args)
     }
     Py_INCREF(v);
     status = list_ass_slice(self, i, i+1, (PyObject *)NULL);
-    assert(status >= 0);
-    /* Use status, so that in a release build compilers don't
-     * complain about the unused name.
-     */
-    (void) status;
-
+    if (status < 0) {
+        Py_DECREF(v);
+        return NULL;
+    }
     return v;
 }