From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH net 1/1 v1] rtnetlink: require unique netns identifier Date: Sat, 3 Feb 2018 11:17:01 -0800 Message-ID: <20180203111701.2ff8d7a3@xeon-e3> References: <20180203132904.11972-1-christian.brauner@ubuntu.com> <20180203132904.11972-2-christian.brauner@ubuntu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, ebiederm@xmission.com, jbenc@redhat.com, nicolas.dichtel@6wind.com, linux-kernel@vger.kernel.org, dsahern@gmail.com, davem@davemloft.net To: Christian Brauner Return-path: Received: from mail-pf0-f181.google.com ([209.85.192.181]:44432 "EHLO mail-pf0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752837AbeBCTRF (ORCPT ); Sat, 3 Feb 2018 14:17:05 -0500 Received: by mail-pf0-f181.google.com with SMTP id 17so17672008pfw.11 for ; Sat, 03 Feb 2018 11:17:04 -0800 (PST) In-Reply-To: <20180203132904.11972-2-christian.brauner@ubuntu.com> Sender: netdev-owner@vger.kernel.org List-ID: On Sat, 3 Feb 2018 14:29:04 +0100 Christian Brauner wrote: > +static int rtnl_ensure_unique_netns_attr(const struct sock *sk, > + struct nlattr *tb[], > + struct netlink_ext_ack *extack) > +{ > + int ret = -EINVAL; > + struct net *net = NULL, *unique_net = NULL; > + > + /* Requests without network namespace ids have been able to specify > + * multiple properties referring to different network namespaces so > + * don't regress them. > + */ > + if (!tb[IFLA_IF_NETNSID]) > + return 0; > + > + if (!tb[IFLA_NET_NS_PID] && !tb[IFLA_NET_NS_FD]) > + return 0; Isn't this an error? > + > + unique_net = get_net_ns_by_id(sock_net(sk), nla_get_s32(tb[IFLA_IF_NETNSID])); > + if (!unique_net) > + return -1; Other paths are returning errno, so why -1 here?