diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2014-11-18 21:10:23 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2014-11-19 12:57:21 -0500 |
commit | 845409b49bcc8fe2ad9f1a948cc4f54c301b742d (patch) | |
tree | 511718e15cfec9c0809ba838f9e48dccd5772baf /fs | |
parent | 81295ce635a83f052497656348e9edc794b4cae6 (diff) | |
download | op-kernel-dev-845409b49bcc8fe2ad9f1a948cc4f54c301b742d.zip op-kernel-dev-845409b49bcc8fe2ad9f1a948cc4f54c301b742d.tar.gz |
gfs2_atomic_open(): simplify the use of finish_no_open()
In ->atomic_open(inode, dentry, file, opened) calling finish_no_open(file, NULL)
is equivalent to dget(dentry); return finish_no_open(file, dentry);
No need to open-code that...
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/gfs2/inode.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index ce0cf9a..6e29174 100644 --- a/fs/gfs2/inode.c +++ b/fs/gfs2/inode.c @@ -1249,11 +1249,8 @@ static int gfs2_atomic_open(struct inode *dir, struct dentry *dentry, if (d != NULL) dentry = d; if (dentry->d_inode) { - if (!(*opened & FILE_OPENED)) { - if (d == NULL) - dget(dentry); - return finish_no_open(file, dentry); - } + if (!(*opened & FILE_OPENED)) + return finish_no_open(file, d); dput(d); return 0; } |