tree e3b2d7e4e7ad159cb1ad1cc63f7ae1c91be33305
parent 43a7985dc9672c420f19d1bcf3248add6d3d6ba7
author Andrew Jeffery <andrew@aj.id.au> 1689561697 +0930
committer Andrew Jeffery <andrew@aj.id.au> 1689747841 +0930

pdr: Hoist record handle overflow test to avoid memory leak

The rework in 572a39507d1e ("pdr: Introduce pldm_pdr_add_check()") left
pldm_pdr_add_check() vulnerable to a memory leak. If the record handle
counter overflowed then the early-exit reporting the fault left the
memory pointed to by `record` allocated.

Hoist the overflow check over the dynamic allocation so that we don't
have to clean up in the error path.

Fixes: 572a39507d1e ("pdr: Introduce pldm_pdr_add_check()")
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I72ba43d009a4e76aa585590909edf6c70d64b7af