From owner-cvs-all  Sat Apr 10 12: 6:36 1999
Delivered-To: cvs-all@freebsd.org
Received: from spinner.netplex.com.au (spinner.netplex.com.au [202.12.86.3])
	by hub.freebsd.org (Postfix) with ESMTP
	id E7C8214E07; Sat, 10 Apr 1999 12:06:31 -0700 (PDT)
	(envelope-from peter@spinner.netplex.com.au)
Received: from spinner.netplex.com.au (localhost [127.0.0.1])
	by spinner.netplex.com.au (Postfix) with ESMTP
	id 3418D1F4D; Sun, 11 Apr 1999 03:04:16 +0800 (WST)
	(envelope-from peter@spinner.netplex.com.au)
X-Mailer: exmh version 2.0.2 2/24/98
To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/nfs nfs_vfsops.c 
In-reply-to: Your message of "Sat, 10 Apr 1999 11:53:30 MST."
             <199904101853.LAA46628@freefall.freebsd.org> 
Date: Sun, 11 Apr 1999 03:04:15 +0800
From: Peter Wemm <peter@spinner.netplex.com.au>
Message-Id: <19990410190418.3418D1F4D@spinner.netplex.com.au>
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk

Peter Wemm wrote:
>   Modified files:
>     sys/nfs              nfs_vfsops.c 
>   Log:
>   Close a potential mbuf and/or mbuf cluster leak in the client-side NFS
>   statfs() code.  Free the whole chain, not just the first one.

The really annoying part is that I've spent 6 hours looking for the exact
opposite problem, ie: freeing something and keeping on using it.  The NFS
code is frightening with all those inter-related macros, scratch variables
and helper functions..

Cheers,
-Peter



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message