On Tue, 3 May 2016 22:18:54 +0200 Linus Lüssing <linus.luessing@xxxxxxxxx> wrote: > diff --git a/net/bridge/br_multicast.c b/net/bridge/br_multicast.c > index 03661d9..7105cdf 100644 > --- a/net/bridge/br_multicast.c > +++ b/net/bridge/br_multicast.c > @@ -1271,6 +1271,7 @@ static int br_ip4_multicast_query(struct net_bridge *br, > unsigned long max_delay; > unsigned long now = jiffies; > __be32 group; > + int offset = skb_transport_offset(skb); shouldn't this be unsigned?