nfsd: remove some dead code in nfsd_create_locked()
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 3 Aug 2016 19:05:00 +0000 (22:05 +0300)
committerJ. Bruce Fields <bfields@redhat.com>
Thu, 4 Aug 2016 21:11:53 +0000 (17:11 -0400)
We changed this around in f135af1041f ('nfsd: reorganize nfsd_create')
so "dchild" can't be an error pointer any more.  Also, dchild can't be
NULL here (and dput would already handle this even if it was).

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/vfs.c

index 6666188c11387ebd43852010e97b0d76564da254..ba944123167b92f3a7460d8acc66b02dc7c53575 100644 (file)
@@ -1141,7 +1141,7 @@ nfsd_create_locked(struct svc_rqst *rqstp, struct svc_fh *fhp,
                char *fname, int flen, struct iattr *iap,
                int type, dev_t rdev, struct svc_fh *resfhp)
 {
-       struct dentry   *dentry, *dchild = NULL;
+       struct dentry   *dentry, *dchild;
        struct inode    *dirp;
        __be32          err;
        __be32          err2;
@@ -1208,8 +1208,7 @@ nfsd_create_locked(struct svc_rqst *rqstp, struct svc_fh *fhp,
        if (!err)
                err = fh_update(resfhp);
 out:
-       if (dchild && !IS_ERR(dchild))
-               dput(dchild);
+       dput(dchild);
        return err;
 
 out_nfserr: