From: Li Qiang Date: Thu, 29 Dec 2016 08:11:26 +0000 (-0500) Subject: virtio-gpu-3d: fix memory leak in resource attach backing X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=33243031dad02d161225ba99d782616da133f689;p=qemu.git virtio-gpu-3d: fix memory leak in resource attach backing If the virgl_renderer_resource_attach_iov function fails the 'res_iovs' will be leaked. Add check of the return value to free the 'res_iovs' when failing. Signed-off-by: Li Qiang Reviewed-by: Marc-André Lureau Message-id: 1482999086-59795-1-git-send-email-liq3ea@gmail.com Signed-off-by: Gerd Hoffmann --- diff --git a/hw/display/virtio-gpu-3d.c b/hw/display/virtio-gpu-3d.c index e29f099bd5..b13ced38fa 100644 --- a/hw/display/virtio-gpu-3d.c +++ b/hw/display/virtio-gpu-3d.c @@ -291,8 +291,11 @@ static void virgl_resource_attach_backing(VirtIOGPU *g, return; } - virgl_renderer_resource_attach_iov(att_rb.resource_id, - res_iovs, att_rb.nr_entries); + ret = virgl_renderer_resource_attach_iov(att_rb.resource_id, + res_iovs, att_rb.nr_entries); + + if (ret != 0) + virtio_gpu_cleanup_mapping_iov(res_iovs, att_rb.nr_entries); } static void virgl_resource_detach_backing(VirtIOGPU *g,