From 7d1da22c828ef92a2761b17d0e2bd4d3e61ddee7 Mon Sep 17 00:00:00 2001
From: temi54c1l8 <temi54c1l8@gmail.com>
Date: Wed, 26 Jun 2024 16:42:19 +0200
Subject: [PATCH] CO_SDOclient: static analysis: essential type of condition of
 'if' statement is 'unsigned8' but should be boolean [MISRA 2012 Rule 14.4,
 required]

---
 301/CO_SDOclient.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/301/CO_SDOclient.c b/301/CO_SDOclient.c
index 3a7fbb7..e545aa1 100644
--- a/301/CO_SDOclient.c
+++ b/301/CO_SDOclient.c
@@ -1389,11 +1389,11 @@ CO_SDO_return_t CO_SDOclientUpload(CO_SDOclient_t *SDO_C,
                         break;
                     }
 
-                    if (SDO_C->CANrxData[0] & 0x02U) {
+                    if ((SDO_C->CANrxData[0] & 0x02U) != 0U) {
                         /* Expedited transfer */
                         size_t count = 4;
                         /* is size indicated? */
-                        if (SDO_C->CANrxData[0] & 0x01U) {
+                        if ((SDO_C->CANrxData[0] & 0x01U) != 0U) {
                             count -= (SDO_C->CANrxData[0] >> 2) & 0x03U;
                         }
                         /* copy data, indicate size and finish */
@@ -1407,7 +1407,7 @@ CO_SDO_return_t CO_SDOclientUpload(CO_SDOclient_t *SDO_C,
                     else {
 #if ((CO_CONFIG_SDO_CLI) & CO_CONFIG_SDO_CLI_SEGMENTED) != 0
                         /* segmented transfer, is size indicated? */
-                        if (SDO_C->CANrxData[0] & 0x01U) {
+                        if ((SDO_C->CANrxData[0] & 0x01U) != 0U) {
                             uint32_t size;
                             (void)memcpy(&size, &SDO_C->CANrxData[4], sizeof(size));
                             SDO_C->sizeInd = CO_SWAP_32(size);
@@ -1465,7 +1465,7 @@ CO_SDO_return_t CO_SDOclientUpload(CO_SDOclient_t *SDO_C,
                     }
 
                     /* If no more segments to be upload, finish */
-                    if (SDO_C->CANrxData[0] & 0x01U) {
+                    if ((SDO_C->CANrxData[0] & 0x01U) != 0U) {
                         /* verify size of data uploaded */
                         if ((SDO_C->sizeInd > 0U)
                             && (SDO_C->sizeTran < SDO_C->sizeInd)
@@ -1501,7 +1501,7 @@ CO_SDO_return_t CO_SDOclientUpload(CO_SDOclient_t *SDO_C,
                     } else {
                         SDO_C->block_crcEnabled = false;
                     }
-                    if (SDO_C->CANrxData[0] & 0x02U) {
+                    if ((SDO_C->CANrxData[0] & 0x02U) != 0U) {
                         uint32_t size;
                         (void)memcpy(&size, &SDO_C->CANrxData[4], sizeof(size));
                         SDO_C->sizeInd = CO_SWAP_32(size);
@@ -1533,11 +1533,11 @@ CO_SDO_return_t CO_SDOclientUpload(CO_SDOclient_t *SDO_C,
                         break;
                     }
 
-                    if (SDO_C->CANrxData[0] & 0x02U) {
+                    if ((SDO_C->CANrxData[0] & 0x02U) != 0U) {
                         /* Expedited transfer */
                         size_t count = 4;
                         /* is size indicated? */
-                        if (SDO_C->CANrxData[0] & 0x01U) {
+                        if ((SDO_C->CANrxData[0] & 0x01U) != 0U) {
                             count -= (SDO_C->CANrxData[0] >> 2) & 0x03U;
                         }
                         /* copy data, indicate size and finish */
@@ -1550,7 +1550,7 @@ CO_SDO_return_t CO_SDOclientUpload(CO_SDOclient_t *SDO_C,
                     }
                     else {
                         /* segmented transfer, is size indicated? */
-                        if (SDO_C->CANrxData[0] & 0x01U) {
+                        if ((SDO_C->CANrxData[0] & 0x01U) != 0U) {
                             uint32_t size;
                             (void)memcpy(&size, &SDO_C->CANrxData[4], sizeof(size));
                             SDO_C->sizeInd = CO_SWAP_32(size);