Commit 357c52ff authored by Jakub Kicinski's avatar Jakub Kicinski
Browse files

selftests: net: netlink-dumps: validation checks 



The sanity checks are going to get silently cast to unsigned
and always pass. Cast the sizeof to signed size.

Reviewed-by: default avatarSimon Horman <horms@kernel.org>
Link: https://patch.msgid.link/20241115003248.733862-1-kuba@kernel.org


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 0de6a472

tools/testing/selftests/net/netlink-dumps.c

+2 −2
Original line number Diff line number Diff line
@@ -56,10 +56,10 @@ TEST(test_sanity) 
	ASSERT_EQ(n, sizeof(dump_policies));



	n = recv(netlink_sock, buf, sizeof(buf), MSG_DONTWAIT);

	ASSERT_GE(n, sizeof(struct nlmsghdr));

	ASSERT_GE(n, (ssize_t)sizeof(struct nlmsghdr));



	n = recv(netlink_sock, buf, sizeof(buf), MSG_DONTWAIT);

	ASSERT_GE(n, sizeof(struct nlmsghdr));

	ASSERT_GE(n, (ssize_t)sizeof(struct nlmsghdr));



	close(netlink_sock);

}