[haiku-commits] haiku: hrev45682 - src/add-ons/kernel/file_systems/bfs

  • From: axeld@xxxxxxxxxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 14 May 2013 22:16:35 +0200 (CEST)

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++;


Other related posts:

  • » [haiku-commits] haiku: hrev45682 - src/add-ons/kernel/file_systems/bfs - axeld