diff options
author | Petri Savolainen <petri.savolainen@nokia.com> | 2021-03-31 17:14:27 +0300 |
---|---|---|
committer | Petri Savolainen <petri.savolainen@nokia.com> | 2021-04-01 09:36:50 +0300 |
commit | cab6899faa97388d3256d6a307302469b550db00 (patch) | |
tree | ef1f45caefbff622a9c5b69928d19ac5f50215cd /example/timer | |
parent | dc00a1fe3ce167c15d9e1f7c64541712ade0c902 (diff) |
example: timer: check function return value
Check timer capability return value and exit on failure.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Diffstat (limited to 'example/timer')
-rw-r--r-- | example/timer/odp_timer_test.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/example/timer/odp_timer_test.c b/example/timer/odp_timer_test.c index a6c8dc73f..ddb2ccfd5 100644 --- a/example/timer/odp_timer_test.c +++ b/example/timer/odp_timer_test.c @@ -253,7 +253,7 @@ static void print_usage(void) * @param argv Argument vector * @param args Test arguments */ -static void parse_args(int argc, char *argv[], test_args_t *args) +static int parse_args(int argc, char *argv[], test_args_t *args) { int opt; int long_index; @@ -273,7 +273,8 @@ static void parse_args(int argc, char *argv[], test_args_t *args) static const char *shortopts = "+c:r:m:x:p:t:h"; /* defaults */ - odp_timer_capability(ODP_CLOCK_CPU, &timer_capa); + if (odp_timer_capability(ODP_CLOCK_CPU, &timer_capa)) + return -1; args->cpu_count = 1; args->resolution_us = MAX(10000, @@ -321,8 +322,9 @@ static void parse_args(int argc, char *argv[], test_args_t *args) if (args->period_us < args->resolution_us) printf("\n\tWarn: timeout is set less then resolution\n"); -} + return 0; +} /** * Test main function @@ -394,7 +396,10 @@ int main(int argc, char *argv[]) gbls->pool = ODP_POOL_INVALID; gbls->tp = ODP_TIMER_POOL_INVALID; - parse_args(argc, argv, &gbls->args); + if (parse_args(argc, argv, &gbls->args)) { + ODPH_ERR("Parse args failed.\n"); + goto err; + } memset(thread_tbl, 0, sizeof(thread_tbl)); |