From: Colin Ian King <colin.king@xxxxxxxxxxxxx> 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@xxxxxxxxxxxxx> --- 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