From owner-freebsd-bugs@FreeBSD.ORG Wed Aug 13 08:40:14 2003 Return-Path: Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8C57037B41A for ; Wed, 13 Aug 2003 08:40:14 -0700 (PDT) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id 27C8243FA3 for ; Wed, 13 Aug 2003 08:40:14 -0700 (PDT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.12.9/8.12.9) with ESMTP id h7DFeEUp025045 for ; Wed, 13 Aug 2003 08:40:14 -0700 (PDT) (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.12.9/8.12.9/Submit) id h7DFeEKw025044; Wed, 13 Aug 2003 08:40:14 -0700 (PDT) Date: Wed, 13 Aug 2003 08:40:14 -0700 (PDT) Message-Id: <200308131540.h7DFeEKw025044@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: Ruslan Ermilov Subject: Re: bin/54854: cvs pserver sig11 crash on 4.8-R X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list Reply-To: Ruslan Ermilov List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Aug 2003 15:40:14 -0000 The following reply was made to PR bin/54854; it has been noted by GNATS. From: Ruslan Ermilov To: Scott Mitchell Cc: bug-followup@FreeBSD.org Subject: Re: bin/54854: cvs pserver sig11 crash on 4.8-R Date: Wed, 13 Aug 2003 18:31:23 +0300 On Wed, Aug 13, 2003 at 02:00:35AM -0700, Scott Mitchell wrote: > > Still seeing these crashes with a stock cvs 1.11.6 (ie. without any of the > FreeBSD extension). The stack traces are very similar - it's interesting > to note that the crash seems to be triggered by bad cleanup from a SIGPIPE, > which I guess is related to the client-server connection closing down... > > Anyway, it looks less likely that this is a FreeBSD problem, although I > still don't have any kind of solution other than using the binary from 4.6. > We're planning to swap out the RAM in this machine within the next couple > of weeks and run all the memory tests again, plus try all the versions of > cvs we've built, in case this changes anything. > See if this patch helps. %%% Index: contrib/cvs/src/server.c =================================================================== RCS file: /home/ncvs/src/contrib/cvs/src/server.c,v retrieving revision 1.13.2.5 diff -u -p -r1.13.2.5 server.c --- contrib/cvs/src/server.c 21 Jan 2003 22:26:46 -0000 1.13.2.5 +++ contrib/cvs/src/server.c 4 Mar 2003 16:47:06 -0000 @@ -4889,11 +4889,14 @@ server_cleanup (sig) * have generated any final output, we shut down BUF_TO_NET. */ - status = buf_shutdown (buf_from_net); - if (status != 0) - error (0, status, "shutting down buffer from client"); - buf_free (buf_from_net); - buf_from_net = NULL; + if (buf_from_net != NULL) + { + status = buf_shutdown (buf_from_net); + if (status != 0) + error (0, status, "shutting down buffer from client"); + buf_free (buf_from_net); + buf_from_net = NULL; + } } if (dont_delete_temp) %%% Cheers, -- Ruslan Ermilov Sysadmin and DBA, ru@sunbay.com Sunbay Software Ltd, ru@FreeBSD.org FreeBSD committer