linux-kernel - Re: [PATCH 1/1] sys_sysfs: Add CONFIG_SYSFS

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] [day] [month] [year] [list]

Message-Id: <20140219204950.f88f409e2bb5a590b1183724@skynet.be>
Date:	Wed, 19 Feb 2014 20:49:50 +0800
From:	Fabian Frederick <fabf@...net.be>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	Dave Jones <davej@...hat.com>, linux-kernel@...r.kernel.org,
	rdunlap <rdunlap@...radead.org>, gregkh@...uxfoundation.org
Subject: Re: [PATCH 1/1] sys_sysfs: Add CONFIG_SYSFS_SYSCALL

On Thu, 13 Feb 2014 14:43:19 -0800
Andrew Morton <akpm@...ux-foundation.org> wrote:

> On Thu, 13 Feb 2014 17:31:41 -0500 Dave Jones <davej@...hat.com> wrote:
> 
> > On Thu, Feb 13, 2014 at 10:09:42PM +0800, Fabian Frederick wrote:
> >  > sys_sysfs is an obsolete system call no longer supported by libc ...
> >  > -This patch adds a default CONFIG_SYSFS_SYSCALL=y
> >  > -Option can be turned off in expert mode.
> >  > -cond_syscall added to kernel/sys_ni.c
> >  > 
> >  > Signed-off-by: Fabian Frederick <fabf@...net.be>
> >  > ---
> >  >  fs/filesystems.c |  2 ++
> >  >  init/Kconfig     | 10 ++++++++++
> >  >  kernel/sys_ni.c  |  1 +
> >  >  3 files changed, 13 insertions(+)
> > 
> > did anyone ever ship userspace that actually used that syscall ?
> > Some ancient version of udev that probably doesn't work on a modern kernel anyway maybe ?
> > 
> > I wonder if it's something we can make -ENOSYS unconditionally at some point,
> > and remove all that code entirely.
> 
> Perhaps we could make it `default n' now and see if we receive nastygrams?

As it seems very old, maybe we wouldn't have any trouble for users in linux-next but it could break something when going in stable (?).Having that patch with functions enabled by default will give us time to test it and have a pointer to it to remove later...

Fabian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/