selftests: net: tls: remove unused variable and code
authorAnders Roxell <anders.roxell@linaro.org>
Fri, 5 Nov 2021 16:45:11 +0000 (17:45 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sun, 7 Nov 2021 19:35:08 +0000 (19:35 +0000)
When building selftests/net with clang, the compiler warn about the
function abs() see below:

tls.c:657:15: warning: variable 'len_compared' set but not used [-Wunused-but-set-variable]
        unsigned int len_compared = 0;
                     ^

Rework to remove the unused variable and the for-loop where the variable
'len_compared' was assinged.

Fixes: 7f657d5bf507 ("selftests: tls: add selftests for TLS sockets")
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/tls.c

index d3047e2..e61fc4c 100644 (file)
@@ -654,7 +654,6 @@ TEST_F(tls, recvmsg_single_max)
 TEST_F(tls, recvmsg_multiple)
 {
        unsigned int msg_iovlen = 1024;
-       unsigned int len_compared = 0;
        struct iovec vec[1024];
        char *iov_base[1024];
        unsigned int iov_len = 16;
@@ -675,8 +674,6 @@ TEST_F(tls, recvmsg_multiple)
        hdr.msg_iovlen = msg_iovlen;
        hdr.msg_iov = vec;
        EXPECT_NE(recvmsg(self->cfd, &hdr, 0), -1);
-       for (i = 0; i < msg_iovlen; i++)
-               len_compared += iov_len;
 
        for (i = 0; i < msg_iovlen; i++)
                free(iov_base[i]);