linux-kernel - Any chance that kernel/uid6.c can go? [Was: [PATCH 22/45] kernel: provide ksys_*() wrappers for syscalls called by kernel/uid16.c]

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]

Message-ID: <20180322102140.GA14294@light.dominikbrodowski.net>
Date:   Thu, 22 Mar 2018 11:21:40 +0100
From:   Dominik Brodowski <linux@...inikbrodowski.net>
To:     linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
        viro@...IV.linux.org.uk, arnd@...db.de, linux-arch@...r.kernel.org
Cc:     "Eric W . Biederman" <ebiederm@...ssion.com>,
        Andrew Morton <akpm@...ux-foundation.org>
Subject: Any chance that kernel/uid6.c can go? [Was: [PATCH 22/45] kernel:
 provide ksys_*() wrappers for syscalls called by kernel/uid16.c]

On Thu, Mar 22, 2018 at 10:00:36AM +0100, Dominik Brodowski wrote:
> Using these helpers allows us to avoid the in-kernel calls to these
> syscalls: sys_setregid(), sys_setgid(), sys_setreuid(), sys_setuid(),
> sys_setresuid(), sys_setresgid(), sys_setfsuid(), and sys_setfsgid().
> 
> The ksys_ prefix denotes that these function are meant as a drop-in
> replacement for the syscall. In particular, they use the same calling
> convention.
> 
> This patch is part of a series which tries to remove in-kernel calls to
> syscalls. On this basis, the syscall entry path can be streamlined.
> 
> Cc: Al Viro <viro@...IV.linux.org.uk>
> Cc: Eric W. Biederman <ebiederm@...ssion.com>
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> Signed-off-by: Dominik Brodowski <linux@...inikbrodowski.net>
> ---
>  kernel/sys.c   | 58 ++++++++++++++++++++++++++++++++++++++++++++++++++--------
>  kernel/uid16.c | 19 ++++++++++---------

In its header, kernel/uid16.c says, since 2.3.39 was released in January
2000:

 *      Wrapper functions for 16bit uid back compatibility. All nicely tied
 *      together in the faint hope we can take the out in five years time.

Are we any closer to removing these wrappers?

Thanks,
	Dominik