diff options
| author | BALATON Zoltan <balaton@eik.bme.hu> | 2020-06-21 13:12:38 +0200 |
|---|---|---|
| committer | Gerd Hoffmann <kraxel@redhat.com> | 2020-06-30 22:54:24 +0200 |
| commit | 41977c65c04a85f177603778cb60e06847efd3af (patch) | |
| tree | e8b499dc6b3c3bd8f9b34232615c9c8daacf657f /hw/display/ati.c | |
| parent | d634c883ca07c28da2cb84c019659694f05d8b3a (diff) | |
| download | qemu-41977c65c04a85f177603778cb60e06847efd3af.zip qemu-41977c65c04a85f177603778cb60e06847efd3af.tar.gz qemu-41977c65c04a85f177603778cb60e06847efd3af.tar.bz2 | |
ati-vga: Do not assert on error
Do not abort on unsupported value just print log and continue. While
display will likely be broken this prevents malicious guest to crash
QEMU causing denial of service.
Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Message-id: 0c13dab5d8e3b7e7479c3edbf53aeac8c09de6de.1592737958.git.balaton@eik.bme.hu
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Diffstat (limited to 'hw/display/ati.c')
| -rw-r--r-- | hw/display/ati.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/display/ati.c b/hw/display/ati.c index 245130d..95fc443 100644 --- a/hw/display/ati.c +++ b/hw/display/ati.c @@ -86,8 +86,8 @@ static void ati_vga_switch_mode(ATIVGAState *s) break; default: qemu_log_mask(LOG_UNIMP, "Unsupported bpp value\n"); + return; } - assert(bpp != 0); DPRINTF("Switching to %dx%d %d %d @ %x\n", h, v, stride, bpp, offs); vbe_ioport_write_index(&s->vga, 0, VBE_DISPI_INDEX_ENABLE); vbe_ioport_write_data(&s->vga, 0, VBE_DISPI_DISABLED); |