diff options
| author | Jakub Kicinski <kuba@kernel.org> | 2024-02-07 02:18:23 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2024-02-10 22:38:19 +0100 |
| commit | 49d821064c44cb5ffdf272905236012ea9ce50e3 (patch) | |
| tree | fc2e0b73076a4a964b2963c9a2305b3f4cb3f8dd /tools | |
| parent | net: tls: fix use-after-free with partial reads and async decrypt (diff) | |
| download | linux-49d821064c44cb5ffdf272905236012ea9ce50e3.tar.xz linux-49d821064c44cb5ffdf272905236012ea9ce50e3.zip | |
selftests: tls: use exact comparison in recv_partial
This exact case was fail for async crypto and we weren't
catching it.
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/testing/selftests/net/tls.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tools/testing/selftests/net/tls.c b/tools/testing/selftests/net/tls.c index 7799e042a971..bc36c91c4480 100644 --- a/tools/testing/selftests/net/tls.c +++ b/tools/testing/selftests/net/tls.c @@ -1002,12 +1002,12 @@ TEST_F(tls, recv_partial) memset(recv_mem, 0, sizeof(recv_mem)); EXPECT_EQ(send(self->fd, test_str, send_len, 0), send_len); - EXPECT_NE(recv(self->cfd, recv_mem, strlen(test_str_first), - MSG_WAITALL), -1); + EXPECT_EQ(recv(self->cfd, recv_mem, strlen(test_str_first), + MSG_WAITALL), strlen(test_str_first)); EXPECT_EQ(memcmp(test_str_first, recv_mem, strlen(test_str_first)), 0); memset(recv_mem, 0, sizeof(recv_mem)); - EXPECT_NE(recv(self->cfd, recv_mem, strlen(test_str_second), - MSG_WAITALL), -1); + EXPECT_EQ(recv(self->cfd, recv_mem, strlen(test_str_second), + MSG_WAITALL), strlen(test_str_second)); EXPECT_EQ(memcmp(test_str_second, recv_mem, strlen(test_str_second)), 0); } |