diff options
author | Akira Fujita <a-fujita@rs.jp.nec.com> | 2011-10-20 18:56:10 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2011-10-20 18:56:10 -0400 |
commit | 09e0834fb0ce1ea2a63885177015bd5d7d2bc22d (patch) | |
tree | 52f8efc4bd21cc93d27cfb9071decc6996065071 /fs/ext4 | |
parent | ee90d57e20bd4749dda3b14397392b18b89dc3ef (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.c | 4 |
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; |