From owner-cvs-all  Fri Jan 11 19:57:29 2002
Delivered-To: cvs-all@freebsd.org
Received: from freefall.freebsd.org (freefall.FreeBSD.org [216.136.204.21])
	by hub.freebsd.org (Postfix) with ESMTP
	id 36FCC37B405; Fri, 11 Jan 2002 19:57:26 -0800 (PST)
Received: (from iedowse@localhost)
	by freefall.freebsd.org (8.11.6/8.11.6) id g0C3vQx26915;
	Fri, 11 Jan 2002 19:57:26 -0800 (PST)
	(envelope-from iedowse)
Message-Id: <200201120357.g0C3vQx26915@freefall.freebsd.org>
From: Ian Dowse <iedowse@FreeBSD.org>
Date: Fri, 11 Jan 2002 19:57:26 -0800 (PST)
To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: cvs commit: src/sys/nfsserver nfs_serv.c
X-FreeBSD-CVS-Branch: HEAD
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

iedowse     2002/01/11 19:57:26 PST

  Modified files:
    sys/nfsserver        nfs_serv.c 
  Log:
  Fix a few NFSv2 issues that slipped in during the big cleanup. The
  semantics of the nfsm_reply() macro were changed so that the caller
  has to explicitly handle the V2 error case, whereas before,
  nfsm_reply() did a `goto nfsmout' then. A few server ops (setattr,
  readlink, create, mkdir) weren't updated to match, so errors in the
  V2 case could cause protocol hangs and leaked mbufs.
  
  Correct some comments that describe the old nfsm_reply behaviour.
  
  [older, harmless nit] Remove the unnecessary `nfsmreply0' label in
  nfsrv_create(), since for its users, the main `ereply' label does
  the same thing.
  
  Revision  Changes    Path
  1.112     +28 -33    src/sys/nfsserver/nfs_serv.c

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