Skip to content

Commit 6869668

Browse files
rluboskartben
authored andcommitted
tests: net: socket: af_packet: Fix test_raw_and_dgram_socket_recv test
The test case did not do what it said, it was supposed to receive data on both RAW and DGRAM packet sockets, but instead it received on RAW socket twice. Signed-off-by: Robert Lubos <[email protected]>
1 parent 9f6ac50 commit 6869668

File tree

1 file changed

+4
-4
lines changed
  • tests/net/socket/af_packet/src

1 file changed

+4
-4
lines changed

tests/net/socket/af_packet/src/main.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -475,7 +475,7 @@ ZTEST(socket_packet, test_raw_and_dgram_socket_recv)
475475

476476
prepare_packet_socket(&packet_sock_1, ud.first, SOCK_DGRAM, htons(ETH_P_ALL));
477477
prepare_packet_socket(&packet_sock_2, ud.second, SOCK_RAW, htons(ETH_P_ALL));
478-
prepare_packet_socket(&packet_sock_3, ud.second, SOCK_RAW, htons(ETH_P_ALL));
478+
prepare_packet_socket(&packet_sock_3, ud.second, SOCK_DGRAM, htons(ETH_P_ALL));
479479

480480
memset(&dst, 0, sizeof(dst));
481481
dst.sll_family = AF_PACKET;
@@ -506,10 +506,10 @@ ZTEST(socket_packet, test_raw_and_dgram_socket_recv)
506506
errno = 0;
507507
ret = zsock_recvfrom(packet_sock_3, data_to_receive, sizeof(data_to_receive),
508508
0, (struct sockaddr *)&src, &addrlen);
509-
zassert_equal(ret, sizeof(expected_payload_raw),
509+
zassert_equal(ret, sizeof(data_to_send),
510510
"Cannot receive all data (%d)", -errno);
511-
zassert_mem_equal(expected_payload_raw, data_to_receive,
512-
sizeof(expected_payload_raw), "Data mismatch");
511+
zassert_mem_equal(data_to_send, data_to_receive, sizeof(data_to_send),
512+
"Data mismatch");
513513
}
514514

515515
static void test_sockets_close(void)

0 commit comments

Comments
 (0)