]> git.baikalelectronics.ru Git - kernel.git/commit
KVM: MTRR: remove MSR 0x2f8
authorAndy Honig <ahonig@google.com>
Tue, 17 May 2016 15:41:47 +0000 (17:41 +0200)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 18 May 2016 16:04:32 +0000 (18:04 +0200)
commit8bc3933f5e91cdd08474366b48e725f8b2b9c8f5
treec744f6a7b442e18f1afc5207fbeee7216e5799b8
parent1cb7ea69681c5f7629cc2d2433999f71bfbd5d3e
KVM: MTRR: remove MSR 0x2f8

MSR 0x2f8 accessed the 124th Variable Range MTRR ever since MTRR support
was introduced by 3a4cece18ae2 ("KVM: MTRR support").

0x2f8 became harmful when 14f200556790 ("KVM: MTRR: exactly define the
size of variable MTRRs") shrinked the array of VR MTRRs from 256 to 8,
which made access to index 124 out of bounds.  The surrounding code only
WARNs in this situation, thus the guest gained a limited read/write
access to struct kvm_arch_vcpu.

0x2f8 is not a valid VR MTRR MSR, because KVM has/advertises only 16 VR
MTRR MSRs, 0x200-0x20f.  Every VR MTRR is set up using two MSRs, 0x2f8
was treated as a PHYSBASE and 0x2f9 would be its PHYSMASK, but 0x2f9 was
not implemented in KVM, therefore 0x2f8 could never do anything useful
and getting rid of it is safe.

This fixes CVE-2016-3713.

Fixes: 14f200556790 ("KVM: MTRR: exactly define the size of variable MTRRs")
Cc: stable@vger.kernel.org
Reported-by: David Matlack <dmatlack@google.com>
Signed-off-by: Andy Honig <ahonig@google.com>
Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/mtrr.c