diff options
| author | Jesper Nilsson <jesper.nilsson@axis.com> | 2008-03-04 14:29:23 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-03-04 16:35:16 -0800 |
| commit | 07f2402b4adbcd0e6822ddc27953b63d4504faec (patch) | |
| tree | d1fca3d0493cf7d08298951a4cf26e6c33f3733e /arch/cris/arch-v32/lib | |
| parent | cee47f5a32a1b5a1c8b148e738249946e3fedb95 (diff) | |
| download | linux-07f2402b4adbcd0e6822ddc27953b63d4504faec.tar.gz linux-07f2402b4adbcd0e6822ddc27953b63d4504faec.tar.bz2 linux-07f2402b4adbcd0e6822ddc27953b63d4504faec.zip | |
cris: correct usage of __user for copy to and from user space in lib/usercopy and uaccess.h
Function __copy_user_zeroing in arch/lib/usercopy.c had the wrong parameter
set as __user, and in include/asm-cris/uaccess.h, it was not set at all for
some of the calling functions.
This will cut the number of warnings quite dramatically when using sparse.
While we're here, remove useless CVS log and correct confusing typo.
Signed-off-by: Jesper Nilsson <jesper.nilsson@axis.com>
Cc: Mikael Starvik <mikael.starvik@axis.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/cris/arch-v32/lib')
| -rw-r--r-- | arch/cris/arch-v32/lib/usercopy.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/cris/arch-v32/lib/usercopy.c b/arch/cris/arch-v32/lib/usercopy.c index 04d0cf35a276..0b5b70d5f58a 100644 --- a/arch/cris/arch-v32/lib/usercopy.c +++ b/arch/cris/arch-v32/lib/usercopy.c @@ -161,7 +161,7 @@ __copy_user (void __user *pdst, const void *psrc, unsigned long pn) inaccessible. */ unsigned long -__copy_user_zeroing (void __user *pdst, const void *psrc, unsigned long pn) +__copy_user_zeroing(void *pdst, const void __user *psrc, unsigned long pn) { /* We want the parameters put in special registers. Make sure the compiler is able to make something useful of this. |