diff options
author | Wei Yongjun <yjwei@cn.fujitsu.com> | 2010-03-11 19:45:39 -0500 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2010-05-17 05:15:30 -0400 |
commit | 5c0d0920a270b9f2aa20c1cecb162703da32e766 (patch) | |
tree | 7494b38a359df05c50fe98d8eed4cd50d8d16866 /arch/ia64/kvm | |
parent | ec68798c8fd0f01cdbd3f3e1a970e76a644cf08e (diff) |
KVM: ia64: fix the error code of ioctl KVM_IA64_VCPU_GET_STACK failure
The ioctl KVM_IA64_VCPU_GET_STACK does not set the error code if
copy_to_user() fail, and 0 will be return, we should use -EFAULT
instead of 0 in this case, so this patch fixed it.
Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Diffstat (limited to 'arch/ia64/kvm')
-rw-r--r-- | arch/ia64/kvm/kvm-ia64.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/arch/ia64/kvm/kvm-ia64.c b/arch/ia64/kvm/kvm-ia64.c index 7f3c0a2e60cd..38d51302f5b3 100644 --- a/arch/ia64/kvm/kvm-ia64.c +++ b/arch/ia64/kvm/kvm-ia64.c | |||
@@ -1535,8 +1535,10 @@ long kvm_arch_vcpu_ioctl(struct file *filp, | |||
1535 | goto out; | 1535 | goto out; |
1536 | 1536 | ||
1537 | if (copy_to_user(user_stack, stack, | 1537 | if (copy_to_user(user_stack, stack, |
1538 | sizeof(struct kvm_ia64_vcpu_stack))) | 1538 | sizeof(struct kvm_ia64_vcpu_stack))) { |
1539 | r = -EFAULT; | ||
1539 | goto out; | 1540 | goto out; |
1541 | } | ||
1540 | 1542 | ||
1541 | break; | 1543 | break; |
1542 | } | 1544 | } |