tests: fix out of bounds access in the rx-event test 62/12762/3
authorMichael Meffie <mmeffie@sinenomine.net>
Mon, 6 Nov 2017 22:37:46 +0000 (17:37 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Thu, 23 Nov 2017 01:50:47 +0000 (20:50 -0500)
commit50a3eb7b7ee94bffaadc98429bd404164e89ec7f
tree446e69c51c7551df18939336f9518be674000027
parent2ae84bf053fe66b73a2c77b5d71305bae2c17587
tests: fix out of bounds access in the rx-event test

Use the NUMEVENTS symbol which defines the array size instead of an
incorrect hard coded number when checking if a second event can be added
to be fired at the same time.  This fixes a potential out of bounds
access of the event test array.

Also update the comment which incorrectly mentions the incorrect number
of events in the test.

Change-Id: I4f993b42e53e7e6a42fa31302fd1baa70e9f5041
Reviewed-on: https://gerrit.openafs.org/12762
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
tests/rx/event-t.c