Stefan Tauner
b6e9107a9a
Refine #585 ( #593 )
...
In MR #585 the initialization of HBcons has been protected from dereferencing stream if it's NULL.
However, the logic is now twisted enough to provoke a maybe-uninitialized warning in GCC 13.
While this is strictly a false positive since the generated code would never dereference HBcons if stream == NULL
this patch refines the code to be easier to read for compilers and humans alike by returning early if stream == NULL.
2025-07-29 15:12:44 +02:00
Cory Van Beek
c7a4d26b38
Fix dereference before NULL check ( #585 )
...
You are right, thank you.
2025-07-15 15:00:55 +02:00
Tilen Majerle
6dfd4ed7f2
define variable before it is being assigned ( #582 )
2025-06-08 17:43:26 +02:00
temi54c1l8
f454a3d82d
change for static analysis for CO_RPDO_process function ( #564 )
2025-03-01 16:54:38 +01:00
Janez
fbd42d46a2
Make OD_requestTPDO() more clear, remove OD_getFlagsPDO().
2024-10-19 12:42:36 +02:00
Vincent Chatelain
9e9aaf5f8d
CO_ODinterface: fix unreachable statement ( #548 )
...
Using a `break` after a `return` triggers warning 111 of Arm Compiler 5. To fix this warning while respecting MISRA rules, the return mechanism of OD_getSub() function was changed to only exit at the end.
2024-10-19 00:51:48 +02:00
Janez
58012aae34
Fix links
2024-08-14 16:49:01 +02:00
Janez
05931a6b76
Fix and update doxygen documentation.
2024-08-07 00:17:39 +02:00
Janez
835bae5664
CO_SYNC: bugfix, if CO_CONFIG_FLAG_OD_DYNAMIC is disabled.
2024-07-13 15:29:58 +02:00
Janez
e85410e923
CO_PDO.c: additional safety check.
2024-07-12 15:56:19 +02:00
Janez
f41bfdcb5d
Format comments in the .h files.
2024-07-09 00:17:15 +02:00
Janez
f46f1e2733
clang-format: ReflowComments: true
2024-07-07 01:05:10 +02:00
Janez
b87fad2195
Format the .h files using clang-format v15.
2024-07-06 21:54:59 +02:00
Janez
82c95a2bc9
Format comments in the .c files.
2024-07-06 21:48:21 +02:00
Janez
95c1705e76
Format the .c files using clang-format v15.
2024-07-06 19:57:06 +02:00
Janez
0d016a0019
Remove /*********...*/ comment delimiters.
2024-07-06 18:53:40 +02:00
Janez
f064eeaa40
Format the license information in files.
2024-07-06 13:03:00 +02:00
Janez
fcd89f87de
Remove comments from arguments in CO_CANrxBufferInit and CO_CANtxBufferInit
2024-07-05 18:38:01 +02:00
Janez
7db21c1f91
CO_SDO_server.c fix missing "bufShifted"
2024-07-05 11:14:37 +02:00
temi54c1l8
675d11a33d
CO_fifo: static analysis: format '%02X' specifies type 'unsigned int' which is nominally inconsistent with argument no. 3 of promoted type 'int'
2024-06-27 16:54:42 +02:00
temi54c1l8
86a5064e2f
CO_fifo: static analisys: use of modifier or type outside of a typedef [MISRA 2012 Directive 4.6, advisory]
2024-06-27 16:43:00 +02:00
temi54c1l8
5a2f24bc4b
static analysis: ANSI/ISO minimum translation limit of 31 'significant characters in an external identifier' exceeded, processing is unaffected
2024-06-27 14:49:37 +02:00
temi54c1l8
1c4a5f6877
CO_fifo: static analysis: multiple loop exits [MISRA 2012 Rule 15.4, advisory]
2024-06-27 13:31:08 +02:00
temi54c1l8
6fef978b69
CO_PDO: static analysis: dependence placed on operator precedence [MISRA 2012 Rule 12.1, advisory]
2024-06-27 13:21:43 +02:00
temi54c1l8
4ae66f9839
CO_SYNC: static analysis: ignoring return value of function 'CO_SYNCsend' [MISRA 2012 Directive 4.7, required], [MISRA 2012 Rule 17.7, required]
2024-06-27 12:58:54 +02:00
temi54c1l8
995fed609d
CO_SYNC: static analysis: an unsigned value and a signed value cannot be used together [MISRA 2012 Rule 10.4, required]
2024-06-27 12:58:02 +02:00
temi54c1l8
d6b3163311
CO_SYNC: static analysis: dependence placed on operator precedence [MISRA 2012 Rule 12.1, advisory]
2024-06-27 12:53:30 +02:00
temi54c1l8
f684b34a03
CO_SYNC: static analysis: no 'else' at end of 'if ... else if' chain [MISRA 2012 Rule 15.7, required]
2024-06-27 12:51:06 +02:00
temi54c1l8
baa9e4e2b2
CO_Emergency: static analysis: argument 1 of type 'void *' is not compatible with argument 2 of type 'uint8_t *' (aka 'unsigned char *') in call to function 'memcpy'
2024-06-27 12:27:01 +02:00
temi54c1l8
a501f961d4
CO_Emergency: ignoring return value of functions [MISRA 2012 Directive 4.7, required], [MISRA 2012 Rule 17.7, required]
2024-06-27 12:24:20 +02:00
temi54c1l8
d9d9044221
CO_Emergency: static analysis: right operand to == is a composite expression of type 'unsigned8' which is smaller than the left operand of type 'unsigned16' [MISRA 2012 Rule 10.7, required]
2024-06-27 12:22:43 +02:00
temi54c1l8
58606174a2
CO_Emergency: static analysis: side effects on right hand of logical operator, '&&' [MISRA 2012 Rule 13.5, required]
2024-06-27 12:21:24 +02:00
temi54c1l8
42c39a2b51
CO_Emergency: static analysis: an unsigned value and a signed value cannot be used together as operands [MISRA 2012 Rule 10.4, required]
2024-06-27 12:19:07 +02:00
temi54c1l8
ce405b1e27
CO_Emergency: static analysis: body should be a compound statement [MISRA 2012 Rule 15.6, required]
2024-06-27 12:12:57 +02:00
temi54c1l8
7bf5065e41
CO_Emergency: static analysis: no 'else' at end of 'if ... else if' chain [MISRA 2012 Rule 15.7, required]
2024-06-27 12:11:19 +02:00
temi54c1l8
ca968aa160
CO_Emergency: static analysis: conditional of #if does not evaluate to 0 or 1 [MISRA 2012 Rule 20.8, required]
2024-06-27 12:08:42 +02:00
temi54c1l8
d76aeb178f
CO_Emergency: static analysis: unsigned integer literal without a 'U' suffix [MISRA 2012 Rule 7.2, required]
2024-06-27 12:07:42 +02:00
temi54c1l8
071f6d7935
CO_Emergency: static analysis: dependence placed on operator precedence [MISRA 2012 Rule 12.1, advisory]
2024-06-27 12:06:07 +02:00
temi54c1l8
2e703328d8
CO_NMT_Heartbeat: static analysis: other minor change
2024-06-27 11:23:27 +02:00
temi54c1l8
b39334d9c3
CO_SDOclient: static analysis: potential out of bounds pointer access [MISRA 2012 Rule 18.1, required]
2024-06-27 08:55:44 +02:00
temi54c1l8
646e367ae1
CO_SDOclient: static anaylsis: loss of precision (assignment) from 32 bits to 8 bits
2024-06-26 18:01:47 +02:00
temi54c1l8
ad14eb5b18
CO_SDOclient: static analysis: argument 1 is not compatible with argument 2 in call to function 'memcpy'
2024-06-26 17:59:01 +02:00
temi54c1l8
b1cba9661b
CO_SDOclient: static analysis: side effects on right hand of logical operator, '||'/'&&' [MISRA 2012 Rule 13.5, required]
2024-06-26 17:53:02 +02:00
temi54c1l8
6b41e85e3d
CO_SDOclient: static analysis: body should be a compound statement [MISRA 2012 Rule 15.6, required]
2024-06-26 17:36:41 +02:00
temi54c1l8
86eb9bfad3
CO_SDOclient: static analysis: an unsigned value and an enum value cannot be used together [MISRA 2012 Rule 10.4, required]
2024-06-26 17:24:43 +02:00
temi54c1l8
53f3c58540
CO_SDOclient: static analysis: body should be a compound statement [MISRA 2012 Rule 15.6, required]
2024-06-26 17:19:05 +02:00
temi54c1l8
2a6b78172d
CO_SDOclient: static analysis: cannot assign a composite expression of type 'unsigned8' to an object of wider type 'unsigned32' [MISRA 2012 Rule 10.6, required]
2024-06-26 17:08:38 +02:00
temi54c1l8
10816afe53
CO_SDOclient: static analysis: right operand to +=/-= is a composite expression of type 'unsigned8' which is smaller than the left operand of type 'unsigned32' [MISRA 2012 Rule 10.7, required]
2024-06-26 17:05:44 +02:00
temi54c1l8
900633ad93
CO_SDOclient: static analysis: cannot assign 'signed8' to different essential type 'unsigned8' [MISRA 2012 Rule 10.3, required]
2024-06-26 16:43:50 +02:00
temi54c1l8
7d1da22c82
CO_SDOclient: static analysis: essential type of condition of 'if' statement is 'unsigned8' but should be boolean [MISRA 2012 Rule 14.4, required]
2024-06-26 16:42:19 +02:00
