diff options
author | mckusick <mckusick@FreeBSD.org> | 2000-03-15 07:18:15 +0000 |
---|---|---|
committer | mckusick <mckusick@FreeBSD.org> | 2000-03-15 07:18:15 +0000 |
commit | 5ce14e7844c41cb937305222fc007b540271f1bb (patch) | |
tree | 36d1867493a02631c715f3013fa5540ceac4fa42 /lib/libpthread/thread/thr_kern.c | |
parent | acdd0d6f535796a21982b0df6183c2a5b350906a (diff) | |
download | FreeBSD-src-5ce14e7844c41cb937305222fc007b540271f1bb.zip FreeBSD-src-5ce14e7844c41cb937305222fc007b540271f1bb.tar.gz |
Bug fixes for currently harmless bugs that could rise to bite
the unwary if the code were called in slightly different ways.
1) In ufs_bmaparray() the code for calculating 'runb' will stop one block
short of the first entry in an indirect block. i.e. if an indirect block
contains N block numbers b[0]..b[N-1] then the code will never check if
b[0] and b[1] are sequential. For reference, compare with the equivalent
code that deals with direct blocks.
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>
Diffstat (limited to 'lib/libpthread/thread/thr_kern.c')
0 files changed, 0 insertions, 0 deletions