diff options
author | Frediano Ziglio <fziglio@redhat.com> | 2015-09-17 15:01:05 +0100 |
---|---|---|
committer | Frediano Ziglio <fziglio@redhat.com> | 2015-10-06 11:11:11 +0100 |
commit | 6e3547f8b192f5b01d478ca222bf46736f5c700c (patch) | |
tree | dff07244d9c20f350d610b382a19c6c77826080a | |
parent | b3be589ab3b32af3e470a9dec19a61fb086f72fc (diff) |
Prevent leak if size from red_get_data_chunks don't match in red_get_image
Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
-rw-r--r-- | server/red_parse_qxl.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/server/red_parse_qxl.c b/server/red_parse_qxl.c index 3ce4431..dd52602 100644 --- a/server/red_parse_qxl.c +++ b/server/red_parse_qxl.c @@ -526,6 +526,7 @@ static SpiceImage *red_get_image(RedMemSlotInfo *slots, int group_id, &chunks, qxl->bitmap.data); spice_assert(size == bitmap_size); if (size != bitmap_size) { + red_put_data_chunks(&chunks); goto error; } red->u.bitmap.data = red_get_image_data_chunked(slots, group_id, @@ -546,6 +547,7 @@ static SpiceImage *red_get_image(RedMemSlotInfo *slots, int group_id, &chunks, (QXLDataChunk *)qxl->quic.data); spice_assert(size == red->u.quic.data_size); if (size != red->u.quic.data_size) { + red_put_data_chunks(&chunks); goto error; } red->u.quic.data = red_get_image_data_chunked(slots, group_id, |