diff options
| author | Russell King <rmk@dyn-67.arm.linux.org.uk> | 2008-10-27 11:24:09 +0000 |
|---|---|---|
| committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2008-11-27 12:37:59 +0000 |
| commit | 59f0cb0fddc14ffc6676ae62e911f8115ebc8ccf (patch) | |
| tree | 1e5fc347287c02e83dce967180c96906f6ed7455 /arch/arm/include/asm/uaccess.h | |
| parent | ed313489badef16d700f5a3be50e8fd8f8294bc8 (diff) | |
| download | lwn-59f0cb0fddc14ffc6676ae62e911f8115ebc8ccf.tar.gz lwn-59f0cb0fddc14ffc6676ae62e911f8115ebc8ccf.zip | |
[ARM] remove memzero()
As suggested by Andrew Morton, remove memzero() - it's not supported
on other architectures so use of it is a potential build breaking bug.
Since the compiler optimizes memset(x,0,n) to __memzero() perfectly
well, we don't miss out on the underlying benefits of memzero().
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/include/asm/uaccess.h')
| -rw-r--r-- | arch/arm/include/asm/uaccess.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h index e98ec60b3400..cbb7a37e2978 100644 --- a/arch/arm/include/asm/uaccess.h +++ b/arch/arm/include/asm/uaccess.h @@ -400,7 +400,7 @@ static inline unsigned long __must_check copy_from_user(void *to, const void __u if (access_ok(VERIFY_READ, from, n)) n = __copy_from_user(to, from, n); else /* security hole - plug it */ - memzero(to, n); + memset(to, 0, n); return n; } |