[PULL,3/4] virtio-gpu: fix memory leak in update_cursor_data_virgl

Commit Message

Gerd Hoffmann Dec. 5, 2016, 11:03 a.m. UTC

From: Li Qiang <[e-mail]>

In update_cursor_data_virgl function, if the 'width'/ 'height'
is not equal to current cursor's width/height it will return
without free the 'data' allocated previously. This will lead
a memory leak issue. This patch fix this issue.

Signed-off-by: Li Qiang <[e-mail]>
Message-id: [e-mail]
Reviewed-by: Marc-André Lureau <[e-mail]>
Signed-off-by: Gerd Hoffmann <[e-mail]>
---
 hw/display/virtio-gpu.c | 1 +
 1 file changed, 1 insertion(+)

Patch

diff --git a/hw/display/virtio-gpu.c b/hw/display/virtio-gpu.c
index 60bce94..5f32e1a 100644
--- a/hw/display/virtio-gpu.c
+++ b/hw/display/virtio-gpu.c
@@ -84,6 +84,7 @@  static void update_cursor_data_virgl(VirtIOGPU *g,
 
     if (width != s->current_cursor->width ||
         height != s->current_cursor->height) {
+        free(data);
         return;
     }