From: Ville Syrjälä <ville.syrj...@linux.intel.com>

The compiler seems to think src/mask xoff/yoff can be used
uninitialized. Zero them to make sure.

Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 src/sna/fb/fbpict.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/sna/fb/fbpict.c b/src/sna/fb/fbpict.c
index abe223f31c88..8e65deb26fe9 100644
--- a/src/sna/fb/fbpict.c
+++ b/src/sna/fb/fbpict.c
@@ -313,8 +313,10 @@ image_from_pict_internal(PicturePtr pict, Bool has_clip, 
int *xoff, int *yoff,
 {
        pixman_image_t *image = NULL;
 
-       if (!pict)
+       if (!pict) {
+               *xoff = *yoff = 0;
                return NULL;
+       }
 
        if (pict->pDrawable) {
                image = create_bits_picture(pict, has_clip, xoff, yoff);
@@ -336,6 +338,8 @@ image_from_pict_internal(PicturePtr pict, Bool has_clip, 
int *xoff, int *yoff,
                                image = create_conical_gradient_image(gradient);
                }
                *xoff = *yoff = 0;
+       } else {
+               *xoff = *yoff = 0;
        }
 
        if (image)
-- 
2.21.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to