Re: [PATCH BUGFIX -rc3] Smack: Don't register smackfs if we're notloaded

From: Ahmed S. Darwish
Date: Wed Mar 05 2008 - 07:55:09 EST


On Wed, Mar 05, 2008 at 02:44:45PM +0200, Ahmed S. Darwish wrote:
> On Tue, Mar 04, 2008 at 09:45:04AM -0800, Casey Schaufler wrote:
> > > From: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
...
> > >
> > > What is the oops? Why does it happen?
> ...
> >
> > One solution would be to tighten the smackfs code so that it
> > handles the uninitialized LSM case properly.
> >
>
> IMHO no smackfs code should ever execute if smack isn't loaded.
>
> This means catching it from the very fist step where it registers
> itself in init_smk_fs instead of doing several if(we're enabled) cases
> in the code path.
>
> The solution should be a _general_ solution, _not_ a SMACK one cause
> SELinux sufferes from exactly the same problem.
>

Not to be misunderstood here, I'm used to writing SMACK in capitals.
I ,ofcourse, don't mean shouting (never will).

Regards,

--

"Better to light a candle, than curse the darkness"

Ahmed S. Darwish
Homepage: http://darwish.07.googlepages.com
Blog: http://darwish-07.blogspot.com

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