<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 18, 2015 at 2:32 PM, Derek Foreman <span dir="ltr"><<a href="mailto:derekf@osg.samsung.com" target="_blank">derekf@osg.samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If we pass negative height or width on to pixman we get error messages<br>
in the log.<br>
<br>
Signed-off-by: Derek Foreman <<a href="mailto:derekf@osg.samsung.com">derekf@osg.samsung.com</a>><br>
---<br>
 src/compositor.c | 3 +++<br>
 1 file changed, 3 insertions(+)<br>
<br>
diff --git a/src/compositor.c b/src/compositor.c<br>
index 666bc15..0efb325 100644<br>
--- a/src/compositor.c<br>
+++ b/src/compositor.c<br>
@@ -2574,6 +2574,9 @@ surface_damage(struct wl_client *client,<br>
 {<br>
        struct weston_surface *surface = wl_resource_get_user_data(resource);<br>
<br>
+       if (width < 0 || height < 0)<br>
+               return;<br></blockquote><div><br></div><div>Might as well quit on <= 0 here, since a zero-sized rectangle won't do anything. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
        pixman_region32_union_rect(&surface->pending.damage,<br>
                                   &surface->pending.damage,<br>
                                   x, y, width, height);<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div> </div></div><br></div></div>