From mboxrd@z Thu Jan 1 00:00:00 1970 Reply-To: kernel-hardening@lists.openwall.com Sender: Vasiliy Kulikov Date: Mon, 20 Jun 2011 18:35:50 +0400 From: Vasiliy Kulikov Message-ID: <20110620143550.GA11835@albatros> References: <20110620103917.GA5230@albatros> <20110620141137.GA31044@openwall.com> <20110620141951.GA11607@albatros> <20110620142555.GA31234@openwall.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110620142555.GA31234@openwall.com> Subject: Re: [kernel-hardening] [RFC 2/5 v4] procfs: add hidepid= and gid= mount options To: kernel-hardening@lists.openwall.com List-ID: Solar, On Mon, Jun 20, 2011 at 18:25 +0400, Solar Designer wrote: > On Mon, Jun 20, 2011 at 06:19:51PM +0400, Vasiliy Kulikov wrote: > > On Mon, Jun 20, 2011 at 18:11 +0400, Solar Designer wrote: > > > On Mon, Jun 20, 2011 at 04:58:10PM +0300, Alexey Dobriyan wrote: > > > > gid= is bad choice because > > > > a) e. g. VFAT uses uid=/gid= mount options to make all inodes to have > > > > certain uid/gid > > > > b) uid=/gid=, IIRC, will be added as generic VFS mount options (like ro) > > > > with semantics described in a) > > > > > > > > so having different semantics for /proc won't be good. > > > > > > I lost track of your proposals/patches. Aren't you currently proposing > > > that gid= would make all inodes have the specified gid? If not, why > > > not? Such semantics sound fine to me. That's what gid= does on procfs > > > on Linux 2.4.x-ow. > > > > With taskstats and similar mechanisms IMO it's better to use sysctls > > instead of procfs mount options as it would influence not only on procfs > > files. > > OK, but why did we receive a comment about the gid= mount option then > (quote above)? Are you still proposing it or not anymore? No, till v4 I assumed there is no way to gather processes' information except procfs: http://www.openwall.com/lists/kernel-hardening/2011/06/16/6 > Is the > question "outdated" (resulting from your earlier proposal, not the > latest one)? Yes. I didn't post a patch with taskstats and sysctl variables to LKML yet (only the changes in ptrace/capabilities code). Thanks, -- Vasiliy