diff options
| author | Dan Carpenter | 2017-05-18 09:38:53 +0200 |
|---|---|---|
| committer | Radim Krčmář | 2017-05-18 14:53:54 +0200 |
| commit | d3e7dec054174fdddae33eaa0032a82c3f42181d (patch) | |
| tree | 3af3c47ff972a64d9745d6801856005045060a43 /arch/x86/kvm/svm.c | |
| parent | Merge tag 'kvm-arm-for-v4.12-rc2' of git://git.kernel.org/pub/scm/linux/kerne... (diff) | |
| download | kernel-qcow2-linux-d3e7dec054174fdddae33eaa0032a82c3f42181d.tar.gz kernel-qcow2-linux-d3e7dec054174fdddae33eaa0032a82c3f42181d.tar.xz kernel-qcow2-linux-d3e7dec054174fdddae33eaa0032a82c3f42181d.zip | |
KVM: Silence underflow warning in avic_get_physical_id_entry()
Smatch complains that we check cap the upper bound of "index" but don't
check for negatives. It's a false positive because "index" is never
negative. But it's also simple enough to make it unsigned which makes
the code easier to audit.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>
Diffstat (limited to 'arch/x86/kvm/svm.c')
| -rw-r--r-- | arch/x86/kvm/svm.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c index c27ac6923a18..183ddb235fb4 100644 --- a/arch/x86/kvm/svm.c +++ b/arch/x86/kvm/svm.c @@ -1272,7 +1272,8 @@ static void init_vmcb(struct vcpu_svm *svm) } -static u64 *avic_get_physical_id_entry(struct kvm_vcpu *vcpu, int index) +static u64 *avic_get_physical_id_entry(struct kvm_vcpu *vcpu, + unsigned int index) { u64 *avic_physical_id_table; struct kvm_arch *vm_data = &vcpu->kvm->arch; |