PDK-4456 Misra fixes
authorPrasad Konnur <prasadkonnur@ti.com>
Wed, 18 Sep 2019 13:09:16 +0000 (18:39 +0530)
committerPrasad Konnur <prasadkonnur@ti.com>
Wed, 18 Sep 2019 13:09:42 +0000 (18:39 +0530)
Signed-off-by: Prasad Konnur <prasadkonnur@ti.com>
include/fvid2_utils.h
src/fvid2_utils.c
src/trace.c

index cd1aaf8666de8c5641be9120afcad362ce193cb4..f584a00c1c44b1f1af9dd97722b21d3c40c02c39 100755 (executable)
@@ -246,7 +246,7 @@ int32_t Fvid2Utils_deInit(void *arg);
  *
  *  \return                 None.
  */
-void Fvid2Utils_memset(void *mem, uint8_t ch, uint32_t byteCount);
+void Fvid2Utils_memset(void *mem, uint8_t ch, size_t byteCount);
 
 /**
  *  Fvid2Utils_memsetw
@@ -273,7 +273,7 @@ void *Fvid2Utils_memsetw(void *mem, uint32_t word, uint32_t wordCount);
  *
  *  \return                 None
  */
-void Fvid2Utils_memcpy(void *dest, const void *src, uint32_t byteCount);
+void Fvid2Utils_memcpy(void *dest, const void *src, size_t byteCount);
 
 /**
  *  Fvid2Utils_memcmp
@@ -765,7 +765,7 @@ static inline void *Fvid2Utils_alloc(Fvid2Utils_PoolParams *params,
     {
         if ((uint32_t) FALSE == params->flag[cnt])
         {
-            tempVal = ((uintptr_t) params->mem) + (params->elemSize * cnt);
+            tempVal = ((uintptr_t) params->mem) + (((uintptr_t) params->elemSize) * ((uintptr_t) cnt));
             allocMem          = (void *) (tempVal);
             params->flag[cnt] = (uint32_t) TRUE;
 
@@ -801,7 +801,7 @@ static inline int32_t Fvid2Utils_free(Fvid2Utils_PoolParams *params,
 
     for (cnt = 0U; cnt < params->numElem; cnt++)
     {
-        tempVal = ((uintptr_t) params->mem) + (params->elemSize * cnt);
+        tempVal = ((uintptr_t) params->mem) + (((uintptr_t)params->elemSize) * ((uintptr_t)cnt));
         if ((void *) (tempVal) == mem)
         {
             /* Check if the memory is already allocated */
index a5f47e1d3544f96d87ad8fce372d28ec50484e22..ad316f3ef6d88c293726021271d086328fcad8c2 100644 (file)
@@ -127,7 +127,7 @@ int32_t Fvid2Utils_deInit(void *arg)
  *
  *  \return                 The destination memory pointer
  */
-void Fvid2Utils_memset(void *mem, uint8_t ch, uint32_t byteCount)
+void Fvid2Utils_memset(void *mem, uint8_t ch, size_t byteCount)
 {
     /* NULL pointer check */
     GT_assert(Fvid2Trace, (NULL_PTR != mem));
@@ -175,7 +175,7 @@ void *Fvid2Utils_memsetw(void *mem, uint32_t word, uint32_t wordCount)
  *
  *  \return                 The destination memory pointer
  */
-void Fvid2Utils_memcpy(void *dest, const void *src, uint32_t byteCount)
+void Fvid2Utils_memcpy(void *dest, const void *src, size_t byteCount)
 {
     /* NULL pointer check */
     GT_assert(Fvid2Trace, (NULL_PTR != dest));
@@ -364,6 +364,7 @@ int32_t Fvid2Utils_linkUniqePriNode(Fvid2Utils_Handle handle,
 {
     Fvid2UtilsLinkListObj *llobj;
     int32_t rtnValue = FVID2_SOK;
+    uint32_t tempPriority = priority;
 
     /* NULL pointer check */
     GT_assert(Fvid2Trace, (NULL_PTR != handle));
@@ -385,7 +386,7 @@ int32_t Fvid2Utils_linkUniqePriNode(Fvid2Utils_Handle handle,
         {
             GT_assert(Fvid2Trace, FALSE);
         }
-        priority = llobj->priorityCnt;
+        tempPriority = llobj->priorityCnt;
     }
 
     if (FVID2UTILS_LLT_CIRCULAR == llobj->listType)
@@ -394,7 +395,7 @@ int32_t Fvid2Utils_linkUniqePriNode(Fvid2Utils_Handle handle,
             llobj->headNode,
             node,
             node->data,
-            priority);
+            tempPriority);
     }
     else if (FVID2UTILS_LLT_DOUBLE == llobj->listType)
     {
@@ -402,7 +403,7 @@ int32_t Fvid2Utils_linkUniqePriNode(Fvid2Utils_Handle handle,
             llobj,
             node,
             node->data,
-            priority,
+            tempPriority,
             (uint32_t) TRUE);
         if (rtnValue == FVID2_SOK)
         {
@@ -896,6 +897,7 @@ static Fvid2Utils_Node *dutilsLinkCircularPri(Fvid2Utils_Node *headNode,
                                               uint32_t         priority)
 {
     Fvid2Utils_Node *curNode;
+    Fvid2Utils_Node *tempHeadNode = headNode;
 
     /* NULL pointer check */
     GT_assert(Fvid2Trace, (NULL_PTR != node));
@@ -904,20 +906,20 @@ static Fvid2Utils_Node *dutilsLinkCircularPri(Fvid2Utils_Node *headNode,
     node->priority = priority;
 
     /* Check if this is the first node. */
-    if (NULL != headNode)
+    if (NULL != tempHeadNode)
     {
         /* Add to the list based on priority */
-        curNode = headNode;
+        curNode = tempHeadNode;
         do
         {
             /* Nodes with the same priority are always added to the bottom
              * of the existing nodes with same priority. */
             if (priority < curNode->priority)
             {
-                if (curNode == headNode)
+                if (curNode == tempHeadNode)
                 {
                     /* Adding to the top of the list */
-                    headNode = node;
+                    tempHeadNode = node;
                 }
                 break;
             }
@@ -926,7 +928,7 @@ static Fvid2Utils_Node *dutilsLinkCircularPri(Fvid2Utils_Node *headNode,
             curNode = curNode->next;
             /* Not supposed to have NULL in circular list */
             GT_assert(Fvid2Trace, (NULL_PTR != curNode));
-        } while (curNode != headNode);
+        } while (curNode != tempHeadNode);
         /* Check if we have come to the end of the list */
 
         /* Add the node before the current node as we have traversed one
@@ -943,12 +945,12 @@ static Fvid2Utils_Node *dutilsLinkCircularPri(Fvid2Utils_Node *headNode,
     else
     {
         /* First node. Add to as head node */
-        headNode   = node;
+        tempHeadNode   = node;
         node->next = node;
         node->prev = node;
     }
 
-    return (headNode);
+    return (tempHeadNode);
 }
 
 /**
@@ -1038,8 +1040,10 @@ static int32_t dutilsLinkDoublePri(Fvid2UtilsLinkListObj *llobj,
 static Fvid2Utils_Node *dutilsUnLinkCircularPri(Fvid2Utils_Node *headNode,
                                                 Fvid2Utils_Node *node)
 {
+    Fvid2Utils_Node *tempHeadNode = headNode;
+
     /* NULL pointer check */
-    GT_assert(Fvid2Trace, (NULL_PTR != headNode));
+    GT_assert(Fvid2Trace, (NULL_PTR != tempHeadNode));
     GT_assert(Fvid2Trace, (NULL_PTR != node));
 
     /* Not supposed to have NULL in circular list */
@@ -1050,21 +1054,21 @@ static Fvid2Utils_Node *dutilsUnLinkCircularPri(Fvid2Utils_Node *headNode,
     node->prev->next = node->next;
     node->next->prev = node->prev;
 
-    if (node == headNode)
+    if (node == tempHeadNode)
     {
         /* Unlinking head node */
-        if ((headNode == headNode->next) || (headNode == headNode->prev))
+        if ((tempHeadNode == tempHeadNode->next) || (tempHeadNode == tempHeadNode->prev))
         {
             /* Unlinking last node */
             GT_assert(Fvid2Trace,
-                      ((headNode == headNode->next) &&
-                       (headNode == headNode->prev)));
-            headNode = NULL;
+                      ((tempHeadNode == tempHeadNode->next) &&
+                       (tempHeadNode == tempHeadNode->prev)));
+            tempHeadNode = NULL;
         }
         else
         {
             /* Make next node as head node */
-            headNode = node->next;
+            tempHeadNode = node->next;
         }
     }
 
@@ -1072,7 +1076,7 @@ static Fvid2Utils_Node *dutilsUnLinkCircularPri(Fvid2Utils_Node *headNode,
     node->next = NULL;
     node->prev = NULL;
 
-    return (headNode);
+    return (tempHeadNode);
 }
 
 /**
index 7a98adf65af0ad2ca95cc57f8607fb74be724f3c..8b37b050e184386176b7df72fdd3cc0d80578bcf 100644 (file)
@@ -440,13 +440,14 @@ void GT_assertLocal(uint32_t    enableMask,
                     const char *fileName,
                     int32_t     lineNum)
 {
-    if ((!(condition)) != 0)
+    volatile uint32_t loop = 1U;
+    if (condition == 0U)
     {
         GT_3trace(
             Fvid2Trace, GT_ERR,
             " Assertion @ Line: %d in %s: %s : failed !!!\n",
             lineNum, fileName, str);
-        while (TRUE)
+        while (loop == 1U)
         {
             ;
         }