in the encode_*_resp() functions we are checking if the response buffer is not allocated/NULL. if so then returning PLDM_ERROR_INVALID_DATA. but we are currently not checking that return code in the responder/handler functions. The responder should check the return type and send appropriate return code back to the caller.
This has to be addressed throughout the repo.
dkodihal
commented
5 years ago
in the encode_*_resp() functions we are checking if the response buffer is not allocated/NULL. if so then returning PLDM_ERROR_INVALID_DATA. but we are currently not checking that return code in the responder/handler functions. The responder should check the return type and send appropriate return code back to the caller. This has to be addressed throughout the repo.