atomic_open: take care of EEXIST in no-open case with O_CREAT|O_EXCL in fs/namei.c
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
33
fs/namei.c
33
fs/namei.c
@@ -2725,16 +2725,6 @@ static int atomic_open(struct nameidata *nd, struct dentry *dentry,
|
||||
goto out;
|
||||
}
|
||||
|
||||
acc_mode = op->acc_mode;
|
||||
if (WARN_ON(excl && !(*opened & FILE_CREATED)))
|
||||
*opened |= FILE_CREATED;
|
||||
|
||||
if (*opened & FILE_CREATED) {
|
||||
WARN_ON(!(open_flag & O_CREAT));
|
||||
fsnotify_create(dir, dentry);
|
||||
acc_mode = MAY_OPEN;
|
||||
}
|
||||
|
||||
if (error) { /* returned 1, that is */
|
||||
if (WARN_ON(file->f_path.dentry == DENTRY_NOT_SET)) {
|
||||
error = -EIO;
|
||||
@@ -2744,10 +2734,19 @@ static int atomic_open(struct nameidata *nd, struct dentry *dentry,
|
||||
dput(dentry);
|
||||
dentry = file->f_path.dentry;
|
||||
}
|
||||
WARN_ON(!dentry->d_inode && (*opened & FILE_CREATED));
|
||||
if (create_error && dentry->d_inode == NULL) {
|
||||
error = create_error;
|
||||
goto out;
|
||||
if (*opened & FILE_CREATED)
|
||||
fsnotify_create(dir, dentry);
|
||||
if (!dentry->d_inode) {
|
||||
WARN_ON(*opened & FILE_CREATED);
|
||||
if (create_error) {
|
||||
error = create_error;
|
||||
goto out;
|
||||
}
|
||||
} else {
|
||||
if (excl && !(*opened & FILE_CREATED)) {
|
||||
error = -EEXIST;
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
goto looked_up;
|
||||
}
|
||||
@@ -2756,6 +2755,12 @@ static int atomic_open(struct nameidata *nd, struct dentry *dentry,
|
||||
* We didn't have the inode before the open, so check open permission
|
||||
* here.
|
||||
*/
|
||||
acc_mode = op->acc_mode;
|
||||
if (*opened & FILE_CREATED) {
|
||||
WARN_ON(!(open_flag & O_CREAT));
|
||||
fsnotify_create(dir, dentry);
|
||||
acc_mode = MAY_OPEN;
|
||||
}
|
||||
error = may_open(&file->f_path, acc_mode, open_flag);
|
||||
if (error)
|
||||
fput(file);
|
||||
|
Reference in New Issue
Block a user