From: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Date: Wed, 25 Jun 2014 12:25:58 +0000 (+0200)
Subject: arch: x86: kvm: x86.c: Cleaning up variable is set more than once
X-Git-Tag: v3.17-rc1~141^2~55
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=9f6226a762c7ae02f6a23a3d4fc552dafa57ea23;p=users%2Fwilly%2Fxarray.git

arch: x86: kvm: x86.c: Cleaning up variable is set more than once

A struct member variable is set to the same value more than once

This was found using a static code analysis program called cppcheck.

Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---

diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index 874607ae0583..5a8691b0ed76 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -4730,7 +4730,6 @@ static void emulator_set_segment(struct x86_emulate_ctxt *ctxt, u16 selector,
 	if (desc->g)
 		var.limit = (var.limit << 12) | 0xfff;
 	var.type = desc->type;
-	var.present = desc->p;
 	var.dpl = desc->dpl;
 	var.db = desc->d;
 	var.s = desc->s;