Received: by 10.223.176.5 with SMTP id f5csp1090807wra; Sat, 3 Feb 2018 18:10:22 -0800 (PST) X-Google-Smtp-Source: AH8x226V6wdoFBS6xSPeJ1w4TYFgLIaXR04i+VfsDskSnVQPuKgwuxo6DX7dan4QqiqR2xpSBvee X-Received: by 10.101.74.5 with SMTP id s5mr9742366pgq.105.1517710222094; Sat, 03 Feb 2018 18:10:22 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1517710222; cv=none; d=google.com; s=arc-20160816; b=ZlnEmQxuJUz0RSeRVkyMBjJ5GB5HSklSdbd0Ctl8FD9zM4KFPf2EcX4t31qLoGgqcF R3hRo/HwHHu8Az8WkNhbznTatbXn+R1rthmzF2Oem4HrrUgWnF1WCHxK6JWZZxtgxs/b eqmPEu7OnKfKNWc36Ojc2P2D8IVD3BcrmiMgf7zfXHmYa267cj0c8SHzBZyI5R5W6t5p sTIAjxzeR7TKEsKf+ZCwF5dAJOlsrUaJdZmSGtooeU7MjYkXlQ8r2WcmvshoYkmHr7eR MgAm8qOD3FmiewSEeclx7zEYg2owe9WEOAvBjO9HgU8xn47OQ8KnNgHTLWciqUnwjEkZ 8p8A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature :arc-authentication-results; bh=r4S8pXBiIDPS2jh8Ln9Ema0hzPOsX8e+Ueft2qGI9h8=; b=vXP1VxK7HTQdAKWTeeXjLe0sgUnsSRPzX25DOuSJ/0FFJsPnK52mdOSUa0PeKDF/vg k4EbC/3Ei3qhMKE/xzt4bANiUzDokTsusFOlEh3Rj/1UgrrZWMzc4c+ajVMRdQvmcLhp 8aav/02FmUXDFwhBnrziEubwYUWj0fdybPqU1xM2egfQYIGZrMKZ14IrIsVec26PqiAu cwVLEADUPVGH0qraiXUvMatKFMzuNg5c4vOjQKt8TvVbzPE96vuh5fhVOhYM3UNp2s1K lTfyTAubOAURXLeVr4VoyeUY12nrVVWRBP5ypkoIEcL5pJ8mDkHyXdDOawkVX4+JXmdY bjvw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Hd2IB/ao; 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=pass (p=NONE sp=NONE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s196si1499996pgc.650.2018.02.03.18.10.06; Sat, 03 Feb 2018 18:10:22 -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; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Hd2IB/ao; 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=pass (p=NONE sp=NONE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751589AbeBDCJn (ORCPT + 99 others); Sat, 3 Feb 2018 21:09:43 -0500 Received: from mail-it0-f43.google.com ([209.85.214.43]:36951 "EHLO mail-it0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750738AbeBDCJj (ORCPT ); Sat, 3 Feb 2018 21:09:39 -0500 Received: by mail-it0-f43.google.com with SMTP id h129so12185078ita.2; Sat, 03 Feb 2018 18:09:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=r4S8pXBiIDPS2jh8Ln9Ema0hzPOsX8e+Ueft2qGI9h8=; b=Hd2IB/aoQ76zconvnVUtbvcSJc6DMdsQxxqiyxJ87IGXZcvM+pc3gCtIZ4acuj3LdD eaLJDZQqXLm6/DyKPpYqPOx/uRtsKPFgPyed0OkQ3nq490FQKzNI3nCKEPuDOVdTUq0q rQ/VN0f5f+zP6df96cQNUZTS13nH+19+RIEcuKwxPXvEAKmoZZ0xYRme4C/GNakjVuQp mY9W7ksU2tadEXcRI+gcPMBuaqzYJCZHBSWXA3yUfbGoesjvHzmg0RihOjd5lMpoh2Cm mg68cUJ0rXznXBq+IDDa5sz3c9i+avTKuQ9Zs0T8tTzcfprWW5xBgQac3ag4420EOg41 X7pg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=r4S8pXBiIDPS2jh8Ln9Ema0hzPOsX8e+Ueft2qGI9h8=; b=Yt4vR/R+nm16mk50zEdB/H3ZuUpy00mmDqESKeWzFbWXR1sJ8jpYfVi7CLPUFoa8Xg yUMqQR1h+VTc+vrWG2qpT0+IfC14Tgb7O1/owE36SnH0B36P0+7Tx+DxvPSMoGtFJ4eH t6QXBTWIrjtUaozIx29d/k5qE8j4vTlUqNzuLZ0Ay54kGhenVfIeUfkixB1m4D51I5V+ 7DMasuRYUlFphsNF8ZOhuLCHMu8o2I9fEHxR3IJGNvj18cnoMyESN/nzZbWH1mPfXz5p /lW/ntRB19kZ1dIhgK6OygI/jwQPaYCGiP/BEJQQzMcwEFEj9+diK6Wa0boMkM6F6geI dZfA== X-Gm-Message-State: AKwxytfYzj6sBR3klcKtIg+UDOD64Yhwhl0r8fyoQ0wjGjDeKYh3h6R/ xY7daxaVFO80BMkLtjjudtI= X-Received: by 10.36.105.17 with SMTP id e17mr52149775itc.84.1517710178912; Sat, 03 Feb 2018 18:09:38 -0800 (PST) Received: from dsa-mb.local ([2601:284:8202:1f70:cc21:745d:5abd:ae63]) by smtp.googlemail.com with ESMTPSA id d3sm3122648itf.39.2018.02.03.18.09.37 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 03 Feb 2018 18:09:38 -0800 (PST) Subject: Re: [PATCH net 1/1 v1] rtnetlink: require unique netns identifier To: Stephen Hemminger , Christian Brauner Cc: netdev@vger.kernel.org, ebiederm@xmission.com, jbenc@redhat.com, nicolas.dichtel@6wind.com, linux-kernel@vger.kernel.org, davem@davemloft.net References: <20180203132904.11972-1-christian.brauner@ubuntu.com> <20180203132904.11972-2-christian.brauner@ubuntu.com> <20180203111701.2ff8d7a3@xeon-e3> From: David Ahern Message-ID: <0e8a86e6-5948-1e91-d937-893eb62ceef1@gmail.com> Date: Sat, 3 Feb 2018 19:09:55 -0700 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <20180203111701.2ff8d7a3@xeon-e3> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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.