diff options
| author |   David Miller <davem@davemloft.net> | 2015-04-07 23:05:42 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-04-08 15:20:50 -0400 |
| commit | c1f866767777d1c6abae0ec57effffcb72017c00 (patch) | |
| tree | b4d26d6d3f93a8a5129d0e2718f3a4a28f71c25c /net/bridge/netfilter | |
| parent | Merge branch 'selinux-nlmsg' (diff) | |
| download | linux-dev-c1f866767777d1c6abae0ec57effffcb72017c00.tar.xz linux-dev-c1f866767777d1c6abae0ec57effffcb72017c00.zip | |
netfilter: Fix switch statement warnings with recent gcc.
More recent GCC warns about two kinds of switch statement uses:
1) Switching on an enumeration, but not having an explicit case
statement for all members of the enumeration. To show the
compiler this is intentional, we simply add a default case
with nothing more than a break statement.
2) Switching on a boolean value. I think this warning is dumb
but nevertheless you get it wholesale with -Wswitch.
This patch cures all such warnings in netfilter.
Signed-off-by: David S. Miller <davem@davemloft.net>
Acked-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net/bridge/netfilter')
| -rw-r--r-- | net/bridge/netfilter/nft_reject_bridge.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/bridge/netfilter/nft_reject_bridge.c b/net/bridge/netfilter/nft_reject_bridge.c index 54a2fdf0f457..ae8141f409d9 100644 --- a/net/bridge/netfilter/nft_reject_bridge.c +++ b/net/bridge/netfilter/nft_reject_bridge.c @@ -371,6 +371,8 @@ static int nft_reject_bridge_dump(struct sk_buff *skb, if (nla_put_u8(skb, NFTA_REJECT_ICMP_CODE, priv->icmp_code)) goto nla_put_failure; break; + default: + break; } return 0; |