Messages in this thread | | | Subject | Re: [PATCH 1/2] module: add syscall to load module from fd | From | Mimi Zohar <> | Date | Wed, 19 Sep 2012 10:41:58 -0400 |
| |
On Wed, 2012-09-19 at 13:08 +0930, Rusty Russell wrote: > Mimi Zohar <zohar@linux.vnet.ibm.com> writes: > > > On Wed, 2012-09-12 at 17:04 +0930, Rusty Russell wrote: > >> "H. Peter Anvin" <hpa@zytor.com> writes: > >> > >> > On 09/06/2012 11:13 AM, Kees Cook wrote: > >> >> Instead of (or in addition to) kernel module signing, being able to reason > >> >> about the origin of a kernel module would be valuable in situations > >> >> where an OS already trusts a specific file system, file, etc, due to > >> >> things like security labels or an existing root of trust to a partition > >> >> through things like dm-verity. > >> >> > >> >> This introduces a new syscall (currently only on x86), similar to > >> >> init_module, that has only two arguments. The first argument is used as > >> >> a file descriptor to the module and the second argument is a pointer to > >> >> the NULL terminated string of module arguments. > >> >> > >> > > >> > Please use the standard naming convention, which is an f- prefix (i.e. > >> > finit_module()). > >> > >> Good point; I just did a replace here. > > > > Have you pushed out the changes? And if so, to where? > > No, I kept them in my patch series but out of linux-next, since I > thought you disliked the placement of the security hooks?
I thought about it some more. The call to security_kernel_module_from_file() from copy_module_from_user() doesn't provide any information, not the buffer contents nor the signature. The only thing IMA-appraisal can do is to fail the request with INTEGRITY_UNKNOWN. This is reflected in the IMA-appraisal patch I posted http://marc.info/?l=linux-security-module&m=134739023306344&w=2.
Please add my Acked-by: Mimi Zohar <zohar@us.ibm.com>
thanks,
Mimi
| |