Received: by 10.223.176.5 with SMTP id f5csp1442111wra; Sun, 4 Feb 2018 04:14:06 -0800 (PST) X-Google-Smtp-Source: AH8x227CuFPbJiQK8YhYPoVSBAGh6NohDXQ1OYLxspF72GMUFnBLPf/pixtvh2RZ9W4AWPDKrlVB X-Received: by 10.99.181.13 with SMTP id y13mr31205795pge.196.1517746446760; Sun, 04 Feb 2018 04:14:06 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1517746446; cv=none; d=google.com; s=arc-20160816; b=X2ANXj8TuhgeE/PzwjsqQCB9A8EDFoYsSriT2pozhHIjojYJK5fmxVKN7KIZ5JYo1a YSM3XfTqgegYEJhUhrdOZ9p1SPexXzHQStordxRD/GlaZdr52or0j/si9HAM9WcLIY8k PQugpeSB4QMyZlayXXZEyBjgfcPHbA37lbJzAekwdi+mSvCD8FWXuur91TE04wHSxqE8 LWjrgNwLLwHWqs01mDEjuo90dYa9KfNm3p7iYroHlaK6cvhDLQg5lgAineXMvVBxGeJP SVSxxFcuopdiLimZrU/xfvt6O5S1xQtIvnX4YeqNPAOFEgRa+hIvw3bahiqXYPIbOBJt mmBw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=KG4VpnzWjdHq6PlB6HAoM4fptvs12uz8/HUuQ0F+K/Q=; b=wLQL5h98SRw/krYkAqF/HkpSaJ18Kptv0+OkWjPL/+e0KVYqjam6jiwzUa4b+1wlF8 RG83gbPs7sZU+ux/hRxk+30rW7tRPKltEwG71blVUv7CAriwaKDr6zl96wn6LHXgK2Z3 QifzAyLQfN5GJvALmiqKt/6cfnwwx0tsEF9qHHm7OC2KdDoad82UHwYmvl6NcbKAd+To Y4olt+tna0oo7t/vE5MLv/zFrznjhdIdix81fF0yIq63fBma/mOts3TdHqN76KE/RB+1 yetuIYpmD234A7iZFP64WIvJZjXgqJDMMQ8U4Q9c6Xa3KfUGm4zaUGtCLxk71yNMm6j+ BpXQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=canonical.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j188si3320518pfb.379.2018.02.04.04.13.52; Sun, 04 Feb 2018 04:14:06 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=canonical.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752015AbeBDMMZ (ORCPT + 99 others); Sun, 4 Feb 2018 07:12:25 -0500 Received: from youngberry.canonical.com ([91.189.89.112]:34031 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751806AbeBDMMQ (ORCPT ); Sun, 4 Feb 2018 07:12:16 -0500 Received: from mail-wm0-f71.google.com ([74.125.82.71]) by youngberry.canonical.com with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.76) (envelope-from ) id 1eiJ9b-0004oN-V6 for linux-kernel@vger.kernel.org; Sun, 04 Feb 2018 12:12:15 +0000 Received: by mail-wm0-f71.google.com with SMTP id f3so6212664wmc.8 for ; Sun, 04 Feb 2018 04:12:15 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=KG4VpnzWjdHq6PlB6HAoM4fptvs12uz8/HUuQ0F+K/Q=; b=MgQvjytn4YGiYtukqRlmDV5fjplRwbt0lfTYl93uXzWZdNRQ+suNqEiicGFmmMYepy xQlVmMF+GotQuWTdHppMUZCedOApYh2HMe6a9IeBQB+zQJt1hvSICWKTZo5U7RjHLTqC dIh+D/8sSONnsoqNzrGYxNnYA2DqvHZi5X746JgsgAR6l6/HkfgGj+O49kdwxB0kql5M /g3k0boUFlfjfvHBjcTj5XuhtB/Q2LtoxeJHSqSPwFdqIprJTz1zyFmY54/NPxYpZOep ev9qoAA2JSN2Y3IcZZXraujqaaei6wj2gR5XbUDFmMR4m6TULxopYbaHwFEktqC+Qy2y y0dw== X-Gm-Message-State: AKwxytcak3SuZkMhtNwq1ea6eHJrZzhXi1ziXGGxsaVdvVFIc7rT0ykm Hgsagny9xyC35qXIl+YR6EqimcFRYo5J2ftVAcrN3GArkpaD9l/8fTJ6KzLhNVz+WZkmAlrIWBt Axr+VXBrVI68O0Y5072pUOTzsdnv5AtLfZFcWdHqakA== X-Received: by 10.80.186.114 with SMTP id 47mr76459536eds.252.1517746335706; Sun, 04 Feb 2018 04:12:15 -0800 (PST) X-Received: by 10.80.186.114 with SMTP id 47mr76459525eds.252.1517746335544; Sun, 04 Feb 2018 04:12:15 -0800 (PST) Received: from gmail.com ([2001:67c:1810:f051:f817:224e:7728:4274]) by smtp.gmail.com with ESMTPSA id f29sm5684003eda.43.2018.02.04.04.12.14 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Sun, 04 Feb 2018 04:12:14 -0800 (PST) Date: Sun, 4 Feb 2018 13:12:08 +0100 From: Christian Brauner To: David Ahern Cc: Stephen Hemminger , Christian Brauner , netdev@vger.kernel.org, ebiederm@xmission.com, jbenc@redhat.com, nicolas.dichtel@6wind.com, linux-kernel@vger.kernel.org, davem@davemloft.net Subject: Re: [PATCH net 1/1 v1] rtnetlink: require unique netns identifier Message-ID: <20180204121207.GB1344@gmail.com> References: <20180203132904.11972-1-christian.brauner@ubuntu.com> <20180203132904.11972-2-christian.brauner@ubuntu.com> <20180203111701.2ff8d7a3@xeon-e3> <0e8a86e6-5948-1e91-d937-893eb62ceef1@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <0e8a86e6-5948-1e91-d937-893eb62ceef1@gmail.com> User-Agent: Mutt/1.9.3 (2018-01-21) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Feb 03, 2018 at 07:09:55PM -0700, David Ahern wrote: > On 2/3/18 12:17 PM, Stephen Hemminger wrote: > > 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? > > > > extack needs to be filled in too. Yeah, it should report that an invalid network namespace identifier has been specified. Thanks! Christian