<div dir="ltr">When you call _intersect_rect and pass it an empty rectangle,<br>it creates an invalid region from that rectangle. The only valid<br>empty region is the one where the data pointer points to<br>pixman_region_empty_data. The region created by _intersect_rect has<br>a null data pointer, and so _not_empty returns true for it even if<br>the extents are empty.<br><br>This patch uses code similar to what's in _union_rect to handle empty<br>rectangles.<br><br>Reviewed-by: Siarhei Siamashka <<a href="mailto:siarhei.siamashka@gmail.com">siarhei.siamashka@gmail.com</a>><br>---<br> pixman/pixman-region.c | 17 +++++++++++++++++<br> 1 file changed, 17 insertions(+)<br><br>diff --git a/pixman/pixman-region.c b/pixman/pixman-region.c<br>index 59bc9c7..69466ed 100644<br>--- a/pixman/pixman-region.c<br>+++ b/pixman/pixman-region.c<br>@@ -1329,16 +1329,33 @@ PREFIX(_intersect_rect) (region_type_t *dest,<br>     region_type_t region;<br> <br>     region.data = NULL;<br>     region.extents.x1 = x;<br>     region.extents.y1 = y;<br>     region.extents.x2 = x + width;<br>     region.extents.y2 = y + height;<br> <br>+    if (!GOOD_RECT (&region.extents))<br>+    {<br>+        if (BAD_RECT (&region.extents))<br>+            _pixman_log_error (FUNC, "Invalid rectangle passed");<br>+        FREE_DATA (dest);<br>+        PREFIX (_init) (dest);<br>+        if (PIXREGION_NAR (source))<br>+        {<br>+            dest->data = pixman_broken_data;<br>+            return FALSE;<br>+        }<br>+        else<br>+        {<br>+            return TRUE;<br>+        }<br>+    }<br>+<br>     return PREFIX(_intersect) (dest, source, &region);<br> }<br> <br> /* Convenience function for performing union of region with a<br>  * single rectangle<br>  */<br> PIXMAN_EXPORT pixman_bool_t<br> PREFIX (_union_rect) (region_type_t *dest,<br>-- <br>2.7.0<br><br></div>