nikon/CANopenNode
1
0
Fork 0
Code Activity Actions

static analysis: essential type of condition of 'if' statement is in not correct [MISRA 2012 Rule 14.4, required]

Browse source
This commit is contained in:
temi54c1l8 2024-06-12 13:57:42 +02:00
parent 7855d35bae
commit 691728f71a
3 changed files with 22 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
301/CO_Emergency.c
View file

@ -564,32 +564,32 @@ void CO_EM_process(CO_EM_t *em,
uint16_t CANerrStChanged = CANerrSt ^ em->CANerrorStatusOld;
em->CANerrorStatusOld = CANerrSt;
if (CANerrStChanged & (CO_CAN_ERRTX_WARNING | CO_CAN_ERRRX_WARNING)) {
if ((CANerrStChanged & (CO_CAN_ERRTX_WARNING | CO_CAN_ERRRX_WARNING)) != 0U) {
CO_error(em,
(CANerrSt & (CO_CAN_ERRTX_WARNING | CO_CAN_ERRRX_WARNING)) != 0U,
CO_EM_CAN_BUS_WARNING, CO_EMC_NO_ERROR, 0);
}
if (CANerrStChanged & CO_CAN_ERRTX_PASSIVE) {
if ((CANerrStChanged & CO_CAN_ERRTX_PASSIVE) != 0U) {
CO_error(em, (CANerrSt & CO_CAN_ERRTX_PASSIVE) != 0U,
CO_EM_CAN_TX_BUS_PASSIVE, CO_EMC_CAN_PASSIVE, 0);
}
if (CANerrStChanged & CO_CAN_ERRTX_BUS_OFF) {
if ((CANerrStChanged & CO_CAN_ERRTX_BUS_OFF) != 0U) {
CO_error(em, (CANerrSt & CO_CAN_ERRTX_BUS_OFF) != 0U,
CO_EM_CAN_TX_BUS_OFF, CO_EMC_BUS_OFF_RECOVERED, 0);
}
if (CANerrStChanged & CO_CAN_ERRTX_OVERFLOW) {
if ((CANerrStChanged & CO_CAN_ERRTX_OVERFLOW) != 0U) {
CO_error(em, (CANerrSt & CO_CAN_ERRTX_OVERFLOW) != 0U,
CO_EM_CAN_TX_OVERFLOW, CO_EMC_CAN_OVERRUN, 0);
}
if (CANerrStChanged & CO_CAN_ERRTX_PDO_LATE) {
if ((CANerrStChanged & CO_CAN_ERRTX_PDO_LATE) != 0U) {
CO_error(em, (CANerrSt & CO_CAN_ERRTX_PDO_LATE) != 0U,
CO_EM_TPDO_OUTSIDE_WINDOW, CO_EMC_COMMUNICATION, 0);
}
if (CANerrStChanged & CO_CAN_ERRRX_PASSIVE) {
if ((CANerrStChanged & CO_CAN_ERRRX_PASSIVE) != 0U) {
CO_error(em, (CANerrSt & CO_CAN_ERRRX_PASSIVE) != 0U,
CO_EM_CAN_RX_BUS_PASSIVE, CO_EMC_CAN_PASSIVE, 0);
}
if (CANerrStChanged & CO_CAN_ERRRX_OVERFLOW) {
if ((CANerrStChanged & CO_CAN_ERRRX_OVERFLOW) != 0U) {
CO_error(em, (CANerrSt & CO_CAN_ERRRX_OVERFLOW) != 0U,
CO_EM_CAN_RXB_OVERFLOW, CO_EMC_CAN_OVERRUN, 0);
}

6
301/CO_SDOserver.c
View file

@ -809,7 +809,7 @@ CO_SDO_return_t CO_SDOserver_process(CO_SDOserver_t *SDO,
if ((SDO->state != CO_SDO_ST_IDLE) && (SDO->state != CO_SDO_ST_ABORT)) {
switch (SDO->state) {
case CO_SDO_ST_DOWNLOAD_INITIATE_REQ: {
if (SDO->CANrxData[0] & 0x02U) {
if ((SDO->CANrxData[0] & 0x02U) != 0U) {
/* Expedited transfer, max 4 bytes of data */
/* Size of OD variable (>0 if indicated) */
@ -817,7 +817,7 @@ CO_SDO_return_t CO_SDOserver_process(CO_SDOserver_t *SDO,
/* Get SDO data size (indicated by SDO client or get from OD) */
OD_size_t dataSizeToWrite = 4;
if (SDO->CANrxData[0] & 0x01U) {
if ((SDO->CANrxData[0] & 0x01U) != 0U) {
dataSizeToWrite -= (SDO->CANrxData[0] >> 2) & 0x03U;
}
else if ((sizeInOd > 0U) && (sizeInOd < 4U)) {
@ -880,7 +880,7 @@ CO_SDO_return_t CO_SDOserver_process(CO_SDOserver_t *SDO,
else {
#if ((CO_CONFIG_SDO_SRV) & CO_CONFIG_SDO_SRV_SEGMENTED) != 0
/* segmented transfer, is size indicated? */
if (SDO->CANrxData[0] & 0x01U) {
if ((SDO->CANrxData[0] & 0x01U) != 0U) {
uint32_t size;
OD_size_t sizeInOd = SDO->OD_IO.stream.dataLength;

24
CANopen.c
View file

@ -1000,7 +1000,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co,
#if ((CO_CONFIG_LEDS) & CO_CONFIG_LEDS_ENABLE) != 0
if (CO_GET_CNT(LEDS) == 1U) {
err = CO_LEDs_init(co->LEDs);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
#endif
@ -1037,7 +1037,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co,
#endif
nodeId,
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
/* NMT_Heartbeat */
@ -1060,7 +1060,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co,
CO_GET_CO(TX_IDX_HB_PROD),
(uint16_t)CO_CAN_ID_HEARTBEAT + nodeId,
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
#if ((CO_CONFIG_HB_CONS) & CO_CONFIG_HB_CONS_ENABLE) != 0
@ -1073,7 +1073,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co,
co->CANmodule,
CO_GET_CO(RX_IDX_HB_CONS),
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
#endif
@ -1088,7 +1088,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co,
co->CANmodule,
CO_GET_CO(TX_IDX_NG_SLV),
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
#endif
#if ((CO_CONFIG_NODE_GUARDING) & CO_CONFIG_NODE_GUARDING_MASTER_ENABLE) != 0
err = CO_nodeGuardingMaster_init(co->NGmaster,
@ -1114,7 +1114,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co,
co->CANmodule,
CO_GET_CO(TX_IDX_SDO_SRV) + i,
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
}
@ -1147,7 +1147,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co,
CO_GET_CO(TX_IDX_TIME),
#endif
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
#endif
@ -1166,7 +1166,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co,
CO_GET_CO(TX_IDX_SYNC),
#endif
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
#endif
@ -1278,7 +1278,7 @@ CO_ReturnError_t CO_CANopenInitPDO(CO_t *co,
co->CANmodule,
CO_GET_CO(RX_IDX_RPDO) + i,
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
}
#endif
@ -1312,7 +1312,7 @@ CO_ReturnError_t CO_CANopenInitPDO(CO_t *co,
co->CANmodule,
CO_GET_CO(TX_IDX_TPDO) + i,
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
}
#endif
@ -1360,7 +1360,7 @@ CO_ReturnError_t CO_CANopenInitSRDO(CO_t *co,
OD_GET(H13FE, OD_H13FE_SRDO_VALID),
OD_GET(H13FF, OD_H13FF_SRDO_CHECKSUM),
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
OD_entry_t *SRDOcomm = OD_GET(H1301, OD_H1301_SRDO_1_PARAM);
OD_entry_t *SRDOmap = OD_GET(H1381, OD_H1381_SRDO_1_MAPPING);
@ -1388,7 +1388,7 @@ CO_ReturnError_t CO_CANopenInitSRDO(CO_t *co,
CANdevTxIdx,
CANdevTxIdx + 1U,
errInfo);
if (err) { return err; }
if (err != CO_ERROR_NO) { return err; }
}
CO_SRDO_init_end(co->SRDOGuard);