diff options
author | Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com> | 2008-11-04 09:10:50 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2008-11-04 09:10:50 -0500 |
commit | ae2d9fb18e575ed37ffc241ece4bf68f0be4ae32 (patch) | |
tree | 89a063362d3cd6ee3ed6bd1f87515fdbaec71cca /fs/ext4/mballoc.c | |
parent | 2423840ded13e6d3b52d88aff8d033bb78fafd08 (diff) | |
download | op-kernel-dev-ae2d9fb18e575ed37ffc241ece4bf68f0be4ae32.zip op-kernel-dev-ae2d9fb18e575ed37ffc241ece4bf68f0be4ae32.tar.gz |
ext4: fix missing ext4_unlock_group in error path
If we try to free a block which is already freed, the code was
returning without first unlocking the group.
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/mballoc.c')
-rw-r--r-- | fs/ext4/mballoc.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c index dfe17a1..444ad99 100644 --- a/fs/ext4/mballoc.c +++ b/fs/ext4/mballoc.c @@ -4441,6 +4441,7 @@ ext4_mb_free_metadata(handle_t *handle, struct ext4_buddy *e4b, else if (block >= (entry->start_blk + entry->count)) n = &(*n)->rb_right; else { + ext4_unlock_group(sb, group); ext4_error(sb, __func__, "Double free of blocks %d (%d %d)\n", block, entry->start_blk, entry->count); |