aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLi Qiang <liqiang6-s@360.cn>2016-11-01 04:06:58 -0700
committerGerd Hoffmann <kraxel@redhat.com>2016-12-05 09:37:52 +0100
commit2d1cd6c7a91a4beb99a0c3a21be529222a708545 (patch)
tree7a5f90e17d4466471002a25fa97ae430f4f0edb8
parent42a8dadc74f8982fc269e54e3c5627b54d9f83d8 (diff)
downloadqemu-2d1cd6c7a91a4beb99a0c3a21be529222a708545.zip
qemu-2d1cd6c7a91a4beb99a0c3a21be529222a708545.tar.gz
qemu-2d1cd6c7a91a4beb99a0c3a21be529222a708545.tar.bz2
virtio-gpu: fix memory leak in update_cursor_data_virgl
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 <liqiang6-s@360.cn> Message-id: 58187760.41d71c0a.cca75.4cb9@mx.google.com Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
-rw-r--r--hw/display/virtio-gpu.c1
1 files changed, 1 insertions, 0 deletions
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;
}