[libvirt] [PATCH 2/2] libxl: Resolve Coverity CHECKED_RETURN
Pavel Hrdina
phrdina at redhat.com
Sat Feb 14 08:09:35 UTC 2015
On Fri, Feb 13, 2015 at 03:21:40PM -0500, John Ferlan wrote:
> Periodically my Coverity scan will return a checked_return failure
> for libxlDomainShutdownThread call to libxlDomainStart. Followed the
> libxlAutostartDomain example in order to check the status, emit a
> message, and continue on.
>
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
> src/libxl/libxl_domain.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/libxl/libxl_domain.c b/src/libxl/libxl_domain.c
> index f0eaf6c..21c41d7 100644
> --- a/src/libxl/libxl_domain.c
> +++ b/src/libxl/libxl_domain.c
> @@ -685,7 +685,11 @@ libxlDomainShutdownThread(void *opaque)
> }
> libxl_domain_destroy(ctx, vm->def->id, NULL);
> libxlDomainCleanupJob(driver, vm, VIR_DOMAIN_SHUTOFF_SHUTDOWN);
> - libxlDomainStart(driver, vm, 0, -1);
> + if (libxlDomainStart(driver, vm, false, -1) < 0) {
> + virErrorPtr err = virGetLastError();
> + VIR_ERROR(_("Failed to restart VM '%s': %s"),
> + vm->def->name, err ? err->message : _("unknown error"));
> + }
>
> cleanup:
> if (vm)
> --
> 2.1.0
ACK
Pavel
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
More information about the libvir-list
mailing list