Fix segfault by checking return value from method.apply()

stefanor · mdboom · commit bed5241d89d7 · 2011-03-16T10:35:31.000-04:00
diff --git a/src/_backend_agg.cpp b/src/_backend_agg.cpp
@@ -358,6 +358,8 @@ GCAgg::_set_hatch_path(const Py::Object& gc)
     Py::Object method_obj = gc.getAttr("get_hatch_path");
     Py::Callable method(method_obj);
     hatchpath = method.apply(Py::Tuple());
+    if (hatchpath.ptr() == NULL)
+        throw Py::Exception();
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -358,6 +358,8 @@ GCAgg::_set_hatch_path(const Py::Object& gc)`
`358`	`358`	`Py::Object method_obj = gc.getAttr("get_hatch_path");`
`359`	`359`	`Py::Callable method(method_obj);`
`360`	`360`	`hatchpath = method.apply(Py::Tuple());`
	`361`	`+ if (hatchpath.ptr() == NULL)`
	`362`	`+ throw Py::Exception();`
`361`	`363`	`}`
`362`	`364`
`363`	`365`