[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3d8471ca0803050523r2a55c91em68021a7b078ef48c@mail.gmail.com>
Date: Wed, 5 Mar 2008 14:23:44 +0100
From: "Guillaume Chazarain" <guichaz@...il.com>
To: "Pawel Plociennik" <paplociennik@...il.com>
Cc: "Pavel Machek" <pavel@....cz>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] chroot= as a new kernel parameter
On Wed, Mar 5, 2008 at 2:38 PM, Pawel Plociennik <paplociennik@...il.com> wrote:
> At last my patch is as simple as possible and it has *only* a *few lines of the code* and it can be used
> as *independent* method to various *hundred* distroes.
Your method requires a certain kernel, mine does not ;-)
> static void run_init_process(char *init_filename)
> {
> + if (chroot_str)
> + if (sys_chroot(chroot_str) < 0)
> + printk(KERN_WARNING "chroot=%s failed\n", chroot_str);
> +
As I said before, this chroot= option can be used for security reason
instead of testing purpose, and in this case, continuing booting after
a failed chroot sounds like a security issue.
--
Guillaume
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists