From: Dan Carpenter Date: Thu, 5 Oct 2017 12:53:47 +0000 (+0300) Subject: selftests/net: rxtimestamp: Fix an off by one X-Git-Tag: v5.15~10263^2~18 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1561b3266ebe029c487a95f92d1a58c03ded84a1;p=platform%2Fkernel%2Flinux-starfive.git selftests/net: rxtimestamp: Fix an off by one The > should be >= so that we don't write one element beyond the end of the array. Fixes: 16e781224198 ("selftests/net: Add a test to validate behavior of rx timestamps") Signed-off-by: Dan Carpenter Signed-off-by: David S. Miller --- diff --git a/tools/testing/selftests/networking/timestamping/rxtimestamp.c b/tools/testing/selftests/networking/timestamping/rxtimestamp.c index 00f2866..dd4162f 100644 --- a/tools/testing/selftests/networking/timestamping/rxtimestamp.c +++ b/tools/testing/selftests/networking/timestamping/rxtimestamp.c @@ -341,7 +341,7 @@ int main(int argc, char **argv) return 0; case 'n': t = atoi(optarg); - if (t > ARRAY_SIZE(test_cases)) + if (t >= ARRAY_SIZE(test_cases)) error(1, 0, "Invalid test case: %d", t); all_tests = false; test_cases[t].enabled = true;