Check unlikely error return from PyDict_DelItemString

virtuald · virtuald · commit 1eee738aaf68 · 2021-07-01T14:16:44.000-04:00
diff --git a/include/pybind11/pybind11.h b/include/pybind11/pybind11.h
@@ -692,7 +692,9 @@ class cpp_function : public function {
                                 kwargs = reinterpret_steal<dict>(PyDict_Copy(kwargs.ptr()));
                                 copied_kwargs = true;
                             }
-                            PyDict_DelItemString(kwargs.ptr(), arg_rec.name);
+                            if (PyDict_DelItemString(kwargs.ptr(), arg_rec.name) == -1) {
+                                throw error_already_set();
+                            }
                         } else if (arg_rec.value) {
                             value = arg_rec.value;
                         }

Original file line number	Diff line number	Diff line change
`@@ -692,7 +692,9 @@ class cpp_function : public function {`
`692`	`692`	`kwargs = reinterpret_steal<dict>(PyDict_Copy(kwargs.ptr()));`
`693`	`693`	`copied_kwargs = true;`
`694`	`694`	`}`
`695`		`- PyDict_DelItemString(kwargs.ptr(), arg_rec.name);`
	`695`	`+ if (PyDict_DelItemString(kwargs.ptr(), arg_rec.name) == -1) {`
	`696`	`+ throw error_already_set();`
	`697`	`+ }`
`696`	`698`	`} else if (arg_rec.value) {`
`697`	`699`	`value = arg_rec.value;`
`698`	`700`	`}`