aio_setup_vectored_rw(): switch to {compat_,}import_iovec()
authorAl Viro <viro@zeniv.linux.org.uk>
Sat, 21 Mar 2015 23:34:53 +0000 (19:34 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Sun, 12 Apr 2015 02:27:11 +0000 (22:27 -0400)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/aio.c

index 3dba5a70ad97969df5380e4a05250dfff3e8c346..3b8467aeb5eeb5ef6061dc10467f40f0d3335801 100644 (file)
--- a/fs/aio.c
+++ b/fs/aio.c
@@ -1357,23 +1357,14 @@ static int aio_setup_vectored_rw(int rw, char __user *buf, size_t len,
                                 bool compat,
                                 struct iov_iter *iter)
 {
-       ssize_t ret;
-
 #ifdef CONFIG_COMPAT
        if (compat)
-               ret = compat_rw_copy_check_uvector(rw,
+               return compat_import_iovec(rw,
                                (struct compat_iovec __user *)buf,
-                               len, UIO_FASTIOV, *iovec, iovec);
-       else
+                               len, UIO_FASTIOV, iovec, iter);
 #endif
-               ret = rw_copy_check_uvector(rw,
-                               (struct iovec __user *)buf,
-                               len, UIO_FASTIOV, *iovec, iovec);
-       if (ret < 0)
-               return ret;
-
-       iov_iter_init(iter, rw, *iovec, len, ret);
-       return 0;
+       return import_iovec(rw, (struct iovec __user *)buf,
+                               len, UIO_FASTIOV, iovec, iter);
 }
 
 /*
@@ -1418,14 +1409,15 @@ rw_common:
                if (opcode == IOCB_CMD_PREADV || opcode == IOCB_CMD_PWRITEV)
                        ret = aio_setup_vectored_rw(rw, buf, len,
                                                &iovec, compat, &iter);
-               else
+               else {
                        ret = import_single_range(rw, buf, len, iovec, &iter);
+                       iovec = NULL;
+               }
                if (!ret)
                        ret = rw_verify_area(rw, file, &req->ki_pos,
                                             iov_iter_count(&iter));
                if (ret < 0) {
-                       if (iovec != inline_vecs)
-                               kfree(iovec);
+                       kfree(iovec);
                        return ret;
                }
 
@@ -1449,6 +1441,7 @@ rw_common:
 
                if (rw == WRITE)
                        file_end_write(file);
+               kfree(iovec);
                break;
 
        case IOCB_CMD_FDSYNC:
@@ -1470,9 +1463,6 @@ rw_common:
                return -EINVAL;
        }
 
-       if (iovec != inline_vecs)
-               kfree(iovec);
-
        if (ret != -EIOCBQUEUED) {
                /*
                 * There's no easy way to restart the syscall since other AIO's