memcpy function shouldn't be run on NULL pointers. Added validation crashes tests because test vectors includes sender_id or id_context set to NULL. If tests vectors force us to handle NULL pointers of these arguments we can:
Return ok after check and do not perform memcpy, set dest pointer to NULL and exit with ok exit code. I don't like to return ok if the operation can be done properly (NULL pointers)
Use special TRY_MEMCPY macro to validate the result with proper handling of wrong_parameter error code.
memcpy function shouldn't be run on NULL pointers. Added validation crashes tests because test vectors includes sender_id or id_context set to NULL. If tests vectors force us to handle NULL pointers of these arguments we can:
@StefanHri what do you think?