Hi,
this patch fixes some overwritten error messages. There are more but
these are the ones I could check easily. O.k. to apply?
Cheers,
-- Guido
>From c18eb1fee9248bc8f09cb937416d43fa30d5c7b8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Guido=20G=C3=BCnther?= <[email protected]>
Date: Mon, 29 Nov 2010 10:23:15 +0100
Subject: [PATCH] OpenVZ: Fix some overwritten error codes
Don't overwrite errors during domain creation/definition to ease
tracking down problems.
---
src/openvz/openvz_driver.c | 10 ++++------
1 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/src/openvz/openvz_driver.c b/src/openvz/openvz_driver.c
index 6189fc1..cbc2888 100644
--- a/src/openvz/openvz_driver.c
+++ b/src/openvz/openvz_driver.c
@@ -59,6 +59,7 @@
#include "bridge.h"
#include "files.h"
#include "intprops.h"
+#include "logging.h"
#define VIR_FROM_THIS VIR_FROM_OPENVZ
@@ -211,8 +212,7 @@ static int openvzSetInitialConfig(virDomainDefPtr vmdef)
else
{
if (openvzDomainDefineCmd(prog, OPENVZ_MAX_ARG, vmdef) < 0) {
- openvzError(VIR_ERR_INTERNAL_ERROR, "%s",
- _("Error creating command for container"));
+ VIR_ERROR0(_("Error creating command for container"));
goto cleanup;
}
@@ -879,8 +879,7 @@ openvzDomainDefineXML(virConnectPtr conn, const char *xml)
vm->persistent = 1;
if (openvzSetInitialConfig(vm->def) < 0) {
- openvzError(VIR_ERR_INTERNAL_ERROR, "%s",
- _("Error creating initial configuration"));
+ VIR_ERROR0(_("Error creating initial configuration"));
goto cleanup;
}
@@ -968,8 +967,7 @@ openvzDomainCreateXML(virConnectPtr conn, const char *xml,
vm->persistent = 1;
if (openvzSetInitialConfig(vm->def) < 0) {
- openvzError(VIR_ERR_INTERNAL_ERROR, "%s",
- _("Error creating initial configuration"));
+ VIR_ERROR0(_("Error creating initial configuration"));
goto cleanup;
}
--
1.7.2.3
--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list