aboutsummaryrefslogtreecommitdiffstats
path: root/fs/ext4
diff options
context:
space:
mode:
authorAkira Fujita <a-fujita@rs.jp.nec.com>2011-10-20 18:56:10 -0400
committerTheodore Ts'o <tytso@mit.edu>2011-10-20 18:56:10 -0400
commit09e0834fb0ce1ea2a63885177015bd5d7d2bc22d (patch)
tree52f8efc4bd21cc93d27cfb9071decc6996065071 /fs/ext4
parentee90d57e20bd4749dda3b14397392b18b89dc3ef (diff)
ext4: fix deadlock in ext4_ordered_write_end()
If ext4_jbd2_file_inode() in ext4_ordered_write_end() fails for some reasons, this function returns to caller without unlocking the page. It leads to the deadlock, and the patch fixes this issue. Signed-off-by: Akira Fujita <a-fujita@rs.jp.nec.com> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
-rw-r--r--fs/ext4/inode.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c
index 081bb25a9ad3..ecb572591924 100644
--- a/fs/ext4/inode.c
+++ b/fs/ext4/inode.c
@@ -959,7 +959,11 @@ static int ext4_ordered_write_end(struct file *file,
959 ext4_orphan_add(handle, inode); 959 ext4_orphan_add(handle, inode);
960 if (ret2 < 0) 960 if (ret2 < 0)
961 ret = ret2; 961 ret = ret2;
962 } else {
963 unlock_page(page);
964 page_cache_release(page);
962 } 965 }
966
963 ret2 = ext4_journal_stop(handle); 967 ret2 = ext4_journal_stop(handle);
964 if (!ret) 968 if (!ret)
965 ret = ret2; 969 ret = ret2;