selftests/bpf: Fix check_mtu using wrong variable type
authorJesper Dangaard Brouer <brouer@redhat.com>
Tue, 6 Jun 2023 11:30:47 +0000 (13:30 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 19 Jul 2023 14:21:07 +0000 (16:21 +0200)
commit54257a763443d2c1af1bf83dc9c5afcf0cfc3a23
tree4d4e79153bb94fdec0980df9e62ec1a6648eeae8
parentda79a0bc36c595dfb95d81ae36d1cd91b4eb55b2
selftests/bpf: Fix check_mtu using wrong variable type

[ Upstream commit 095641817e1bf6aa2560e025e47575188ee3edaf ]

Dan Carpenter found via Smatch static checker, that unsigned 'mtu_lo' is
never less than zero.

Variable mtu_lo should have been an 'int', because read_mtu_device_lo()
uses minus as error indications.

Fixes: b62eba563229 ("selftests/bpf: Tests using bpf_check_mtu BPF-helper")
Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Link: https://lore.kernel.org/bpf/168605104733.3636467.17945947801753092590.stgit@firesoul
Signed-off-by: Sasha Levin <sashal@kernel.org>
tools/testing/selftests/bpf/prog_tests/check_mtu.c