diff options
author | David S. Miller <davem@davemloft.net> | 2011-04-17 17:06:15 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-04-17 17:06:15 -0700 |
commit | b169f6db40605d0907458d1ff78ceac2b194a44f (patch) | |
tree | 685955d2bb913f026612e5b200529179033d5e82 /net/ipv6 | |
parent | f3c85dd560f26ceae1351e6f83e83f1322761ead (diff) | |
download | op-kernel-dev-b169f6db40605d0907458d1ff78ceac2b194a44f.zip op-kernel-dev-b169f6db40605d0907458d1ff78ceac2b194a44f.tar.gz |
netfilter: ip6table_mangle: Fix set-but-unused variables.
The variable 'flowlabel' is set but unused in ip6t_mangle_out().
The intention here was to compare this key to the header value after
mangling, and trigger a route lookup on mismatch.
Make it so.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6')
-rw-r--r-- | net/ipv6/netfilter/ip6table_mangle.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/ipv6/netfilter/ip6table_mangle.c b/net/ipv6/netfilter/ip6table_mangle.c index 679a0a3..00d1917 100644 --- a/net/ipv6/netfilter/ip6table_mangle.c +++ b/net/ipv6/netfilter/ip6table_mangle.c @@ -64,7 +64,8 @@ ip6t_mangle_out(struct sk_buff *skb, const struct net_device *out) (memcmp(&ipv6_hdr(skb)->saddr, &saddr, sizeof(saddr)) || memcmp(&ipv6_hdr(skb)->daddr, &daddr, sizeof(daddr)) || skb->mark != mark || - ipv6_hdr(skb)->hop_limit != hop_limit)) + ipv6_hdr(skb)->hop_limit != hop_limit || + flowlabel != *((u_int32_t *)ipv6_hdr(skb)))) return ip6_route_me_harder(skb) == 0 ? ret : NF_DROP; return ret; |