summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2017-10-05 15:53:47 +0300
committerDavid S. Miller <davem@davemloft.net>2017-10-05 21:28:53 -0700
commit1561b3266ebe029c487a95f92d1a58c03ded84a1 (patch)
treead44a7ff063a407ea42c2fb6b7587e18e1be9d1d
parent7a92616c0bac849e790283723b36c399668a1d9f (diff)
downloadlinux-1561b3266ebe029c487a95f92d1a58c03ded84a1.tar.bz2
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 <dan.carpenter@oracle.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--tools/testing/selftests/networking/timestamping/rxtimestamp.c2
1 files changed, 1 insertions, 1 deletions
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;