commit c3c4f69424db0760239762d36d0b1b6ae524008b
author Al Viro <viro@zeniv.linux.org.uk> Sat Jun 23 22:49:45 2012 +0400
committer Al Viro <viro@zeniv.linux.org.uk> Sat Jul 14 16:35:50 2012 +0400
tree b8a6a5eb446d22604d362b7b718c2ff52c7812de
parent 85d7d618c17a09cfd824c1ad4483c19e6f9637ff

do_dentry_open(): close the race with mark_files_ro() in failure exit

we want to take it out of mark_files_ro() reach *before* we start
checking if we ought to drop write access.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

fs/open.c

diff --git a/fs/open.c b/fs/open.c
index 124ccb1..764cc9c 100644
--- a/fs/open.c
+++ b/fs/open.c
@@ -727,6 +727,7 @@
 
 cleanup_all:
 	fops_put(f->f_op);
+	file_sb_list_del(f);
 	if (f->f_mode & FMODE_WRITE) {
 		put_write_access(inode);
 		if (!special_file(inode->i_mode)) {
@@ -740,7 +741,6 @@
 			mnt_drop_write(f->f_path.mnt);
 		}
 	}
-	file_sb_list_del(f);
 cleanup_file:
 	path_put(&f->f_path);
 	f->f_path.mnt = NULL;