summaryrefslogtreecommitdiffstats
path: root/fs/namei.c
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2012-07-30 11:50:30 +0400
committerAl Viro <viro@zeniv.linux.org.uk>2012-07-30 11:50:30 +0400
commitf8310c59201b183ebee2e3fe0c7242f5729be0af (patch)
tree534606455dd90b2ec0ee9aa53bc4e51ce102b609 /fs/namei.c
parentbf8848918d751c1fb86f6514a75bf8d406f1c3c3 (diff)
downloadop-kernel-dev-f8310c59201b183ebee2e3fe0c7242f5729be0af.zip
op-kernel-dev-f8310c59201b183ebee2e3fe0c7242f5729be0af.tar.gz
fix O_EXCL handling for devices
O_EXCL without O_CREAT has different semantics; it's "fail if already opened", not "fail if already exists". commit 71574865 broke that... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/namei.c')
-rw-r--r--fs/namei.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/namei.c b/fs/namei.c
index 618d353..e133bf3 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -2418,7 +2418,7 @@ static int atomic_open(struct nameidata *nd, struct dentry *dentry,
if ((open_flag & O_CREAT) && !IS_POSIXACL(dir))
mode &= ~current_umask();
- if (open_flag & O_EXCL) {
+ if ((open_flag & (O_EXCL | O_CREAT)) == (O_EXCL | O_CREAT)) {
open_flag &= ~O_TRUNC;
*opened |= FILE_CREATED;
}
@@ -2742,7 +2742,7 @@ retry_lookup:
}
error = -EEXIST;
- if (open_flag & O_EXCL)
+ if ((open_flag & (O_EXCL | O_CREAT)) == (O_EXCL | O_CREAT))
goto exit_dput;
error = follow_managed(path, nd->flags);
OpenPOWER on IntegriCloud