red_get_image_data_flat: allocate mem after sanity check

This patch prevents possible memory leak.

Found by coverity.

Signed-off-by: Uri Lublin <uril@redhat.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
This commit is contained in:
Uri Lublin 2016-12-06 18:06:28 +02:00 committed by Frediano Ziglio
parent dad108edb1
commit 547f9f4387

View File

@ -371,13 +371,16 @@ static SpiceChunks *red_get_image_data_flat(RedMemSlotInfo *slots, int group_id,
{
SpiceChunks *data;
int error;
unsigned long bitmap_virt;
data = spice_chunks_new(1);
data->data_size = size;
data->chunk[0].data = (void*)memslot_get_virt(slots, addr, size, group_id, &error);
bitmap_virt = memslot_get_virt(slots, addr, size, group_id, &error);
if (error) {
return 0;
}
data = spice_chunks_new(1);
data->data_size = size;
data->chunk[0].data = (void*)bitmap_virt;
data->chunk[0].len = size;
return data;
}