-
Notifications
You must be signed in to change notification settings - Fork 245
Closed
Description
As described at #241 , it is undefined behavior to call memcpy with NULL pointer, even if length is 0.
erpc/erpcgen/src/templates/c_coders.template
Lines 66 to 73 in 6756f38
| {$indent}if (({$info.name} == NULL) && ({$info.sizeTemp} > 0)) | |
| {$indent}{ | |
| {$indent} codec->updateStatus(kErpcStatus_MemoryError); | |
| {$indent}} | |
| {$indent}else | |
| {$indent}{ | |
| {$indent} memcpy({$info.name}, {$info.dataTemp}, {$info.sizeTemp}); | |
| {$indent}} |
Here, if {$info.name} is NULL and {$info.sizeTemp} is 0 this undefined behavior will be reached.