diff options
| author | Keir Fraser <keir.fraser@citrix.com> | 2009-05-27 11:17:40 +0100 |
|---|---|---|
| committer | Keir Fraser <keir.fraser@citrix.com> | 2009-05-27 11:17:40 +0100 |
| commit | 7f75b4d5a3b1405c8689b6374bbb7ed5af6130b0 (patch) | |
| tree | 0600701ce4ebae00bb1e51b4d9773c09aa7a7782 /xen/arch/x86/hvm/vlapic.c | |
| parent | d655e37925bdb9fb3b50046248f9992ca8828077 (diff) | |
| download | xen-7f75b4d5a3b1405c8689b6374bbb7ed5af6130b0.tar.gz xen-7f75b4d5a3b1405c8689b6374bbb7ed5af6130b0.tar.bz2 xen-7f75b4d5a3b1405c8689b6374bbb7ed5af6130b0.zip | |
x86/hvm: fix off-by-one errors in vcpuid range checks
Signed-off-by: Jan Beulich <jbeulich@novell.com>
Diffstat (limited to 'xen/arch/x86/hvm/vlapic.c')
| -rw-r--r-- | xen/arch/x86/hvm/vlapic.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xen/arch/x86/hvm/vlapic.c b/xen/arch/x86/hvm/vlapic.c index 68e9b27632..6e30a2e2af 100644 --- a/xen/arch/x86/hvm/vlapic.c +++ b/xen/arch/x86/hvm/vlapic.c @@ -913,7 +913,7 @@ static int lapic_load_hidden(struct domain *d, hvm_domain_context_t *h) /* Which vlapic to load? */ vcpuid = hvm_load_instance(h); - if ( vcpuid > MAX_VIRT_CPUS || (v = d->vcpu[vcpuid]) == NULL ) + if ( vcpuid >= MAX_VIRT_CPUS || (v = d->vcpu[vcpuid]) == NULL ) { gdprintk(XENLOG_ERR, "HVM restore: domain has no vlapic %u\n", vcpuid); return -EINVAL; @@ -936,7 +936,7 @@ static int lapic_load_regs(struct domain *d, hvm_domain_context_t *h) /* Which vlapic to load? */ vcpuid = hvm_load_instance(h); - if ( vcpuid > MAX_VIRT_CPUS || (v = d->vcpu[vcpuid]) == NULL ) + if ( vcpuid >= MAX_VIRT_CPUS || (v = d->vcpu[vcpuid]) == NULL ) { gdprintk(XENLOG_ERR, "HVM restore: domain has no vlapic %u\n", vcpuid); return -EINVAL; |