If the lost+found directory is created by fsck, it will do a cacheino()

which sets the inoinfo's i_parent and i_dotdot to 0, but they never get
set to ROOTINO. This means that propagate will never find lost+found and
its descendents, subdirectories will remain DSTATE (instead of DFOUND)
even though they *are* correctly linked in, and pass4.c will try to
clear them unsuccessfully, thinking that there is no link count from the
DSTATE directory's parent. The result is that you need to run fsck twice
and get link count increasing errors (which are unexpected and fatal
when running in preen mode). The fix is to set i_parent and i_dotdot to
"parent" after the second cacheino() call in dir.c:allocdir().

Obtained from:	"Ethan Solomita" <ethan@geocast.com> (of the NetBSD Project)
This commit is contained in:
Kirk McKusick 2000-07-24 19:50:20 +00:00
parent 2eaaa4dea8
commit 91ea161570
3 changed files with 15 additions and 3 deletions

View File

@ -669,7 +669,8 @@ allocdir(parent, request, mode)
ino_t ino;
char *cp;
struct dinode *dp;
register struct bufarea *bp;
struct bufarea *bp;
struct inoinfo *inp;
struct dirtemplate *dirp;
ino = allocino(request, IFDIR|mode);
@ -704,6 +705,9 @@ allocdir(parent, request, mode)
return (0);
}
cacheino(dp, ino);
inp = getinoinfo(ino);
inp->i_parent = parent;
inp->i_dotdot = parent;
inoinfo(ino)->ino_state = inoinfo(parent)->ino_state;
if (inoinfo(ino)->ino_state == DSTATE) {
inoinfo(ino)->ino_linkcnt = dp->di_nlink;

View File

@ -669,7 +669,8 @@ allocdir(parent, request, mode)
ino_t ino;
char *cp;
struct dinode *dp;
register struct bufarea *bp;
struct bufarea *bp;
struct inoinfo *inp;
struct dirtemplate *dirp;
ino = allocino(request, IFDIR|mode);
@ -704,6 +705,9 @@ allocdir(parent, request, mode)
return (0);
}
cacheino(dp, ino);
inp = getinoinfo(ino);
inp->i_parent = parent;
inp->i_dotdot = parent;
inoinfo(ino)->ino_state = inoinfo(parent)->ino_state;
if (inoinfo(ino)->ino_state == DSTATE) {
inoinfo(ino)->ino_linkcnt = dp->di_nlink;

View File

@ -669,7 +669,8 @@ allocdir(parent, request, mode)
ino_t ino;
char *cp;
struct dinode *dp;
register struct bufarea *bp;
struct bufarea *bp;
struct inoinfo *inp;
struct dirtemplate *dirp;
ino = allocino(request, IFDIR|mode);
@ -704,6 +705,9 @@ allocdir(parent, request, mode)
return (0);
}
cacheino(dp, ino);
inp = getinoinfo(ino);
inp->i_parent = parent;
inp->i_dotdot = parent;
inoinfo(ino)->ino_state = inoinfo(parent)->ino_state;
if (inoinfo(ino)->ino_state == DSTATE) {
inoinfo(ino)->ino_linkcnt = dp->di_nlink;