diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2010-06-04 17:14:47 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-06-04 18:21:44 -0400 |
commit | 7cbe17701a0379c7b05a79a6df4f24e41d2afde8 (patch) | |
tree | 914879cce805da0525f6d0c7c730f4945f1586f5 | |
parent | eaa6e4dd4bf243a357056448e54d7c673cd44acb (diff) |
fs/compat_rw_copy_check_uvector: add missing compat_ptr call
A call to access_ok is missing a compat_ptr conversion. Introduced with
b83733639a494d5f42fa00a2506563fbd2d3015d "compat: factor out
compat_rw_copy_check_uvector from compat_do_readv_writev"
fs/compat.c: In function 'compat_rw_copy_check_uvector':
fs/compat.c:629: warning: passing argument 1 of '__access_ok' makes pointer from integer without a cast
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Reviewed-by: Jeff Moyer <jmoyer@redhat.com>
Cc: <stable@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/compat.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/compat.c b/fs/compat.c index f0b391c50552..6490d2134ff3 100644 --- a/fs/compat.c +++ b/fs/compat.c | |||
@@ -626,7 +626,7 @@ ssize_t compat_rw_copy_check_uvector(int type, | |||
626 | tot_len += len; | 626 | tot_len += len; |
627 | if (tot_len < tmp) /* maths overflow on the compat_ssize_t */ | 627 | if (tot_len < tmp) /* maths overflow on the compat_ssize_t */ |
628 | goto out; | 628 | goto out; |
629 | if (!access_ok(vrfy_dir(type), buf, len)) { | 629 | if (!access_ok(vrfy_dir(type), compat_ptr(buf), len)) { |
630 | ret = -EFAULT; | 630 | ret = -EFAULT; |
631 | goto out; | 631 | goto out; |
632 | } | 632 | } |