summaryrefslogtreecommitdiffstats
path: root/arch/x86/kvm/x86.c
diff options
context:
space:
mode:
authorPaolo Bonzini2014-11-24 14:35:24 +0100
committerPaolo Bonzini2014-11-24 16:53:50 +0100
commit2b4a273b4266d9928d5b20154fea96f09ea5cb9a (patch)
tree521f290d8ade86e3524565513314abc99b6d3540 /arch/x86/kvm/x86.c
parentKVM: x86: move device assignment out of kvm_host.h (diff)
downloadkernel-qcow2-linux-2b4a273b4266d9928d5b20154fea96f09ea5cb9a.tar.gz
kernel-qcow2-linux-2b4a273b4266d9928d5b20154fea96f09ea5cb9a.tar.xz
kernel-qcow2-linux-2b4a273b4266d9928d5b20154fea96f09ea5cb9a.zip
kvm: x86: avoid warning about potential shift wrapping bug
cs.base is declared as a __u64 variable and vector is a u32 so this causes a static checker warning. The user indeed can set "sipi_vector" to any u32 value in kvm_vcpu_ioctl_x86_set_vcpu_events(), but the value should really have 8-bit precision only. Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'arch/x86/kvm/x86.c')
-rw-r--r--arch/x86/kvm/x86.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index c42bca47f7f5..08b5657e57ed 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -7033,7 +7033,7 @@ void kvm_vcpu_reset(struct kvm_vcpu *vcpu)
kvm_x86_ops->vcpu_reset(vcpu);
}
-void kvm_vcpu_deliver_sipi_vector(struct kvm_vcpu *vcpu, unsigned int vector)
+void kvm_vcpu_deliver_sipi_vector(struct kvm_vcpu *vcpu, u8 vector)
{
struct kvm_segment cs;