tree bf7da602aa0d8820e5c90cadbe38977d25471b27
parent 9dfcae8b94a42a1d4eda735a3de92f92c401aca9
author Paul Elliott <paul.elliott@arm.com> 1592234493 +0100
committer Paul Elliott <paul.elliott@arm.com> 1592314495 +0100

Remove Dangerous Parameter Passing

Another coverity bug - #350039

When this test discovers a key of the wrong type, it still throws it
through the export function in order to check that it too will detect
this as a not permitted action. For the buffer and buffer length
arguments it passes in a local pointer (which will most likely be NULL),
and the sizeof that pointer, as it knows that they will never be used.
Coverity rightly (imho) flagged this as suspicious - if we are going to
pass in incorrect parameters, at least make them obviously incorrect, and
ones that will not potentially cause errors if the code later changes.
There is, for example safety checks for zero length buffer, but less
protection for an insufficiently sized one.

Signed-off-by: Paul Elliott <paul.elliott@arm.com>