Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 91e1f7b0 authored by Owen Hofmann's avatar Owen Hofmann Committed by Greg Kroah-Hartman
Browse files

kvm: x86: Check memopp before dereference (CVE-2016-8630) 



commit d9092f52d7e61dd1557f2db2400ddb430e85937e upstream.

Commit 41061cdb ("KVM: emulate: do not initialize memopp") removes a
check for non-NULL under incorrect assumptions. An undefined instruction
with a ModR/M byte with Mod=0 and R/M-5 (e.g. 0xc7 0x15) will attempt
to dereference a null pointer here.

Fixes: 41061cdb
Message-Id: <1477592752-126650-2-git-send-email-osh@google.com>
Signed-off-by: default avatarOwen Hofmann <osh@google.com>
Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 62fa839b

arch/x86/kvm/emulate.c

+1 −1
Original line number Diff line number Diff line
@@ -5033,7 +5033,7 @@ done_prefixes: 
	/* Decode and fetch the destination operand: register or memory. */

	rc = decode_operand(ctxt, &ctxt->dst, (ctxt->d >> DstShift) & OpMask);



	if (ctxt->rip_relative)

	if (ctxt->rip_relative && likely(ctxt->memopp))

		ctxt->memopp->addr.mem.ea = address_mask(ctxt,

					ctxt->memopp->addr.mem.ea + ctxt->_eip);