All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrange" <berrange-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: "Eric W. Biederman" <ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org>
Cc: containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Oleg Nesterov <oleg-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	Michael Kerrisk
	<mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Tejun Heo <tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Subject: Re: [PATCH] Forbid invocation of kexec_load() outside initial PID namespace
Date: Fri, 3 Aug 2012 13:52:10 +0100	[thread overview]
Message-ID: <20120803125210.GD12870@redhat.com> (raw)
In-Reply-To: <cfa2e2c9-db52-40cd-979b-7a6084427190-2ueSQiBKiTY7tOexoI0I+QC/G2K4zDHf@public.gmane.org>

On Fri, Aug 03, 2012 at 05:45:40AM -0700, Eric W. Biederman wrote:
> The solution is to use user namespaces and to only test ns_capable on the magic reboot path.
> 
> For the 3.7 timeframe that should be a realistic solution.

Hmm, that would imply that if LXC wants to allow reboot()/CAP_SYS_BOOT
they will be forced to use CLONE_NEWUSER. I was rather looking for a way
to allow the container to keep CAP_SYS_BOOT, without also mandating use
of user namespaces.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

WARNING: multiple messages have this Message-ID (diff)
From: "Daniel P. Berrange" <berrange@redhat.com>
To: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: linux-kernel@vger.kernel.org,
	containers@lists.linux-foundation.org,
	Serge Hallyn <serge.hallyn@canonical.com>,
	Daniel Lezcano <daniel.lezcano@free.fr>,
	Michael Kerrisk <mtk.manpages@gmail.com>,
	Tejun Heo <tj@kernel.org>, Oleg Nesterov <oleg@redhat.com>
Subject: Re: [PATCH] Forbid invocation of kexec_load() outside initial PID namespace
Date: Fri, 3 Aug 2012 13:52:10 +0100	[thread overview]
Message-ID: <20120803125210.GD12870@redhat.com> (raw)
In-Reply-To: <cfa2e2c9-db52-40cd-979b-7a6084427190@email.android.com>

On Fri, Aug 03, 2012 at 05:45:40AM -0700, Eric W. Biederman wrote:
> The solution is to use user namespaces and to only test ns_capable on the magic reboot path.
> 
> For the 3.7 timeframe that should be a realistic solution.

Hmm, that would imply that if LXC wants to allow reboot()/CAP_SYS_BOOT
they will be forced to use CLONE_NEWUSER. I was rather looking for a way
to allow the container to keep CAP_SYS_BOOT, without also mandating use
of user namespaces.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

  parent reply	other threads:[~2012-08-03 12:52 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-03 10:53 [PATCH] Forbid invocation of kexec_load() outside initial PID namespace Daniel P. Berrange
2012-08-03 10:53 ` Daniel P. Berrange
     [not found] ` <1343991184-3619-1-git-send-email-berrange-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2012-08-03 11:25   ` richard -rw- weinberger
2012-08-03 11:25     ` richard -rw- weinberger
2012-08-03 12:45   ` Eric W. Biederman
2012-08-03 12:45     ` Eric W. Biederman
     [not found]     ` <cfa2e2c9-db52-40cd-979b-7a6084427190-2ueSQiBKiTY7tOexoI0I+QC/G2K4zDHf@public.gmane.org>
2012-08-03 12:52       ` Daniel P. Berrange [this message]
2012-08-03 12:52         ` Daniel P. Berrange
     [not found]         ` <20120803125210.GD12870-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2012-08-03 13:07           ` Eric W. Biederman
2012-08-03 13:07             ` Eric W. Biederman
2012-08-04 23:15             ` Serge Hallyn
2012-08-06 19:20               ` Serge E. Hallyn
2012-08-06 19:20                 ` Serge E. Hallyn
     [not found]             ` <bef31b2b-429b-4b2b-981b-b230f9c6bfad-2ueSQiBKiTY7tOexoI0I+QC/G2K4zDHf@public.gmane.org>
2012-08-04 23:15               ` Serge Hallyn
2012-08-06 19:00   ` Serge E. Hallyn
2012-08-06 19:00     ` Serge E. Hallyn
     [not found]     ` <20120806190014.GA15267-7LNsyQBKDXoIagZqoN9o3w@public.gmane.org>
2012-08-06 19:16       ` Eric W. Biederman
2012-08-06 19:16         ` Eric W. Biederman
     [not found]         ` <87r4rjn84y.fsf-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org>
2012-08-06 19:24           ` Serge E. Hallyn
2012-08-06 19:24             ` Serge E. Hallyn

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120803125210.GD12870@redhat.com \
    --to=berrange-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
    --cc=ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=oleg-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.