]> xenbits.xensource.com Git - libvirt.git/commitdiff
virstring: Unify string list function names
authorMichal Privoznik <mprivozn@redhat.com>
Fri, 25 Nov 2016 08:18:35 +0000 (09:18 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Fri, 25 Nov 2016 12:54:05 +0000 (13:54 +0100)
We have couple of functions that operate over NULL terminated
lits of strings. However, our naming sucks:

virStringJoin
virStringFreeList
virStringFreeListCount
virStringArrayHasString
virStringGetFirstWithPrefix

We can do better:

virStringListJoin
virStringListFree
virStringListFreeCount
virStringListHasString
virStringListGetFirstWithPrefix

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
54 files changed:
daemon/remote.c
src/bhyve/bhyve_command.c
src/bhyve/bhyve_parse_command.c
src/conf/domain_capabilities.c
src/cpu/cpu_ppc64.c
src/cpu/cpu_x86.c
src/libvirt_private.syms
src/lxc/lxc_container.c
src/lxc/lxc_native.c
src/openvz/openvz_conf.c
src/qemu/qemu_agent.c
src/qemu/qemu_capabilities.c
src/qemu/qemu_conf.c
src/qemu/qemu_domain.c
src/qemu/qemu_driver.c
src/qemu/qemu_monitor_json.c
src/qemu/qemu_monitor_text.c
src/qemu/qemu_parse_command.c
src/qemu/qemu_process.c
src/remote/remote_driver.c
src/storage/storage_backend_sheepdog.c
src/storage/storage_backend_zfs.c
src/storage/storage_driver.c
src/util/vircgroup.c
src/util/vircommand.c
src/util/virconf.c
src/util/virfile.c
src/util/virfirewall.c
src/util/virfirmware.c
src/util/virlog.c
src/util/virpolkit.c
src/util/virprocess.c
src/util/virstoragefile.c
src/util/virstring.c
src/util/virstring.h
src/util/viruri.c
src/vbox/vbox_common.c
src/vbox/vbox_snapshot_conf.c
src/vz/vz_sdk.c
tests/domainsnapshotxml2xmltest.c
tests/qemumonitorjsontest.c
tests/qemuxml2argvtest.c
tests/vboxsnapshotxmltest.c
tests/virconftest.c
tests/virfiletest.c
tests/virpolkittest.c
tests/virstringtest.c
tools/virsh-domain.c
tools/virsh-nodedev.c
tools/virsh-pool.c
tools/virsh-snapshot.c
tools/virt-host-validate-common.c
tools/virt-login-shell.c
tools/vsh.c

index e414f920e1b5dd89b87d4cc9843ebd9fc154757c..46773da63407a8fa03b5b8994682e826748f5ada 100644 (file)
@@ -5425,7 +5425,7 @@ remoteDispatchConnectGetCPUModelNames(virNetServerPtr server ATTRIBUTE_UNUSED,
  cleanup:
     if (rv < 0)
         virNetMessageSaveError(rerr);
-    virStringFreeList(models);
+    virStringListFree(models);
     return rv;
 }
 
index bb5c45c3a9c8eac57f83a6608998b283009320c4..8a29977ff1f17a54d50c240c2222e94cecd22975 100644 (file)
@@ -323,7 +323,7 @@ virAppendBootloaderArgs(virCommandPtr cmd, virDomainDefPtr def)
     /* XXX: Handle quoted? */
     blargs = virStringSplit(def->os.bootloaderArgs, " ", 0);
     virCommandAddArgSet(cmd, (const char * const *)blargs);
-    virStringFreeList(blargs);
+    virStringListFree(blargs);
 }
 
 static virCommandPtr
index 0ae7a834ce8769f200f3cd8ad2e6d0a1f01468c6..6190042983133bab239176fc54b789655d51434d 100644 (file)
@@ -246,7 +246,7 @@ bhyveCommandLineToArgv(const char *nativeConfig,
         } else {
             /* To prevent a use-after-free here, only free the argument list
              * when it is definitely not going to be used */
-            virStringFreeList(arglist);
+            virStringListFree(arglist);
         }
     }
 
@@ -254,13 +254,13 @@ bhyveCommandLineToArgv(const char *nativeConfig,
     if (!(*bhyve_argv = _bhyve_argv))
         goto error;
 
-    virStringFreeList(lines);
+    virStringListFree(lines);
     return 0;
 
  error:
     VIR_FREE(_loader_argv);
     VIR_FREE(_bhyve_argv);
-    virStringFreeList(lines);
+    virStringListFree(lines);
     return -1;
 }
 
@@ -804,7 +804,7 @@ bhyveParseBhyveLoadCommandLine(virDomainDefPtr def,
         if (VIR_STRDUP(def->os.bootloader, argv[0]) < 0)
            goto error;
 
-        def->os.bootloaderArgs = virStringJoin((const char**) &argv[1], " ");
+        def->os.bootloaderArgs = virStringListJoin((const char**) &argv[1], " ");
     }
 
     if (argc != parser->optind) {
@@ -841,7 +841,7 @@ bhyveParseCustomLoaderCommandLine(virDomainDefPtr def,
     if (VIR_STRDUP(def->os.bootloader, argv[0]) < 0)
        goto error;
 
-    def->os.bootloaderArgs = virStringJoin((const char**) &argv[1], " ");
+    def->os.bootloaderArgs = virStringListJoin((const char**) &argv[1], " ");
 
     return 0;
  error:
@@ -893,8 +893,8 @@ bhyveParseCommandLineString(const char* nativeConfig,
     }
 
  cleanup:
-    virStringFreeList(loader_argv);
-    virStringFreeList(bhyve_argv);
+    virStringListFree(loader_argv);
+    virStringListFree(bhyve_argv);
     return def;
  error:
     virDomainDefFree(def);
index beedd7076296f365b018171fb1423243f7d1a67f..fb2d696449680068c08c9ce50ed4f19a64d6baa5 100644 (file)
@@ -185,7 +185,7 @@ virDomainCapsCPUModelsFilter(virDomainCapsCPUModelsPtr old,
         return NULL;
 
     for (i = 0; i < old->nmodels; i++) {
-        if (models && !virStringArrayHasString(models, old->models[i].name))
+        if (models && !virStringListHasString(models, old->models[i].name))
             continue;
 
         if (virDomainCapsCPUModelsAdd(cpuModels,
index 225fb6d9694b1c9c46e6ab07d6684e1a4ee718fd..180ad87d18c951fc3ffa30ea322577861188fff1 100644 (file)
@@ -888,7 +888,7 @@ virCPUppc64DriverGetModels(char ***models)
 
  error:
     if (models) {
-        virStringFreeList(*models);
+        virStringListFree(*models);
         *models = NULL;
     }
     goto cleanup;
index e91850f3ff2ebde8b51aae0a351665eb0c548be4..c79308a71512a14a14e2b7acff3fdab13a02e109 100644 (file)
@@ -2675,7 +2675,7 @@ virCPUx86GetModels(char ***models)
 
  error:
     if (models) {
-        virStringFreeList(*models);
+        virStringListFree(*models);
         *models = NULL;
     }
     return -1;
index a04ba23d3deb16cd3567406dfcaea4dc573d1017..7cc7bf88587d4c25fcaef9513ed0b956c75c46aa 100644 (file)
@@ -2453,16 +2453,16 @@ virSkipSpacesAndBackslash;
 virSkipSpacesBackwards;
 virStrcpy;
 virStrdup;
-virStringArrayHasString;
 virStringBufferIsPrintable;
 virStringEncodeBase64;
-virStringFreeList;
-virStringFreeListCount;
-virStringGetFirstWithPrefix;
 virStringHasControlChars;
 virStringIsEmpty;
 virStringIsPrintable;
-virStringJoin;
+virStringListFree;
+virStringListFreeCount;
+virStringListGetFirstWithPrefix;
+virStringListHasString;
+virStringListJoin;
 virStringListLength;
 virStringReplace;
 virStringSearch;
index 5357df48de587375ca1b308d20c3276ebc21736b..dd013dfcee6b791badac6163a27521763da44804 100644 (file)
@@ -607,7 +607,7 @@ static int lxcContainerUnmountSubtree(const char *prefix,
     ret = 0;
 
  cleanup:
-    virStringFreeList(mounts);
+    virStringListFree(mounts);
 
     return ret;
 }
index ce62df90a88a84590ef5ea107647230810da248f..8f44168e4dff6482f48c9011a636ab9a8687d692 100644 (file)
@@ -129,13 +129,13 @@ static char ** lxcStringSplit(const char *string)
     }
 
     VIR_FREE(tmp);
-    virStringFreeList(parts);
+    virStringListFree(parts);
     return result;
 
  error:
     VIR_FREE(tmp);
-    virStringFreeList(parts);
-    virStringFreeList(result);
+    virStringListFree(parts);
+    virStringListFree(result);
     return NULL;
 }
 
@@ -160,13 +160,13 @@ lxcParseFstabLine(char *fstabLine)
             VIR_STRDUP(fstab->options, parts[3]) < 0)
         goto error;
 
-    virStringFreeList(parts);
+    virStringListFree(parts);
 
     return fstab;
 
  error:
     lxcFstabFree(fstab);
-    virStringFreeList(parts);
+    virStringListFree(parts);
     return NULL;
 }
 
@@ -299,7 +299,7 @@ lxcAddFstabLine(virDomainDefPtr def, lxcFstabPtr fstab)
         type = VIR_DOMAIN_FS_TYPE_BLOCK;
 
     /* Do we have ro in options? */
-    readonly = virStringArrayHasString((const char **) options, "ro");
+    readonly = virStringListHasString((const char **) options, "ro");
 
     if (lxcAddFSDef(def, type, src, dst, readonly, usage) < 0)
         goto cleanup;
@@ -308,7 +308,7 @@ lxcAddFstabLine(virDomainDefPtr def, lxcFstabPtr fstab)
 
  cleanup:
     VIR_FREE(dst);
-    virStringFreeList(options);
+    virStringListFree(options);
     return ret;
 }
 
@@ -618,12 +618,12 @@ lxcNetworkWalkCallback(const char *name, virConfValuePtr value, void *data)
             virReportError(VIR_ERR_INVALID_ARG,
                            _("Invalid CIDR address: '%s'"), value->str);
 
-            virStringFreeList(ipparts);
+            virStringListFree(ipparts);
             VIR_FREE(ip);
             return -1;
         }
 
-        virStringFreeList(ipparts);
+        virStringListFree(ipparts);
 
         if (VIR_APPEND_ELEMENT(parseData->ips, parseData->nips, ip) < 0) {
             VIR_FREE(ip);
@@ -943,7 +943,7 @@ lxcBlkioDeviceWalkCallback(const char *name, virConfValuePtr value, void *data)
     ret = 0;
 
  cleanup:
-    virStringFreeList(parts);
+    virStringListFree(parts);
     VIR_FREE(path);
 
     return ret;
@@ -982,13 +982,13 @@ lxcSetCapDrop(virDomainDefPtr def, virConfPtr properties)
     for (i = 0; i < VIR_DOMAIN_CAPS_FEATURE_LAST; i++) {
         capString = virDomainCapsFeatureTypeToString(i);
         if (toDrop != NULL &&
-            virStringArrayHasString((const char **) toDrop, capString))
+            virStringListHasString((const char **) toDrop, capString))
             def->caps_features[i] = VIR_TRISTATE_SWITCH_OFF;
     }
 
     def->features[VIR_DOMAIN_FEATURE_CAPABILITIES] = VIR_DOMAIN_CAPABILITIES_POLICY_ALLOW;
 
-    virStringFreeList(toDrop);
+    virStringListFree(toDrop);
 }
 
 virDomainDefPtr
index 65c441e6d6037532ee0c0e02e4669170602338ae..b02e1aab8b86f3bc74fb3567f1d94b99add3d0e9 100644 (file)
@@ -151,7 +151,7 @@ openvzParseBarrierLimit(const char* value,
 
     ret = 0;
  error:
-    virStringFreeListCount(tmp, ntmp);
+    virStringListFreeCount(tmp, ntmp);
     return ret;
 }
 
index ec8d47e0586c929986c51f5acdc736fd33496589..1d677f74f1f8e3aeacb1024e5835204300e7266f 100644 (file)
@@ -2115,7 +2115,7 @@ qemuAgentGetInterfaces(qemuAgentPtr mon,
         }
 
         /* Has to be freed for each interface. */
-        virStringFreeList(ifname);
+        virStringListFree(ifname);
 
         /* as well as IP address which - moreover -
          * can be presented multiple times */
@@ -2201,7 +2201,7 @@ qemuAgentGetInterfaces(qemuAgentPtr mon,
             virDomainInterfaceFree(ifaces_ret[i]);
     }
     VIR_FREE(ifaces_ret);
-    virStringFreeList(ifname);
+    virStringListFree(ifname);
 
     goto cleanup;
 }
index f9e39a34ac3efd70844fd15e2402a0f7bfdb8eb6..8901e7b2c367fc005c055d95859ced47cbf647c9 100644 (file)
@@ -1842,7 +1842,7 @@ virQEMUCapsProcessProps(virQEMUCapsPtr qemuCaps,
 
         for (j = 0; j < nvalues; j++) {
             if (STREQ(values[j], props[i].prop)) {
-                if (virStringArrayHasString(props[i].objects, object))
+                if (virStringListHasString(props[i].objects, object))
                     virQEMUCapsSet(qemuCaps, props[i].flag);
                 break;
             }
@@ -2369,7 +2369,7 @@ virQEMUCapsGetCPUDefinitions(virQEMUCapsPtr qemuCaps,
     return 0;
 
  error:
-    virStringFreeListCount(models, i);
+    virStringListFreeCount(models, i);
     return -1;
 }
 
@@ -2777,12 +2777,12 @@ virQEMUCapsProbeQMPTPM(virQEMUCapsPtr qemuCaps,
         for (i = 0; i < ARRAY_CARDINALITY(virQEMUCapsTPMModelsToCaps); i++) {
             const char *needle = virDomainTPMModelTypeToString(
                 virQEMUCapsTPMModelsToCaps[i].type);
-            if (virStringArrayHasString((const char **) entries, needle))
+            if (virStringListHasString((const char **) entries, needle))
                 virQEMUCapsSet(qemuCaps,
                                virQEMUCapsTPMModelsToCaps[i].caps);
         }
     }
-    virStringFreeList(entries);
+    virStringListFree(entries);
 
     if ((nentries = qemuMonitorGetTPMTypes(mon, &entries)) < 0)
         return -1;
@@ -2791,11 +2791,11 @@ virQEMUCapsProbeQMPTPM(virQEMUCapsPtr qemuCaps,
         for (i = 0; i < ARRAY_CARDINALITY(virQEMUCapsTPMTypesToCaps); i++) {
             const char *needle = virDomainTPMBackendTypeToString(
                 virQEMUCapsTPMTypesToCaps[i].type);
-            if (virStringArrayHasString((const char **) entries, needle))
+            if (virStringListHasString((const char **) entries, needle))
                 virQEMUCapsSet(qemuCaps, virQEMUCapsTPMTypesToCaps[i].caps);
         }
     }
-    virStringFreeList(entries);
+    virStringListFree(entries);
 
     return 0;
 }
@@ -2887,7 +2887,7 @@ virQEMUCapsProbeQMPCommandLine(virQEMUCapsPtr qemuCaps,
                 break;
             }
         }
-        virStringFreeList(values);
+        virStringListFree(values);
     }
 
     return 0;
@@ -3849,13 +3849,13 @@ virQEMUCapsQMPSchemaGetByPath(const char *query,
 
     if (!*elems) {
         virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("malformed query string"));
-        virStringFreeList(elems);
+        virStringListFree(elems);
         return -1;
     }
 
     *entry = virQEMUCapsQMPSchemaTraverse(*elems, elems + 1, schema);
 
-    virStringFreeList(elems);
+    virStringListFree(elems);
     return 0;
 }
 
@@ -4644,7 +4644,7 @@ virQEMUCapsFillDomainCPUCaps(virCapsPtr caps,
         if (virCPUGetModels(domCaps->arch, &models) >= 0) {
             filtered = virDomainCapsCPUModelsFilter(qemuCaps->cpuDefinitions,
                                                     (const char **) models);
-            virStringFreeList(models);
+            virStringListFree(models);
         }
         domCaps->cpu.custom = filtered;
     }
index fbac3ea8c24b0dd5c5cf4cbc791c0221e9d662d4..ccefbe89077c665e67e3a33e11cc9f8de5d451c5 100644 (file)
@@ -350,7 +350,7 @@ static void virQEMUDriverConfigDispose(void *obj)
     virQEMUDriverConfigPtr cfg = obj;
 
 
-    virStringFreeList(cfg->cgroupDeviceACL);
+    virStringListFree(cfg->cgroupDeviceACL);
 
     VIR_FREE(cfg->configBaseDir);
     VIR_FREE(cfg->configDir);
@@ -392,7 +392,7 @@ static void virQEMUDriverConfigDispose(void *obj)
     VIR_FREE(cfg->dumpImageFormat);
     VIR_FREE(cfg->autoDumpPath);
 
-    virStringFreeList(cfg->securityDriverNames);
+    virStringListFree(cfg->securityDriverNames);
 
     VIR_FREE(cfg->lockManagerName);
 
@@ -801,9 +801,9 @@ int virQEMUDriverConfigLoadFile(virQEMUDriverConfigPtr cfg,
     ret = 0;
 
  cleanup:
-    virStringFreeList(controllers);
-    virStringFreeList(hugetlbfs);
-    virStringFreeList(nvram);
+    virStringListFree(controllers);
+    virStringListFree(hugetlbfs);
+    virStringListFree(nvram);
     VIR_FREE(corestr);
     VIR_FREE(user);
     VIR_FREE(group);
index 137d4d51958443ca0cdaa172951f6b0009fc8390..47332a82c5db5e4d9d15aaaa144df4041a7dbfa8 100644 (file)
@@ -1549,7 +1549,7 @@ qemuDomainObjPrivateFree(void *data)
     VIR_FREE(priv->lockState);
     VIR_FREE(priv->origname);
 
-    virStringFreeList(priv->qemuDevices);
+    virStringListFree(priv->qemuDevices);
     virChrdevFree(priv->devs);
 
     /* This should never be non-NULL if we get here, but just in case... */
@@ -1963,7 +1963,7 @@ qemuDomainObjPrivateXMLParse(xmlXPathContextPtr ctxt,
     priv->monConfig = NULL;
     VIR_FREE(nodes);
     VIR_FREE(tmp);
-    virStringFreeList(priv->qemuDevices);
+    virStringListFree(priv->qemuDevices);
     priv->qemuDevices = NULL;
     virObjectUnref(qemuCaps);
     virObjectUnref(caps);
@@ -5263,7 +5263,7 @@ qemuDomainUpdateDeviceList(virQEMUDriverPtr driver,
     if (rc < 0)
         return -1;
 
-    virStringFreeList(priv->qemuDevices);
+    virStringListFree(priv->qemuDevices);
     priv->qemuDevices = aliases;
     return 0;
 }
index fdfe9121267ca3c082f9f6f235a9b4ac9510df5a..76f965d58b736e4136e34ac56d4091ee5c66c99c 100644 (file)
@@ -1383,7 +1383,7 @@ qemuGetSchedInfo(unsigned long long *cpuWait,
  cleanup:
     VIR_FREE(data);
     VIR_FREE(proc);
-    virStringFreeList(lines);
+    virStringListFree(lines);
     return ret;
 }
 
index 90d74d546651bd01689e4f3cf25f375d1d0a8c36..03a5e1e4d7efa731890deb4714110a36ca4cf76b 100644 (file)
@@ -4987,7 +4987,7 @@ int qemuMonitorJSONGetCommands(qemuMonitorPtr mon,
 
 
  cleanup:
-    virStringFreeList(commandlist);
+    virStringListFree(commandlist);
     virJSONValueFree(cmd);
     virJSONValueFree(reply);
     return ret;
@@ -5051,7 +5051,7 @@ int qemuMonitorJSONGetEvents(qemuMonitorPtr mon,
     eventlist = NULL;
 
  cleanup:
-    virStringFreeList(eventlist);
+    virStringListFree(eventlist);
     virJSONValueFree(cmd);
     virJSONValueFree(reply);
     return ret;
@@ -5173,7 +5173,7 @@ qemuMonitorJSONGetCommandLineOptionParameters(qemuMonitorPtr mon,
     if (!qemuMonitorGetOptions(mon))
         qemuMonitorSetOptions(mon, virJSONValueNewArray());
 
-    virStringFreeList(paramlist);
+    virStringListFree(paramlist);
     virJSONValueFree(cmd);
     virJSONValueFree(reply);
     return ret;
@@ -5276,7 +5276,7 @@ int qemuMonitorJSONGetObjectTypes(qemuMonitorPtr mon,
     typelist = NULL;
 
  cleanup:
-    virStringFreeList(typelist);
+    virStringListFree(typelist);
     virJSONValueFree(cmd);
     virJSONValueFree(reply);
     return ret;
@@ -5577,7 +5577,7 @@ int qemuMonitorJSONGetObjectProps(qemuMonitorPtr mon,
     proplist = NULL;
 
  cleanup:
-    virStringFreeList(proplist);
+    virStringListFree(proplist);
     virJSONValueFree(cmd);
     virJSONValueFree(reply);
     return ret;
@@ -5683,7 +5683,7 @@ qemuMonitorJSONGetMigrationCapabilities(qemuMonitorPtr mon,
     list = NULL;
 
  cleanup:
-    virStringFreeList(list);
+    virStringListFree(list);
     virJSONValueFree(cmd);
     virJSONValueFree(reply);
     return ret;
@@ -5701,9 +5701,9 @@ qemuMonitorJSONGetMigrationCapability(qemuMonitorPtr mon,
     if (qemuMonitorJSONGetMigrationCapabilities(mon, &capsList) < 0)
         return -1;
 
-    ret = virStringArrayHasString((const char **) capsList, cap);
+    ret = virStringListHasString((const char **) capsList, cap);
 
-    virStringFreeList(capsList);
+    virStringListFree(capsList);
     return ret;
 }
 
@@ -6067,7 +6067,7 @@ qemuMonitorJSONGetStringArray(qemuMonitorPtr mon, const char *qmpCmd,
     list = NULL;
 
  cleanup:
-    virStringFreeList(list);
+    virStringListFree(list);
     virJSONValueFree(cmd);
     virJSONValueFree(reply);
     return ret;
index 4692d53a4c44b17966fa768573669aa2cfac4641..a2a5885e00d866918cd47a9c99dc728c951404fc 100644 (file)
@@ -971,15 +971,15 @@ qemuMonitorTextGetAllBlockStatsInfo(qemuMonitorPtr mon,
             goto cleanup;
         stats = NULL;
 
-        virStringFreeList(values);
+        virStringListFree(values);
         values = NULL;
     }
 
     ret = maxstats;
 
  cleanup:
-    virStringFreeList(lines);
-    virStringFreeList(values);
+    virStringListFree(lines);
+    virStringListFree(values);
     VIR_FREE(stats);
     VIR_FREE(info);
     return ret;
index cfebcc739ec9f220b67f1ed8baf7c30fee507fd1..405e6554995765053eb92665d40e9b64a6821b87 100644 (file)
@@ -360,7 +360,7 @@ static int qemuStringToArgvEnv(const char *args,
  error:
     VIR_FREE(progenv);
     VIR_FREE(progargv);
-    virStringFreeList(arglist);
+    virStringListFree(arglist);
     return -1;
 }
 
@@ -1588,7 +1588,7 @@ qemuParseCommandLineCPU(virDomainDefPtr dom,
             case VIR_DOMAIN_HYPERV_LAST:
                 break;
             }
-            virStringFreeList(hv_tokens);
+            virStringListFree(hv_tokens);
             hv_tokens = NULL;
         } else if (STREQ(tokens[i], "kvm=off")) {
              dom->features[VIR_DOMAIN_FEATURE_KVM] = VIR_TRISTATE_SWITCH_ON;
@@ -1619,8 +1619,8 @@ qemuParseCommandLineCPU(virDomainDefPtr dom,
 
  cleanup:
     VIR_FREE(model);
-    virStringFreeList(tokens);
-    virStringFreeList(hv_tokens);
+    virStringListFree(tokens);
+    virStringListFree(hv_tokens);
     return ret;
 
  syntax:
@@ -2196,7 +2196,7 @@ qemuParseCommandLine(virCapsPtr caps,
                         def->keywrap->dea = VIR_TRISTATE_SWITCH_ABSENT;
                 }
             }
-            virStringFreeList(list);
+            virStringListFree(list);
             list = NULL;
         } else if (STREQ(arg, "-serial")) {
             WANT_VALUE();
@@ -2658,7 +2658,7 @@ qemuParseCommandLine(virCapsPtr caps,
     virDomainDiskDefFree(disk);
     qemuDomainCmdlineDefFree(cmd);
     virDomainDefFree(def);
-    virStringFreeList(list);
+    virStringListFree(list);
     VIR_FREE(nics);
     if (monConfig) {
         virDomainChrSourceDefFree(*monConfig);
@@ -2688,8 +2688,8 @@ virDomainDefPtr qemuParseCommandLineString(virCapsPtr caps,
                                pidfile, monConfig, monJSON);
 
  cleanup:
-    virStringFreeList(progargv);
-    virStringFreeList(progenv);
+    virStringListFree(progargv);
+    virStringListFree(progenv);
 
     return def;
 }
@@ -2736,7 +2736,7 @@ static int qemuParseProcFileStrings(int pid_value,
 
  cleanup:
     if (ret < 0)
-        virStringFreeList(str);
+        virStringListFree(str);
     VIR_FREE(data);
     VIR_FREE(path);
     return ret;
@@ -2780,7 +2780,7 @@ virDomainDefPtr qemuParseCommandLinePid(virCapsPtr caps,
 
  cleanup:
     VIR_FREE(exepath);
-    virStringFreeList(progargv);
-    virStringFreeList(progenv);
+    virStringListFree(progargv);
+    virStringListFree(progenv);
     return def;
 }
index f8f379a589546dffeec042c57beb35e9c18d32dd..ab0c2c8bfacd4e3e9397e945cbbfdeae86933288 100644 (file)
@@ -3164,7 +3164,7 @@ qemuProcessUpdateDevices(virQEMUDriverPtr driver,
     qemuDevices = (const char **) priv->qemuDevices;
     if ((tmp = old)) {
         while (*tmp) {
-            if (!virStringArrayHasString(qemuDevices, *tmp) &&
+            if (!virStringListHasString(qemuDevices, *tmp) &&
                 virDomainDefFindDevice(vm->def, *tmp, &dev, false) == 0 &&
                 qemuDomainRemoveDevice(driver, vm, &dev) < 0) {
                 goto cleanup;
@@ -3175,7 +3175,7 @@ qemuProcessUpdateDevices(virQEMUDriverPtr driver,
     ret = 0;
 
  cleanup:
-    virStringFreeList(old);
+    virStringListFree(old);
     return ret;
 }
 
@@ -5085,7 +5085,7 @@ qemuProcessUpdateGuestCPU(virDomainDefPtr def,
     ret = 0;
 
  cleanup:
-    virStringFreeListCount(models, nmodels);
+    virStringListFreeCount(models, nmodels);
     return ret;
 }
 
@@ -6067,7 +6067,7 @@ void qemuProcessStop(virQEMUDriverPtr driver,
         VIR_FREE(vm->def->seclabels[i]->imagelabel);
     }
 
-    virStringFreeList(priv->qemuDevices);
+    virStringListFree(priv->qemuDevices);
     priv->qemuDevices = NULL;
 
     qemuHostdevReAttachDomainDevices(driver, vm->def);
index db2bdd4e551be9d61e06421205242ac5a2d297ee..888052045eeb3a39d0e498a4c136fb2b30b1ac72 100644 (file)
@@ -6221,7 +6221,7 @@ remoteConnectGetCPUModelNames(virConnectPtr conn,
     rv = ret.ret;
 
  cleanup:
-    virStringFreeList(retmodels);
+    virStringListFree(retmodels);
 
     xdr_free((xdrproc_t) xdr_remote_connect_get_cpu_model_names_ret, (char *) &ret);
 
index 001d16f607a6d7fdd8990752d7883432d55ff97a..ad2d7562969e20542c5a690bcb872b6ede266d04 100644 (file)
@@ -173,7 +173,7 @@ virStorageBackendSheepdogRefreshAllVol(virConnectPtr conn ATTRIBUTE_UNUSED,
                 goto cleanup;
         }
 
-        virStringFreeList(cells);
+        virStringListFree(cells);
         cells = NULL;
     }
 
@@ -181,8 +181,8 @@ virStorageBackendSheepdogRefreshAllVol(virConnectPtr conn ATTRIBUTE_UNUSED,
 
  cleanup:
     virCommandFree(cmd);
-    virStringFreeList(lines);
-    virStringFreeList(cells);
+    virStringListFree(lines);
+    virStringListFree(cells);
     VIR_FREE(output);
     return ret;
 }
index a3477e28e925d29e3a5b2b3f953cc8793f676d60..3a43be41c117a2d199bd65321a379f6e04770e56 100644 (file)
@@ -168,8 +168,8 @@ virStorageBackendZFSParseVol(virStoragePoolObjPtr pool,
 
     ret = 0;
  cleanup:
-    virStringFreeList(tokens);
-    virStringFreeList(name_tokens);
+    virStringListFree(tokens);
+    virStringListFree(name_tokens);
     if (is_new_vol)
         virStorageVolDefFree(volume);
     return ret;
@@ -219,7 +219,7 @@ virStorageBackendZFSFindVols(virStoragePoolObjPtr pool,
 
  cleanup:
     virCommandFree(cmd);
-    virStringFreeList(lines);
+    virStringListFree(lines);
     VIR_FREE(volumes_list);
 
     return 0;
@@ -264,7 +264,7 @@ virStorageBackendZFSRefreshPool(virConnectPtr conn ATTRIBUTE_UNUSED,
         if (STREQ(lines[i], ""))
             continue;
 
-        virStringFreeList(tokens);
+        virStringListFree(tokens);
         if (!(tokens = virStringSplitCount(lines[i], "\t", 0, &count)))
             goto cleanup;
 
@@ -294,8 +294,8 @@ virStorageBackendZFSRefreshPool(virConnectPtr conn ATTRIBUTE_UNUSED,
 
  cleanup:
     virCommandFree(cmd);
-    virStringFreeList(lines);
-    virStringFreeList(tokens);
+    virStringListFree(lines);
+    virStringListFree(tokens);
     VIR_FREE(zpool_props);
 
     return 0;
index 9900596572d156826b3ed2f16bf126f59afb7b1b..df6580781d5cd03444747e7e4ade7e013ff8992d 100644 (file)
@@ -3392,7 +3392,7 @@ virStorageAddISCSIPoolSourceHost(virDomainDiskDefPtr def,
     ret = 0;
 
  cleanup:
-    virStringFreeList(tokens);
+    virStringListFree(tokens);
     return ret;
 }
 
index 92329949452f3c8fcc18774988c62d3c793d5c8e..f151193401584f7298946d514a71b53e719d4ef7 100644 (file)
@@ -863,14 +863,14 @@ virCgroupGetValueForBlkDev(virCgroupPtr group,
     if (!(lines = virStringSplit(str, "\n", -1)))
         goto error;
 
-    if (VIR_STRDUP(*value, virStringGetFirstWithPrefix(lines, prefix)) < 0)
+    if (VIR_STRDUP(*value, virStringListGetFirstWithPrefix(lines, prefix)) < 0)
         goto error;
 
     ret = 0;
  error:
     VIR_FREE(str);
     VIR_FREE(prefix);
-    virStringFreeList(lines);
+    virStringListFree(lines);
     return ret;
 }
 
@@ -1275,13 +1275,13 @@ virCgroupSetPartitionSuffix(const char *path, char **res)
             goto cleanup;
     }
 
-    if (!(*res = virStringJoin((const char **)tokens, "/")))
+    if (!(*res = virStringListJoin((const char **)tokens, "/")))
         goto cleanup;
 
     ret = 0;
 
  cleanup:
-    virStringFreeList(tokens);
+    virStringListFree(tokens);
     return ret;
 }
 
index 2a59bd1a4112f1cb2ee99d7edcb2bfa1e102a081..e1bbc05264d0e3ebe217dd606d3226c84f730a57 100644 (file)
@@ -3026,7 +3026,7 @@ virCommandRunRegex(virCommandPtr cmd,
 
     ret = 0;
  cleanup:
-    virStringFreeList(lines);
+    virStringListFree(lines);
     VIR_FREE(outbuf);
     if (groups) {
         for (j = 0; j < totgroups; j++)
index 3e49f41b603ed9752626e671cf0d449587c04273..a85a3079cbf5382853de701746e3a0577a6fd14f 100644 (file)
@@ -983,7 +983,7 @@ int virConfGetValueStringList(virConfPtr conf,
     if (!cval)
         return 0;
 
-    virStringFreeList(*values);
+    virStringListFree(*values);
     *values = NULL;
 
     switch (cval->type) {
@@ -1003,7 +1003,7 @@ int virConfGetValueStringList(virConfPtr conf,
 
         for (len = 0, eval = cval->list; eval; len++, eval = eval->next) {
             if (VIR_STRDUP((*values)[len], eval->str) < 0) {
-                virStringFreeList(*values);
+                virStringListFree(*values);
                 *values = NULL;
                 return -1;
             }
index f006abfcee4191d7d3f441738716b713debc466f..1fb89ce1ca85d950dc24f12013bc95fe854002f9 100644 (file)
@@ -1877,7 +1877,7 @@ virFileGetMountSubtreeImpl(const char *mtabpath,
 
  cleanup:
     if (ret < 0)
-        virStringFreeList(mounts);
+        virStringListFree(mounts);
     endmntent(procmnt);
     return ret;
 }
@@ -1906,7 +1906,7 @@ virFileGetMountSubtreeImpl(const char *mtabpath ATTRIBUTE_UNUSED,
  * the path @prefix, sorted from shortest to longest path.
  *
  * The @mountsret array will be NULL terminated and should
- * be freed with virStringFreeList
+ * be freed with virStringListFree
  *
  * Returns 0 on success, -1 on error
  */
@@ -1930,7 +1930,7 @@ int virFileGetMountSubtree(const char *mtabpath,
  * ie opposite order to which they appear in @mtabpath
  *
  * The @mountsret array will be NULL terminated and should
- * be freed with virStringFreeList
+ * be freed with virStringListFree
  *
  * Returns 0 on success, -1 on error
  */
index f26fd865c97bb2c595b962ae510f5795e0531bbc..3f976186a6714a1f51135a56e1f6f83c47e0b8b9 100644 (file)
@@ -878,7 +878,7 @@ virFirewallApplyRule(virFirewallPtr firewall,
 
     ret = 0;
  cleanup:
-    virStringFreeList(lines);
+    virStringListFree(lines);
     VIR_FREE(output);
     return ret;
 }
index 6b20c062fa5e211c7b5c47aa2484f64470cfeb48..86f8b684d6c1ecd62c5663b44d170298d97050f3 100644 (file)
@@ -87,7 +87,7 @@ virFirmwareParse(const char *str, virFirmwarePtr firmware)
 
     ret = 0;
  cleanup:
-    virStringFreeList(token);
+    virStringListFree(token);
     return ret;
 }
 
@@ -132,6 +132,6 @@ virFirmwareParseList(const char *list,
 
     ret = 0;
  cleanup:
-    virStringFreeList(token);
+    virStringListFree(token);
     return ret;
 }
index 8f831fc2b6c19b00c9f6b4cb2b82cf65615c55b6..77bc6b3c223fcbdf649e778d8b854769a9672533 100644 (file)
@@ -1530,7 +1530,7 @@ virLogParseOutput(const char *src)
     }
 
  cleanup:
-    virStringFreeList(tokens);
+    virStringListFree(tokens);
     return ret;
 }
 
@@ -1607,7 +1607,7 @@ virLogParseFilter(const char *src)
         goto cleanup;
 
  cleanup:
-    virStringFreeList(tokens);
+    virStringListFree(tokens);
     return ret;
 }
 
@@ -1667,7 +1667,7 @@ virLogParseOutputs(const char *src, virLogOutputPtr **outputs)
     *outputs = list;
     list = NULL;
  cleanup:
-    virStringFreeList(strings);
+    virStringListFree(strings);
     return ret;
 }
 
@@ -1717,7 +1717,7 @@ virLogParseFilters(const char *src, virLogFilterPtr **filters)
     *filters = list;
     list = NULL;
  cleanup:
-    virStringFreeList(strings);
+    virStringListFree(strings);
     return ret;
 }
 
index e7e46b8cac9fb9bda71d3dcf937f74a917291802..c735ca9b1fa61e9342c189caf4a221a7f4238f5a 100644 (file)
@@ -137,7 +137,7 @@ int virPolkitCheckAuth(const char *actionid,
     }
 
  cleanup:
-    virStringFreeListCount(retdetails, nretdetails);
+    virStringListFreeCount(retdetails, nretdetails);
     return ret;
 }
 
index 3cacc89789351156b0e4e3828759ebfd4b52a257..7db97bc5335616bab763fe86bf1e9054f4677181 100644 (file)
@@ -1014,7 +1014,7 @@ int virProcessGetStartTime(pid_t pid,
     ret = 0;
 
  cleanup:
-    virStringFreeList(tokens);
+    virStringListFree(tokens);
     VIR_FREE(filename);
     VIR_FREE(buf);
     return ret;
index f6a1b5d30f2fb067b6df91ab2c91483169eb80fc..1011bd063f13ef32e50f1dc58f75b8d76208ef57 100644 (file)
@@ -1476,7 +1476,7 @@ virStorageFileParseChainIndex(const char *diskTarget,
     *chainIndex = idx;
 
  cleanup:
-    virStringFreeList(strings);
+    virStringListFree(strings);
     return ret;
 }
 
@@ -2350,7 +2350,7 @@ virStorageSourceParseBackingURI(virStorageSourcePtr src,
 
  cleanup:
     virURIFree(uri);
-    virStringFreeList(scheme);
+    virStringListFree(scheme);
     return ret;
 }
 
@@ -2385,8 +2385,8 @@ virStorageSourceRBDAddHost(virStorageSourcePtr src,
     parts = virStringSplit(hostport, "\\:", 0);
     if (!parts)
         goto error;
-    src->hosts[src->nhosts-1].name = virStringJoin((const char **)parts, ":");
-    virStringFreeList(parts);
+    src->hosts[src->nhosts-1].name = virStringListJoin((const char **)parts, ":");
+    virStringListFree(parts);
     if (!src->hosts[src->nhosts-1].name)
         goto error;
 
@@ -2576,7 +2576,7 @@ virStorageSourceParseNBDColonString(const char *nbdstr,
     ret = 0;
 
  cleanup:
-    virStringFreeList(backing);
+    virStringListFree(backing);
 
     return ret;
 }
@@ -3272,7 +3272,7 @@ virStorageFileCanonicalizeInjectSymlink(const char *path,
     ret = 0;
 
  cleanup:
-    virStringFreeListCount(tmp, ntmp);
+    virStringListFreeCount(tmp, ntmp);
     return ret;
 }
 
@@ -3420,7 +3420,7 @@ virStorageFileCanonicalizePath(const char *path,
 
  cleanup:
     virHashFree(cycle);
-    virStringFreeListCount(components, ncomponents);
+    virStringListFreeCount(components, ncomponents);
     VIR_FREE(linkpath);
     VIR_FREE(currentpath);
 
@@ -3534,6 +3534,6 @@ virStorageFileCheckCompat(const char *compat)
     ret = 0;
 
  cleanup:
-    virStringFreeList(version);
+    virStringListFree(version);
     return ret;
 }
index f93d6b1259db2232179f587577db4f62a592ae39..2ab252cbd298a60d555fcff4b8c46f591b4b8d34 100644 (file)
@@ -38,7 +38,7 @@
 VIR_LOG_INIT("util.string");
 
 /*
- * The following virStringSplit & virStringJoin methods
+ * The following virStringSplit & virStringListJoin methods
  * are derived from g_strsplit / g_strjoin in glib2,
  * also available under the LGPLv2+ license terms
  */
@@ -66,7 +66,7 @@ VIR_LOG_INIT("util.string");
  * before calling virStringSplit().
  *
  * Return value: a newly-allocated NULL-terminated array of strings. Use
- *    virStringFreeList() to free it.
+ *    virStringListFree() to free it.
  */
 char **
 virStringSplitCount(const char *string,
@@ -137,7 +137,7 @@ virStringSplit(const char *string,
 
 
 /**
- * virStringJoin:
+ * virStringListJoin:
  * @strings: a NULL-terminated array of strings to join
  * @delim: a string to insert between each of the strings
  *
@@ -148,8 +148,8 @@ virStringSplit(const char *string,
  * Returns: a newly-allocated string containing all of the strings joined
  *     together, with @delim between them
  */
-char *virStringJoin(const char **strings,
-                    const char *delim)
+char *virStringListJoin(const char **strings,
+                        const char *delim)
 {
     char *ret;
     virBuffer buf = VIR_BUFFER_INITIALIZER;
@@ -169,13 +169,13 @@ char *virStringJoin(const char **strings,
 
 
 /**
- * virStringFreeList:
+ * virStringListFree:
  * @str_array: a NULL-terminated array of strings to free
  *
  * Frees a NULL-terminated array of strings, and the array itself.
- * If called on a NULL value, virStringFreeList() simply returns.
+ * If called on a NULL value, virStringListFree() simply returns.
  */
-void virStringFreeList(char **strings)
+void virStringListFree(char **strings)
 {
     char **tmp = strings;
     while (tmp && *tmp) {
@@ -187,14 +187,14 @@ void virStringFreeList(char **strings)
 
 
 /**
- * virStringFreeListCount:
+ * virStringListFreeCount:
  * @strings: array of strings to free
  * @count: number of elements in the array
  *
  * Frees a string array of @count length.
  */
 void
-virStringFreeListCount(char **strings,
+virStringListFreeCount(char **strings,
                        size_t count)
 {
     size_t i;
@@ -210,8 +210,8 @@ virStringFreeListCount(char **strings,
 
 
 bool
-virStringArrayHasString(const char **strings,
-                        const char *needle)
+virStringListHasString(const char **strings,
+                       const char *needle)
 {
     size_t i = 0;
 
@@ -227,7 +227,8 @@ virStringArrayHasString(const char **strings,
 }
 
 char *
-virStringGetFirstWithPrefix(char **strings, const char *prefix)
+virStringListGetFirstWithPrefix(char **strings,
+                                const char *prefix)
 {
     size_t i = 0;
 
@@ -814,7 +815,7 @@ int virStringSortRevCompare(const void *a, const void *b)
  * @result: pointer to an array to be filled with NULL terminated list of matches
  *
  * Performs a POSIX extended regex search against a string and return all matching substrings.
- * The @result value should be freed with virStringFreeList() when no longer
+ * The @result value should be freed with virStringListFree() when no longer
  * required.
  *
  * @code
@@ -834,7 +835,7 @@ int virStringSortRevCompare(const void *a, const void *b)
  *  // matches[2] == "bbb3c75c-d60f-43b0-b802-fd56b84a4222"
  *  // matches[3] == NULL;
  *
- *  virStringFreeList(matches);
+ *  virStringListFree(matches);
  * @endcode
  *
  * Returns: -1 on error, or number of matches
@@ -902,7 +903,7 @@ virStringSearch(const char *str,
  cleanup:
     regfree(&re);
     if (ret < 0) {
-        virStringFreeList(*matches);
+        virStringListFree(*matches);
         *matches = NULL;
     }
     return ret;
index 8854d5fa624b4e10e02797f1f37cc20dcbdc6e65..f6801fcd1484a44a4094a513cfa33ef4fc7de530 100644 (file)
@@ -37,15 +37,18 @@ char **virStringSplit(const char *string,
                       size_t max_tokens)
     ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);
 
-char *virStringJoin(const char **strings,
-                    const char *delim)
+char *virStringListJoin(const char **strings,
+                        const char *delim)
     ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);
 
-void virStringFreeList(char **strings);
-void virStringFreeListCount(char **strings, size_t count);
+void virStringListFree(char **strings);
+void virStringListFreeCount(char **strings,
+                            size_t count);
 
-bool virStringArrayHasString(const char **strings, const char *needle);
-char *virStringGetFirstWithPrefix(char **strings, const char *prefix)
+bool virStringListHasString(const char **strings,
+                            const char *needle);
+char *virStringListGetFirstWithPrefix(char **strings,
+                                      const char *prefix)
     ATTRIBUTE_NONNULL(2);
 
 char *virArgvToString(const char *const *argv);
index 2f252ac31b2238f765c7a4d7e6a0eae7d02a5162..b80d9249e6383ac1d7435ec1b04f135a084d0880 100644 (file)
@@ -386,7 +386,7 @@ virURIResolveAlias(virConfPtr conf, const char *alias, char **uri)
 
     if (aliases && *aliases) {
         ret = virURIFindAliasMatch(aliases, alias, uri);
-        virStringFreeList(aliases);
+        virStringListFree(aliases);
     } else {
         ret = 0;
     }
index 7ae3dc7d9102dfb15b9c84b3bf4faa3004e2efbd..af8a9f77ad23987831a4494a034e143bddcfe22d 100644 (file)
@@ -4933,7 +4933,7 @@ vboxSnapshotRedefine(virDomainPtr dom,
         tmp = virStringReplace(newSnapshotPtr->storageController,
                                searchResultTab[it],
                                uuidReplacing);
-        virStringFreeList(searchResultTab);
+        virStringListFree(searchResultTab);
         searchResultTab = NULL;
         VIR_FREE(newSnapshotPtr->storageController);
         if (!tmp)
@@ -5341,9 +5341,9 @@ vboxSnapshotRedefine(virDomainPtr dom,
     VIR_FREE(currentSnapshotXmlFilePath);
     VBOX_UTF16_FREE(machineNameUtf16);
     VBOX_UTF8_FREE(machineName);
-    virStringFreeList(realReadOnlyDisksPath);
-    virStringFreeList(realReadWriteDisksPath);
-    virStringFreeList(searchResultTab);
+    virStringListFree(realReadOnlyDisksPath);
+    virStringListFree(realReadWriteDisksPath);
+    virStringListFree(searchResultTab);
     VIR_FREE(newSnapshotPtr);
     VIR_FREE(machineLocationPath);
     VIR_FREE(nameTmpUse);
@@ -7187,7 +7187,7 @@ vboxDomainSnapshotDeleteMetadataOnly(virDomainSnapshotPtr snapshot)
     VBOX_RELEASE(machine);
     VBOX_UTF16_FREE(settingsFilePathUtf16);
     VBOX_UTF8_FREE(settingsFilepath);
-    virStringFreeList(searchResultTab);
+    virStringListFree(searchResultTab);
     VIR_FREE(snapshotMachineDesc);
     VBOX_UTF16_FREE(machineNameUtf16);
     VBOX_UTF8_FREE(machineName);
index e1307e1009b4f28dc68d002b8edde829f1ba815f..8bf7ef327215abbf1bb2dbe4745d854e69ddc904 100644 (file)
@@ -107,7 +107,7 @@ virVBoxSnapshotConfCreateVBoxSnapshotConfHardDiskPtr(xmlNodePtr diskNode,
     VIR_FREE(nodes);
     VIR_FREE(location);
     VIR_FREE(tmp);
-    virStringFreeList(searchTabResult);
+    virStringListFree(searchTabResult);
     if (result < 0) {
         virVboxSnapshotConfHardDiskFree(hardDisk);
         hardDisk = NULL;
@@ -276,7 +276,7 @@ virVBoxSnapshotConfRetrieveSnapshot(xmlNodePtr snapshotNode,
     }
     VIR_FREE(nodes);
     VIR_FREE(uuid);
-    virStringFreeList(searchTabResult);
+    virStringListFree(searchTabResult);
     return snapshot;
 }
 
@@ -458,8 +458,8 @@ virVBoxSnapshotConfSerializeSnapshot(xmlNodePtr node,
         xmlUnlinkNode(snapshotsNode);
         xmlFreeNode(snapshotsNode);
     }
-    virStringFreeList(firstRegex);
-    virStringFreeList(secondRegex);
+    virStringListFree(firstRegex);
+    virStringListFree(secondRegex);
     VIR_FREE(uuid);
     VIR_FREE(timeStamp);
     return result;
@@ -739,7 +739,7 @@ virVBoxSnapshotConfLoadVboxFile(const char *filePath,
 
     VIR_FREE(currentStateModifiedString);
     VIR_FREE(currentSnapshotAttribute);
-    virStringFreeList(searchResultTab);
+    virStringListFree(searchResultTab);
     if (ret < 0) {
         virVBoxSnapshotConfMachineFree(machineDescription);
         machineDescription = NULL;
@@ -1235,8 +1235,8 @@ virVBoxSnapshotConfSaveVboxFile(virVBoxSnapshotConfMachinePtr machine,
 
     xmlFreeDoc(xml);
 
-    virStringFreeList(firstRegex);
-    virStringFreeList(secondRegex);
+    virStringListFree(firstRegex);
+    virStringListFree(secondRegex);
     return ret;
 }
 
@@ -1324,7 +1324,7 @@ virVBoxSnapshotConfGetRWDisksPathsFromLibvirtXML(const char *filePath,
     xmlFreeDoc(xml);
     xmlXPathFreeContext(xPathContext);
     if (result < 0) {
-        virStringFreeList(ret);
+        virStringListFree(ret);
         nodeSize = -1;
     } else {
         *rwDisksPath = ret;
@@ -1385,7 +1385,7 @@ virVBoxSnapshotConfGetRODisksPathsFromLibvirtXML(const char *filePath,
     xmlFreeDoc(xml);
     xmlXPathFreeContext(xPathContext);
     if (result < 0) {
-        virStringFreeList(ret);
+        virStringListFree(ret);
         nodeSize = -1;
     } else {
         *roDisksPath = ret;
index d61bccf06f2467b0b16d80719d52ad04cc4fc968..d7b754e1def571b23cc592297648d999b4c7e946 100644 (file)
@@ -730,7 +730,7 @@ prlsdkGetFSInfo(PRL_HANDLE prldisk,
 
  cleanup:
     VIR_FREE(buf);
-    virStringFreeList(matches);
+    virStringListFree(matches);
     return ret;
 }
 
index 650c56532daa463ff444d9c4b13af07b9be3b7ef..f5fec47520ff0eafb49c0cbe02d3756e48f9b9ca 100644 (file)
@@ -65,7 +65,7 @@ testFilterXML(char *xml)
 
  cleanup:
     virBufferFreeAndReset(&buf);
-    virStringFreeList(xmlLines);
+    virStringListFree(xmlLines);
     return ret;
 }
 
index a7d4d5a1455ee1e4dd30bc6dfc86a53e1d3894ca..f729b2eda00e94561d5c65817c38e9d531053a50 100644 (file)
@@ -591,7 +591,7 @@ testQemuMonitorJSONGetTPMModels(const void *data)
 
  cleanup:
     qemuMonitorTestFree(test);
-    virStringFreeList(tpmmodels);
+    virStringListFree(tpmmodels);
     return ret;
 }
 
@@ -651,7 +651,7 @@ testQemuMonitorJSONGetCommandLineOptionParameters(const void *data)
 
 #undef CHECK
 
-    virStringFreeList(params);
+    virStringListFree(params);
     params = NULL;
 
     /* present but empty */
@@ -677,7 +677,7 @@ testQemuMonitorJSONGetCommandLineOptionParameters(const void *data)
         goto cleanup;
     }
 
-    virStringFreeList(params);
+    virStringListFree(params);
     params = NULL;
 
     /* no such option */
@@ -707,7 +707,7 @@ testQemuMonitorJSONGetCommandLineOptionParameters(const void *data)
 
  cleanup:
     qemuMonitorTestFree(test);
-    virStringFreeList(params);
+    virStringListFree(params);
     return ret;
 }
 
@@ -1151,20 +1151,20 @@ testQemuMonitorJSONGetDeviceAliases(const void *data)
 
     ret = 0;
     for (alias = (const char **) aliases; *alias; alias++) {
-        if (!virStringArrayHasString(expected, *alias)) {
+        if (!virStringListHasString(expected, *alias)) {
             fprintf(stderr, "got unexpected device alias '%s'\n", *alias);
             ret = -1;
         }
     }
     for (alias = expected; *alias; alias++) {
-        if (!virStringArrayHasString((const char **) aliases, *alias)) {
+        if (!virStringListHasString((const char **) aliases, *alias)) {
             fprintf(stderr, "missing expected alias '%s'\n", *alias);
             ret = -1;
         }
     }
 
  cleanup:
-    virStringFreeList(aliases);
+    virStringListFree(aliases);
     qemuMonitorTestFree(test);
     return ret;
 }
index 19546969938563f85343d0c9735b8542255059d9..e0ef47f09e5a1f49378eccda4281b82eec204d2f 100644 (file)
@@ -160,7 +160,7 @@ fakeStorageVolLookupByName(virStoragePoolPtr pool,
                            NULL, NULL);
 
  cleanup:
-    virStringFreeList(volinfo);
+    virStringListFree(volinfo);
     return ret;
 
  fallback:
index fa7bbfe4a30d790a539745743ce7ad8d22e5e95d..f84d199e6077c35589640de4c42be70f3309fa02 100644 (file)
@@ -47,7 +47,7 @@ testFilterXML(char *xml)
 
  cleanup:
    virBufferFreeAndReset(&buf);
-   virStringFreeList(xmlLines);
+   virStringListFree(xmlLines);
    return ret;
 }
 
index dbf6ed669333dd0bf7e513d3475bd4fb12bf20e7..4fc7993ddc2b04bef798480bce0618cd6f8d1e0a 100644 (file)
@@ -394,7 +394,7 @@ static int testConfParseStringList(const void *opaque ATTRIBUTE_UNUSED)
 
     ret = 0;
  cleanup:
-    virStringFreeList(str);
+    virStringListFree(str);
     virConfFree(conf);
     return ret;
 }
index a6458df5b87f69012963cfeaf3cbea1b91c11b0d..05148e94ad048ca47dec8333a0b13f3e3735c360 100644 (file)
@@ -84,7 +84,7 @@ static int testFileGetMountSubtree(const void *opaque)
                               data->mounts, data->nmounts);
 
  cleanup:
-    virStringFreeList(gotmounts);
+    virStringListFree(gotmounts);
     return ret;
 }
 #endif /* ! defined HAVE_MNTENT_H && defined HAVE_GETMNTENT_R */
index 421ffee5260b2fc1e134777fad0664a12bb72beb..a51dd1c2d41946da6e2c7c062e2d9533fb959156 100644 (file)
@@ -124,7 +124,7 @@ VIR_MOCK_WRAP_RET_ARGS(dbus_connection_send_with_reply_and_block,
         VIR_FREE(uidkey);
         VIR_FREE(actionid);
         VIR_FREE(cancellationId);
-        virStringFreeListCount(details, detailslen);
+        virStringListFreeCount(details, detailslen);
 
         if (virDBusCreateReply(&reply,
                                "(bba&{ss})",
index f7e1e92be4185612e4c224c86f0fd5546abd29ff..a11d7c5d87055e21a970378e6caa392bbd03f1e0 100644 (file)
@@ -94,7 +94,7 @@ static int testSplit(const void *args)
 
     ret = 0;
  cleanup:
-    virStringFreeList(got);
+    virStringListFree(got);
 
     return ret;
 }
@@ -106,7 +106,7 @@ static int testJoin(const void *args)
     char *got;
     int ret = -1;
 
-    if (!(got = virStringJoin(data->tokens, data->delim))) {
+    if (!(got = virStringListJoin(data->tokens, data->delim))) {
         VIR_DEBUG("Got no result");
         return -1;
     }
@@ -354,7 +354,7 @@ testStringSearch(const void *opaque)
     ret = 0;
 
  cleanup:
-    virStringFreeList(matches);
+    virStringListFree(matches);
     return ret;
 }
 
@@ -502,7 +502,7 @@ testStringToLong(const void *opaque)
 /* The point of this test is to check whether all members of the array are
  * freed. The test has to be checked using valgrind. */
 static int
-testVirStringFreeListCount(const void *opaque ATTRIBUTE_UNUSED)
+testVirStringListFreeCount(const void *opaque ATTRIBUTE_UNUSED)
 {
     char **list;
 
@@ -513,7 +513,7 @@ testVirStringFreeListCount(const void *opaque ATTRIBUTE_UNUSED)
     ignore_value(VIR_STRDUP(list[2], "test2"));
     ignore_value(VIR_STRDUP(list[3], "test3"));
 
-    virStringFreeListCount(list, 4);
+    virStringListFreeCount(list, 4);
 
     return 0;
 }
@@ -779,8 +779,8 @@ mymain(void)
     TEST_STRTOL("-18446744073709551616", NULL, 0, -1, 0U, -1,
                 0LL, -1, 0ULL, -1);
 
-    /* test virStringFreeListCount */
-    if (virTestRun("virStringFreeListCount", testVirStringFreeListCount,
+    /* test virStringListFreeCount */
+    if (virTestRun("virStringListFreeCount", testVirStringListFreeCount,
                    NULL) < 0)
         ret = -1;
 
index 242f15ef9768e2d8dae108edf0fa90ba5d2ac1e8..e359bfc8328b62b514243b03415725d9b1d5849d 100644 (file)
@@ -878,7 +878,7 @@ virshParseRateStr(vshControl *ctl,
 
     ret = 0;
  cleanup:
-    virStringFreeList(tok);
+    virStringListFree(tok);
     return ret;
 }
 
@@ -4043,14 +4043,14 @@ cmdStartGetFDs(vshControl *ctl,
         fds[nfds - 1] = fd;
     }
 
-    virStringFreeList(fdlist);
+    virStringListFree(fdlist);
 
     *fdsret = fds;
     *nfdsret = nfds;
     return 0;
 
  error:
-    virStringFreeList(fdlist);
+    virStringListFree(fdlist);
     VIR_FREE(fds);
     return -1;
 }
@@ -5570,7 +5570,7 @@ cmdShutdown(vshControl *ctl, const vshCmd *cmd)
  cleanup:
     if (dom)
         virDomainFree(dom);
-    virStringFreeList(modes);
+    virStringListFree(modes);
     return ret;
 }
 
@@ -5650,7 +5650,7 @@ cmdReboot(vshControl *ctl, const vshCmd *cmd)
  cleanup:
     if (dom)
         virDomainFree(dom);
-    virStringFreeList(modes);
+    virStringListFree(modes);
     return ret;
 }
 
@@ -8827,7 +8827,7 @@ virshParseEventStr(const char *event,
 
     ret = 0;
  cleanup:
-    virStringFreeList(tok);
+    virStringListFree(tok);
     return ret;
 }
 
index 9446664e386000be5b24f1e00be8aa9b56a4f447..f76cade74d47143604b29183607ac8da12c13a73 100644 (file)
@@ -156,7 +156,7 @@ cmdNodeDeviceDestroy(vshControl *ctl, const vshCmd *cmd)
 
     ret = true;
  cleanup:
-    virStringFreeList(arr);
+    virStringListFree(arr);
     if (dev)
         virNodeDeviceFree(dev);
     return ret;
@@ -498,7 +498,7 @@ cmdNodeListDevices(vshControl *ctl, const vshCmd *cmd ATTRIBUTE_UNUSED)
     }
 
  cleanup:
-    virStringFreeList(caps);
+    virStringListFree(caps);
     virshNodeDeviceListFree(list);
     return ret;
 }
@@ -567,7 +567,7 @@ cmdNodeDeviceDumpXML(vshControl *ctl, const vshCmd *cmd)
 
     ret = true;
  cleanup:
-    virStringFreeList(arr);
+    virStringListFree(arr);
     VIR_FREE(xml);
     if (device)
         virNodeDeviceFree(device);
index 7118a6cc18893654e95830d6d26d373da09cbb02..8313be8684421e48d91ca3b48abd7d95c137092d 100644 (file)
@@ -1125,7 +1125,7 @@ cmdPoolList(vshControl *ctl, const vshCmd *cmd ATTRIBUTE_UNUSED)
         for (i = 0; i < npoolTypes; i++) {
             if ((poolType = virStoragePoolTypeFromString(poolTypes[i])) < 0) {
                 vshError(ctl, _("Invalid pool type '%s'"), poolTypes[i]);
-                virStringFreeList(poolTypes);
+                virStringListFree(poolTypes);
                 return false;
             }
 
@@ -1170,7 +1170,7 @@ cmdPoolList(vshControl *ctl, const vshCmd *cmd ATTRIBUTE_UNUSED)
                 break;
             }
         }
-        virStringFreeList(poolTypes);
+        virStringListFree(poolTypes);
     }
 
     if (!(list = virshStoragePoolListCollect(ctl, flags)))
index e4d0df7a2b0d5f5b6f17829e52dcc239dd4c3fb4..5c844a5ea8446d75dd2df9ff227c1e4650a2e342 100644 (file)
@@ -255,7 +255,7 @@ virshParseSnapshotMemspec(vshControl *ctl, virBufferPtr buf, const char *str)
  cleanup:
     if (ret < 0)
         vshError(ctl, _("unable to parse memspec: %s"), str);
-    virStringFreeList(array);
+    virStringListFree(array);
     return ret;
 }
 
@@ -306,7 +306,7 @@ virshParseSnapshotDiskspec(vshControl *ctl, virBufferPtr buf, const char *str)
  cleanup:
     if (ret < 0)
         vshError(ctl, _("unable to parse diskspec: %s"), str);
-    virStringFreeList(array);
+    virStringListFree(array);
     return ret;
 }
 
index 4bfc52af324d9d0edb651fdf450a82bfc539e6d8..6faed04586d7261769d6084e21a3b79cb773301d 100644 (file)
@@ -245,7 +245,7 @@ virBitmapPtr virHostValidateGetCPUFlags(void)
                 ignore_value(virBitmapSetBit(flags, value));
         }
 
-        virStringFreeListCount(tokens, ntokens);
+        virStringListFreeCount(tokens, ntokens);
     } while (1);
 
  cleanup:
@@ -345,7 +345,7 @@ static int virHostValidateCGroupSupport(const char *hvname,
                 matched = true;
         }
 
-        virStringFreeListCount(cgroups, ncgroups);
+        virStringListFreeCount(cgroups, ncgroups);
     }
 
     VIR_FREE(line);
@@ -394,7 +394,7 @@ static int virHostValidateCGroupMount(const char *hvname,
                 matched = true;
         }
 
-        virStringFreeListCount(opts, nopts);
+        virStringListFreeCount(opts, nopts);
     }
     endmntent(fp);
     if (!matched)
index a2b32ac0c4cb2cf381bdef708bd521afc7a8625a..016e2b6d4e196094fa59340d34736fc93610bc5e 100644 (file)
@@ -90,7 +90,7 @@ static int virLoginShellAllowedUser(virConfPtr conf,
                          name, conf_file);
  cleanup:
     VIR_FREE(gname);
-    virStringFreeList(users);
+    virStringListFree(users);
     return ret;
 }
 
@@ -308,7 +308,7 @@ main(int argc, char **argv)
     if (autoshell) {
         tmp = virGetUserShell(uid);
         if (tmp) {
-            virStringFreeList(shargv);
+            virStringListFree(shargv);
             shargvlen = 1;
             if (VIR_ALLOC_N(shargv[0], shargvlen + 1) < 0) {
                 VIR_FREE(tmp);
@@ -394,7 +394,7 @@ main(int argc, char **argv)
         virDomainFree(dom);
     if (conn)
         virConnectClose(conn);
-    virStringFreeList(shargv);
+    virStringListFree(shargv);
     VIR_FREE(shcmd);
     VIR_FREE(term);
     VIR_FREE(name);
index f94eaa33d386cfed4311d0d3a61e2e69be1930a3..c1fc041cf8ffc1ee9e9206e5208c9a464415884e 100644 (file)
@@ -179,7 +179,7 @@ vshNameSorter(const void *a, const void *b)
 /*
  * Convert the strings separated by ',' into array. The returned
  * array is a NULL terminated string list. The caller has to free
- * the array using virStringFreeList or a similar method.
+ * the array using virStringListFree or a similar method.
  *
  * Returns the length of the filled array on success, or -1
  * on error.
@@ -2835,7 +2835,7 @@ vshReadlineParse(const char *text, int state)
                 return res;
             }
             res = NULL;
-            virStringFreeList(completed_list);
+            virStringListFree(completed_list);
             completed_list_index = 0;
         }
     }