Commit 97ac63b1 authored May 26, 2021 by Kees Cook Committed by Greg Kroah-Hartman Oct 10, 2023

selftests/tls: Add {} to avoid static checker warning



[ Upstream commit f50688b47c5858d2ff315d020332bf4cb6710837 ]

This silences a static checker warning due to the unusual macro
construction of EXPECT_*() by adding explicit {}s around the enclosing
while loop.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Fixes: 7f657d5b ("selftests: tls: add selftests for TLS sockets")
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Stable-dep-of: c326ca98446e ("selftests: tls: swap the TX and RX sockets in some tests")
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent a477402d

tools/testing/selftests/net/tls.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -202,8 +202,9 @@ TEST_F(tls, sendmsg_large)
		EXPECT_EQ(sendmsg(self->cfd, &msg, 0), send_len);
		}

		while (recvs++ < sends)
		while (recvs++ < sends) {
		EXPECT_NE(recv(self->fd, mem, send_len, 0), -1);
		}

		free(mem);
		}