From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: Re: [patch] ipvs: off by one in set_sctp_state() Date: Mon, 22 Apr 2013 12:30:16 +0300 Message-ID: <20130422093016.GB8936@mwanda> References: <20130420112455.GA30198@elgon.mountain> <20130422041151.GM15680@verge.net.au> <51750016.5080600@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Julian Anastasov , Simon Horman , Wensong Zhang , Pablo Neira Ayuso , Patrick McHardy , "David S. Miller" , netdev@vger.kernel.org, lvs-devel@vger.kernel.org, netfilter-devel@vger.kernel.org, netfilter@vger.kernel.org, coreteam@netfilter.org To: Moahn Reddy Return-path: Received: from aserp1040.oracle.com ([141.146.126.69]:33153 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754561Ab3DVJaq (ORCPT ); Mon, 22 Apr 2013 05:30:46 -0400 Content-Disposition: inline In-Reply-To: <51750016.5080600@gmail.com> Sender: netfilter-devel-owner@vger.kernel.org List-ID: On Mon, Apr 22, 2013 at 02:47:10PM +0530, Moahn Reddy wrote: > Hi, > > I see there is no problem in the code regarding the state change. > And the thing why I took 255 in the sctp_events array is that as per > the sctp specification, the 255 message is reserved, so I thought 0 > to 254 messages are enough. The issue is can chunk_type in set_sctp_state() be 255? How is this prevented in the code? regards, dan carpenter