ARM: KVM: prevent NULL pointer dereferences with KVM VCPU ioctl
authorAndre Przywara <andre.przywara@linaro.org>
Wed, 8 May 2013 22:28:06 +0000 (00:28 +0200)
committerGleb Natapov <gleb@redhat.com>
Mon, 3 Jun 2013 07:58:56 +0000 (10:58 +0300)
commite8180dcaa8470ceca21109f143876fdcd9fe050a
treee5c4f6dc1e1b0fc3aa34e76a15ccefbdd86ac983
parented829857b36bc0155d85b661ab227df57ac898f3
ARM: KVM: prevent NULL pointer dereferences with KVM VCPU ioctl

Some ARM KVM VCPU ioctls require the vCPU to be properly initialized
with the KVM_ARM_VCPU_INIT ioctl before being used with further
requests. KVM_RUN checks whether this initialization has been
done, but other ioctls do not.
Namely KVM_GET_REG_LIST will dereference an array with index -1
without initialization and thus leads to a kernel oops.
Fix this by adding checks before executing the ioctl handlers.

 [ Removed superflous comment from static function - Christoffer ]

Changes from v1:
 * moved check into a static function with a meaningful name

Signed-off-by: Andre Przywara <andre.przywara@linaro.org>
Signed-off-by: Christoffer Dall <cdall@cs.columbia.edu>
arch/arm/kvm/arm.c