Fix nasa#2100, Resolve UT uninit var static analysis warnings

skliper · May 12, 2022 · e03f5e5 · e03f5e5
1 parent 81416f8
commit e03f5e5
Show file tree

Hide file tree

Showing 5 changed files with 171 additions and 122 deletions.
diff --git a/modules/es/ut-coverage/es_UT.c b/modules/es/ut-coverage/es_UT.c
@@ -558,6 +558,9 @@ int32 ES_UT_SetupOSCleanupHook(void *UserObj, int32 StubRetcode, uint32 CallCoun
      */
     if (CallCount == 0)
     {
+        /* Initialize to avoid static analysis warnings */
+        memset(ObjList, 0, sizeof(ObjList));
+
         OS_TaskCreate(&ObjList[0], NULL, NULL, OSAL_TASK_STACK_ALLOCATE, 0, 0, 0);
         OS_QueueCreate(&ObjList[1], NULL, 0, 0, 0);
         OS_MutSemCreate(&ObjList[2], NULL, 0);

diff --git a/modules/evs/ut-coverage/evs_UT.c b/modules/evs/ut-coverage/evs_UT.c
@@ -760,6 +760,8 @@ void Test_Ports(void)
     CFE_EVS_BitMaskCmd_t           bitmaskcmd;
     UT_SoftwareBusSnapshot_Entry_t LocalSnapshotData = {.MsgId = CFE_SB_MSGID_WRAP_VALUE(CFE_EVS_LONG_EVENT_MSG_MID)};
 
+    memset(&bitmaskcmd, 0, sizeof(bitmaskcmd));
+
     UtPrintf("Begin Test Ports");
 
     CFE_EVS_Global.EVS_TlmPkt.Payload.MessageFormatMode = CFE_EVS_MsgFormat_LONG;
@@ -1683,6 +1685,8 @@ void Test_InvalidCmd(void)
 {
     CFE_MSG_CommandHeader_t cmd;
 
+    memset(&cmd, 0, sizeof(cmd));
+
     UtPrintf("Begin Test Invalid Command");
 
     /* Test invalid msg id event */

diff --git a/modules/msg/ut-coverage/test_msg_pri_not.c b/modules/msg/ut-coverage/test_msg_pri_not.c
@@ -29,14 +29,14 @@
 
 unsigned int Test_MSG_Pri_NotZero(const CFE_MSG_Message_t *MsgPtr)
 {
-    unsigned int               bits = 0;
-    CFE_MSG_HeaderVersion_t    hdrver;
-    bool                       hassec;
-    CFE_MSG_Type_t             type;
-    CFE_MSG_ApId_t             apid;
-    CFE_MSG_SegmentationFlag_t segflag;
-    CFE_MSG_SequenceCount_t    seqcnt;
-    CFE_MSG_Size_t             size;
+    unsigned int               bits    = 0;
+    CFE_MSG_HeaderVersion_t    hdrver  = 0;
+    bool                       hassec  = false;
+    CFE_MSG_Type_t             type    = 0;
+    CFE_MSG_ApId_t             apid    = 0;
+    CFE_MSG_SegmentationFlag_t segflag = 0;
+    CFE_MSG_SequenceCount_t    seqcnt  = 0;
+    CFE_MSG_Size_t             size    = 0;
 
     CFE_MSG_GetHeaderVersion(MsgPtr, &hdrver);
     if (hdrver != 0)
@@ -65,14 +65,14 @@ unsigned int Test_MSG_Pri_NotZero(const CFE_MSG_Message_t *MsgPtr)
 
 unsigned int Test_MSG_Pri_NotF(const CFE_MSG_Message_t *MsgPtr)
 {
-    unsigned int               bits = 0;
-    CFE_MSG_HeaderVersion_t    hdrver;
-    bool                       hassec;
-    CFE_MSG_Type_t             type;
-    CFE_MSG_ApId_t             apid;
-    CFE_MSG_SegmentationFlag_t segflag;
-    CFE_MSG_SequenceCount_t    seqcnt;
-    CFE_MSG_Size_t             size;
+    unsigned int               bits    = 0;
+    CFE_MSG_HeaderVersion_t    hdrver  = 0;
+    bool                       hassec  = false;
+    CFE_MSG_Type_t             type    = 0;
+    CFE_MSG_ApId_t             apid    = 0;
+    CFE_MSG_SegmentationFlag_t segflag = 0;
+    CFE_MSG_SequenceCount_t    seqcnt  = 0;
+    CFE_MSG_Size_t             size    = 0;
 
     CFE_MSG_GetHeaderVersion(MsgPtr, &hdrver);
     if (hdrver != 0x7)