diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-10-05 15:53:47 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-10-05 21:28:53 -0700 |
| commit | 1561b3266ebe029c487a95f92d1a58c03ded84a1 (patch) | |
| tree | ad44a7ff063a407ea42c2fb6b7587e18e1be9d1d /tools/testing | |
| parent | 7a92616c0bac849e790283723b36c399668a1d9f (diff) | |
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>
Diffstat (limited to 'tools/testing')
| -rw-r--r-- | tools/testing/selftests/networking/timestamping/rxtimestamp.c | 2 |
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; |