Re: [PATCH net] selftests/net: rxtimestamp: Fix an off by one

[David Miller]

From: Dan Carpenter 
Date: Thu, 5 Oct 2017 15:53:47 +0300

> 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 

Applied.

[PATCH net] selftests/net: rxtimestamp: Fix an off by one

[Dan Carpenter]

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 

diff --git a/tools/testing/selftests/networking/timestamping/rxtimestamp.c 
b/tools/testing/selftests/networking/timestamping/rxtimestamp.c
index 00f286661dcd..dd4162fc0419 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;