From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH net] bridge: fix igmp / mld query parsing Date: Tue, 3 May 2016 13:26:23 -0700 Message-ID: <20160503132623.59cf54fb@xeon-e3> References: <1462306734-4183-1-git-send-email-linus.luessing@c0d3.blue> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: Simon Wunderlich , netdev@vger.kernel.org, bridge@lists.linux-foundation.org, "David S . Miller" , linux-kernel@vger.kernel.org To: Linus =?UTF-8?B?TMO8c3Npbmc=?= Return-path: In-Reply-To: <1462306734-4183-1-git-send-email-linus.luessing@c0d3.blue> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: bridge-bounces@lists.linux-foundation.org Errors-To: bridge-bounces@lists.linux-foundation.org List-Id: netdev.vger.kernel.org On Tue, 3 May 2016 22:18:54 +0200 Linus L=C3=BCssing 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 =3D jiffies; > __be32 group; > + int offset =3D skb_transport_offset(skb); shouldn't this be unsigned?