Up to [cvs.NetBSD.org] / src / sys / ufs / ufs
Request diff between arbitrary revisions
Default branch: MAIN
Current tag: thorpej_scsipi_base
Revision 1.33 / (download) - annotate - [select for diffs], Mon Feb 26 20:25:11 2001 UTC (23 years, 1 month ago) by fvdl
Branch: MAIN
CVS Tags: thorpej_scsipi_nbase,
thorpej_scsipi_beforemerge,
thorpej_scsipi_base,
thorpej-devvp-base3,
thorpej-devvp-base2,
thorpej-devvp-base,
thorpej-devvp,
pre-chs-ubcperf,
post-chs-ubcperf,
perseant-lfsv2-base,
perseant-lfsv2
Branch point for: thorpej-mips-cache,
nathanw_sa,
kqueue
Changes since 1.32: +5 -5
lines
Diff to previous 1.32 (colored)
Some bugfixes from rev 1.33 and 1.34 of this file in FreeBSD (some in effect cosmetic). Original FreeBSD commit messages: == date: 2000/03/15 07:18:15; author: mckusick; state: Exp; lines: +4 -4 Bug fixes for currently harmless bugs that could rise to bite the unwary if the code were called in slightly different ways. [...] 2) In ufs_lookup() there is an off-by-one error in the test that checks if dp->i_diroff is outside the range of the the current directory size. This is completely harmless, since the following while-loop condition 'dp->i_offset < endsearch' is never met, so the code immediately does a second pass starting at dp->i_offset = 0. 3) Again in ufs_lookup(), the condition in a sanity check is wrong for directories that are longer than one block. This bug means that the sanity check is only effective for small directories. Submitted by: Ian Dowse <iedowse@maths.tcd.ie> == date: 2000/03/09 18:54:59; author: dillon; state: Exp; lines: +2 -2 In the 'found' case for ufs_lookup() the underlying bp's data was being accessed after the bp had been releaed. A simple move of the brelse() solves the problem. Approved by: jkh Submitted by: Ian Dowse <iedowse@maths.tcd.ie> ==