vfs: nameidata_to_filp(): don't throw away file on error
If open fails, don't put the file. This allows it to be reused if open needs to be retried. Signed-off-by: Miklos Szeredi <mszeredi@suse.cz> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
parent
91daee988d
commit
50ee93afca
1 changed files with 5 additions and 3 deletions
|
@ -824,10 +824,11 @@ struct file *nameidata_to_filp(struct nameidata *nd)
|
|||
|
||||
/* Pick up the filp from the open intent */
|
||||
filp = nd->intent.open.file;
|
||||
nd->intent.open.file = NULL;
|
||||
|
||||
/* Has the filesystem initialised the file for us? */
|
||||
if (filp->f_path.dentry == NULL) {
|
||||
if (filp->f_path.dentry != NULL) {
|
||||
nd->intent.open.file = NULL;
|
||||
} else {
|
||||
struct file *res;
|
||||
|
||||
path_get(&nd->path);
|
||||
|
@ -836,6 +837,7 @@ struct file *nameidata_to_filp(struct nameidata *nd)
|
|||
if (!IS_ERR(res)) {
|
||||
int error;
|
||||
|
||||
nd->intent.open.file = NULL;
|
||||
BUG_ON(res != filp);
|
||||
|
||||
error = open_check_o_direct(filp);
|
||||
|
@ -844,7 +846,7 @@ struct file *nameidata_to_filp(struct nameidata *nd)
|
|||
filp = ERR_PTR(error);
|
||||
}
|
||||
} else {
|
||||
put_filp(filp);
|
||||
/* Allow nd->intent.open.file to be recycled */
|
||||
filp = res;
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue