qemu: Do not access stale data in virDomainBlockStats
authorJiri Denemark <jdenemar@redhat.com>
Thu, 19 Dec 2013 21:10:04 +0000 (22:10 +0100)
committerEric Blake <eblake@redhat.com>
Wed, 15 Jan 2014 18:30:38 +0000 (11:30 -0700)
commit561b03f9165a860139edd3c03bb3e35a2c2f85ca
tree330e29a41de428e3684bf57dd748015a58c4e0cc
parent939edc41821880ef3c4272a52c88f88d8eea083b
qemu: Do not access stale data in virDomainBlockStats

CVE-2013-6458
https://bugzilla.redhat.com/show_bug.cgi?id=1043069

When virDomainDetachDeviceFlags is called concurrently to
virDomainBlockStats: libvirtd may crash because qemuDomainBlockStats
finds a disk in vm->def before getting a job on a domain and uses the
disk pointer after getting the job. However, the domain in unlocked
while waiting on a job condition and thus data behind the disk pointer
may disappear. This happens when thread 1 runs
virDomainDetachDeviceFlags and enters monitor to actually remove the
disk. Then another thread starts running virDomainBlockStats, finds the
disk in vm->def, and while it's waiting on the job condition (owned by
the first thread), the first thread finishes the disk removal. When the
second thread gets the job, the memory pointed to be the disk pointer is
already gone.

That said, every API that is going to begin a job should do that before
fetching data from vm->def.

(cherry picked from commit db86da5ca2109e4006c286a09b6c75bfe10676ad)

Conflicts:
src/qemu/qemu_driver.c - context: no ACLs
src/qemu/qemu_driver.c