tree d074f5626d4b57a1f0175944c3a4f3a45ef78fa9
parent 7cd16ff869cc68009295795b47285fd88ac45df4
author Andrew Jeffery <andrew@aj.id.au> 1687953582 +0930
committer Andrew Jeffery <andrew@aj.id.au> 1688554851 +0930

pdr: Always uphold expectations of get_next_record_handle()

pldm_pdr_get_next_record() uses assert() to validate its arguments.
While this is usually a concern, get_next_record_handle() is internal
and can instead be protected at the public API boundary.

Add tests for the validity of parameters to pldm_pdr_find_record() and
pldm_pdr_get_next_record(), the callers of get_next_record_handle().
Both functions return pointers, and already return NULL on error.
Correct invocation must therefore already validate the returned pointer,
thus it is valid to also return NULL in the case of invalid arguments.

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I07596acc9da7427d785b8b9db11eb84219fb8a5d