orangefs_bufmap_copy_from_iovec(): fix EFAULT handling

Al Viro · Al Viro · commit 890559e34eac · 2017-04-17T14:23:20.000-04:00
short copy here should mean instant EFAULT, not "move to the
next page and hope it fails there, this time with nothing
copied"

Signed-off-by: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
diff --git a/fs/orangefs/orangefs-bufmap.c b/fs/orangefs/orangefs-bufmap.c
@@ -521,13 +521,11 @@ int orangefs_bufmap_copy_from_iovec(struct iov_iter *iter,
 		size_t n = size;
 		if (n > PAGE_SIZE)
 			n = PAGE_SIZE;
-		n = copy_page_from_iter(page, 0, n, iter);
-		if (!n)
+		if (copy_page_from_iter(page, 0, n, iter) != n)
 			return -EFAULT;
 		size -= n;
 	}
 	return 0;
-
 }
 
 /*

Original file line number	Diff line number	Diff line change
`@@ -521,13 +521,11 @@ int orangefs_bufmap_copy_from_iovec(struct iov_iter *iter,`
`521`	`521`	`size_t n = size;`
`522`	`522`	`if (n > PAGE_SIZE)`
`523`	`523`	`n = PAGE_SIZE;`
`524`		`- n = copy_page_from_iter(page, 0, n, iter);`
`525`		`- if (!n)`
	`524`	`+ if (copy_page_from_iter(page, 0, n, iter) != n)`
`526`	`525`	`return -EFAULT;`
`527`	`526`	`size -= n;`
`528`	`527`	`}`
`529`	`528`	`return 0;`
`530`		`-`
`531`	`529`	`}`
`532`	`530`
`533`	`531`	`/*`