From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pablo Neira Ayuso Subject: Re: [PATCH net-next 1/2] netfilter: xt_bpf: don't include linux/filter.h from uapi header Date: Tue, 29 Jul 2014 18:05:49 +0200 Message-ID: <20140729160549.GA4546@salvia> References: <1406648188-3681-1-git-send-email-pablo@netfilter.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netfilter-devel , David Miller , Network Development , Alexei Starovoitov , Daniel Borkmann , keescook@chromium.org To: Willem de Bruijn Return-path: Received: from mail.us.es ([193.147.175.20]:56115 "EHLO mail.us.es" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752559AbaG2QFi (ORCPT ); Tue, 29 Jul 2014 12:05:38 -0400 Content-Disposition: inline In-Reply-To: Sender: netfilter-devel-owner@vger.kernel.org List-ID: On Tue, Jul 29, 2014 at 11:43:57AM -0400, Willem de Bruijn wrote: > > In e6f30c7 ("netfilter: x_tables: add xt_bpf match"), the internal > > linux/filter.h header slipped through in the user exposed xt_bpf.h > > header as included file. > > is that true? #include should include > include/uapi/linux/filter.h in userspace builds. You're right. I'm going to send a v2. We should also cache the linux/filter.h header in the iptables sources, would you send a patch for that? Thanks.