hrev45682 adds 1 changeset to branch 'master' old head: d2be966d2b09930d393196da77292a278d435ef2 new head: c9e66680b9b025f38dd7f37549c61b3646975f82 overview: http://cgit.haiku-os.org/haiku/log/?qt=range&q=c9e6668+%5Ed2be966 ---------------------------------------------------------------------------- c9e6668: Incorrect checking of already set double indirect blocks Wrong variable usage in inner loop caused some double indirect stream runs to be checked twice when block size was smaller than DOUBLE_INDIRECT_ARRAY_SIZE, while some were incorrectly marked as unallocated in the bitmap. Signed-off-by: Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> [ ahenriksson <sausageboy@xxxxxxxxx> ] ---------------------------------------------------------------------------- Revision: hrev45682 Commit: c9e66680b9b025f38dd7f37549c61b3646975f82 URL: http://cgit.haiku-os.org/haiku/commit/?id=c9e6668 Author: ahenriksson <sausageboy@xxxxxxxxx> Date: Mon Jul 23 12:35:39 2012 UTC Committer: Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> Commit-Date: Tue May 14 20:13:50 2013 UTC ---------------------------------------------------------------------------- 1 file changed, 1 insertion(+), 1 deletion(-) src/add-ons/kernel/file_systems/bfs/BlockAllocator.cpp | 2 +- ---------------------------------------------------------------------------- diff --git a/src/add-ons/kernel/file_systems/bfs/BlockAllocator.cpp b/src/add-ons/kernel/file_systems/bfs/BlockAllocator.cpp index 68c4e03..a6aceca 100644 --- a/src/add-ons/kernel/file_systems/bfs/BlockAllocator.cpp +++ b/src/add-ons/kernel/file_systems/bfs/BlockAllocator.cpp @@ -2013,7 +2013,7 @@ BlockAllocator::_CheckInodeBlocks(Inode* inode, const char* name) fCheckCookie->control.stats.double_indirect_block_runs++; fCheckCookie->control.stats.blocks_in_double_indirect += runs[index % runsPerBlock].Length(); - } while ((++index % runsPerArray) != 0); + } while ((++index % runsPerBlock) != 0); } fCheckCookie->control.stats.double_indirect_array_blocks++;