Wu Fengguang wrote:
> On Mon, Nov 24, 2008 at 02:23:16PM +0200, Patrick McHardy wrote:
>> +	/* Avoid bogus warning, gcc doesn't realize do_basic_checks()
>> +	 * guarantees that there is at least one SCTP chunk.
>> +	 */
>> +	if (unlikely(new_state == SCTP_CONNTRACK_MAX))
>> +		goto out;
>> +
> 
> If do_basic_checks() guarantees that, why not Initialize new_state to 0?

Good point. I've replaced the patch by this one: