summaryrefslogtreecommitdiffstats
path: root/drivers/xen
diff options
context:
space:
mode:
authorJeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>2010-08-25 12:19:53 -0700
committerJeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>2010-08-25 12:19:53 -0700
commitfb27cfbcbd2865b0e731c4aae47df71778da805e (patch)
treebc321b782bb4234a7d6a5cb019c31c3843ab2506 /drivers/xen
parent6280f190da4dd083f14f704be6b3314311a7eacb (diff)
downloadop-kernel-dev-fb27cfbcbd2865b0e731c4aae47df71778da805e.zip
op-kernel-dev-fb27cfbcbd2865b0e731c4aae47df71778da805e.tar.gz
xenfs/xenbus: report partial reads/writes correctly
copy_(to|from)_user return the number of uncopied bytes, so a successful return is 0, and any non-zero result indicates some degree of failure. Reported-by: "Jun Zhu (Intern)" <Jun.Zhu@citrix.com> Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Diffstat (limited to 'drivers/xen')
-rw-r--r--drivers/xen/xenfs/xenbus.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/xen/xenfs/xenbus.c b/drivers/xen/xenfs/xenbus.c
index 9d5b519..d2a9058 100644
--- a/drivers/xen/xenfs/xenbus.c
+++ b/drivers/xen/xenfs/xenbus.c
@@ -142,7 +142,7 @@ static ssize_t xenbus_file_read(struct file *filp,
i += sz - ret;
rb->cons += sz - ret;
- if (ret != sz) {
+ if (ret != 0) {
if (i == 0)
i = -EFAULT;
goto out;
@@ -453,7 +453,7 @@ static ssize_t xenbus_file_write(struct file *filp,
ret = copy_from_user(u->u.buffer + u->len, ubuf, len);
- if (ret == len) {
+ if (ret != 0) {
rc = -EFAULT;
goto out;
}
OpenPOWER on IntegriCloud