Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752468AbdLKQNz (ORCPT ); Mon, 11 Dec 2017 11:13:55 -0500 Received: from mx2.suse.de ([195.135.220.15]:46558 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750707AbdLKQNw (ORCPT ); Mon, 11 Dec 2017 11:13:52 -0500 Date: Mon, 11 Dec 2017 17:13:50 +0100 From: Michal Kubecek To: David Miller Cc: cernekee@chromium.org, johannes.berg@intel.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, daniel@iogearbox.net, dsahern@gmail.com Subject: Re: [PATCH V2] netlink: Add netns check on taps Message-ID: <20171211161350.s5ki6peee2oxpric@unicorn.suse.cz> References: <1512591147-29618-1-git-send-email-cernekee@chromium.org> <20171206.155714.908446002721815545.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171206.155714.908446002721815545.davem@davemloft.net> User-Agent: NeoMutt/20170421 (1.8.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1142 Lines: 32 On Wed, Dec 06, 2017 at 03:57:14PM -0500, David Miller wrote: > From: Kevin Cernekee > Date: Wed, 6 Dec 2017 12:12:27 -0800 > > > Currently, a nlmon link inside a child namespace can observe systemwide > > netlink activity. Filter the traffic so that nlmon can only sniff > > netlink messages from its own netns. > > > > Test case: > > > > vpnns -- bash -c "ip link add nlmon0 type nlmon; \ > > ip link set nlmon0 up; \ > > tcpdump -i nlmon0 -q -w /tmp/nlmon.pcap -U" & > > sudo ip xfrm state add src 10.1.1.1 dst 10.1.1.2 proto esp \ > > spi 0x1 mode transport \ > > auth sha1 0x6162633132330000000000000000000000000000 \ > > enc aes 0x00000000000000000000000000000000 > > grep --binary abc123 /tmp/nlmon.pcap > > > > Signed-off-by: Kevin Cernekee > > Applied and queued up for -stable, thanks Kevin. David, this patch is marked as accepted in patchworks and listed in http://patchwork.ozlabs.org/bundle/davem/stable/?state=* but it's not in the net tree. Is there a problem with it? Michal Kubecek