summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2012-06-23 22:49:45 +0400
committerAl Viro <viro@zeniv.linux.org.uk>2012-07-14 16:35:50 +0400
commitc3c4f69424db0760239762d36d0b1b6ae524008b (patch)
treeb8a6a5eb446d22604d362b7b718c2ff52c7812de
parent85d7d618c17a09cfd824c1ad4483c19e6f9637ff (diff)
downloadop-kernel-dev-c3c4f69424db0760239762d36d0b1b6ae524008b.zip
op-kernel-dev-c3c4f69424db0760239762d36d0b1b6ae524008b.tar.gz
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>
-rw-r--r--fs/open.c2
1 files changed, 1 insertions, 1 deletions
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 @@ static int do_dentry_open(struct file *f,
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 @@ cleanup_all:
mnt_drop_write(f->f_path.mnt);
}
}
- file_sb_list_del(f);
cleanup_file:
path_put(&f->f_path);
f->f_path.mnt = NULL;
OpenPOWER on IntegriCloud