LWN.net Logo

Re: [PATCH 1/2] module: allow loading module from fd

From:  Rusty Russell <rusty-AT-rustcorp.com.au>
To:  Kees Cook <keescook-AT-chromium.org>, linux-kernel-AT-vger.kernel.org, "Linus Torvalds" <torvalds-AT-linux-foundation.org>
Subject:  Re: [PATCH 1/2] module: allow loading module from fd
Date:  Thu, 06 Sep 2012 10:44:08 +0930
Message-ID:  <87y5kohs0v.fsf@rustcorp.com.au>
Cc:  Serge Hallyn <serge.hallyn-AT-canonical.com>, James Morris <james.l.morris-AT-oracle.com>, Al Viro <viro-AT-zeniv.linux.org.uk>, Eric Paris <eparis-AT-redhat.com>, Kees Cook <keescook-AT-chromium.org>, Jiri Kosina <jkosina-AT-suse.cz>, linux-security-module-AT-vger.kernel.org
Archive-link:  Article, Thread

Kees Cook <keescook@chromium.org> writes:
> This changes the init_module syscall so that when the first argument
> (blob address) is NULL, the second argument is used as a file descriptor
> to the module (instead of length). The third argument (module arguments)
> remains unchanged.

Do you know why Linus hates ioctls?  It's because they encourage
casual introduction of new ABIs.

I see you managed the same feat with a different system call.  It avoids
the agony of debate a new system call would entail.  Very clever.

Now do it properly.
Rusty.


(Log in to post comments)

Copyright © 2012, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds