Re: [PATCH XENSTORE v1 04/10] xenstore_client: handle memory on error

[Jürgen Groß <jgross@xxxxxxxx>]

On 26.02.21 15:41, Norbert Manthey wrote:
> In case a command fails, also free the memory. As this is for the CLI
> client, currently the leaked memory is freed right after receiving the
> error, as the application terminates next.
>
> Similarly, if the allocation fails, do not use the NULL pointer
> afterwards, but instead error out.
>
> This bug was discovered and resolved using Coverity Static Analysis
> Security Testing (SAST) by Synopsys, Inc.
>
> Signed-off-by: Norbert Manthey <nmanthey@xxxxxxxxx>
> Reviewed-by: Thomas Friebel <friebelt@xxxxxxxxx>
> Reviewed-by: Julien Grall <jgrall@xxxxxxxxxxxx>
Reviewed-by: Juergen Gross <jgross@xxxxxxxx>


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: application/pgp-keys

Attachment: OpenPGP_signature
Description: OpenPGP digital signature