From cbaaf5a970a3488031aab5c79bd2cfba0b245c9d Mon Sep 17 00:00:00 2001 From: Joseph Hickey Date: Tue, 16 Jun 2020 12:30:25 -0400 Subject: [PATCH] Fix #505, warnings in time base API test Add requisite cast to printf. Also use sizeof() rather than hardcoded size of 12. --- src/tests/time-base-api-test/time-base-api-test.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/tests/time-base-api-test/time-base-api-test.c b/src/tests/time-base-api-test/time-base-api-test.c index cf8aee607..dfb5f1dba 100644 --- a/src/tests/time-base-api-test/time-base-api-test.c +++ b/src/tests/time-base-api-test/time-base-api-test.c @@ -103,11 +103,11 @@ void TestTimeBaseApi(void) /* Checking OS_MAX_TIMEBASES + 1 */ for ( int i = 0; i < OS_MAX_TIMEBASES; i++ ) { - snprintf(TimeBaseIter[i], 12, "TimeBase%d", i); + snprintf(TimeBaseIter[i], sizeof(TimeBaseIter[i]), "TimeBase%d", i); tbc_results[i] = OS_TimeBaseCreate(&tb_id[i], TimeBaseIter[i], 0); } TimeBaseNum = OS_MAX_TIMEBASES+1; - snprintf(overMaxTimeBase, 12, "TimeBase%d", TimeBaseNum); + snprintf(overMaxTimeBase, sizeof(overMaxTimeBase), "TimeBase%d", (int)TimeBaseNum); expected = OS_ERR_NO_FREE_IDS; actual= OS_TimeBaseCreate(&time_base_id, "overMaxTimeBase", 0); UtAssert_True(actual == expected, "OS_TimeBaseCreate() (%ld) == OS_ERR_NO_FREE_IDS", (long)actual);