diff options
author | Keir Fraser <keir.fraser@citrix.com> | 2007-12-27 23:01:50 +0000 |
---|---|---|
committer | Keir Fraser <keir.fraser@citrix.com> | 2007-12-27 23:01:50 +0000 |
commit | f53513b012968240ba490efe873982d213a85d46 (patch) | |
tree | 1647617c7d400cdf17d827e0bb7595f4ee6625bd | |
parent | f8dc2a2051c619d20c725701b0274df40a9f3f45 (diff) | |
download | xen-f53513b012968240ba490efe873982d213a85d46.tar.gz xen-f53513b012968240ba490efe873982d213a85d46.tar.bz2 xen-f53513b012968240ba490efe873982d213a85d46.zip |
hvm: copy_{to,from}_user_hvm shoudl return appropriate error codes.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
-rw-r--r-- | xen/arch/x86/hvm/platform.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/xen/arch/x86/hvm/platform.c b/xen/arch/x86/hvm/platform.c index 273a2ed08a..e6ea6d9b99 100644 --- a/xen/arch/x86/hvm/platform.c +++ b/xen/arch/x86/hvm/platform.c @@ -1310,26 +1310,30 @@ DEFINE_PER_CPU(int, guest_handles_in_xen_space); deal with this. */ unsigned long copy_to_user_hvm(void *to, const void *from, unsigned len) { + int rc; + if ( this_cpu(guest_handles_in_xen_space) ) { memcpy(to, from, len); return 0; } - return hvm_copy_to_guest_virt_nofault( - (unsigned long)to, (void *)from, len); + rc = hvm_copy_to_guest_virt_nofault((unsigned long)to, (void *)from, len); + return rc ? len : 0; /* fake a copy_to_user() return code */ } unsigned long copy_from_user_hvm(void *to, const void *from, unsigned len) { + int rc; + if ( this_cpu(guest_handles_in_xen_space) ) { memcpy(to, from, len); return 0; } - return hvm_copy_from_guest_virt_nofault( - to, (unsigned long)from, len); + rc = hvm_copy_from_guest_virt_nofault(to, (unsigned long)from, len); + return rc ? len : 0; /* fake a copy_from_user() return code */ } /* |