| Message ID | 00bb6e21-484b-47d6-82fa-85c787d71a86@moroto.mountain (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | btrfs: return negative -EFAULT instead of positive | expand |
On Tue, Nov 28, 2023 at 05:40:33PM +0300, Dan Carpenter wrote: > There is a typo here and the '-' character was accidentally left off. > > Fixes: 2dc8b96809b2 ("btrfs: allow extent buffer helpers to skip cross-page handling") > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Folded to the patch, thanks.
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c index f9b47f5d7e3d..62963bc6f61b 100644 --- a/fs/btrfs/extent_io.c +++ b/fs/btrfs/extent_io.c @@ -4085,7 +4085,7 @@ int read_extent_buffer_to_user_nofault(const struct extent_buffer *eb, if (eb->addr) { if (copy_to_user_nofault(dstv, eb->addr + start, len)) - ret = EFAULT; + ret = -EFAULT; return ret; }
There is a typo here and the '-' character was accidentally left off. Fixes: 2dc8b96809b2 ("btrfs: allow extent buffer helpers to skip cross-page handling") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> --- fs/btrfs/extent_io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)