netdev - Re: netfilter: nf_tables: add reject module for NFPROTO

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20140212142754.GA303@macbook.localnet>
Date:	Wed, 12 Feb 2014 14:27:54 +0000
From:	Patrick McHardy <kaber@...sh.net>
To:	Dave Jones <davej@...hat.com>
Cc:	netdev@...r.kernel.org, pablo@...filter.org
Subject: Re: netfilter: nf_tables: add reject module for NFPROTO_INET

On Wed, Feb 12, 2014 at 09:18:27AM -0500, Dave Jones wrote:
> On Tue, Feb 11, 2014 at 08:33:59PM +0000, Linux Kernel wrote:
>  > Gitweb:     http://git.kernel.org/linus/;a=commit;h=05513e9e33dbded8124567466a444d32173eecc6
>  > Commit:     05513e9e33dbded8124567466a444d32173eecc6
>  > Parent:     cc4723ca316742891954efa346298e7c747c0d17
>  > Author:     Patrick McHardy <kaber@...sh.net>
>  > AuthorDate: Wed Feb 5 15:03:39 2014 +0000
>  > Committer:  Pablo Neira Ayuso <pablo@...filter.org>
>  > CommitDate: Thu Feb 6 09:44:18 2014 +0100
>  > 
>  >     netfilter: nf_tables: add reject module for NFPROTO_INET
>  >     
>  >     Add a reject module for NFPROTO_INET. It does nothing but dispatch
>  >     to the AF-specific modules based on the hook family.
>  >     
>  >     Signed-off-by: Patrick McHardy <kaber@...sh.net>
>  >     Signed-off-by: Pablo Neira Ayuso <pablo@...filter.org>
>  > ---
> 
> ....
> 
>  > +static void nft_reject_inet_eval(const struct nft_expr *expr,
>  > +				 struct nft_data data[NFT_REG_MAX + 1],
>  > +				 const struct nft_pktinfo *pkt)
>  > +{
>  > +	switch (pkt->ops->pf) {
>  > +	case NFPROTO_IPV4:
>  > +		nft_reject_ipv4_eval(expr, data, pkt);
>  > +	case NFPROTO_IPV6:
>  > +		nft_reject_ipv6_eval(expr, data, pkt);
>  > +	}
>  > +}
> 
> Is the fallthrough intentional here, or is there a missing break ?

Crap. Thanks for catching this, I'll send a fix later.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html