From cdf065537f7e490760a367f327f181437623b810 Mon Sep 17 00:00:00 2001 From: Marc Hartmayer Date: Fri, 5 Aug 2016 18:04:27 +0200 Subject: [PATCH] virsystemd: Fix error check Commit b3e4401dc620 introduced a check to ignore an error if the guest is already terminated. However the check accidentally compared error.code with VIR_ERR_ERROR, which is an error level, not an error code. Because of this, almost every error got silently ignored. Fixes: b3e4401dc620 ("systemd: don't report an error if the guest is already terminated") Signed-off-by: Marc Hartmayer Reviewed-by: Sascha Silbe Reviewed-by: Boris Fiuczynski --- src/util/virsystemd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/virsystemd.c b/src/util/virsystemd.c index 7d6985b0e3..604dcdd0ad 100644 --- a/src/util/virsystemd.c +++ b/src/util/virsystemd.c @@ -468,7 +468,7 @@ int virSystemdTerminateMachine(const char *name) name) < 0) goto cleanup; - if (error.code == VIR_ERR_ERROR && + if (error.level == VIR_ERR_ERROR && STRNEQ_NULLABLE("org.freedesktop.machine1.NoSuchMachine", error.str1)) { virReportErrorObject(&error); -- GitLab