OSDN Git Service

NFSD: Remove redundant assignment to variable host_err
authorColin Ian King <colin.i.king@gmail.com>
Mon, 10 Oct 2022 20:24:23 +0000 (21:24 +0100)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 28 Nov 2022 17:54:44 +0000 (12:54 -0500)
Variable host_err is assigned a value that is never read, it is being
re-assigned a value in every different execution path in the following
switch statement. The assignment is redundant and can be removed.

Cleans up clang-scan warning:
warning: Value stored to 'host_err' is never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/vfs.c

index 849a720..d6d90d6 100644 (file)
@@ -1305,7 +1305,6 @@ nfsd_create_locked(struct svc_rqst *rqstp, struct svc_fh *fhp,
                iap->ia_mode &= ~current_umask();
 
        err = 0;
-       host_err = 0;
        switch (type) {
        case S_IFREG:
                host_err = vfs_create(&init_user_ns, dirp, dchild, iap->ia_mode, true);