diff options
Diffstat (limited to 'fs/reiserfs/bitmap.c')
-rw-r--r-- | fs/reiserfs/bitmap.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/fs/reiserfs/bitmap.c b/fs/reiserfs/bitmap.c index a6f39fe2b664..edc8ef78b63f 100644 --- a/fs/reiserfs/bitmap.c +++ b/fs/reiserfs/bitmap.c | |||
@@ -513,9 +513,17 @@ static void __discard_prealloc(struct reiserfs_transaction_handle *th, | |||
513 | "inode has negative prealloc blocks count."); | 513 | "inode has negative prealloc blocks count."); |
514 | #endif | 514 | #endif |
515 | while (ei->i_prealloc_count > 0) { | 515 | while (ei->i_prealloc_count > 0) { |
516 | reiserfs_free_prealloc_block(th, inode, ei->i_prealloc_block); | 516 | b_blocknr_t block_to_free; |
517 | ei->i_prealloc_block++; | 517 | |
518 | /* | ||
519 | * reiserfs_free_prealloc_block can drop the write lock, | ||
520 | * which could allow another caller to free the same block. | ||
521 | * We can protect against it by modifying the prealloc | ||
522 | * state before calling it. | ||
523 | */ | ||
524 | block_to_free = ei->i_prealloc_block++; | ||
518 | ei->i_prealloc_count--; | 525 | ei->i_prealloc_count--; |
526 | reiserfs_free_prealloc_block(th, inode, block_to_free); | ||
519 | dirty = 1; | 527 | dirty = 1; |
520 | } | 528 | } |
521 | if (dirty) | 529 | if (dirty) |