[libvirt] [PATCH] Minor cleanup of error path for c_oneVmInfo.

Chris Lalancette clalance at redhat.com
Fri Aug 7 09:06:19 UTC 2009


Signed-off-by: Chris Lalancette <clalance at redhat.com>
---
 src/opennebula/one_client.c |   16 ++++++----------
 1 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/src/opennebula/one_client.c b/src/opennebula/one_client.c
index d1641bf..21f303a 100644
--- a/src/opennebula/one_client.c
+++ b/src/opennebula/one_client.c
@@ -176,6 +176,7 @@ int c_oneVmInfo(int vmid, char* ret_info,int length)
     xmlrpc_value *resultP;
     int return_code;
     char *return_string;
+    int retval = -1;
 
     resultP = xmlrpc_client_call(&one_client.env, one_client.url,
         "one.vmget_info", "(si)", one_client.session, vmid);
@@ -188,18 +189,13 @@ int c_oneVmInfo(int vmid, char* ret_info,int length)
         strncpy(ret_info, return_string, length-1);
         ret_info[length-1] = '\0';
 
-        xmlrpc_DECREF(resultP);
-        free(return_string);
-
-        return 0;
+        retval = 0;
     }
-    else
-    {
-        xmlrpc_DECREF(resultP);
-        free(return_string);
 
-        return -1;
-    }
+    xmlrpc_DECREF(resultP);
+    free(return_string);
+
+    return retval;
 }
 
 void c_oneFree()
-- 
1.6.0.6




More information about the libvir-list mailing list