linux-kernel - Re: [PATCH v2] devtmpfs: mount with noexec and nosuid

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: <20121120211312.57f1b63d@pyramind.ukuu.org.uk>
Date:	Tue, 20 Nov 2012 21:13:12 +0000
From:	Alan Cox <alan@...rguk.ukuu.org.uk>
To:	Kees Cook <keescook@...omium.org>
Cc:	linux-kernel@...r.kernel.org,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	ellyjones@...omium.org, Kay Sievers <kay@...y.org>,
	Roland Eggner <edvx1@...temanalysen.net>
Subject: Re: [PATCH v2] devtmpfs: mount with noexec and nosuid

On Tue, 20 Nov 2012 12:42:38 -0800
Kees Cook <keescook@...omium.org> wrote:

> Since devtmpfs is writable, make the default noexec,nosuid as well. This
> protects from the case of a privileged process having an arbitrary file
> write flaw and an argumentless arbitrary execution (i.e. it would lack
> the ability to run "mount -o remount,exec,suid /dev").

Ok this looks crap on two levels.

1. Why not just have your userspace mount -o remount the file system this
way already in early boot. (and if you trojanned boot that early then any
supposed security gain is already lost)

2. If you want to do this right then you need to work out what you are
trying to prevent. Your devtmpfs can force file permissions on the
underlying device nodes by having its own operation handling for chmod.

At that point you can force permissions on anything that you want to
avoid floating around that filesystem with other rights, while not
touching it on device or directory nodes where the meaning is different.

In its current form however it appears to be a kernel implementation of
"mount is too hard".

Alan
--
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/