From 8c60bc169ff2c52b5f3677563c889acf0cb39dc0 Mon Sep 17 00:00:00 2001
From: "Daniel P. Berrange" <berrange@redhat.com>
Date: Tue, 29 Nov 2011 15:22:53 +0000
Subject: [PATCH] Remove pointless strdup in node suspend code

The command name for the suspend action does not need to be
strdup'd. The constant string can be used directly. This
also means the code can be trivially rearranged to make the
switch clearer

* src/util/virnodesuspend.c: Remove strdup of cmdString
---
 src/util/virnodesuspend.c | 49 ++++++++++++++-------------------------
 1 file changed, 17 insertions(+), 32 deletions(-)

diff --git a/src/util/virnodesuspend.c b/src/util/virnodesuspend.c
index 39ce56dd42..6eb91bc80f 100644
--- a/src/util/virnodesuspend.c
+++ b/src/util/virnodesuspend.c
@@ -127,9 +127,7 @@ cleanup:
  */
 static void virNodeSuspend(void *cmdString)
 {
-    virCommandPtr suspendCmd = virCommandNew((char *)cmdString);
-
-    VIR_FREE(cmdString);
+    virCommandPtr suspendCmd = virCommandNew((const char *)cmdString);
 
     /*
      * Delay for sometime so that the function nodeSuspendForDuration()
@@ -183,7 +181,7 @@ int nodeSuspendForDuration(virConnectPtr conn ATTRIBUTE_UNUSED,
                            unsigned int flags)
 {
     static virThread thread;
-    char *cmdString = NULL;
+    const char *cmdString = NULL;
     int ret = -1;
     unsigned int supported;
 
@@ -209,40 +207,28 @@ int nodeSuspendForDuration(virConnectPtr conn ATTRIBUTE_UNUSED,
     /* Check if the host supports the requested suspend target */
     switch (target) {
     case VIR_NODE_SUSPEND_TARGET_MEM:
-        if (supported & (1 << VIR_NODE_SUSPEND_TARGET_MEM)) {
-            cmdString = strdup("pm-suspend");
-            if (cmdString == NULL) {
-                virReportOOMError();
-                goto cleanup;
-            }
-            break;
+        if (!(supported & (1 << VIR_NODE_SUSPEND_TARGET_MEM))) {
+            virNodeSuspendError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s", _("Suspend-to-RAM"));
+            goto cleanup;
         }
-        virNodeSuspendError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s", _("Suspend-to-RAM"));
-        goto cleanup;
+        cmdString = "pm-suspend";
+        break;
 
     case VIR_NODE_SUSPEND_TARGET_DISK:
-        if (supported & (1 << VIR_NODE_SUSPEND_TARGET_DISK)) {
-            cmdString = strdup("pm-hibernate");
-            if (cmdString == NULL) {
-                virReportOOMError();
-                goto cleanup;
-            }
-            break;
+        if (!(supported & (1 << VIR_NODE_SUSPEND_TARGET_DISK))) {
+            virNodeSuspendError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s", _("Suspend-to-Disk"));
+            goto cleanup;
         }
-        virNodeSuspendError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s", _("Suspend-to-Disk"));
-        goto cleanup;
+        cmdString = "pm-hibernate";
+        break;
 
     case VIR_NODE_SUSPEND_TARGET_HYBRID:
-        if (supported & (1 << VIR_NODE_SUSPEND_TARGET_HYBRID)) {
-            cmdString = strdup("pm-suspend-hybrid");
-            if (cmdString == NULL) {
-                virReportOOMError();
-                goto cleanup;
-            }
-            break;
+        if (!(supported & (1 << VIR_NODE_SUSPEND_TARGET_HYBRID))) {
+            virNodeSuspendError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s", _("Hybrid-Suspend"));
+            goto cleanup;
         }
-        virNodeSuspendError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s", _("Hybrid-Suspend"));
-        goto cleanup;
+        cmdString = "pm-suspend-hybrid";
+        break;
 
     default:
         virNodeSuspendError(VIR_ERR_INVALID_ARG, "%s", _("Invalid suspend target"));
@@ -262,7 +248,6 @@ int nodeSuspendForDuration(virConnectPtr conn ATTRIBUTE_UNUSED,
     ret = 0;
 cleanup:
     virNodeSuspendUnlock();
-    VIR_FREE(cmdString);
     return ret;
 }
 
-- 
2.47.2