lkml.org 
[lkml]   [2011]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] debugfs: only allow root access to debugging interfaces
On Tue, Feb 22, 2011 at 11:25:33AM -0800, Kees Cook wrote:
> Hi Greg,
>
> On Tue, Feb 22, 2011 at 11:14:54AM -0800, Greg KH wrote:
> > On Tue, Feb 22, 2011 at 10:47:26AM -0800, Kees Cook wrote:
> > > On Tue, Feb 22, 2011 at 10:32:19AM -0800, David Daney wrote:
> > > > On 02/22/2011 10:16 AM, Kees Cook wrote:
> > > > >Har har, I forgot --compose to "git send-email".
> > > > >
> > > > >Anyway, with the continuing deluge of bugs in the "debug" filesystem, I
> > > > >would like to make that filesystem's root directory mode 0700 by default
> > > > >since it's filled with crazy stuff that regular users do not need to see.
> > > > >
> > > > >Better to try to just close the door completely on all the stuff in there.
> > > > >It is, after all, supposed to only be used for debugging, right?
> > > > >
> > > >
> > > > It depends if you consider use of ftrace and kprobes 'debugging'.
> > > > In any event, you really have to be root to be able to manipulate
> > > > them.
> > > >
> > > > I can currently do 'cat /sys/kernel/debug/tracing/trace' as a normal
> > > > user. With your change I don't think it would be possible. This is
> > > > not something I often (ever) do, but it is a change.
> > >
> > > Right, my thinking is that all the manipulations on this tree should be
> > > root-only (non-root stuff shouldn't live in something named "debug"),
> > > so we might as well lock it down a bit more so we can avoid all the CVEs[1]
> > > being assigned for glitches in this tree.
> >
> > The CVEs are for non-root writes to debugfs, same thing for sysfs. It's
> > just stupid mistakes being made here, don't try to lock down the whole
> > filesystem for just a handfull of bugs.
> >
> > So I really can't accept these patches, sorry.
>
> What system do you proposed to keep these "stupid mistakes" from
> continuing to happen?

The newly added rule to checkpatch.pl for sysfs files could be extended
to also check for debugfs files with these incorrect permissions.
That's the best thing for all new patches as lots of people run that
tool to find problems.

> If debugfs had already been mode 0700, we could have
> avoided all of these CVEs, including the full-blown local root escalation.

Same could be said for sysfs :)

> The "no rules" approach to debugfs is not a good idea, IMO.

Sorry you don't like it, but that's the way it is.

And as Alan said, you can convince your distro to just not enable it if
you really don't like it, or mount it with this permission.

thanks,

greg k-h


\
 
 \ /
  Last update: 2011-02-22 20:51    [from the cache]
©2003-2011 Jasper Spaans