KVM: ia64: fix the error code of ioctl KVM_IA64_VCPU_GET_STACK failure
authorWei Yongjun <yjwei@cn.fujitsu.com>
Fri, 12 Mar 2010 00:45:39 +0000 (08:45 +0800)
committerAvi Kivity <avi@redhat.com>
Mon, 17 May 2010 09:15:30 +0000 (12:15 +0300)
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>
arch/ia64/kvm/kvm-ia64.c

index 7f3c0a2..38d5130 100644 (file)
@@ -1535,8 +1535,10 @@ long kvm_arch_vcpu_ioctl(struct file *filp,
                        goto out;
 
                if (copy_to_user(user_stack, stack,
-                                sizeof(struct kvm_ia64_vcpu_stack)))
+                                sizeof(struct kvm_ia64_vcpu_stack))) {
+                       r = -EFAULT;
                        goto out;
+               }
 
                break;
        }