diff options
| author | Keir Fraser <keir.fraser@citrix.com> | 2009-04-08 14:43:12 +0100 |
|---|---|---|
| committer | Keir Fraser <keir.fraser@citrix.com> | 2009-04-08 14:43:12 +0100 |
| commit | 75c3d290f10b1b82bdec72ea803f76d22c4d4e8e (patch) | |
| tree | 23b0bb70ed43d3d30d88709b030d6c3a7e547584 | |
| parent | 7e2b6626e8a6692b1024c52a09ad6ff955e16b05 (diff) | |
| download | xen-75c3d290f10b1b82bdec72ea803f76d22c4d4e8e.tar.gz xen-75c3d290f10b1b82bdec72ea803f76d22c4d4e8e.tar.bz2 xen-75c3d290f10b1b82bdec72ea803f76d22c4d4e8e.zip | |
x86 hvm: Remove the extra IS_PRIV_FOR() checks I added to do_hvm_op().
Actually this is already done by rcu_lock_target_domain_by_id(). Oops.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
| -rw-r--r-- | xen/arch/x86/hvm/hvm.c | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c index b8b1579041..82b99ab761 100644 --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -2439,10 +2439,6 @@ long do_hvm_op(unsigned long op, XEN_GUEST_HANDLE(void) arg) if ( rc != 0 ) return rc; - rc = -EPERM; - if ( (curr_d != d) && !IS_PRIV_FOR(curr_d, d) ) - goto param_fail; - rc = -EINVAL; if ( !is_hvm_domain(d) ) goto param_fail; @@ -2621,10 +2617,6 @@ long do_hvm_op(unsigned long op, XEN_GUEST_HANDLE(void) arg) if ( rc != 0 ) return rc; - rc = -EPERM; - if ( !IS_PRIV_FOR(curr_d, d) ) - goto param_fail2; - rc = -EINVAL; if ( !is_hvm_domain(d) ) goto param_fail2; @@ -2663,10 +2655,6 @@ long do_hvm_op(unsigned long op, XEN_GUEST_HANDLE(void) arg) if ( rc != 0 ) return rc; - rc = -EPERM; - if ( !IS_PRIV_FOR(curr_d, d) ) - goto param_fail3; - rc = -EINVAL; if ( !is_hvm_domain(d) ) goto param_fail3; @@ -2723,10 +2711,6 @@ long do_hvm_op(unsigned long op, XEN_GUEST_HANDLE(void) arg) if ( rc != 0 ) return rc; - rc = -EPERM; - if ( !IS_PRIV_FOR(curr_d, d) ) - goto param_fail4; - rc = -EINVAL; if ( !is_hvm_domain(d) ) goto param_fail4; |