summaryrefslogtreecommitdiffstats
path: root/net/bridge
diff options
context:
space:
mode:
authorKulikov Vasiliy <segooon@gmail.com>2010-07-15 08:47:33 +0000
committerDavid S. Miller <davem@davemloft.net>2010-07-15 20:27:58 -0700
commitbb7a0bd600ac2e09a8747ef89e692a2967ed8c97 (patch)
tree43ccfd0f128c53ee588bd538e27afc491e6cee1b /net/bridge
parent2540ddb5124f883bba3d4af5d8920d44dd66e794 (diff)
downloadop-kernel-dev-bb7a0bd600ac2e09a8747ef89e692a2967ed8c97.zip
op-kernel-dev-bb7a0bd600ac2e09a8747ef89e692a2967ed8c97.tar.gz
net: bridge: fix sign bug
ipv6_skip_exthdr() can return error code that is below zero. 'offset' is unsigned, so it makes no sense. ipv6_skip_exthdr() returns 'int' so we can painlessly change type of offset to int. Signed-off-by: Kulikov Vasiliy <segooon@gmail.com> Acked-by: Stephen Hemminger <shemminger@vyatta.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bridge')
-rw-r--r--net/bridge/br_multicast.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/bridge/br_multicast.c b/net/bridge/br_multicast.c
index 27ae946..85afcda 100644
--- a/net/bridge/br_multicast.c
+++ b/net/bridge/br_multicast.c
@@ -1435,7 +1435,7 @@ static int br_multicast_ipv6_rcv(struct net_bridge *br,
struct icmp6hdr *icmp6h;
u8 nexthdr;
unsigned len;
- unsigned offset;
+ int offset;
int err;
if (!pskb_may_pull(skb, sizeof(*ip6h)))
OpenPOWER on IntegriCloud