Re: [PATCH] target/i386: Fix memory leak in sev_read_file_base64()

[Paolo Bonzini]

On 20/08/21 19:05, Philippe Mathieu-Daudé wrote:
> On 8/20/21 6:56 PM, Peter Maydell wrote:
> > In sev_read_file_base64() we call g_file_get_contents(), which
> > allocates memory for the file contents.  We then base64-decode the
> > contents (which allocates another buffer for the decoded data), but
> > forgot to free the memory for the original file data.
> >
> > Use g_autofree to ensure that the file data is freed.
> >
> > Fixes: Coverity CID 1459997
> > Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> > ---
> > Tested with 'make/make check' only...
> >
> >   target/i386/sev.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Queued, thanks.

Paolo