From: Emil Velikov <emil.veli...@collabora.com> Earlier commit added support for 'front_buffers', erroneously adding a return in vl_dri3_screen_destroy. Effectively leaking a lot of state.
Fixes: 8d7ac0a4e4d ("vl/dri3: implement DRI3 BufferFromPixmap") Cc: Leo Liu <leo....@amd.com> Cc: Christian König <christian.koe...@amd.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- Gents, can you please double-check this. I haven't really looked at the implementation, although the return seems bogus here. Thanks --- src/gallium/auxiliary/vl/vl_winsys_dri3.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri3.c b/src/gallium/auxiliary/vl/vl_winsys_dri3.c index 8e3c4a0e04d..df2c9c0e50c 100644 --- a/src/gallium/auxiliary/vl/vl_winsys_dri3.c +++ b/src/gallium/auxiliary/vl/vl_winsys_dri3.c @@ -712,7 +712,6 @@ vl_dri3_screen_destroy(struct vl_screen *vscreen) if (scrn->front_buffer) { dri3_free_front_buffer(scrn, scrn->front_buffer); scrn->front_buffer = NULL; - return; } for (i = 0; i < BACK_BUFFER_NUM; ++i) { -- 2.18.0 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev