On Thu, Feb 02, 2017 at 03:46:36PM +0000, Wei Liu wrote:
> Otherwise it is leaked. Found by code inspection.
>
> Signed-off-by: Wei Liu <wei.l...@citrix.com>
> ---
> Cc: Ian Jackson <ian.jack...@eu.citrix.com>
> Cc: Fatih Acar <fatih.a...@gandi.net>
>
> Backport candidate.
> ---
> tools/libxl/xl_cmdimpl.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
> index 7e8a8ae5c4..9bf10dfcb2 100644
> --- a/tools/libxl/xl_cmdimpl.c
> +++ b/tools/libxl/xl_cmdimpl.c
> @@ -3145,6 +3145,7 @@ start:
> case DOMAIN_RESTART_RENAME:
> if (domid_soft_reset == INVALID_DOMID &&
> !preserve_domain(&domid, event, &d_config)) {
> + libxl_event_free(event);
Stupid me.
This should have been
libxl_event_free(ctx, event);
> /* If we fail then exit leaving the old domain in place.
> */
> ret = -1;
> goto out;
> --
> 2.11.0
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
