From 05af21e98d737826b12a654d94c1b61e57edcb38 Mon Sep 17 00:00:00 2001 From: Jacob Hageman Date: Mon, 16 May 2022 10:20:44 -0600 Subject: [PATCH] Fix #342, Resolve uninit var static analysis warning --- fsw/pc-linux/src/cfe_psp_start.c | 2 ++ .../timebase_vxworks/src/coveragetest-timebase_vxworks.c | 8 ++++++++ 2 files changed, 10 insertions(+) diff --git a/fsw/pc-linux/src/cfe_psp_start.c b/fsw/pc-linux/src/cfe_psp_start.c index a7c906f3..82b2a9ce 100644 --- a/fsw/pc-linux/src/cfe_psp_start.c +++ b/fsw/pc-linux/src/cfe_psp_start.c @@ -153,6 +153,8 @@ int32 CFE_PSP_OS_EventHandler(OS_Event_t event, osal_id_t object_id, void *data) { char taskname[OS_MAX_API_NAME]; + memset(taskname, 0, sizeof(taskname)); + switch (event) { case OS_EVENT_RESOURCE_ALLOCATED: diff --git a/unit-test-coverage/modules/timebase_vxworks/src/coveragetest-timebase_vxworks.c b/unit-test-coverage/modules/timebase_vxworks/src/coveragetest-timebase_vxworks.c index b172ffad..4f09d71b 100644 --- a/unit-test-coverage/modules/timebase_vxworks/src/coveragetest-timebase_vxworks.c +++ b/unit-test-coverage/modules/timebase_vxworks/src/coveragetest-timebase_vxworks.c @@ -82,6 +82,8 @@ void Test_Nominal(void) OS_time_t OsTime; PSP_VxWorks_TimeBaseVal_t VxTime; + memset(&OsTime, 0, sizeof(OsTime)); + /* Nominal test with a simple 1:1 ratio */ UT_PSP_TIMEBASE_VXWORKS_TESTCONFIG.PeriodNumerator = 1; UT_PSP_TIMEBASE_VXWORKS_TESTCONFIG.PeriodDenominator = 1; @@ -103,6 +105,8 @@ void Test_Non_Reducible(void) PSP_VxWorks_TimeBaseVal_t VxTime; int64 TestTime; + memset(&OsTime, 0, sizeof(OsTime)); + /* Use an oddball ratio of of some primes, will not be reducible */ /* Ratio is 43*3 / 53*2 => 129/106 */ /* This translates to about ~1.217ns per tick */ @@ -136,6 +140,8 @@ void Test_Reducible_1(void) PSP_VxWorks_TimeBaseVal_t VxTime; int64 TestTime; + memset(&OsTime, 0, sizeof(OsTime)); + /* Test with a ratio that is also 1:1, but can be reduced */ UT_PSP_TIMEBASE_VXWORKS_TESTCONFIG.PeriodNumerator = 1000; UT_PSP_TIMEBASE_VXWORKS_TESTCONFIG.PeriodDenominator = 1000; @@ -164,6 +170,8 @@ void Test_Reducible_2(void) PSP_VxWorks_TimeBaseVal_t VxTime; int64 TestTime; + memset(&OsTime, 0, sizeof(OsTime)); + /* Test with a ratio that can be reduced */ /* Final reduced ratio should be 12:5 with 100ns OS ticks */ UT_PSP_TIMEBASE_VXWORKS_TESTCONFIG.PeriodNumerator = 84000;