d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

MFufs 1.33:

Browse Source 
In the 'found' case for ext2_lookup() the underlying bp's data was
    being accessed after the bp had been releaed.  A simple move of the
    brelse() solves the problem.

The PR reports that this caused panics running the GDB testsuite unless
NO_GEOM is configured.

PR:		44060
Reported by:	Mark Kettenis <kettenis@chello.nl>
MFC after:	3 days
This commit is contained in:
Bruce Evans 2002-10-18 21:41:41 +00:00
parent 1a6b414672
commit eb5b0bc42e
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=105420
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sys/gnu/ext2fs/ext2_lookup.c
View File

@ -474,7 +474,6 @@ ext2_lookup(ap)
*/
dp->i_ino = ep->inode;
dp->i_reclen = ep->rec_len;
brelse(bp);
goto found;
}
}
@ -569,6 +568,7 @@ ext2_lookup(ap)
dp->i_size = entryoffsetinblock+EXT2_DIR_REC_LEN(ep->name_len);
dp->i_flag |= IN_CHANGE | IN_UPDATE;
}
brelse(bp);
/*
* Found component in pathname.

2
sys/gnu/fs/ext2fs/ext2_lookup.c
View File

@ -474,7 +474,6 @@ ext2_lookup(ap)
*/
dp->i_ino = ep->inode;
dp->i_reclen = ep->rec_len;
brelse(bp);
goto found;
}
}
@ -569,6 +568,7 @@ ext2_lookup(ap)
dp->i_size = entryoffsetinblock+EXT2_DIR_REC_LEN(ep->name_len);
dp->i_flag |= IN_CHANGE | IN_UPDATE;
}
brelse(bp);
/*
* Found component in pathname.