diff options
author | Jan Kiszka <jan.kiszka@siemens.com> | 2011-06-11 06:23:55 -0400 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2011-07-12 06:16:18 -0400 |
commit | 9f3191aec595ef9f3c80bc96664fd7aef57ef5be (patch) | |
tree | 53be5ad0a1b677fa9a43bd8b2c22e1c5bad76a54 /virt | |
parent | 1dda606c5f94b14a8f36c220d1d8844bab68a720 (diff) |
KVM: Fix off-by-one in overflow check of KVM_ASSIGN_SET_MSIX_NR
KVM_MAX_MSIX_PER_DEV implies that up to that many MSI-X entries can be
requested. But the kernel so far rejected already the upper limit.
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
Diffstat (limited to 'virt')
-rw-r--r-- | virt/kvm/assigned-dev.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/virt/kvm/assigned-dev.c b/virt/kvm/assigned-dev.c index 6cc4b97ec458..4e9eaeb518c7 100644 --- a/virt/kvm/assigned-dev.c +++ b/virt/kvm/assigned-dev.c | |||
@@ -617,7 +617,7 @@ static int kvm_vm_ioctl_set_msix_nr(struct kvm *kvm, | |||
617 | if (adev->entries_nr == 0) { | 617 | if (adev->entries_nr == 0) { |
618 | adev->entries_nr = entry_nr->entry_nr; | 618 | adev->entries_nr = entry_nr->entry_nr; |
619 | if (adev->entries_nr == 0 || | 619 | if (adev->entries_nr == 0 || |
620 | adev->entries_nr >= KVM_MAX_MSIX_PER_DEV) { | 620 | adev->entries_nr > KVM_MAX_MSIX_PER_DEV) { |
621 | r = -EINVAL; | 621 | r = -EINVAL; |
622 | goto msix_nr_out; | 622 | goto msix_nr_out; |
623 | } | 623 | } |