Re: [PATCH] userns,pidns: Verify the userns for new pid namespaces

From	ebiederm@xmission.com (Eric W. Biederman)
Newsgroups	linux.kernel
Subject	Re: [PATCH] userns,pidns: Verify the userns for new pid namespaces
Date	2017-04-30 06:50 +0200
Message-ID	<tBOY1-79n-3@gated-at.bofh.it> (permalink)
References	(4 earlier) <tBG4q-1Ko-3@gated-at.bofh.it> <tBG4q-1Ko-1@gated-at.bofh.it> <tBGnM-1QW-25@gated-at.bofh.it> <tBHDb-2vx-11@gated-at.bofh.it> <tBOY1-79n-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

ebiederm@xmission.com (Eric W. Biederman) writes:

> "Serge E. Hallyn" <serge@hallyn.com> writes:
>
>> Quoting Eric W. Biederman (ebiederm@xmission.com):
>>> 
>>> It is pointless and confusing to allow a pid namespace hierarchy and
>>> the user namespace hierarchy to get out of sync.  The owner of a child
>>> pid namespace should be the owner of the parent pid namespace or
>>> a descendant of the owner of the parent pid namespace.
>>> 
>>> Otherwise it is possible to construct scenarios where it is legal to
>>> do something in a parent pid namespace but in a child pid namespace.
>>
>> Hi,
>>
>> did you mean 'but not in a child...' above?
>
> Actually I believe I meant:
>
>>> Otherwise it is possible to construct scenarios where it is not legal
>>> to do something in a parent pid namespace but it is legal a child pid
>>> namespace.
>
> I definitely need to fix that wording thank you.

Looking at some more I mean:

Otherwise it is possible to construct scenarios where a process has a
capability in a over a parent pid namespace but does not have the
capability over a child pid namespace.  Which confusingly makes
permission checks non-transitive.


Eric

Thread

Re: [PATCH v3] pid_ns: Introduce ioctl to set vector of ns_last_pid's on ns hierarhy ebiederm@xmission.com (Eric W. Biederman) - 2017-04-27 17:30 +0200
  Re: [PATCH v3] pid_ns: Introduce ioctl to set vector of ns_last_pid's on ns hierarhy ebiederm@xmission.com (Eric W. Biederman) - 2017-04-27 18:20 +0200
    Re: [PATCH v3] pid_ns: Introduce ioctl to set vector of ns_last_pid's on ns hierarhy ebiederm@xmission.com (Eric W. Biederman) - 2017-04-29 21:20 +0200
      [PATCH] userns,pidns: Verify the userns for new pid namespaces ebiederm@xmission.com (Eric W. Biederman) - 2017-04-29 21:40 +0200
        Re: [PATCH] userns,pidns: Verify the userns for new pid namespaces "Serge E. Hallyn" <serge@hallyn.com> - 2017-04-29 23:00 +0200
          Re: [PATCH] userns,pidns: Verify the userns for new pid namespaces ebiederm@xmission.com (Eric W. Biederman) - 2017-04-30 06:50 +0200
          Re: [PATCH] userns,pidns: Verify the userns for new pid namespaces ebiederm@xmission.com (Eric W. Biederman) - 2017-04-30 06:50 +0200
        Re: [PATCH] userns,pidns: Verify the userns for new pid namespaces ebiederm@xmission.com (Eric W. Biederman) - 2017-05-02 22:50 +0200
      Re: [PATCH v3] pid_ns: Introduce ioctl to set vector of ns_last_pid's on ns hierarhy ebiederm@xmission.com (Eric W. Biederman) - 2017-05-02 23:40 +0200

csiph-web