Messages in this thread Patch in this message | | | From | Colin King <> | Subject | [PATCH] afs: remove redundant assignment to variable ret | Date | Wed, 22 Jul 2020 16:22:24 +0100 |
| |
From: Colin Ian King <colin.king@canonical.com>
The variable ret is being assigned a value that is never read because the next statment returns from the function. The assignment is redundant and can be removed.
Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King <colin.king@canonical.com> --- fs/afs/server.c | 1 - 1 file changed, 1 deletion(-)
diff --git a/fs/afs/server.c b/fs/afs/server.c index e82e452e2612..429cd226e4da 100644 --- a/fs/afs/server.c +++ b/fs/afs/server.c @@ -701,7 +701,6 @@ bool afs_check_server_record(struct afs_operation *op, struct afs_server *server retries++; if (retries == 4) { _leave(" = f [stale]"); - ret = -ESTALE; return false; } goto retry; -- 2.27.0
| |