]> xenbits.xensource.com Git - qemu-xen-traditional.git/commit
cirrus: add blit_is_unsafe call to cirrus_bitblt_cputovideo xen-4.7.2
authorGerd Hoffmann <kraxel@redhat.com>
Wed, 22 Feb 2017 16:25:19 +0000 (16:25 +0000)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Wed, 22 Feb 2017 16:28:13 +0000 (16:28 +0000)
commit0d5d265f1b81019bff48d2b666032041253f89e9
tree4145790333c9fa14d0aaa81e6b7616d6db7b65ab
parenta20cf3a6edd051f2f2eb734a7bcda6c6246740ff
cirrus: add blit_is_unsafe call to cirrus_bitblt_cputovideo

CIRRUS_BLTMODE_MEMSYSSRC blits do NOT check blit destination
and blit width, at all.  Oops.  Fix it.

Security impact: high.

The missing blit destination check allows to write to host memory.
Basically same as CVE-2014-8106 for the other blit variants.

The missing blit width check allows to overflow cirrus_bltbuf,
with the attractive target cirrus_srcptr (current cirrus_bltbuf write
position) being located right after cirrus_bltbuf in CirrusVGAState.

Due to cirrus emulation writing cirrus_bltbuf bytewise the attacker
hasn't full control over cirrus_srcptr though, only one byte can be
changed.  Once the first byte has been modified further writes land
elsewhere.

[ This is CVE-2017-2620 / XSA-209  - Ian Jackson ]

Fixed compilation by removing extra parameter to blit_is_unsafe. -iwj

Reported-by: Gerd Hoffmann <ghoffman@redhat.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
(cherry picked from commit 83256b312acfddda34d96e9bba30e93387481091)
(cherry picked from commit b77d58b59ff0688e4aeb933450d07716ca0fb1f2)
hw/cirrus_vga.c