From: Adam Jackson Date: Tue, 26 Jan 2021 19:17:52 +0000 (-0500) Subject: vl: Silence a warning at -Og X-Git-Tag: upstream/21.2.3~7666 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=109f9f39106de13880381aae020add77a54823f7;p=platform%2Fupstream%2Fmesa.git vl: Silence a warning at -Og ../src/gallium/auxiliary/vl/vl_winsys_dri.c: In function ‘vl_dri2_screen_texture_from_drawable’: ../src/gallium/auxiliary/vl/vl_winsys_dri.c:239:34: warning: ‘back_left’ may be used uninitialized in this function [-Wmaybe-uninitialized] 239 | dri2_handle.stride = back_left->pitch; The server would be broken if it failed to inform us of the back left buffer, but let's leave nothing to chance. Reviewed-by: Eric Anholt Part-of: --- diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri.c b/src/gallium/auxiliary/vl/vl_winsys_dri.c index 1f160ec..091395d 100644 --- a/src/gallium/auxiliary/vl/vl_winsys_dri.c +++ b/src/gallium/auxiliary/vl/vl_winsys_dri.c @@ -186,7 +186,7 @@ vl_dri2_screen_texture_from_drawable(struct vl_screen *vscreen, void *drawable) struct pipe_resource templ, *tex; xcb_dri2_get_buffers_reply_t *reply; - xcb_dri2_dri2_buffer_t *buffers, *back_left; + xcb_dri2_dri2_buffer_t *buffers, *back_left = NULL; unsigned depth = ((xcb_screen_t *)(vscreen->xcb_screen))->root_depth; unsigned i; @@ -217,7 +217,7 @@ vl_dri2_screen_texture_from_drawable(struct vl_screen *vscreen, void *drawable) } } - if (i == reply->count) { + if (i == reply->count || !back_left) { free(reply); return NULL; }