ext4: fix deadlock in ext4_ordered_write_end()
authorAkira Fujita <a-fujita@rs.jp.nec.com>
Thu, 20 Oct 2011 22:56:10 +0000 (18:56 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Thu, 20 Oct 2011 22:56:10 +0000 (18:56 -0400)
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>
fs/ext4/inode.c

index 081bb25a9ad3ceb442b10e0f7ec10c5fd36eec16..ecb5725919245a4ef10ec96cdafc3b13969b952a 100644 (file)
@@ -959,7 +959,11 @@ static int ext4_ordered_write_end(struct file *file,
                        ext4_orphan_add(handle, inode);
                if (ret2 < 0)
                        ret = ret2;
+       } else {
+               unlock_page(page);
+               page_cache_release(page);
        }
+
        ret2 = ext4_journal_stop(handle);
        if (!ret)
                ret = ret2;