Use g_strdup instead of ignoring VIR_STRDUP's value

Replace all the occurrences of
  ignore_value(VIR_STRDUP(a, b));
with
  a = g_strdup(b);

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

diff --git a/src/conf/node_device_util.c b/src/conf/node_device_util.c
index 859ff9ebbbc5ac9e46c75b3de8cce181033c57f6..3b1e8e487fc167418f1b030f09e53e950ae397e4 100644 (file)
--- a/src/conf/node_device_util.c
+++ b/src/conf/node_device_util.c
@@ -54,7 +54,7 @@ virNodeDeviceGetParentName(virConnectPtr conn,
         return NULL;
     }
 
-    ignore_value(VIR_STRDUP(parent, virNodeDeviceGetParent(device)));
+    parent = g_strdup(virNodeDeviceGetParent(device));
     virObjectUnref(device);
 
     return parent;

diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c
index cdf925d62c7e62962916f26031f3b3dc62f70513..6c80e972208deb390075eaee7f05125c6675bbd3 100644 (file)
--- a/src/esx/esx_driver.c
+++ b/src/esx/esx_driver.c
@@ -1856,7 +1856,7 @@ esxDomainGetOSType(virDomainPtr domain G_GNUC_UNUSED)
 {
     char *osType;
 
-    ignore_value(VIR_STRDUP(osType, "hvm"));
+    osType = g_strdup("hvm");
     return osType;
 }
 

diff --git a/src/esx/esx_storage_backend_iscsi.c b/src/esx/esx_storage_backend_iscsi.c
index e9a30876bfa3b70dbdf32f1ff3ebb5afd1414b7d..a7e1f1b34019a3e288a10e001ef34d589067cbd5 100644 (file)
--- a/src/esx/esx_storage_backend_iscsi.c
+++ b/src/esx/esx_storage_backend_iscsi.c
@@ -766,7 +766,7 @@ esxStorageVolGetPath(virStorageVolPtr volume)
 {
     char *path;
 
-    ignore_value(VIR_STRDUP(path, volume->name));
+    path = g_strdup(volume->name);
     return path;
 }
 

diff --git a/src/hyperv/hyperv_driver.c b/src/hyperv/hyperv_driver.c
index 4e99721d374b1f6c5f03adf430672157ed306fc4..d69542b5006e5ec3392b293976c7223ad1539834 100644 (file)
--- a/src/hyperv/hyperv_driver.c
+++ b/src/hyperv/hyperv_driver.c
@@ -221,7 +221,7 @@ hypervConnectGetHostname(virConnectPtr conn)
         goto cleanup;
     }
 
-    ignore_value(VIR_STRDUP(hostname, computerSystem->data.common->DNSHostName));
+    hostname = g_strdup(computerSystem->data.common->DNSHostName);
 
  cleanup:
     hypervFreeObject(priv, (hypervObject *)computerSystem);
@@ -602,7 +602,7 @@ hypervDomainGetOSType(virDomainPtr domain G_GNUC_UNUSED)
 {
     char *osType;
 
-    ignore_value(VIR_STRDUP(osType, "hvm"));
+    osType = g_strdup("hvm");
     return osType;
 }
 

diff --git a/src/libxl/libxl_domain.c b/src/libxl/libxl_domain.c
index 1e4d900ac5f1b776189d63ad495bcc8f818a483a..25228af7a6493a8971bd6314d7aa24a08c473a4a 100644 (file)
--- a/src/libxl/libxl_domain.c
+++ b/src/libxl/libxl_domain.c
@@ -1132,10 +1132,8 @@ libxlConsoleCallback(libxl_ctx *ctx, libxl_event *ev, void *for_callback)
                                         &console);
             if (!ret) {
                 VIR_FREE(chr->source->data.file.path);
-                if (console && console[0] != '\0') {
-                    ignore_value(VIR_STRDUP(chr->source->data.file.path,
-                                            console));
-                }
+                if (console && console[0] != '\0')
+                    chr->source->data.file.path = g_strdup(console);
             }
             VIR_FREE(console);
         }
@@ -1153,10 +1151,8 @@ libxlConsoleCallback(libxl_ctx *ctx, libxl_event *ev, void *for_callback)
                                         &console);
             if (!ret) {
                 VIR_FREE(chr->source->data.file.path);
-                if (console && console[0] != '\0') {
-                    ignore_value(VIR_STRDUP(chr->source->data.file.path,
-                                            console));
-                }
+                if (console && console[0] != '\0')
+                    chr->source->data.file.path = g_strdup(console);
             }
             VIR_FREE(console);
         }
@@ -1242,8 +1238,7 @@ libxlDomainCreateChannelPTY(virDomainDefPtr def, libxl_ctx *ctx)
         if (!ret && channelinfo.u.pty.path &&
             *channelinfo.u.pty.path != '\0') {
                 VIR_FREE(chr->source->data.file.path);
-                ignore_value(VIR_STRDUP(chr->source->data.file.path,
-                                        channelinfo.u.pty.path));
+                chr->source->data.file.path = g_strdup(channelinfo.u.pty.path);
             }
     }
 

diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c
index 64b21f20731c6d1adb592fe481e9e6afab2e5712..14bd62b383f707897ccd9c9ccd12016975981c48 100644 (file)
--- a/src/libxl/libxl_driver.c
+++ b/src/libxl/libxl_driver.c
@@ -4607,7 +4607,7 @@ libxlDomainGetSchedulerType(virDomainPtr dom, int *nparams)
         goto cleanup;
     }
 
-    ignore_value(VIR_STRDUP(ret, name));
+    ret = g_strdup(name);
 
  cleanup:
     virDomainObjEndAPI(&vm);

diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c
index 204a3ed522aeb72f7d5282aa7ff36521e68fa675..7c2e6cf56126d5a6b129cd169fe411e63a7d43c1 100644 (file)
--- a/src/lxc/lxc_driver.c
+++ b/src/lxc/lxc_driver.c
@@ -1792,7 +1792,7 @@ static char *lxcDomainGetSchedulerType(virDomainPtr dom,
     if (!virDomainObjIsActive(vm)) {
         if (nparams)
             *nparams = 3;
-        ignore_value(VIR_STRDUP(ret, "posix"));
+        ret = g_strdup("posix");
         goto cleanup;
     }
 
@@ -1809,7 +1809,7 @@ static char *lxcDomainGetSchedulerType(virDomainPtr dom,
             *nparams = 1;
     }
 
-    ignore_value(VIR_STRDUP(ret, "posix"));
+    ret = g_strdup("posix");
 
  cleanup:
     virDomainObjEndAPI(&vm);

diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
index 6862818698e0cee29c235728ab1b3331ec306e2e..9dce161937b0384484474c27471bf5d7740d22dd 100644 (file)
--- a/src/network/bridge_driver.c
+++ b/src/network/bridge_driver.c
@@ -4190,7 +4190,7 @@ networkGetBridgeName(virNetworkPtr net)
         goto cleanup;
     }
 
-    ignore_value(VIR_STRDUP(bridge, def->bridge));
+    bridge = g_strdup(def->bridge);
 
  cleanup:
     virNetworkObjEndAPI(&obj);

diff --git a/src/node_device/node_device_udev.c b/src/node_device/node_device_udev.c
index f6729308d7663ef666b081ffc4409639f0cfdd90..c9449ace9ecfc29d29cde4f69e95f031b7f8693d 100644 (file)
--- a/src/node_device/node_device_udev.c
+++ b/src/node_device/node_device_udev.c
@@ -665,34 +665,34 @@ udevGetSCSIType(virNodeDeviceDefPtr def G_GNUC_UNUSED,
 
     switch (type) {
     case TYPE_DISK:
-        ignore_value(VIR_STRDUP(*typestring, "disk"));
+        *typestring = g_strdup("disk");
         break;
     case TYPE_TAPE:
-        ignore_value(VIR_STRDUP(*typestring, "tape"));
+        *typestring = g_strdup("tape");
         break;
     case TYPE_PROCESSOR:
-        ignore_value(VIR_STRDUP(*typestring, "processor"));
+        *typestring = g_strdup("processor");
         break;
     case TYPE_WORM:
-        ignore_value(VIR_STRDUP(*typestring, "worm"));
+        *typestring = g_strdup("worm");
         break;
     case TYPE_ROM:
-        ignore_value(VIR_STRDUP(*typestring, "cdrom"));
+        *typestring = g_strdup("cdrom");
         break;
     case TYPE_SCANNER:
-        ignore_value(VIR_STRDUP(*typestring, "scanner"));
+        *typestring = g_strdup("scanner");
         break;
     case TYPE_MOD:
-        ignore_value(VIR_STRDUP(*typestring, "mod"));
+        *typestring = g_strdup("mod");
         break;
     case TYPE_MEDIUM_CHANGER:
-        ignore_value(VIR_STRDUP(*typestring, "changer"));
+        *typestring = g_strdup("changer");
         break;
     case TYPE_ENCLOSURE:
-        ignore_value(VIR_STRDUP(*typestring, "enclosure"));
+        *typestring = g_strdup("enclosure");
         break;
     case TYPE_RAID:
-        ignore_value(VIR_STRDUP(*typestring, "raid"));
+        *typestring = g_strdup("raid");
         break;
     case TYPE_NO_LUN:
     default:

diff --git a/src/nwfilter/nwfilter_dhcpsnoop.c b/src/nwfilter/nwfilter_dhcpsnoop.c
index 90f3933959e2a16f08ca8e9fbda6aa4a854994a1..2c7a1a1522064a16d635b95af40c453455fa4b94 100644 (file)
--- a/src/nwfilter/nwfilter_dhcpsnoop.c
+++ b/src/nwfilter/nwfilter_dhcpsnoop.c
@@ -1395,7 +1395,7 @@ virNWFilterDHCPSnoopThread(void *req0)
             fds[i].fd = pcap_fileno(pcapConf[i].handle);
         }
         tmp = virNetDevGetIndex(req->binding->portdevname, &ifindex);
-        ignore_value(VIR_STRDUP(threadkey, req->threadkey));
+        threadkey = g_strdup(req->threadkey);
         worker = virThreadPoolNew(1, 1, 0,
                                   virNWFilterDHCPDecodeWorker,
                                   req);

diff --git a/src/openvz/openvz_conf.c b/src/openvz/openvz_conf.c
index be5f89ea45a1cbff6dc27d5c058313bc5ea00cc9..e5ef426329a1eafcca550e0e5045d94485bb1bca 100644 (file)
--- a/src/openvz/openvz_conf.c
+++ b/src/openvz/openvz_conf.c
@@ -878,7 +878,7 @@ openvzLocateConfDir(void)
 
     while (conf_dir_list[i]) {
         if (virFileExists(conf_dir_list[i])) {
-            ignore_value(VIR_STRDUP(ret, conf_dir_list[i]));
+            ret = g_strdup(conf_dir_list[i]);
             goto cleanup;
         }
         i++;

diff --git a/src/openvz/openvz_driver.c b/src/openvz/openvz_driver.c
index 408b393e3096d2e61f65d136584a5a999993ef6b..d1153edd009c105197f98643b154facb13d6653e 100644 (file)
--- a/src/openvz/openvz_driver.c
+++ b/src/openvz/openvz_driver.c
@@ -373,7 +373,7 @@ static char *openvzDomainGetOSType(virDomainPtr dom)
     if (!(vm = openvzDomObjFromDomain(driver, dom->uuid)))
         return NULL;
 
-    ignore_value(VIR_STRDUP(ret, virDomainOSTypeToString(vm->def->os.type)));
+    ret = g_strdup(virDomainOSTypeToString(vm->def->os.type));
 
     virDomainObjEndAPI(&vm);
     return ret;
@@ -716,7 +716,7 @@ openvzGenerateContainerVethName(int veid)
 
     /* try to get line "^NETIF=..." from config */
     if (openvzReadVPSConfigParam(veid, "NETIF", &temp) <= 0) {
-        ignore_value(VIR_STRDUP(name, "eth0"));
+        name = g_strdup("eth0");
     } else {
         char *saveptr = NULL;
         char *s;

diff --git a/src/qemu/qemu_alias.c b/src/qemu/qemu_alias.c
index d294963d35b9d635a0b861dbb47d7b7726fc1a10..5af3725bf5e3f808d85aa8938f91e50df31e6b20 100644 (file)
--- a/src/qemu/qemu_alias.c
+++ b/src/qemu/qemu_alias.c
@@ -758,7 +758,7 @@ qemuDomainGetMasterKeyAlias(void)
 {
     char *alias;
 
-    ignore_value(VIR_STRDUP(alias, "masterKey0"));
+    alias = g_strdup("masterKey0");
 
     return alias;
 }

diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c
index 4dc4f2922db7801fdbe4276c978e2f97b157b852..515bcd3d591c0fa41b7cada37b3eaca039a49d28 100644 (file)
--- a/src/qemu/qemu_block.c
+++ b/src/qemu/qemu_block.c
@@ -1934,7 +1934,7 @@ qemuBlockGetBackingStoreString(virStorageSourcePtr src)
     char *ret = NULL;
 
     if (virStorageSourceIsLocalStorage(src)) {
-        ignore_value(VIR_STRDUP(ret, src->path));
+        ret = g_strdup(src->path);
         return ret;
     }
 

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index a97bf65e7f9b7d759a1f739e97acf19538c8596f..3ef84cc9a6c9fcda81d353951758564a8afd1284 100644 (file)
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -10197,7 +10197,7 @@ qemuDomainStorageAlias(const char *device, int depth)
     device = qemuAliasDiskDriveSkipPrefix(device);
 
     if (!depth)
-        ignore_value(VIR_STRDUP(alias, device));
+        alias = g_strdup(device);
     else
         ignore_value(virAsprintf(&alias, "%s.%d", device, depth));
     return alias;
@@ -14719,7 +14719,7 @@ qemuDomainDiskBackingStoreGetName(virDomainDiskDefPtr disk,
     if (idx)
         ignore_value(virAsprintf(&ret, "%s[%d]", disk->dst, idx));
     else
-        ignore_value(VIR_STRDUP(ret, disk->dst));
+        ret = g_strdup(disk->dst);
 
     return ret;
 }

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index bf436f7dc382ac26a327a660a1fdbba0ae174f91..421a21818e70dec3a0b3d15b2bd78a68d8dcb703 100644 (file)
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -2281,7 +2281,7 @@ static char *qemuDomainGetOSType(virDomainPtr dom) {
     if (virDomainGetOSTypeEnsureACL(dom->conn, vm->def) < 0)
         goto cleanup;
 
-    ignore_value(VIR_STRDUP(type, virDomainOSTypeToString(vm->def->os.type)));
+    type = g_strdup(virDomainOSTypeToString(vm->def->os.type));
 
  cleanup:
     virDomainObjEndAPI(&vm);
@@ -4062,7 +4062,7 @@ qemuDomainScreenshot(virDomainPtr dom,
         goto endjob;
     }
 
-    ignore_value(VIR_STRDUP(ret, "image/x-portable-pixmap"));
+    ret = g_strdup("image/x-portable-pixmap");
 
  endjob:
     VIR_FORCE_CLOSE(tmp_fd);
@@ -9236,7 +9236,7 @@ static char *qemuDomainGetSchedulerType(virDomainPtr dom,
     if (!virDomainObjIsActive(vm)) {
         if (nparams)
             *nparams = 9;
-        ignore_value(VIR_STRDUP(ret, "posix"));
+        ret = g_strdup("posix");
         goto cleanup;
     }
 
@@ -9253,7 +9253,7 @@ static char *qemuDomainGetSchedulerType(virDomainPtr dom,
             *nparams = 1;
     }
 
-    ignore_value(VIR_STRDUP(ret, "posix"));
+    ret = g_strdup("posix");
 
  cleanup:
     virDomainObjEndAPI(&vm);

diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index c65491f9d6b71f943129564e268719370d98d143..fed95a285a1e01ab0df5b12821c838d71ecf4d28 100644 (file)
--- a/src/qemu/qemu_migration.c
+++ b/src/qemu/qemu_migration.c
@@ -1841,7 +1841,7 @@ qemuMigrationDstGetURI(const char *migrateFrom,
     if (STREQ(migrateFrom, "stdio"))
         ignore_value(virAsprintf(&uri, "fd:%d", migrateFd));
     else
-        ignore_value(VIR_STRDUP(uri, migrateFrom));
+        uri = g_strdup(migrateFrom);
 
     return uri;
 }

diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index 02308dffe6f2ec4f75ef7362657da9b460180cfb..4b765da0b14fc79ba75b00c8ab7ff622ca3ecd68 100644 (file)
--- a/src/qemu/qemu_monitor_json.c
+++ b/src/qemu/qemu_monitor_json.c
@@ -6719,7 +6719,7 @@ qemuMonitorJSONGetTargetArch(qemuMonitorPtr mon)
         goto cleanup;
     }
 
-    ignore_value(VIR_STRDUP(ret, arch));
+    ret = g_strdup(arch);
 
  cleanup:
     virJSONValueFree(cmd);

diff --git a/src/rpc/virnetsaslcontext.c b/src/rpc/virnetsaslcontext.c
index f579162aa2184a7dcb0f04dd462f3c8e245abf49..01ff41b7789f74e4dc501c52e6a380ff99a8ff4a 100644 (file)
--- a/src/rpc/virnetsaslcontext.c
+++ b/src/rpc/virnetsaslcontext.c
@@ -415,7 +415,7 @@ char *virNetSASLSessionListMechanisms(virNetSASLSessionPtr sasl)
                        _("no SASL mechanisms are available"));
         goto cleanup;
     }
-    ignore_value(VIR_STRDUP(ret, mechlist));
+    ret = g_strdup(mechlist);
 
  cleanup:
     virObjectUnlock(sasl);

diff --git a/src/security/security_apparmor.c b/src/security/security_apparmor.c
index 75203cc43a14a4484ec81b11c092af6cef779430..dd8f7d3808f3400aeec0759e024d9af27a39dc18 100644 (file)
--- a/src/security/security_apparmor.c
+++ b/src/security/security_apparmor.c
@@ -1158,7 +1158,7 @@ AppArmorGetMountOptions(virSecurityManagerPtr mgr G_GNUC_UNUSED,
 {
     char *opts;
 
-    ignore_value(VIR_STRDUP(opts, ""));
+    opts = g_strdup("");
     return opts;
 }
 

diff --git a/src/security/security_nop.c b/src/security/security_nop.c
index 5e7eed8e7cc528966e4c17109b5b4d1f0350b6f7..c1856eb4214beb7e202bf4571a187a511e998a3b 100644 (file)
--- a/src/security/security_nop.c
+++ b/src/security/security_nop.c
@@ -196,7 +196,7 @@ virSecurityDomainGetMountOptionsNop(virSecurityManagerPtr mgr G_GNUC_UNUSED,
 {
     char *opts;
 
-    ignore_value(VIR_STRDUP(opts, ""));
+    opts = g_strdup("");
     return opts;
 }
 

diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c
index 0812def9227fe13f552842fbe766d15a4e7042f3..4988659044beaf262bc1f517b155e71b688e833d 100644 (file)
--- a/src/security/security_selinux.c
+++ b/src/security/security_selinux.c
@@ -552,7 +552,7 @@ virSecuritySELinuxContextAddRange(security_context_t src,
         goto cleanup;
     }
 
-    ignore_value(VIR_STRDUP(ret, str));
+    ret = g_strdup(str);
 
  cleanup:
     if (srccon) context_free(srccon);

diff --git a/src/storage/storage_backend_scsi.c b/src/storage/storage_backend_scsi.c
index fd243776f8ffc6cfc371409d82d4ae5ad1c50191..98d9d92ac5989058166545d34b2568232f1c631b 100644 (file)
--- a/src/storage/storage_backend_scsi.c
+++ b/src/storage/storage_backend_scsi.c
@@ -184,7 +184,7 @@ getAdapterName(virStorageAdapterPtr adapter)
                                                         unique_id)))
                 return NULL;
         } else {
-            ignore_value(VIR_STRDUP(name, scsi_host->name));
+            name = g_strdup(scsi_host->name);
         }
     } else if (adapter->type == VIR_STORAGE_ADAPTER_TYPE_FC_HOST) {
         virStorageAdapterFCHostPtr fchost = &adapter->data.fchost;

diff --git a/src/storage/storage_backend_sheepdog.c b/src/storage/storage_backend_sheepdog.c
index f03c5c7e05545335ba8e60890aec26b598225919..443c22422250cd7a53a87a57e3accd25158ff879 100644 (file)
--- a/src/storage/storage_backend_sheepdog.c
+++ b/src/storage/storage_backend_sheepdog.c
@@ -343,7 +343,7 @@ virStorageBackendSheepdogRefreshVol(virStoragePoolObjPtr pool,
         return -1;
 
     VIR_FREE(vol->target.path);
-    ignore_value(VIR_STRDUP(vol->target.path, vol->name));
+    vol->target.path = g_strdup(vol->name);
 
     return 0;
 }

diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c
index da09d7de131aa8b99f8aeab733b1f1776d222ec3..ac215c7036413c15876c3c77d3fa86231436e880 100644 (file)
--- a/src/storage/storage_driver.c
+++ b/src/storage/storage_driver.c
@@ -1657,7 +1657,7 @@ storageVolLookupByPathCallback(virStoragePoolObjPtr obj,
         case VIR_STORAGE_POOL_SHEEPDOG:
         case VIR_STORAGE_POOL_ZFS:
         case VIR_STORAGE_POOL_LAST:
-            ignore_value(VIR_STRDUP(stable_path, data->path));
+            stable_path = g_strdup(data->path);
             break;
     }
 
@@ -2760,7 +2760,7 @@ storageVolGetPath(virStorageVolPtr vol)
                                       voldef) < 0)
         goto cleanup;
 
-    ignore_value(VIR_STRDUP(ret, voldef->target.path));
+    ret = g_strdup(voldef->target.path);
 
  cleanup:
     virStoragePoolObjEndAPI(&obj);

diff --git a/src/storage/storage_util.c b/src/storage/storage_util.c
index 410d412ef159a339c9de307a23359ddc5e7b0413..a4c8a5e971a16edafb5dad5f938f76a13a3f6f59 100644 (file)
--- a/src/storage/storage_util.c
+++ b/src/storage/storage_util.c
@@ -1986,7 +1986,7 @@ virStorageBackendStablePath(virStoragePoolObjPtr pool,
      * the original non-stable dev path
      */
 
-    ignore_value(VIR_STRDUP(stablepath, devpath));
+    stablepath = g_strdup(devpath);
 
     return stablepath;
 }
@@ -3641,7 +3641,7 @@ virStorageBackendSCSISerial(const char *dev,
     if (rc == -2)
         return NULL;
 
-    ignore_value(VIR_STRDUP(serial, dev));
+    serial = g_strdup(dev);
     return serial;
 }
 

diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index ab7e5fc02a1785109779491564acd232d89ce501..bd095604c76e8b5551afc7c78a40ce21937437e1 100644 (file)
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -725,7 +725,7 @@ static char *testBuildFilename(const char *relativeTo,
     if (!filename || filename[0] == '\0')
         return NULL;
     if (filename[0] == '/') {
-        ignore_value(VIR_STRDUP(ret, filename));
+        ret = g_strdup(filename);
         return ret;
     }
 
@@ -742,7 +742,7 @@ static char *testBuildFilename(const char *relativeTo,
         strcat(absFile, filename);
         return absFile;
     } else {
-        ignore_value(VIR_STRDUP(ret, filename));
+        ret = g_strdup(filename);
         return ret;
     }
 }
@@ -1585,7 +1585,7 @@ testConnectGetSysinfo(virConnectPtr conn G_GNUC_UNUSED,
 
     virCheckFlags(0, NULL);
 
-    ignore_value(VIR_STRDUP(ret, sysinfo));
+    ret = g_strdup(sysinfo);
     return ret;
 }
 
@@ -2561,7 +2561,7 @@ testDomainGetOSType(virDomainPtr dom G_GNUC_UNUSED)
 {
     char *ret;
 
-    ignore_value(VIR_STRDUP(ret, "linux"));
+    ret = g_strdup("linux");
     return ret;
 }
 
@@ -4889,7 +4889,7 @@ static char *testDomainGetSchedulerType(virDomainPtr domain G_GNUC_UNUSED,
     if (nparams)
         *nparams = 1;
 
-    ignore_value(VIR_STRDUP(type, "fair"));
+    type = g_strdup("fair");
 
     return type;
 }
@@ -5611,7 +5611,7 @@ testNetworkGetBridgeName(virNetworkPtr net)
         goto cleanup;
     }
 
-    ignore_value(VIR_STRDUP(bridge, def->bridge));
+    bridge = g_strdup(def->bridge);
 
  cleanup:
     virNetworkObjEndAPI(&obj);
@@ -6367,7 +6367,7 @@ testConnectFindStoragePoolSources(virConnectPtr conn G_GNUC_UNUSED,
     switch (pool_type) {
 
     case VIR_STORAGE_POOL_LOGICAL:
-        ignore_value(VIR_STRDUP(ret, defaultPoolSourcesLogicalXML));
+        ret = g_strdup(defaultPoolSourcesLogicalXML);
         return ret;
 
     case VIR_STORAGE_POOL_NETFS:
@@ -7241,7 +7241,7 @@ testStorageVolGetPath(virStorageVolPtr vol)
     if (!(privvol = testStorageVolDefFindByName(obj, vol->name)))
         goto cleanup;
 
-    ignore_value(VIR_STRDUP(ret, privvol->target.path));
+    ret = g_strdup(privvol->target.path);
 
  cleanup:
     virStoragePoolObjEndAPI(&obj);
@@ -7362,7 +7362,7 @@ testNodeDeviceGetParent(virNodeDevicePtr dev)
     def = virNodeDeviceObjGetDef(obj);
 
     if (def->parent) {
-        ignore_value(VIR_STRDUP(ret, def->parent));
+        ret = g_strdup(def->parent);
     } else {
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        "%s", _("no parent for this device"));

diff --git a/src/util/virbitmap.c b/src/util/virbitmap.c
index 5ac489004de96072810cdcdd83b1097f226771e7..a9d8e6c3d58a6b93c55fadded70dfa96bda15615 100644 (file)
--- a/src/util/virbitmap.c
+++ b/src/util/virbitmap.c
@@ -418,7 +418,7 @@ virBitmapFormat(virBitmapPtr bitmap)
 
     if (!bitmap || (cur = virBitmapNextSetBit(bitmap, -1)) < 0) {
         char *ret;
-        ignore_value(VIR_STRDUP(ret, ""));
+        ret = g_strdup("");
         return ret;
     }
 

diff --git a/src/util/virhash.c b/src/util/virhash.c
index a7fc6205677bd5cc7a2dc961d6f43f2898595976..447b7cfabfb10c6146d13ab3b64f1a939306e85b 100644 (file)
--- a/src/util/virhash.c
+++ b/src/util/virhash.c
@@ -94,7 +94,7 @@ static bool virHashStrEqual(const void *namea, const void *nameb)
 static void *virHashStrCopy(const void *name)
 {
     char *ret;
-    ignore_value(VIR_STRDUP(ret, name));
+    ret = g_strdup(name);
     return ret;
 }
 

diff --git a/src/util/virlockspace.c b/src/util/virlockspace.c
index c8e520c672d88510313c5c58ed549dcf490430c7..d3837461834d84ec34e841c1d15781079dc88e65 100644 (file)
--- a/src/util/virlockspace.c
+++ b/src/util/virlockspace.c
@@ -69,7 +69,7 @@ static char *virLockSpaceGetResourcePath(virLockSpacePtr lockspace,
     if (lockspace->dir)
         ignore_value(virAsprintf(&ret, "%s/%s", lockspace->dir, resname));
     else
-        ignore_value(VIR_STRDUP(ret, resname));
+        ret = g_strdup(resname);
     return ret;
 }
 

diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c
index d8ae5fbc5fac9370155fd5c896c68dcf230c90a2..5391f0030be41eae64680c6f27b6a73a8c054836 100644 (file)
--- a/src/util/virnetdev.c
+++ b/src/util/virnetdev.c
@@ -836,7 +836,7 @@ char *virNetDevGetName(int ifindex)
         goto cleanup;
     }
 
-   ignore_value(VIR_STRDUP(ifname, name));
+   ifname = g_strdup(name);
 
  cleanup:
      return ifname;

diff --git a/src/util/virscsi.c b/src/util/virscsi.c
index 17e8aa1daa932214a42f8745e71d79dd72c814d6..af8692eb5b44e867f898395633e752ff7a5863fa 100644 (file)
--- a/src/util/virscsi.c
+++ b/src/util/virscsi.c
@@ -127,7 +127,7 @@ virSCSIDeviceGetSgName(const char *sysfs_prefix,
 
     while (virDirRead(dir, &entry, path) > 0) {
         /* Assume a single directory entry */
-        ignore_value(VIR_STRDUP(sg, entry->d_name));
+        sg = g_strdup(entry->d_name);
         break;
     }
 
@@ -165,7 +165,7 @@ virSCSIDeviceGetDevName(const char *sysfs_prefix,
         goto cleanup;
 
     while (virDirRead(dir, &entry, path) > 0) {
-        ignore_value(VIR_STRDUP(name, entry->d_name));
+        name = g_strdup(entry->d_name);
         break;
     }
 

diff --git a/src/util/virscsihost.c b/src/util/virscsihost.c
index a5e8583b2b2feb227c55e129e926364dca7a37be..10524f49fa7594b5f75943c0c1de42594357052b 100644 (file)
--- a/src/util/virscsihost.c
+++ b/src/util/virscsihost.c
@@ -157,7 +157,7 @@ virSCSIHostFindByPCI(const char *sysfs_prefix,
             continue;
         }
 
-        ignore_value(VIR_STRDUP(ret, entry->d_name));
+        ret = g_strdup(entry->d_name);
         break;
     }
 

diff --git a/src/util/virstoragefile.c b/src/util/virstoragefile.c
index acf26721e7450b61a45b29d45cc1702d83c9582c..098d2684ccaea968cf61ab8cf39a4af2e6eda5d4 100644 (file)
--- a/src/util/virstoragefile.c
+++ b/src/util/virstoragefile.c
@@ -4014,7 +4014,7 @@ virStorageFileCanonicalizeFormatPath(char **components,
 
     /* if the output string is empty just return an empty string */
     if (!(ret = virBufferContentAndReset(&buf)))
-        ignore_value(VIR_STRDUP(ret, ""));
+        ret = g_strdup("");
 
     return ret;
 }

diff --git a/src/util/virstring.c b/src/util/virstring.c
index 6453a23ada982fe3e9ff3bedbfbfdfe6c3f927e9..fcae16c8feb7786c1fa82036b4e4fbd337eb5015 100644 (file)
--- a/src/util/virstring.c
+++ b/src/util/virstring.c
@@ -160,7 +160,7 @@ char *virStringListJoin(const char **strings,
         return NULL;
     ret = virBufferContentAndReset(&buf);
     if (!ret)
-        ignore_value(VIR_STRDUP(ret, ""));
+        ret = g_strdup("");
     return ret;
 }
 

diff --git a/src/util/virtpm.c b/src/util/virtpm.c
index c7702c8c25b21f83eaca83115adc177b59e8e407..9244697902b5a2a5f62f0063d728893a666db765 100644 (file)
--- a/src/util/virtpm.c
+++ b/src/util/virtpm.c
@@ -75,7 +75,7 @@ virTPMCreateCancelPath(const char *devpath)
                 VIR_FREE(path);
             }
             if (!path)
-                ignore_value(VIR_STRDUP(path, "/dev/null"));
+                path = g_strdup("/dev/null");
         } else {
             virReportError(VIR_ERR_INTERNAL_ERROR,
                            _("TPM device path %s is invalid"), devpath);
@@ -116,7 +116,7 @@ virTPMGetSwtpm(void)
         return NULL;
 
     virMutexLock(&swtpm_tools_lock);
-    ignore_value(VIR_STRDUP(s, swtpm_path));
+    s = g_strdup(swtpm_path);
     virMutexUnlock(&swtpm_tools_lock);
 
     return s;
@@ -131,7 +131,7 @@ virTPMGetSwtpmSetup(void)
         return NULL;
 
     virMutexLock(&swtpm_tools_lock);
-    ignore_value(VIR_STRDUP(s, swtpm_setup));
+    s = g_strdup(swtpm_setup);
     virMutexUnlock(&swtpm_tools_lock);
 
     return s;
@@ -146,7 +146,7 @@ virTPMGetSwtpmIoctl(void)
         return NULL;
 
     virMutexLock(&swtpm_tools_lock);
-    ignore_value(VIR_STRDUP(s, swtpm_ioctl));
+    s = g_strdup(swtpm_ioctl);
     virMutexUnlock(&swtpm_tools_lock);
 
     return s;

diff --git a/src/util/virtypedparam.c b/src/util/virtypedparam.c
index 2cd7e81bdd9925486f1e4c5ee95a970e33cf2c40..c2dadfd41095116578daa154c71ebf1001058a16 100644 (file)
--- a/src/util/virtypedparam.c
+++ b/src/util/virtypedparam.c
@@ -194,7 +194,7 @@ virTypedParameterToString(virTypedParameterPtr param)
         ignore_value(virAsprintf(&value, "%d", param->value.b));
         break;
     case VIR_TYPED_PARAM_STRING:
-        ignore_value(VIR_STRDUP(value, param->value.s));
+        value = g_strdup(param->value.s);
         break;
     default:
         virReportError(VIR_ERR_INTERNAL_ERROR,

diff --git a/src/util/virutil.c b/src/util/virutil.c
index cf3a397516094ef721ad592de90f603f216e5b6c..637d19e46cb264e72a831ae5229e19b5e32ecbdb 100644 (file)
--- a/src/util/virutil.c
+++ b/src/util/virutil.c
@@ -714,7 +714,7 @@ static char *virGetGroupEnt(gid_t gid)
         return NULL;
     }
 
-    ignore_value(VIR_STRDUP(ret, gr->gr_name));
+    ret = g_strdup(gr->gr_name);
     VIR_FREE(strbuf);
     return ret;
 }

diff --git a/src/util/virvhba.c b/src/util/virvhba.c
index ac80d9bd79efcb94045ddc402bc6d7dceb37ebd6..dbbd8d0f52a1ae7060bad7684d002f3674d2253a 100644 (file)
--- a/src/util/virvhba.c
+++ b/src/util/virvhba.c
@@ -148,7 +148,7 @@ virVHBAGetConfig(const char *sysfs_prefix,
     else
         p = buf;
 
-    ignore_value(VIR_STRDUP(result, p));
+    result = g_strdup(p);
 
  cleanup:
     VIR_FREE(sysfs_path);
@@ -222,7 +222,7 @@ virVHBAFindVportHost(const char *sysfs_prefix)
         if ((strlen(max_vports) >= strlen(vports)) ||
             ((strlen(max_vports) == strlen(vports)) &&
              strcmp(max_vports, vports) > 0)) {
-            ignore_value(VIR_STRDUP(ret, entry->d_name));
+            ret = g_strdup(entry->d_name);
             goto cleanup;
         }
 
@@ -403,7 +403,7 @@ virVHBAGetHostByWWN(const char *sysfs_prefix,
         if (rc == 0)
             continue;
 
-        ignore_value(VIR_STRDUP(ret, entry->d_name));
+        ret = g_strdup(entry->d_name);
         break;
     }
 
@@ -454,7 +454,7 @@ virVHBAGetHostByFabricWWN(const char *sysfs_prefix,
         if (rc == 0)
             continue;
 
-        ignore_value(VIR_STRDUP(ret, entry->d_name));
+        ret = g_strdup(entry->d_name);
         break;
     }
 

diff --git a/src/util/virxml.c b/src/util/virxml.c
index 46c79e2d946c0fb5109f956ed833d0779dce3b93..209c83b6cc7d5c2d5bc853015b1b55d61105ffd2 100644 (file)
--- a/src/util/virxml.c
+++ b/src/util/virxml.c
@@ -91,7 +91,7 @@ virXPathString(const char *xpath,
         xmlXPathFreeObject(obj);
         return NULL;
     }
-    ignore_value(VIR_STRDUP(ret, (char *) obj->stringval));
+    ret = g_strdup((char *)obj->stringval);
     xmlXPathFreeObject(obj);
     return ret;
 }
@@ -995,7 +995,7 @@ virXMLNodeToString(xmlDocPtr doc,
         goto cleanup;
     }
 
-    ignore_value(VIR_STRDUP(ret, (const char *)xmlBufferContent(xmlbuf)));
+    ret = g_strdup((const char *)xmlBufferContent(xmlbuf));
 
  cleanup:
     xmlBufferFree(xmlbuf);

diff --git a/src/vbox/vbox_common.c b/src/vbox/vbox_common.c
index 56c3c8d896e7d15c49e35518354e875bb76d7266..02c0a894e7124676f38083f75443698b4003bcd3 100644 (file)
--- a/src/vbox/vbox_common.c
+++ b/src/vbox/vbox_common.c
@@ -2701,7 +2701,7 @@ static char *vboxDomainGetOSType(virDomainPtr dom G_GNUC_UNUSED) {
      */
     char *osType;
 
-    ignore_value(VIR_STRDUP(osType, "hvm"));
+    osType = g_strdup("hvm");
     return osType;
 }
 

diff --git a/src/vbox/vbox_storage.c b/src/vbox/vbox_storage.c
index ba5b876ffba655475f3d834b71d47d9d80bf361c..6929a9a2f3340e93730b0679f9fb8fd1d8659826 100644 (file)
--- a/src/vbox/vbox_storage.c
+++ b/src/vbox/vbox_storage.c
@@ -843,7 +843,7 @@ static char *vboxStorageVolGetPath(virStorageVolPtr vol)
     if (!hddLocationUtf8)
         goto cleanup;
 
-    ignore_value(VIR_STRDUP(ret, hddLocationUtf8));
+    ret = g_strdup(hddLocationUtf8);
 
     VIR_DEBUG("Storage Volume Name: %s", vol->name);
     VIR_DEBUG("Storage Volume Path: %s", hddLocationUtf8);

diff --git a/src/vmware/vmware_conf.c b/src/vmware/vmware_conf.c
index 701062e440f7e31352983571b66c91fdebdb82e6..207af8fe81d2ac597d96aee065655915b7339311 100644 (file)
--- a/src/vmware/vmware_conf.c
+++ b/src/vmware/vmware_conf.c
@@ -541,6 +541,6 @@ vmwareCopyVMXFileName(const char *datastorePath, void *opaque G_GNUC_UNUSED)
 {
     char *path;
 
-    ignore_value(VIR_STRDUP(path, datastorePath));
+    path = g_strdup(datastorePath);
     return path;
 }

diff --git a/src/vmware/vmware_driver.c b/src/vmware/vmware_driver.c
index 724345330731c6baef811dbccf706d7fb30afa7d..912827a921112dae019a2289a4f3e003547416f9 100644 (file)
--- a/src/vmware/vmware_driver.c
+++ b/src/vmware/vmware_driver.c
@@ -844,7 +844,7 @@ vmwareDomainGetOSType(virDomainPtr dom)
     if (!(vm = vmwareDomObjFromDomain(driver, dom->uuid)))
         return NULL;
 
-    ignore_value(VIR_STRDUP(ret, virDomainOSTypeToString(vm->def->os.type)));
+    ret = g_strdup(virDomainOSTypeToString(vm->def->os.type));
 
     virDomainObjEndAPI(&vm);
     return ret;

diff --git a/src/vmx/vmx.c b/src/vmx/vmx.c
index fa7aa79484a85c420c955467611c4a77c2ed3072..7d20351a837ec223a4fe3aa8f9094d34efdc76aa 100644 (file)
--- a/src/vmx/vmx.c
+++ b/src/vmx/vmx.c
@@ -3595,7 +3595,7 @@ virVMXFormatDisk(virVMXContext *ctx, virDomainDiskDefPtr def,
 
             fileName = ctx->formatFileName(src, ctx->opaque);
         } else if (def->device == VIR_DOMAIN_DISK_DEVICE_CDROM) {
-            ignore_value(VIR_STRDUP(fileName, "emptyBackingString"));
+            fileName = g_strdup("emptyBackingString");
         }
 
         if (fileName == NULL)

diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c
index 21425346ac7a1e10887c209fe42f683b9aece412..76b23788b82ee15db71ca03f808c0112250e4f94 100644 (file)
--- a/src/vz/vz_driver.c
+++ b/src/vz/vz_driver.c
@@ -677,7 +677,7 @@ vzDomainGetOSType(virDomainPtr domain)
     if (virDomainGetOSTypeEnsureACL(domain->conn, dom->def) < 0)
         goto cleanup;
 
-    ignore_value(VIR_STRDUP(ret, virDomainOSTypeToString(dom->def->os.type)));
+    ret = g_strdup(virDomainOSTypeToString(dom->def->os.type));
 
  cleanup:
     virDomainObjEndAPI(&dom);

diff --git a/tests/qemuxml2argvmock.c b/tests/qemuxml2argvmock.c
index cc9a3ccbff8dd8a693aa35a2ccd6872149d100f1..2b9e5e7a492429ee4936cde71f39689a61a12e50 100644 (file)
--- a/tests/qemuxml2argvmock.c
+++ b/tests/qemuxml2argvmock.c
@@ -101,7 +101,7 @@ virTPMCreateCancelPath(const char *devpath)
     char *path;
     (void)devpath;
 
-    ignore_value(VIR_STRDUP(path, "/sys/class/misc/tpm0/device/cancel"));
+    path = g_strdup("/sys/class/misc/tpm0/device/cancel");
 
     return path;
 }
@@ -125,7 +125,7 @@ virSCSIDeviceGetSgName(const char *sysfs_prefix G_GNUC_UNUSED,
 {
     char *ret;
 
-    ignore_value(VIR_STRDUP(ret, "sg0"));
+    ret = g_strdup("sg0");
     return ret;
 }
 
@@ -199,7 +199,7 @@ virHostGetDRMRenderNode(void)
 {
     char *dst = NULL;
 
-    ignore_value(VIR_STRDUP(dst, "/dev/dri/foo"));
+    dst = g_strdup("/dev/dri/foo");
     return dst;
 }
 

diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
index 90edd7a8441de0bb31ac91c947fd278878f5607a..7bbc44451d6b277b5aaefcdd8697334096bb434d 100644 (file)
--- a/tests/qemuxml2argvtest.c
+++ b/tests/qemuxml2argvtest.c
@@ -1215,7 +1215,7 @@ mymain(void)
 
     driver.config->vncSASL = 1;
     VIR_FREE(driver.config->vncSASLdir);
-    ignore_value(VIR_STRDUP(driver.config->vncSASLdir, "/root/.sasl2"));
+    driver.config->vncSASLdir = g_strdup("/root/.sasl2");
     DO_TEST("graphics-vnc-sasl", QEMU_CAPS_VNC, QEMU_CAPS_DEVICE_CIRRUS_VGA);
     driver.config->vncTLS = 1;
     driver.config->vncTLSx509verify = 1;
@@ -1247,7 +1247,7 @@ mymain(void)
     DO_TEST("graphics-spice-no-args",
             QEMU_CAPS_SPICE, QEMU_CAPS_DEVICE_CIRRUS_VGA);
     driver.config->spiceSASL = 1;
-    ignore_value(VIR_STRDUP(driver.config->spiceSASLdir, "/root/.sasl2"));
+    driver.config->spiceSASLdir = g_strdup("/root/.sasl2");
     DO_TEST("graphics-spice-sasl",
             QEMU_CAPS_SPICE,
             QEMU_CAPS_DEVICE_QXL);

diff --git a/tests/testutils.c b/tests/testutils.c
index 15fe561c6bc9061fec1034224fe3b739e47a6fe6..1c89d503d962a6276d11c316dbb30cc5de72ae46 100644 (file)
--- a/tests/testutils.c
+++ b/tests/testutils.c
@@ -767,7 +767,7 @@ virTestLogContentAndReset(void)
         return NULL;
     ret = virBufferContentAndReset(&testLog.buf);
     if (!ret)
-        ignore_value(VIR_STRDUP(ret, ""));
+        ret = g_strdup("");
     return ret;
 }
 

diff --git a/tests/viriscsitest.c b/tests/viriscsitest.c
index 4722591b866b7bfe66fcb51dfa9a91f816a66186..76f0aa29465f68a6eadb185eb5b0af3ec15b57fc 100644 (file)
--- a/tests/viriscsitest.c
+++ b/tests/viriscsitest.c
@@ -87,9 +87,9 @@ static void testIscsiadmCb(const char *const*args,
         args[2] && STREQ(args[2], "session") &&
         args[3] == NULL) {
         if (data->output_version)
-            ignore_value(VIR_STRDUP(*output, iscsiadmSessionOutputNonFlash));
+            *output = g_strdup(iscsiadmSessionOutputNonFlash);
         else
-            ignore_value(VIR_STRDUP(*output, iscsiadmSessionOutput));
+            *output = g_strdup(iscsiadmSessionOutput);
     } else if (args[0] && STREQ(args[0], ISCSIADM) &&
                args[1] && STREQ(args[1], "--mode") &&
                args[2] && STREQ(args[2], "discovery") &&
@@ -100,7 +100,7 @@ static void testIscsiadmCb(const char *const*args,
                args[7] && STREQ(args[7], "--op") &&
                args[8] && STREQ(args[8], "nonpersistent") &&
                args[9] == NULL) {
-        ignore_value(VIR_STRDUP(*output, iscsiadmSendtargetsOutput));
+        *output = g_strdup(iscsiadmSendtargetsOutput);
     } else if (args[0] && STREQ(args[0], ISCSIADM) &&
                args[1] && STREQ(args[1], "--mode") &&
                args[2] && STREQ(args[2], "node") &&
@@ -125,9 +125,9 @@ static void testIscsiadmCb(const char *const*args,
                args[2] && STREQ(args[2], "iface") &&
                args[3] == NULL) {
         if (data->iface_created)
-            ignore_value(VIR_STRDUP(*output, iscsiadmIfaceIfaceOutput));
+            *output = g_strdup(iscsiadmIfaceIfaceOutput);
         else
-            ignore_value(VIR_STRDUP(*output, iscsiadmIfaceDefaultOutput));
+            *output = g_strdup(iscsiadmIfaceDefaultOutput);
     } else if (args[0] && STREQ(args[0], ISCSIADM) &&
                args[1] && STREQ(args[1], "--mode") &&
                args[2] && STREQ(args[2], "iface") &&
@@ -171,7 +171,7 @@ static void testIscsiadmCb(const char *const*args,
                args[8] && STREQ(args[8], "libvirt-iface-03020100") &&
                args[9] == NULL &&
                data->iface_created) {
-        ignore_value(VIR_STRDUP(*output, iscsiadmSendtargetsOutput));
+        *output = g_strdup(iscsiadmSendtargetsOutput);
     } else if (args[0] && STREQ(args[0], ISCSIADM) &&
                args[1] && STREQ(args[1], "--mode") &&
                args[2] && STREQ(args[2], "node") &&

diff --git a/tests/virstringtest.c b/tests/virstringtest.c
index 43c76ccda0f11c719e67d177170178538e0f018a..5d010f7f59f9d2314f96814c54ca3dc5b2e65f6d 100644 (file)
--- a/tests/virstringtest.c
+++ b/tests/virstringtest.c
@@ -697,9 +697,9 @@ testVirStringListFreeCount(const void *opaque G_GNUC_UNUSED)
     if (VIR_ALLOC_N(list, 4) < 0)
         return -1;
 
-    ignore_value(VIR_STRDUP(list[0], "test1"));
-    ignore_value(VIR_STRDUP(list[2], "test2"));
-    ignore_value(VIR_STRDUP(list[3], "test3"));
+    list[0] = g_strdup("test1");
+    list[2] = g_strdup("test2");
+    list[3] = g_strdup("test3");
 
     virStringListFreeCount(list, 4);
 

diff --git a/tests/vmx2xmltest.c b/tests/vmx2xmltest.c
index b2c56a15fe6c06b2e0f024adef1a832bee79f17a..4c5d41aee553dc8e1db06366cf1ef66794b4a7b0 100644 (file)
--- a/tests/vmx2xmltest.c
+++ b/tests/vmx2xmltest.c
@@ -157,7 +157,7 @@ testParseVMXFileName(const char *fileName, void *opaque G_GNUC_UNUSED)
             goto cleanup;
     } else if (STRPREFIX(fileName, "/")) {
         /* Found absolute path referencing a file outside a datastore */
-        ignore_value(VIR_STRDUP(src, fileName));
+        src = g_strdup(fileName);
     } else if (strchr(fileName, '/') != NULL) {
         /* Found relative path, this is not supported */
         src = NULL;

diff --git a/tests/xml2vmxtest.c b/tests/xml2vmxtest.c
index 2078f5c7cf9cdb4bedf00150c13a164b947c007b..0cfdb1facf7681e82b5b47a767f10bb91cf6d71a 100644 (file)
--- a/tests/xml2vmxtest.c
+++ b/tests/xml2vmxtest.c
@@ -175,7 +175,7 @@ testFormatVMXFileName(const char *src, void *opaque G_GNUC_UNUSED)
             goto cleanup;
     } else if (STRPREFIX(src, "/")) {
         /* Found absolute path */
-        ignore_value(VIR_STRDUP(absolutePath, src));
+        absolutePath = g_strdup(src);
     } else {
         /* Found relative path, this is not supported */
         goto cleanup;