[libvirt] [PATCH] conf: use virDomainDeviceDefFree free dev
Daniel Henrique Barboza
danielhb413 at gmail.com
Thu Sep 19 13:33:53 UTC 2019
On 9/19/19 5:01 AM, Xu Yandong wrote:
> In function virDomainDeviceDefParse, we shoud use virDomainDeviceDefFree
> free data structure avoid potential memory leak.
>
> Signed-off-by: Xu Yandong <xuyandong2 at huawei.com>
> ---
> src/conf/domain_conf.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index 848c831330..8fb9480827 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -16504,7 +16504,8 @@ virDomainDeviceDefParse(const char *xmlStr,
> return dev;
>
> error:
> - VIR_FREE(dev);
> + virDomainDeviceDefFree(dev);
> + dev = NULL;
> goto cleanup;
Just noticed that the current master code does not have this potential
leak anymore. It was fixed by commit 475777c9ec, introducing
VIR_AUTOFREE() for virDomainDeviceDefPtr.
Thanks,
DHB
> }
>
More information about the libvir-list
mailing list