Error check PyNumber_AsSsize_t in rect_impl.h

Starbuck5 · Starbuck5 · commit b7eed5f40eb1 · 2024-11-26T23:07:50.000-08:00
diff --git a/src_c/rect_impl.h b/src_c/rect_impl.h
@@ -2161,6 +2161,9 @@ RectExport_subscript(RectObject *self, PyObject *op)
 
     if (PyIndex_Check(op)) {
         Py_ssize_t i = PyNumber_AsSsize_t(op, NULL);
+        if (i == -1 && PyErr_Occurred()) {
+            return NULL;
+        }
         return RectExport_item(self, i);
     }
     else if (op == Py_Ellipsis) {
@@ -2207,6 +2210,9 @@ RectExport_assSubscript(RectObject *self, PyObject *op, PyObject *value)
     }
     if (PyIndex_Check(op)) {
         Py_ssize_t i = PyNumber_AsSsize_t(op, NULL);
+        if (i == -1 && PyErr_Occurred()) {
+            return -1;
+        }
         return RectExport_assItem(self, i, value);
     }
     else if (op == Py_Ellipsis) {

Original file line number	Diff line number	Diff line change
`@@ -2161,6 +2161,9 @@ RectExport_subscript(RectObject self, PyObject op)`
`2161`	`2161`
`2162`	`2162`	`if (PyIndex_Check(op)) {`
`2163`	`2163`	`Py_ssize_t i = PyNumber_AsSsize_t(op, NULL);`
	`2164`	`+ if (i == -1 && PyErr_Occurred()) {`
	`2165`	`+ return NULL;`
	`2166`	`+ }`
`2164`	`2167`	`return RectExport_item(self, i);`
`2165`	`2168`	`}`
`2166`	`2169`	`else if (op == Py_Ellipsis) {`
`@@ -2207,6 +2210,9 @@ RectExport_assSubscript(RectObject self, PyObject op, PyObject *value)`
`2207`	`2210`	`}`
`2208`	`2211`	`if (PyIndex_Check(op)) {`
`2209`	`2212`	`Py_ssize_t i = PyNumber_AsSsize_t(op, NULL);`
	`2213`	`+ if (i == -1 && PyErr_Occurred()) {`
	`2214`	`+ return -1;`
	`2215`	`+ }`
`2210`	`2216`	`return RectExport_assItem(self, i, value);`
`2211`	`2217`	`}`
`2212`	`2218`	`else if (op == Py_Ellipsis) {`