[libvirt] [PATCH 23/30] one: Remove virConnectPtr from oneError

Matthias Bolte matthias.bolte at googlemail.com
Sun Apr 4 17:36:46 UTC 2010


---
 src/opennebula/one_conf.c   |    7 ++--
 src/opennebula/one_conf.h   |    6 ++--
 src/opennebula/one_driver.c |   66 +++++++++++++++++++++---------------------
 3 files changed, 39 insertions(+), 40 deletions(-)

diff --git a/src/opennebula/one_conf.c b/src/opennebula/one_conf.c
index a2f7073..029d475 100644
--- a/src/opennebula/one_conf.c
+++ b/src/opennebula/one_conf.c
@@ -135,8 +135,7 @@ no_memory:
  * @return the OpenNebula ID for the new VM or -1 in case of error
  */
 
-int oneSubmitVM(virConnectPtr    conn,
-                one_driver_t*    driver ATTRIBUTE_UNUSED,
+int oneSubmitVM(one_driver_t*    driver ATTRIBUTE_UNUSED,
                 virDomainObjPtr  vm)
 {
     char* templ;
@@ -146,8 +145,8 @@ int oneSubmitVM(virConnectPtr    conn,
         return -1;
 
     if ((oneid = c_oneAllocateTemplate(templ)) < 0) {
-        oneError(conn, NULL, VIR_ERR_OPERATION_FAILED,
-                 "%s", _("Error submitting virtual machine to OpenNebula"));
+        oneError(VIR_ERR_OPERATION_FAILED, "%s",
+                 _("Error submitting virtual machine to OpenNebula"));
         VIR_FREE(templ);
         return -1;
     }
diff --git a/src/opennebula/one_conf.h b/src/opennebula/one_conf.h
index 0d45bff..c53090c 100644
--- a/src/opennebula/one_conf.h
+++ b/src/opennebula/one_conf.h
@@ -43,12 +43,12 @@ typedef struct one_driver one_driver_t;
 
 virCapsPtr oneCapsInit(void);
 
-int oneSubmitVM(virConnectPtr conn ,one_driver_t* driver, virDomainObjPtr  vm);
+int oneSubmitVM(one_driver_t* driver, virDomainObjPtr  vm);
 
 char* xmlOneTemplate(virDomainDefPtr def);
 
-# define oneError(conn, dom, code, ...)                                  \
-    virReportErrorHelper(conn, VIR_FROM_ONE, code, __FILE__,            \
+# define oneError(code, ...)                                            \
+    virReportErrorHelper(NULL, VIR_FROM_ONE, code, __FILE__,            \
                          __FUNCTION__, __LINE__, __VA_ARGS__)
 
 #endif /* ONE_CONF_H */
diff --git a/src/opennebula/one_driver.c b/src/opennebula/one_driver.c
index 91d7459..77d7d00 100644
--- a/src/opennebula/one_driver.c
+++ b/src/opennebula/one_driver.c
@@ -117,7 +117,7 @@ static virDomainPtr oneDomainLookupByID(virConnectPtr conn,
     oneDriverUnlock(driver);
 
     if (!vm) {
-        oneError(conn, NULL, VIR_ERR_NO_DOMAIN, NULL);
+        oneError(VIR_ERR_NO_DOMAIN, NULL);
         goto return_point;
     }
 
@@ -145,7 +145,7 @@ static virDomainPtr oneDomainLookupByUUID(virConnectPtr conn,
     vm = virDomainFindByUUID(&driver->domains, uuid);
     oneDriverUnlock(driver);
     if (!vm) {
-        oneError(conn, NULL, VIR_ERR_NO_DOMAIN, NULL);
+        oneError(VIR_ERR_NO_DOMAIN, NULL);
         goto return_point;
     }
 
@@ -174,7 +174,7 @@ static virDomainPtr oneDomainLookupByName(virConnectPtr conn,
     oneDriverUnlock(driver);
 
     if (!vm) {
-        oneError(conn, NULL, VIR_ERR_NO_DOMAIN, NULL);
+        oneError(VIR_ERR_NO_DOMAIN, NULL);
         goto return_point;
     }
 
@@ -279,14 +279,14 @@ static int oneDomainUndefine(virDomainPtr dom)
     oneDriverLock(driver);
     vm =virDomainFindByUUID(&driver->domains, dom->uuid);
     if (!vm) {
-        oneError(dom->conn, dom, VIR_ERR_INVALID_DOMAIN,
-                 "%s", _("no domain with matching uuid"));
+        oneError(VIR_ERR_INVALID_DOMAIN, "%s",
+                 _("no domain with matching uuid"));
         goto return_point;
     }
 
     if (!vm->persistent) {
-        oneError(dom->conn, dom, VIR_ERR_INTERNAL_ERROR,
-                 "%s", _("cannot undefine transient domain"));
+        oneError(VIR_ERR_INTERNAL_ERROR, "%s",
+                 _("cannot undefine transient domain"));
         goto return_point;
     }
     virDomainRemoveInactive(&driver->domains, vm);
@@ -310,14 +310,14 @@ static int oneDomainGetInfo(virDomainPtr dom,
     oneDriverUnlock(driver);
 
     if (!vm) {
-        oneError(dom->conn,dom, VIR_ERR_INVALID_DOMAIN,
-                 "%s", _("no domain with matching uuid"));
+        oneError(VIR_ERR_INVALID_DOMAIN, "%s",
+                 _("no domain with matching uuid"));
         return -1;
     }
 
     if(gettimeofday(&tv,NULL)<0) {
-        oneError(dom->conn,dom, VIR_ERR_INTERNAL_ERROR,
-                 "%s",_("getting time of day"));
+        oneError(VIR_ERR_INTERNAL_ERROR, "%s",
+                 _("getting time of day"));
         virDomainObjUnlock(vm);
         return -1;
     }
@@ -385,8 +385,8 @@ static char *oneGetOSType(virDomainPtr dom)
     vm =virDomainFindByUUID(&driver->domains, dom->uuid);
     oneDriverUnlock(driver);
     if (!vm) {
-        oneError(dom->conn,dom, VIR_ERR_INVALID_DOMAIN,
-                 "%s", _("no domain with matching uuid"));
+        oneError(VIR_ERR_INVALID_DOMAIN, "%s",
+                 _("no domain with matching uuid"));
         goto cleanup;
     }
 
@@ -412,11 +412,11 @@ static int oneDomainStart(virDomainPtr dom)
     vm = virDomainFindByName(&driver->domains, dom->name);
 
     if (!vm) {
-        oneError(conn, dom, VIR_ERR_INVALID_DOMAIN,
+        oneError(VIR_ERR_INVALID_DOMAIN,
                  _("no domain named %s"), dom->name);
         goto return_point;
     }
-    if((oneid = oneSubmitVM(dom->conn,driver,vm)) < 0) {
+    if((oneid = oneSubmitVM(driver, vm)) < 0) {
         goto return_point;
     }
     vm->pid=oneid;
@@ -449,7 +449,7 @@ oneDomainCreateAndStart(virConnectPtr conn,
 
     vm = virDomainFindByName(&driver->domains, def->name);
     if (vm) {
-        oneError(conn,NULL, VIR_ERR_OPERATION_FAILED,
+        oneError(VIR_ERR_OPERATION_FAILED,
                  _("Already an OpenNebula VM active with the name: '%s' id: %d "),
                  def->name,def->id);
         goto return_point;
@@ -460,7 +460,7 @@ oneDomainCreateAndStart(virConnectPtr conn,
         virDomainDefFree(def);
         goto return_point;
     }
-    if ((oneid = oneSubmitVM(conn, driver, vm)) < 0) {
+    if ((oneid = oneSubmitVM(driver, vm)) < 0) {
         virDomainRemoveInactive(&driver->domains, vm);
         vm=NULL;
         goto return_point;
@@ -492,14 +492,14 @@ static int oneDomainShutdown(virDomainPtr dom)
 
     oneDriverLock(driver);
     if (!(vm=virDomainFindByID(&driver->domains, dom->id))) {
-        oneError(dom->conn,dom, VIR_ERR_INVALID_DOMAIN,
+        oneError(VIR_ERR_INVALID_DOMAIN,
                  _("no domain with id %d"), dom->id);
         goto return_point;
     }
 
     if (c_oneShutdown(vm->pid)) {
-        oneError(dom->conn, dom, VIR_ERR_OPERATION_INVALID,
-                 "%s", _("Wrong state to perform action"));
+        oneError(VIR_ERR_OPERATION_INVALID, "%s",
+                 _("Wrong state to perform action"));
         goto return_point;
     }
     vm->state=VIR_DOMAIN_SHUTDOWN;
@@ -527,15 +527,15 @@ static int oneDomainDestroy(virDomainPtr dom)
     oneDriverLock(driver);
     vm= virDomainFindByID(&driver->domains, dom->id);
     if (!vm) {
-        oneError(dom->conn, dom, VIR_ERR_INVALID_DOMAIN,
+        oneError(VIR_ERR_INVALID_DOMAIN,
                  _("no domain with id %d"), dom->id);
         goto return_point;
     }
     if(c_oneCancel(vm->pid)) {
         /* VM not running, delete the instance at ONE DB */
         if(c_oneFinalize(vm->pid)){
-            oneError(dom->conn, dom, VIR_ERR_OPERATION_INVALID,
-                     "%s", _("Wrong state to perform action"));
+            oneError(VIR_ERR_OPERATION_INVALID, "%s",
+                     _("Wrong state to perform action"));
             goto return_point;
         }
     }
@@ -569,14 +569,14 @@ static int oneDomainSuspend(virDomainPtr dom)
                 ret=0;
                 goto return_point;
             }
-            oneError(dom->conn, dom, VIR_ERR_OPERATION_INVALID,
-                     "%s", _("Wrong state to perform action"));
+            oneError(VIR_ERR_OPERATION_INVALID, "%s",
+                     _("Wrong state to perform action"));
             goto return_point;
         }
-        oneError(dom->conn,dom, VIR_ERR_OPERATION_INVALID,
-                 "%s", _("domain is not running"));
+        oneError(VIR_ERR_OPERATION_INVALID, "%s",
+                 _("domain is not running"));
     } else {
-        oneError(dom->conn, dom, VIR_ERR_INVALID_DOMAIN,
+        oneError(VIR_ERR_INVALID_DOMAIN,
                  _("no domain with matching id %d"), dom->id);
     }
 
@@ -602,14 +602,14 @@ static int oneDomainResume(virDomainPtr dom)
                 ret=0;
                 goto return_point;
             }
-            oneError(dom->conn, dom, VIR_ERR_OPERATION_INVALID,
-                     "%s", _("Wrong state to perform action"));
+            oneError(VIR_ERR_OPERATION_INVALID, "%s",
+                     _("Wrong state to perform action"));
             goto return_point;
         }
-        oneError(dom->conn,dom, VIR_ERR_OPERATION_INVALID,
-                 "%s", _("domain is not paused"));
+        oneError(VIR_ERR_OPERATION_INVALID, "%s",
+                 _("domain is not paused"));
     } else {
-        oneError(dom->conn, dom, VIR_ERR_INVALID_DOMAIN,
+        oneError(VIR_ERR_INVALID_DOMAIN,
                  _("no domain with matching id %d"), dom->id);
     }
 
-- 
1.6.3.3




More information about the libvir-list mailing list