Re: [PATCH try #2] security: Convert LSM into a static interface

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Crispin Cowan
Date: Tuesday, June 26, 2007 - 5:57 pm

Kyle Moffett wrote:
Hmmm. You seem to be mostly concerned with safely rmmod'ing modules. In
contrast, my main concern with the proposed patch is that it removes the
ability to *insert* a module.

Consider the use case of joe admin who is running enterprise-supported
RHEL or SLES, and wants to try some newfangled LSM FooSecureMod thingie.
So he grabs a machine, config's selinux=0 or apparmor=0 and loads his
own module on boot, and plays with it. He even likes FooSecure, better
than SELinux or AppArmor, and wants to roll it out across his data center.

Without James's patch, he can do that, and at worst has a tainted
kernel. RH or Novell or his favorite distro vendor can fix that with a
wave of the hand and bless FooSecure as a module. With James's patch, he
has to patch his kernels, and then enterprise support is hopeless, to
say nothing of the barrier to entry that "patch and rebuild kernel" is
more than many admins are willing to do.

So to solve the problem James & Kyle are concerned with, and preserve
user choice, how about we *only* remove the ability to rmmod, and leave
in place the ability to modprobe? Or even easier, LSMs that don't want
to be unloaded can just block rmmod, and simple LSMs that can be
unloaded safely can permit it.

Crispin

-- 
Crispin Cowan, Ph.D.               http://crispincowan.com/~crispin/
Director of Software Engineering   http://novell.com
	AppArmor Chat: irc.oftc.net/#apparmor

-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: implement-file-posix-capabilities.patch, Serge E. Hallyn, (Thu Jun 21, 9:00 am)
Re: implement-file-posix-capabilities.patch, Andrew Morgan, (Sat Jun 23, 1:13 am)
Re: implement-file-posix-capabilities.patch, Serge E. Hallyn, (Sun Jun 24, 8:51 am)
Re: implement-file-posix-capabilities.patch, James Morris, (Sun Jun 24, 9:18 am)
Re: [PATCH try #2] security: Convert LSM into a static int ..., Andreas Gruenbacher, (Mon Jun 25, 1:37 pm)
Re: [PATCH try #2] security: Convert LSM into a static int ..., Crispin Cowan, (Tue Jun 26, 5:57 pm)
Re: implement-file-posix-capabilities.patch, Andrew Morgan, (Tue Jun 26, 10:00 pm)
Re: implement-file-posix-capabilities.patch, Serge E. Hallyn, (Wed Jun 27, 6:16 am)
Re: implement-file-posix-capabilities.patch, Andrew Morgan, (Wed Jun 27, 11:19 pm)
Re: implement-file-posix-capabilities.patch, Serge E. Hallyn, (Thu Jun 28, 6:36 am)
Re: implement-file-posix-capabilities.patch, Casey Schaufler, (Thu Jun 28, 8:14 am)
Re: implement-file-posix-capabilities.patch, Serge E. Hallyn, (Thu Jun 28, 8:38 am)
Re: implement-file-posix-capabilities.patch, Andrew Morgan, (Thu Jun 28, 8:50 am)
Re: implement-file-posix-capabilities.patch, Casey Schaufler, (Thu Jun 28, 8:56 am)
Re: implement-file-posix-capabilities.patch, Andrew Morgan, (Thu Jun 28, 10:30 pm)
Re: implement-file-posix-capabilities.patch, Serge E. Hallyn, (Fri Jun 29, 6:24 am)
Re: implement-file-posix-capabilities.patch, Casey Schaufler, (Fri Jun 29, 7:46 am)
Re: implement-file-posix-capabilities.patch, Serge E. Hallyn, (Mon Jul 2, 7:38 am)
Re: implement-file-posix-capabilities.patch, Andrew Morgan, (Wed Jul 4, 2:29 pm)
Re: implement-file-posix-capabilities.patch, Casey Schaufler, (Wed Jul 4, 4:00 pm)