summaryrefslogtreecommitdiffstats
path: root/net/handshake
diff options
context:
space:
mode:
authorColin Ian King <colin.i.king@gmail.com>2024-04-15 12:07:13 +0200
committerJakub Kicinski <kuba@kernel.org>2024-04-17 02:14:55 +0200
commitc2b640529ec7595b4f82f1630da1ac9697a0c927 (patch)
treeb8272ac23ff5b5e785d82dbec783b1165030ff14 /net/handshake
parentnet: dsa: microchip: drop unneeded MODULE_ALIAS (diff)
downloadlinux-c2b640529ec7595b4f82f1630da1ac9697a0c927.tar.xz
linux-c2b640529ec7595b4f82f1630da1ac9697a0c927.zip
net/handshake: remove redundant assignment to variable ret
The variable is being assigned an value and then is being re-assigned a new value in the next statement. The assignment is redundant and can be removed. Cleans up clang scan build warning: net/handshake/tlshd.c:216:2: warning: Value stored to 'ret' is never read [deadcode.DeadStores] Signed-off-by: Colin Ian King <colin.i.king@gmail.com> Reviewed-by: Jason Xing <kerneljasonxing@gmail.com> Acked-by: Chuck Lever <chuck.lever@oracle.com> Link: https://lore.kernel.org/r/20240415100713.483399-1-colin.i.king@gmail.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'net/handshake')
-rw-r--r--net/handshake/tlshd.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/net/handshake/tlshd.c b/net/handshake/tlshd.c
index d697f68c598c..d6f52839827e 100644
--- a/net/handshake/tlshd.c
+++ b/net/handshake/tlshd.c
@@ -213,7 +213,6 @@ static int tls_handshake_accept(struct handshake_req *req,
if (!hdr)
goto out_cancel;
- ret = -EMSGSIZE;
ret = nla_put_s32(msg, HANDSHAKE_A_ACCEPT_SOCKFD, fd);
if (ret < 0)
goto out_cancel;