diff options
| author | Donatas Abraitis <donatas.abraitis@gmail.com> | 2021-10-07 12:22:18 +0300 |
|---|---|---|
| committer | Donatas Abraitis <donatas.abraitis@gmail.com> | 2021-10-08 08:56:42 +0300 |
| commit | 1a8e5aea78aba2d38de0e7bc7bdbe0e4b5a6960d (patch) | |
| tree | d7bdf142989da2972b80d4a4432dad096206f6a5 | |
| parent | 89748f2704ee69177e5adb5ca08f5c1fd00bb50e (diff) | |
tests: Do not explicitly set the thread pointer to NULL
FRR should only ever use the appropriate THREAD_ON/THREAD_OFF
semantics. This is espacially true for the functions we
end up calling the thread for.
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
| -rw-r--r-- | tests/lib/test_timer_correctness.c | 1 | ||||
| -rw-r--r-- | tests/lib/test_timer_performance.c | 1 |
2 files changed, 0 insertions, 2 deletions
diff --git a/tests/lib/test_timer_correctness.c b/tests/lib/test_timer_correctness.c index 0ae9761b11..1756d87a6f 100644 --- a/tests/lib/test_timer_correctness.c +++ b/tests/lib/test_timer_correctness.c @@ -153,7 +153,6 @@ int main(int argc, char **argv) XFREE(MTYPE_TMP, timers[index]->arg); thread_cancel(&timers[index]); - timers[index] = NULL; timers_pending--; } diff --git a/tests/lib/test_timer_performance.c b/tests/lib/test_timer_performance.c index 23c044c7c1..f9d634b6a6 100644 --- a/tests/lib/test_timer_performance.c +++ b/tests/lib/test_timer_performance.c @@ -77,7 +77,6 @@ int main(int argc, char **argv) index = prng_rand(prng) % SCHEDULE_TIMERS; thread_cancel(&timers[index]); - timers[index] = NULL; } monotime(&tv_stop); |