x86: sys32_kill and sys32_mprotect are pointless
authorAl Viro <viro@zeniv.linux.org.uk>
Sun, 24 Feb 2013 19:00:48 +0000 (14:00 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 4 Mar 2013 04:00:33 +0000 (23:00 -0500)
their argument types are identical to those of sys_kill and sys_mprotect
resp., so we are not doing any kind of argument validation, etc. in those -
they turn into unconditional branches to corresponding syscalls.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
arch/x86/ia32/sys_ia32.c
arch/x86/include/asm/sys_ia32.h
arch/x86/syscalls/syscall_32.tbl

index c0df976b0b7151e2b9b0a69210fe061526de209d..4e4907c67d92d0940270de5c7c55c731681afaf5 100644 (file)
@@ -166,12 +166,6 @@ asmlinkage long sys32_mmap(struct mmap_arg_struct32 __user *arg)
                               a.offset>>PAGE_SHIFT);
 }
 
-asmlinkage long sys32_mprotect(unsigned long start, size_t len,
-                              unsigned long prot)
-{
-       return sys_mprotect(start, len, prot);
-}
-
 asmlinkage long sys32_waitpid(compat_pid_t pid, unsigned int __user *stat_addr,
                              int options)
 {
@@ -198,11 +192,6 @@ asmlinkage long sys32_pwrite(unsigned int fd, const char __user *ubuf,
  * Some system calls that need sign extended arguments. This could be
  * done by a generic wrapper.
  */
-long sys32_kill(int pid, int sig)
-{
-       return sys_kill(pid, sig);
-}
-
 long sys32_fadvise64_64(int fd, __u32 offset_low, __u32 offset_high,
                        __u32 len_low, __u32 len_high, int advice)
 {
index df8ad3b3920a1108db55aebb9107cccca15cc37e..5e17241766265e3a137d6fbb3fe21093c3dc9ef1 100644 (file)
@@ -30,7 +30,6 @@ asmlinkage long sys32_fstatat(unsigned int, const char __user *,
                              struct stat64 __user *, int);
 struct mmap_arg_struct32;
 asmlinkage long sys32_mmap(struct mmap_arg_struct32 __user *);
-asmlinkage long sys32_mprotect(unsigned long, size_t, unsigned long);
 
 asmlinkage long sys32_alarm(unsigned int);
 
@@ -42,7 +41,6 @@ asmlinkage long sys32_pwrite(unsigned int, const char __user *, u32, u32, u32);
 
 asmlinkage long sys32_personality(unsigned long);
 
-long sys32_kill(int, int);
 long sys32_fadvise64_64(int, __u32, __u32, __u32, __u32, int);
 long sys32_vm86_warning(void);
 
index 0f6f5becab0d555a3dfbd874a3f5b784a0ccf47f..d0d59bfbccce01e2e019ffd64b7d57e3ddad9622 100644 (file)
@@ -43,7 +43,7 @@
 34     i386    nice                    sys_nice
 35     i386    ftime
 36     i386    sync                    sys_sync
-37     i386    kill                    sys_kill                        sys32_kill
+37     i386    kill                    sys_kill
 38     i386    rename                  sys_rename
 39     i386    mkdir                   sys_mkdir
 40     i386    rmdir                   sys_rmdir
 122    i386    uname                   sys_newuname
 123    i386    modify_ldt              sys_modify_ldt
 124    i386    adjtimex                sys_adjtimex                    compat_sys_adjtimex
-125    i386    mprotect                sys_mprotect                    sys32_mprotect
+125    i386    mprotect                sys_mprotect
 126    i386    sigprocmask             sys_sigprocmask                 compat_sys_sigprocmask
 127    i386    create_module
 128    i386    init_module             sys_init_module
This page took 0.029295 seconds and 5 git commands to generate.