From owner-cvs-all  Mon Jun  8 17:43:24 1998
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Received: (from majordom@localhost)
          by hub.freebsd.org (8.8.8/8.8.8) id RAA15175
          for cvs-all-outgoing; Mon, 8 Jun 1998 17:43:24 -0700 (PDT)
          (envelope-from owner-cvs-all@FreeBSD.ORG)
Received: from godzilla.zeta.org.au (godzilla.zeta.org.au [203.15.68.22])
          by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id RAA14993;
          Mon, 8 Jun 1998 17:41:21 -0700 (PDT)
          (envelope-from bde@godzilla.zeta.org.au)
Received: (from bde@localhost)
	by godzilla.zeta.org.au (8.8.7/8.8.7) id KAA11579;
	Tue, 9 Jun 1998 10:41:20 +1000
Date: Tue, 9 Jun 1998 10:41:20 +1000
From: Bruce Evans <bde@zeta.org.au>
Message-Id: <199806090041.KAA11579@godzilla.zeta.org.au>
To: cvs-all@FreeBSD.ORG, cvs-committers@FreeBSD.ORG, cvs-sys@FreeBSD.ORG,
        julian@FreeBSD.ORG
Subject: Re: cvs commit: src/sys/ufs/ufs ufs_vnops.c
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk

>  Modified files:
>    sys/ufs/ufs          ufs_vnops.c 
>  Log:
>  The version of the softdep changes in FreeBSD broke the
>  (doingdirectory && !newparent) case of ufs_rename().
>  rename("D1/X/", "D2/Y/") gives a wrong link count for D2.
>  
>  Submitted by: Bruce Evans <bde@zeta.org.au>
>  Reviewed by: Kirk McKusick <mckusick@McKusick.COM>

It actually broke the (doingdirectory && newparent) case, as
shown in the example.  D2's link count became -1 (if Y was the
only subdirectory in it to begin with), so it could not be
removed and it took an fsck to fix the error.

Bruce

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