Fix #3304.

mdboom · mdboom · commit d797bc18b482 · 2014-07-29T12:27:55.000-04:00
Fix crash in tostring_rgba_minimized when the region has zero-size. Thanks to @neothemachine for tracking this down.
diff --git a/src/_backend_agg.cpp b/src/_backend_agg.cpp
@@ -2420,6 +2420,12 @@ RendererAgg::tostring_rgba_minimized(const Py::Tuple& args)
                 *dst = src[y * width + x];
             }
         }
+    } else {
+        data = PyBytes_FromStringAndSize(NULL, 0);
+        if (data == NULL)
+        {
+            throw Py::MemoryError("RendererAgg::tostring_minimized could not allocate memory");
+        }
     }
 
     Py::Tuple bounds(4);

Original file line number	Diff line number	Diff line change
`@@ -2420,6 +2420,12 @@ RendererAgg::tostring_rgba_minimized(const Py::Tuple& args)`
`2420`	`2420`	`dst = src[y width + x];`
`2421`	`2421`	`}`
`2422`	`2422`	`}`
	`2423`	`+ } else {`
	`2424`	`+ data = PyBytes_FromStringAndSize(NULL, 0);`
	`2425`	`+ if (data == NULL)`
	`2426`	`+ {`
	`2427`	`+ throw Py::MemoryError("RendererAgg::tostring_minimized could not allocate memory");`
	`2428`	`+ }`
`2423`	`2429`	`}`
`2424`	`2430`
`2425`	`2431`	`Py::Tuple bounds(4);`