netdev - Re: [PATCH iproute2-next] man: remove ppp from list of devices not allowed to change netns

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sat, 9 Nov 2019 12:02:18 -0800
From:   Stephen Hemminger <stephen@...workplumber.org>
To:     Guillaume Nault <gnault@...hat.com>
Cc:     David Ahern <dsahern@...il.com>, netdev@...r.kernel.org
Subject: Re: [PATCH iproute2-next] man: remove ppp from list of devices not
 allowed to change netns

On Sat, 9 Nov 2019 18:01:23 +0100
Guillaume Nault <gnault@...hat.com> wrote:

> On Fri, Nov 08, 2019 at 03:31:59PM -0800, Stephen Hemminger wrote:
> > On Fri, 8 Nov 2019 22:28:04 +0100
> > Guillaume Nault <gnault@...hat.com> wrote:  
> > > diff --git a/man/man8/ip-link.8.in b/man/man8/ip-link.8.in
> > > index 9629a649..939e2ad4 100644
> > > --- a/man/man8/ip-link.8.in
> > > +++ b/man/man8/ip-link.8.in
> > > @@ -1921,7 +1921,7 @@ move the device to the network namespace associated with name
> > >  .RI process " PID".
> > >  
> > >  Some devices are not allowed to change network namespace: loopback, bridge,
> > > -ppp, wireless. These are network namespace local devices. In such case
> > > +wireless. These are network namespace local devices. In such case
> > >  .B ip
> > >  tool will return "Invalid argument" error. It is possible to find out
> > >  if device is local to a single network namespace by checking  
> > 
> > This doesn't have to wait for net-next  
> Right, I should have posted it for -iproute2, sorry. The patch applies
> cleanly there though. Do you want to take it as is, or do you prefer
> that I submit it again?
> 

I will merge it as is