diff options
author | Kulikov Vasiliy <segooon@gmail.com> | 2010-07-15 08:47:33 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-07-15 20:27:58 -0700 |
commit | bb7a0bd600ac2e09a8747ef89e692a2967ed8c97 (patch) | |
tree | 43ccfd0f128c53ee588bd538e27afc491e6cee1b /net/bridge | |
parent | 2540ddb5124f883bba3d4af5d8920d44dd66e794 (diff) | |
download | op-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.c | 2 |
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))) |