NFSv4: Remove redundant check for FMODE_EXEC in nfs_finish_open
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 29 May 2013 15:53:29 +0000 (11:53 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Thu, 6 Jun 2013 20:24:41 +0000 (16:24 -0400)
We already check the EXEC access mode in the lower layers.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/dir.c

index e093e73178b71467aa15eafa1d5327714eff284e..bf4e5f47d686731bd58e58e00f5ca78b6b4519bd 100644 (file)
@@ -1362,13 +1362,6 @@ static int nfs_finish_open(struct nfs_open_context *ctx,
                ctx->dentry = dget(dentry);
        }
 
-       /* If the open_intent is for execute, we have an extra check to make */
-       if (ctx->mode & FMODE_EXEC) {
-               err = nfs_may_open(dentry->d_inode, ctx->cred, open_flags);
-               if (err < 0)
-                       goto out;
-       }
-
        err = finish_open(file, dentry, do_open, opened);
        if (err)
                goto out;