summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlon Levy <alevy@redhat.com>2012-10-07 17:03:35 +0200
committerDoug Goldstein <cardoe@cardoe.com>2012-11-21 15:22:55 -0600
commit3278dc83135415e75bd7f77436e1953326d13e2c (patch)
treec18ed31a02081162c392384d5dc0edc80bda98f7
parentuhci: Raise interrupt when requested even for non active tds (diff)
downloadqemu-kvm-3278dc83135415e75bd7f77436e1953326d13e2c.tar.gz
qemu-kvm-3278dc83135415e75bd7f77436e1953326d13e2c.tar.bz2
qemu-kvm-3278dc83135415e75bd7f77436e1953326d13e2c.zip
hw/qxl: qxl_dirty_surfaces: use uintptr_t
As suggested by Paolo Bonzini, to avoid possible integer overflow issues. Signed-off-by: Alon Levy <alevy@redhat.com> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> (cherry picked from commit c5825ac6c861bfe1a4adfa27517931b56079e298)
-rw-r--r--hw/qxl.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/hw/qxl.c b/hw/qxl.c
index 3da339993..b971ac703 100644
--- a/hw/qxl.c
+++ b/hw/qxl.c
@@ -1603,7 +1603,7 @@ static void qxl_hw_text_update(void *opaque, console_ch_t *chardata)
static void qxl_dirty_surfaces(PCIQXLDevice *qxl)
{
- intptr_t vram_start;
+ uintptr_t vram_start;
int i;
if (qxl->mode != QXL_MODE_NATIVE && qxl->mode != QXL_MODE_COMPAT) {
@@ -1614,7 +1614,7 @@ static void qxl_dirty_surfaces(PCIQXLDevice *qxl)
qxl_set_dirty(&qxl->vga.vram, qxl->shadow_rom.draw_area_offset,
qxl->shadow_rom.surface0_area_size);
- vram_start = (intptr_t)memory_region_get_ram_ptr(&qxl->vram_bar);
+ vram_start = (uintptr_t)memory_region_get_ram_ptr(&qxl->vram_bar);
/* dirty the off-screen surfaces */
for (i = 0; i < NUM_SURFACES; i++) {