From owner-cvs-src-old@FreeBSD.ORG Mon May 4 20:48:18 2009 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 DD59710656DB for ; Mon, 4 May 2009 20:48:18 +0000 (UTC) (envelope-from emax@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 7CEB88FC19 for ; Mon, 4 May 2009 20:48:18 +0000 (UTC) (envelope-from emax@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id n44KmIxT023244 for ; Mon, 4 May 2009 20:48:18 GMT (envelope-from emax@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n44KmInd023243 for cvs-src-old@freebsd.org; Mon, 4 May 2009 20:48:18 GMT (envelope-from emax@repoman.freebsd.org) Message-Id: <200905042048.n44KmInd023243@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to emax@repoman.freebsd.org using -f From: Maksim Yevmenkin Date: Mon, 4 May 2009 20:48:07 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: RELENG_7 Subject: cvs commit: src/sys/kern uipc_sockbuf.c 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: Mon, 04 May 2009 20:48:20 -0000 emax 2009-05-04 20:48:07 UTC FreeBSD src repository Modified files: (Branch: RELENG_7) sys/kern uipc_sockbuf.c Log: SVN rev 191796 on 2009-05-04 20:48:07Z by emax MFC r191366 Fix sbappendrecord_locked(). The main problem is that sbappendrecord_locked() relies on sbcompress() to set sb_mbtail. This will not happen if sbappendrecord_locked() is called with mbuf chain made of exactly one mbuf (i.e. m0->m_next == NULL). In this case sbcompress() will be called with m == NULL and will do nothing. I'm not entirely sure if m == NULL is a valid argument for sbcompress(), and, it rather pointless to call it like that, but keep calling it so it can do SBLASTMBUFCHK(). PR: kern/126742 Investigated by: pluknet < pluknet -at- gmail -dot- com > No response from: freebsd-current@, freebsd-bluetooth@ Revision Changes Path 1.171.2.5 +2 -8 src/sys/kern/uipc_sockbuf.c