KVM: arm64: vgic-its: Fix handling of indirect tables
authorMarc Zyngier <marc.zyngier@arm.com>
Mon, 18 Jul 2016 15:16:26 +0000 (16:16 +0100)
committerMarc Zyngier <marc.zyngier@arm.com>
Mon, 18 Jul 2016 17:15:16 +0000 (18:15 +0100)
The current code will fail on valid indirect tables, and happily
use the ones that are pointing out of the guest RAM. Funny what a
small "!" can do for you...

Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
virt/kvm/arm/vgic/vgic-its.c

index f427fa2f7263f4705dfc8686ce9039ff30e6f248..d6697c4d81ec5c6084ecb0b3dd596160ffac1a70 100644 (file)
@@ -702,9 +702,9 @@ static bool vgic_its_check_device_id(struct kvm *kvm, struct vgic_its *its,
                return false;
 
        /* Each 1st level entry is represented by a 64-bit value. */
-       if (!kvm_read_guest(kvm,
-                           BASER_ADDRESS(r) + index * sizeof(indirect_ptr),
-                           &indirect_ptr, sizeof(indirect_ptr)))
+       if (kvm_read_guest(kvm,
+                          BASER_ADDRESS(r) + index * sizeof(indirect_ptr),
+                          &indirect_ptr, sizeof(indirect_ptr)))
                return false;
 
        /* check the valid bit of the first level entry */