diff options
| author | YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org> | 2010-03-13 12:27:21 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-03-13 12:27:21 -0800 |
| commit | bec68ff1637ca00bb1585a03a7be8a13380084de (patch) | |
| tree | 2e1f14359ab479d1429bbdfcecb2897b5fa4e667 /net/bridge/br_multicast.c | |
| parent | c251c7f738cd94eb3a1febda318078c661eccb4d (diff) | |
| download | lwn-bec68ff1637ca00bb1585a03a7be8a13380084de.tar.gz lwn-bec68ff1637ca00bb1585a03a7be8a13380084de.zip | |
bridge: ensure to unlock in error path in br_multicast_query().
Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bridge/br_multicast.c')
| -rw-r--r-- | net/bridge/br_multicast.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/net/bridge/br_multicast.c b/net/bridge/br_multicast.c index 12ce1eaa4f3e..fd96a8dc97f4 100644 --- a/net/bridge/br_multicast.c +++ b/net/bridge/br_multicast.c @@ -823,6 +823,7 @@ static int br_multicast_query(struct net_bridge *br, unsigned long max_delay; unsigned long now = jiffies; __be32 group; + int err = 0; spin_lock(&br->multicast_lock); if (!netif_running(br->dev) || @@ -841,12 +842,14 @@ static int br_multicast_query(struct net_bridge *br, group = 0; } } else { - if (!pskb_may_pull(skb, sizeof(struct igmpv3_query))) - return -EINVAL; + if (!pskb_may_pull(skb, sizeof(struct igmpv3_query))) { + err = -EINVAL; + goto out; + } ih3 = igmpv3_query_hdr(skb); if (ih3->nsrcs) - return 0; + goto out; max_delay = ih3->code ? 1 : IGMPV3_MRC(ih3->code) * (HZ / IGMP_TIMER_SCALE); @@ -876,7 +879,7 @@ static int br_multicast_query(struct net_bridge *br, out: spin_unlock(&br->multicast_lock); - return 0; + return err; } static void br_multicast_leave_group(struct net_bridge *br, |