linux-kernel - Re: [PATCH 13/19] Unify sys

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 8 Dec 2009 10:36:29 +0800
From:	liqin.chen@...plusct.com
To:	Al Viro <viro@....linux.org.uk>
Cc:	linux-arch@...r.kernel.org, linux-arch-owner@...r.kernel.org,
	linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org
Subject: Re: [PATCH 13/19] Unify sys_mmap*

linux-arch-owner@...r.kernel.org 写于 2009-12-07 11:54:00:
> 
> New helper - sys_mmap_pgoff(); switch syscalls to using it.
> 
> Signed-off-by: Al Viro <viro@...iv.linux.org.uk>
> ---
>  arch/alpha/kernel/osf_sys.c               |   19 ++-----
>  arch/arm/kernel/entry-common.S            |    4 +-
>  arch/arm/kernel/sys_arm.c                 |   30 +----------
>  arch/avr32/include/asm/syscalls.h         |    4 --
>  arch/avr32/kernel/sys_avr32.c             |   31 -----------
>  arch/avr32/kernel/syscall-stubs.S         |    2 +-
>  arch/blackfin/kernel/sys_bfin.c           |   33 -----------
>  arch/blackfin/mach-common/entry.S         |    2 +-
>  arch/cris/kernel/sys_cris.c               |   30 +---------
>  arch/frv/kernel/sys_frv.c                 |   66 
+----------------------
>  arch/h8300/kernel/sys_h8300.c             |   83 
+----------------------------
>  arch/h8300/kernel/syscalls.S              |    2 +-
>  arch/ia64/kernel/sys_ia64.c               |   37 +------------
>  arch/m32r/kernel/sys_m32r.c               |   24 --------
>  arch/m32r/kernel/syscall_table.S          |    2 +-
>  arch/m68k/kernel/sys_m68k.c               |   79 
++--------------------------
>  arch/m68knommu/kernel/sys_m68k.c          |   38 +------------
>  arch/m68knommu/kernel/syscalltable.S      |    2 +-
>  arch/microblaze/kernel/sys_microblaze.c   |   38 +------------
>  arch/microblaze/kernel/syscall_table.S    |    2 +-
>  arch/mips/kernel/linux32.c                |   19 +------
>  arch/mips/kernel/syscall.c                |   29 +---------
>  arch/mn10300/kernel/entry.S               |    2 +-
>  arch/mn10300/kernel/sys_mn10300.c         |   31 +----------
>  arch/parisc/kernel/sys_parisc.c           |   30 ++---------
>  arch/powerpc/kernel/syscalls.c            |   15 +-----
>  arch/s390/kernel/compat_linux.c           |   32 +----------
>  arch/s390/kernel/sys_s390.c               |   30 +----------
>  arch/score/kernel/sys_score.c             |   21 +------
>  arch/sh/kernel/sys_sh.c                   |   28 +---------
>  arch/sparc/kernel/sys_sparc_32.c          |   31 ++---------
>  arch/sparc/kernel/sys_sparc_64.c          |   22 ++------
>  arch/um/kernel/syscall.c                  |   28 +---------
>  arch/um/sys-i386/shared/sysdep/syscalls.h |    4 --
>  arch/x86/ia32/ia32entry.S                 |    2 +-
>  arch/x86/ia32/sys_ia32.c                  |   43 +---------------
>  arch/x86/include/asm/sys_ia32.h           |    3 -
>  arch/x86/include/asm/syscalls.h           |    2 -
>  arch/x86/kernel/sys_i386_32.c             |   27 +---------
>  arch/x86/kernel/sys_x86_64.c              |   17 +------
>  arch/x86/kernel/syscall_table_32.S        |    2 +-
>  arch/xtensa/include/asm/syscall.h         |    2 -
>  arch/xtensa/include/asm/unistd.h          |    2 +-
>  arch/xtensa/kernel/syscall.c              |   25 ---------
>  include/linux/syscalls.h                  |    4 ++
>  mm/util.c                                 |   29 ++++++++++
>  46 files changed, 105 insertions(+), 903 deletions(-)
> 
> diff --git a/arch/score/kernel/sys_score.c 
b/arch/score/kernel/sys_score.c
> index 0012494..5299a47 100644
> --- a/arch/score/kernel/sys_score.c
> +++ b/arch/score/kernel/sys_score.c
> @@ -36,33 +36,18 @@ asmlinkage long
>  sys_mmap2(unsigned long addr, unsigned long len, unsigned long prot,
>       unsigned long flags, unsigned long fd, unsigned long pgoff)
>  {
> -   int error = -EBADF;
> -   struct file *file = NULL;
> -
>     if (pgoff & (~PAGE_MASK >> 12))
>        return -EINVAL;
> 
> -   flags &= ~(MAP_EXECUTABLE | MAP_DENYWRITE);
> -   if (!(flags & MAP_ANONYMOUS)) {
> -      file = fget(fd);
> -      if (!file)
> -         return error;
> -   }
> -
> -   down_write(&current->mm->mmap_sem);
> -   error = do_mmap_pgoff(file, addr, len, prot, flags, pgoff);
> -   up_write(&current->mm->mmap_sem);
> -
> -   if (file)
> -      fput(file);
> -
> -   return error;
> +   return sys_mmap_pgoff(addr, len, prot, flags, fd, pgoff);
> +   /* sic - almost certainly should shift pgoff as well */
>  }
> 
>  asmlinkage long
>  sys_mmap(unsigned long addr, unsigned long len, unsigned long prot,
>     unsigned long flags, unsigned long fd, off_t pgoff)
>  {
> +   /* where's the alignment check? */
>     return sys_mmap2(addr, len, prot, flags, fd, pgoff >> PAGE_SHIFT);
>  }
> 

It's ok for your update, even 
     if (pgoff & (~PAGE_MASK >> 12))
        return -EINVAL;
code haven't use anymore, you could remove it.

In addition, the sys_mmap should like this.

asmlinkage long
sys_mmap(unsigned long addr, unsigned long len, unsigned long prot,
    unsigned long flags, unsigned long fd, off_t pgoff)
{
        unsigned long result;

        result = -EINVAL;
        if (pgoff & ~PAGE_MASK)
                goto out;

        result = sys_mmap2(addr, len, prot, flags, fd, pgoff >> 
PAGE_SHIFT);
out:
        return result;
} 

Thanks
                liqin