From owner-cvs-all@FreeBSD.ORG Mon Jul 2 19:22:23 2007 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 27A8916A468; Mon, 2 Jul 2007 19:22:23 +0000 (UTC) (envelope-from rrs@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [69.147.83.41]) by mx1.freebsd.org (Postfix) with ESMTP id 1563A13C46C; Mon, 2 Jul 2007 19:22:23 +0000 (UTC) (envelope-from rrs@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.13.8/8.13.8) with ESMTP id l62JMM8C088768; Mon, 2 Jul 2007 19:22:22 GMT (envelope-from rrs@repoman.freebsd.org) Received: (from rrs@localhost) by repoman.freebsd.org (8.13.8/8.13.8/Submit) id l62JMM84088767; Mon, 2 Jul 2007 19:22:22 GMT (envelope-from rrs) Message-Id: <200707021922.l62JMM84088767@repoman.freebsd.org> From: Randall Stewart Date: Mon, 2 Jul 2007 19:22:22 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: HEAD Cc: Subject: cvs commit: src/sys/netinet sctp_indata.c sctp_input.c sctp_input.h sctp_output.c sctp_pcb.c sctp_var.h sctputil.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 02 Jul 2007 19:22:23 -0000 rrs 2007-07-02 19:22:22 UTC FreeBSD src repository Modified files: sys/netinet sctp_indata.c sctp_input.c sctp_input.h sctp_output.c sctp_pcb.c sctp_var.h sctputil.c Log: - Consolidate the code that free's chunks to actually also call the sctp_free_remote_address() function. - Assure that when we allocate a chunk the whoTo is NULL, also when we free it and place it into the cache we NULL it (that way the consolidation code will always work). - Fix a small race, when a empty data holder is left on the stream out queue, and both sides do a shutdown, the empty data holder would prevent us from sending a SHUTDOWN-ACK and at the same time we never would cleanup the empty holder (since nothing was ever in queue). We now add a utility function that a) cleans up empty holders and b) properly determines if there are still pending data chunks on the stream out wheel. Approved by: re@freebsd.org (Ken Smith) Revision Changes Path 1.36 +15 -11 src/sys/netinet/sctp_indata.c 1.48 +64 -15 src/sys/netinet/sctp_input.c 1.7 +3 -0 src/sys/netinet/sctp_input.h 1.44 +10 -10 src/sys/netinet/sctp_output.c 1.46 +0 -1 src/sys/netinet/sctp_pcb.c 1.14 +5 -0 src/sys/netinet/sctp_var.h 1.50 +0 -6 src/sys/netinet/sctputil.c