]> git.ipfire.org Git - thirdparty/qemu.git/commit
vga: make sure vga register setup for vbe stays intact (CVE-2016-3712).
authorGerd Hoffmann <kraxel@redhat.com>
Tue, 26 Apr 2016 12:48:06 +0000 (14:48 +0200)
committerMichael Roth <mdroth@linux.vnet.ibm.com>
Mon, 9 May 2016 01:56:43 +0000 (20:56 -0500)
commit44b86aa32e4147c727fadd9a0f0bc503a5dedb72
tree5ed5f50d2c700237b7af9107c8bb0eda8dbd5eab
parenta6e5e5dd4bbc022acbd10ebcf415a6a57418d09e
vga: make sure vga register setup for vbe stays intact (CVE-2016-3712).

Call vbe_update_vgaregs() when the guest touches GFX, SEQ or CRT
registers, to make sure the vga registers will always have the
values needed by vbe mode.  This makes sure the sanity checks
applied by vbe_fixup_regs() are effective.

Without this guests can muck with shift_control, can turn on planar
vga modes or text mode emulation while VBE is active, making qemu
take code paths meant for CGA compatibility, but with the very
large display widths and heigts settable using VBE registers.

Which is good for one or another buffer overflow.  Not that
critical as they typically read overflows happening somewhere
in the display code.  So guests can DoS by crashing qemu with a
segfault, but it is probably not possible to break out of the VM.

Fixes: CVE-2016-3712
Reported-by: Zuozhi Fzz <zuozhi.fzz@alibaba-inc.com>
Reported-by: P J P <ppandit@redhat.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
hw/display/vga.c