From owner-cvs-src-old@FreeBSD.ORG Tue Mar 23 01:37:15 2010 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2D947106564A for ; Tue, 23 Mar 2010 01:37:15 +0000 (UTC) (envelope-from rrs@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 1AEE38FC1D for ; Tue, 23 Mar 2010 01:37:15 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id o2N1bE9W027300 for ; Tue, 23 Mar 2010 01:37:14 GMT (envelope-from rrs@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id o2N1bEDV027299 for cvs-src-old@freebsd.org; Tue, 23 Mar 2010 01:37:14 GMT (envelope-from rrs@repoman.freebsd.org) Message-Id: <201003230137.o2N1bEDV027299@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to rrs@repoman.freebsd.org using -f From: Randall Stewart Date: Tue, 23 Mar 2010 01:36:50 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/netinet sctp_indata.c sctputil.c sctputil.h X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 23 Mar 2010 01:37:15 -0000 rrs 2010-03-23 01:36:50 UTC FreeBSD src repository Modified files: sys/netinet sctp_indata.c sctputil.c sctputil.h Log: SVN rev 205502 on 2010-03-23 01:36:50Z by rrs Fixes a bug where SACKs in the face of mapping_array expansion would break. Basically once we expanded the array we no longer had both mapping arrays in sync which the sack processing code depends on. This would mean we were randomly referring to memory that was probably not there. This mostly just gave us bad sack results going back to the peer. If INVARIENTS was on of course we would hit the panic routine in the sack_check call. We also add a print routine for the place where one would panic in invarients so one can see what the main mapping array holds. Reviewed by: tuexen@freebsd.org MFC after: 2 weeks Revision Changes Path 1.70 +1 -35 src/sys/netinet/sctp_indata.c 1.102 +33 -14 src/sys/netinet/sctputil.c 1.39 +1 -1 src/sys/netinet/sctputil.h