Re: [PATCH] chroot= as a new kernel parameter

From: Guillaume Chazarain
Date: Sun Mar 02 2008 - 11:00:48 EST


On Sat, Mar 1, 2008 at 8:09 PM, Pawel Plociennik <paplociennik@xxxxxxxxx> wrote:
> In the short this patch has added a new chroot= kernel parameter which can be used
> to changing a chroot of an init process before it will has been runed.

Wouldn't booting with 'init=/usr/sbin/chroot /path /sbin/init' achieve
the same effect?
Haven't tested it, just wondering.

> + if(chroot_str) {
> + if(sys_chroot(chroot_str) < 0)
> + printk("chroot=%s failed\n", chroot_str);
> + else
> + printk("chroot=%s successed\n",chroot_str);
> + }

Continuing booting normally when the chroot failed sounds like a
security issue to me.

Regards.

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