selftests: tls: use exact comparison in recv_partial
[ Upstream commit 49d821064c44cb5ffdf272905236012ea9ce50e3 ] 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> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Sasha Levin

parent
90c445799f
commit
8e2113f61d
@@ -653,12 +653,12 @@ TEST_F(tls, recv_partial)
|
|||||||
|
|
||||||
memset(recv_mem, 0, sizeof(recv_mem));
|
memset(recv_mem, 0, sizeof(recv_mem));
|
||||||
EXPECT_EQ(send(self->fd, test_str, send_len, 0), send_len);
|
EXPECT_EQ(send(self->fd, test_str, send_len, 0), send_len);
|
||||||
EXPECT_NE(recv(self->cfd, recv_mem, strlen(test_str_first),
|
EXPECT_EQ(recv(self->cfd, recv_mem, strlen(test_str_first),
|
||||||
MSG_WAITALL), -1);
|
MSG_WAITALL), strlen(test_str_first));
|
||||||
EXPECT_EQ(memcmp(test_str_first, recv_mem, strlen(test_str_first)), 0);
|
EXPECT_EQ(memcmp(test_str_first, recv_mem, strlen(test_str_first)), 0);
|
||||||
memset(recv_mem, 0, sizeof(recv_mem));
|
memset(recv_mem, 0, sizeof(recv_mem));
|
||||||
EXPECT_NE(recv(self->cfd, recv_mem, strlen(test_str_second),
|
EXPECT_EQ(recv(self->cfd, recv_mem, strlen(test_str_second),
|
||||||
MSG_WAITALL), -1);
|
MSG_WAITALL), strlen(test_str_second));
|
||||||
EXPECT_EQ(memcmp(test_str_second, recv_mem, strlen(test_str_second)),
|
EXPECT_EQ(memcmp(test_str_second, recv_mem, strlen(test_str_second)),
|
||||||
0);
|
0);
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user