Re: [PATCH] ptrace: it is fun to strace /sbin/init

[Pavel Machek]

> > > @@ -803,6 +803,8 @@ and is between 256 and 4096 characters. 
> > >  			Run specified binary instead of /sbin/init as init
> > >  			process.
> > >  
> > > +	init_ptrace	[KNL] Allows to ptrace init.
> > > +
> > 
> > No words about it being dangerous, but I believe it is.
> 
> it is, admin should know what he does,
> 
> > If it is not, lets do the patch, but not optional.
> 
> This will change the default historical behaviour, I can't predict
> does this matter for (say) SELinux or not. Otherwise I'd personally
> prefer to always allow to ptrace init.
> 
> > If it is dangerous, option does not make it better.
> 
> ptrace() is always dangerous. ptracer can crash oracle and lose data.
> 
> /sbin/init is important, but there are other important (and sometimes
> much more important) services. Why it is so special so that we can't
> debug/strace it?

Maybe. Let's kill /sbin/init protection in 2.6.26. But making it
optional is wrong.
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
pomozte zachranit klanovicky les:  http://www.ujezdskystrom.info/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/