Skip to content

Commit 49d8210

Browse files
kuba-moodavem330
authored andcommitted
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>
1 parent 32b55c5 commit 49d8210

File tree

1 file changed

+4
-4
lines changed
  • tools/testing/selftests/net

1 file changed

+4
-4
lines changed

tools/testing/selftests/net/tls.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1002,12 +1002,12 @@ TEST_F(tls, recv_partial)
10021002

10031003
memset(recv_mem, 0, sizeof(recv_mem));
10041004
EXPECT_EQ(send(self->fd, test_str, send_len, 0), send_len);
1005-
EXPECT_NE(recv(self->cfd, recv_mem, strlen(test_str_first),
1006-
MSG_WAITALL), -1);
1005+
EXPECT_EQ(recv(self->cfd, recv_mem, strlen(test_str_first),
1006+
MSG_WAITALL), strlen(test_str_first));
10071007
EXPECT_EQ(memcmp(test_str_first, recv_mem, strlen(test_str_first)), 0);
10081008
memset(recv_mem, 0, sizeof(recv_mem));
1009-
EXPECT_NE(recv(self->cfd, recv_mem, strlen(test_str_second),
1010-
MSG_WAITALL), -1);
1009+
EXPECT_EQ(recv(self->cfd, recv_mem, strlen(test_str_second),
1010+
MSG_WAITALL), strlen(test_str_second));
10111011
EXPECT_EQ(memcmp(test_str_second, recv_mem, strlen(test_str_second)),
10121012
0);
10131013
}

0 commit comments

Comments
 (0)