diff --git a/src/bhyve/bhyve_capabilities.c b/src/bhyve/bhyve_capabilities.c index e9b4e5176d..efd6a59aec 100644 --- a/src/bhyve/bhyve_capabilities.c +++ b/src/bhyve/bhyve_capabilities.c @@ -154,10 +154,7 @@ virBhyveDomainCapsBuild(bhyveConnPtr conn, if (virDirOpenIfExists(&dir, firmware_dir) > 0) { while ((virDirRead(dir, &entry, firmware_dir)) > 0) { - if (VIR_RESIZE_N(firmwares->values, - firmwares_alloc, firmwares->nvalues, 1) < 0) - goto cleanup; - + VIR_RESIZE_N(firmwares->values, firmwares_alloc, firmwares->nvalues, 1); firmwares->values[firmwares->nvalues] = g_strdup_printf("%s/%s", firmware_dir, entry->d_name); firmwares->nvalues++; diff --git a/src/bhyve/bhyve_parse_command.c b/src/bhyve/bhyve_parse_command.c index 2762b7e921..8d9a21e671 100644 --- a/src/bhyve/bhyve_parse_command.c +++ b/src/bhyve/bhyve_parse_command.c @@ -153,10 +153,7 @@ bhyveCommandLineToArgv(const char *nativeConfig, else line = g_strdup(curr); - if (VIR_RESIZE_N(lines, lines_alloc, line_count, 2) < 0) { - VIR_FREE(line); - goto error; - } + VIR_RESIZE_N(lines, lines_alloc, line_count, 2); if (*line) lines[line_count++] = line; @@ -203,11 +200,7 @@ bhyveCommandLineToArgv(const char *nativeConfig, if (next && (*next == '\'' || *next == '"')) next++; - if (VIR_RESIZE_N(arglist, args_alloc, args_count, 2) < 0) { - VIR_FREE(arg); - goto error; - } - + VIR_RESIZE_N(arglist, args_alloc, args_count, 2); arglist[args_count++] = arg; arglist[args_count] = NULL; diff --git a/src/conf/capabilities.c b/src/conf/capabilities.c index f610975ae5..14f0d48930 100644 --- a/src/conf/capabilities.c +++ b/src/conf/capabilities.c @@ -280,10 +280,8 @@ int virCapabilitiesAddHostFeature(virCapsPtr caps, const char *name) { - if (VIR_RESIZE_N(caps->host.features, caps->host.nfeatures_max, - caps->host.nfeatures, 1) < 0) - return -1; - + VIR_RESIZE_N(caps->host.features, caps->host.nfeatures_max, + caps->host.nfeatures, 1); caps->host.features[caps->host.nfeatures] = g_strdup(name); caps->host.nfeatures++; @@ -301,10 +299,8 @@ int virCapabilitiesAddHostMigrateTransport(virCapsPtr caps, const char *name) { - if (VIR_RESIZE_N(caps->host.migrateTrans, caps->host.nmigrateTrans_max, - caps->host.nmigrateTrans, 1) < 0) - return -1; - + VIR_RESIZE_N(caps->host.migrateTrans, caps->host.nmigrateTrans_max, + caps->host.nmigrateTrans, 1); caps->host.migrateTrans[caps->host.nmigrateTrans] = g_strdup(name); caps->host.nmigrateTrans++; @@ -447,9 +443,7 @@ virCapabilitiesAddGuest(virCapsPtr caps, guest->arch.defaultInfo.emulator = g_strdup(emulator); guest->arch.defaultInfo.loader = g_strdup(loader); - if (VIR_RESIZE_N(caps->guests, caps->nguests_max, - caps->nguests, 1) < 0) - goto error; + VIR_RESIZE_N(caps->guests, caps->nguests_max, caps->nguests, 1); caps->guests[caps->nguests++] = guest; if (nmachines) { @@ -458,10 +452,6 @@ virCapabilitiesAddGuest(virCapsPtr caps, } return guest; - - error: - virCapabilitiesFreeGuest(guest); - return NULL; } @@ -493,9 +483,8 @@ virCapabilitiesAddGuestDomain(virCapsGuestPtr guest, dom->info.emulator = g_strdup(emulator); dom->info.loader = g_strdup(loader); - if (VIR_RESIZE_N(guest->arch.domains, guest->arch.ndomains_max, - guest->arch.ndomains, 1) < 0) - goto error; + VIR_RESIZE_N(guest->arch.domains, guest->arch.ndomains_max, + guest->arch.ndomains, 1); guest->arch.domains[guest->arch.ndomains] = dom; guest->arch.ndomains++; @@ -505,10 +494,6 @@ virCapabilitiesAddGuestDomain(virCapsGuestPtr guest, } return dom; - - error: - virCapabilitiesFreeGuestDomain(dom); - return NULL; } @@ -803,15 +788,10 @@ virCapabilitiesAddStoragePool(virCapsPtr caps, pool->type = poolType; - if (VIR_RESIZE_N(caps->pools, caps->npools_max, caps->npools, 1) < 0) - goto error; + VIR_RESIZE_N(caps->pools, caps->npools_max, caps->npools, 1); caps->pools[caps->npools++] = pool; return 0; - - error: - virCapabilitiesFreeStoragePool(pool); - return -1; } diff --git a/src/conf/cpu_conf.c b/src/conf/cpu_conf.c index 8d800bd80d..380a74691d 100644 --- a/src/conf/cpu_conf.c +++ b/src/conf/cpu_conf.c @@ -593,10 +593,7 @@ virCPUDefParseXML(xmlXPathContextPtr ctxt, return -1; } - if (VIR_RESIZE_N(def->features, def->nfeatures_max, - def->nfeatures, n) < 0) - return -1; - + VIR_RESIZE_N(def->features, def->nfeatures_max, def->nfeatures, n); def->nfeatures = n; } @@ -898,12 +895,8 @@ virCPUDefAddFeatureInternal(virCPUDefPtr def, } } - if (VIR_RESIZE_N(def->features, def->nfeatures_max, - def->nfeatures, 1) < 0) - return -1; - + VIR_RESIZE_N(def->features, def->nfeatures_max, def->nfeatures, 1); def->features[def->nfeatures].name = g_strdup(name); - def->features[def->nfeatures].policy = policy; def->nfeatures++; diff --git a/src/conf/domain_capabilities.c b/src/conf/domain_capabilities.c index fc48cc3fba..e0d7eddd2c 100644 --- a/src/conf/domain_capabilities.c +++ b/src/conf/domain_capabilities.c @@ -191,9 +191,8 @@ virDomainCapsCPUModelsAdd(virDomainCapsCPUModelsPtr cpuModels, nameCopy = g_strdup(name); - if (VIR_RESIZE_N(cpuModels->models, cpuModels->nmodels_max, - cpuModels->nmodels, 1) < 0) - return -1; + VIR_RESIZE_N(cpuModels->models, cpuModels->nmodels_max, + cpuModels->nmodels, 1); cpu = cpuModels->models + cpuModels->nmodels; cpuModels->nmodels++; diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c index 96cc8bda0d..9d73bc6de8 100644 --- a/src/esx/esx_driver.c +++ b/src/esx/esx_driver.c @@ -4917,8 +4917,7 @@ esxConnectListAllDomains(virConnectPtr conn, continue; } - if (VIR_RESIZE_N(doms, ndoms, count, 2) < 0) - goto cleanup; + VIR_RESIZE_N(doms, ndoms, count, 2); /* Only running/suspended virtual machines have an ID != -1 */ if (powerState == esxVI_VirtualMachinePowerState_PoweredOff) diff --git a/src/hyperv/hyperv_driver.c b/src/hyperv/hyperv_driver.c index e4f537bd12..17f5be1f0d 100644 --- a/src/hyperv/hyperv_driver.c +++ b/src/hyperv/hyperv_driver.c @@ -3524,8 +3524,7 @@ hypervConnectListAllDomains(virConnectPtr conn, continue; } - if (VIR_RESIZE_N(doms, ndoms, count, 2) < 0) - goto cleanup; + VIR_RESIZE_N(doms, ndoms, count, 2); domain = NULL; diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index f6bd5403ae..5f0c7f0531 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -4287,10 +4287,8 @@ qemuDomainDefCPUPostParse(virDomainDefPtr def, sveFeature->policy = VIR_CPU_FEATURE_REQUIRE; } } else { - if (VIR_RESIZE_N(def->cpu->features, def->cpu->nfeatures_max, - def->cpu->nfeatures, 1) < 0) { - return -1; - } + VIR_RESIZE_N(def->cpu->features, def->cpu->nfeatures_max, + def->cpu->nfeatures, 1); def->cpu->features[def->cpu->nfeatures].name = g_strdup("sve"); def->cpu->features[def->cpu->nfeatures].policy = VIR_CPU_FEATURE_REQUIRE; @@ -7340,10 +7338,9 @@ qemuDomainCleanupAdd(virDomainObjPtr vm, return 0; } - if (VIR_RESIZE_N(priv->cleanupCallbacks, - priv->ncleanupCallbacks_max, - priv->ncleanupCallbacks, 1) < 0) - return -1; + VIR_RESIZE_N(priv->cleanupCallbacks, + priv->ncleanupCallbacks_max, + priv->ncleanupCallbacks, 1); priv->cleanupCallbacks[priv->ncleanupCallbacks++] = cb; return 0; diff --git a/src/util/virbitmap.c b/src/util/virbitmap.c index b2569b0b03..9242af6fb5 100644 --- a/src/util/virbitmap.c +++ b/src/util/virbitmap.c @@ -139,9 +139,8 @@ virBitmapExpand(virBitmapPtr map, /* resize the memory if necessary */ if (map->map_len < new_len) { - if (VIR_RESIZE_N(map->map, map->map_alloc, map->map_len, - new_len - map->map_len) < 0) - return -1; + VIR_RESIZE_N(map->map, map->map_alloc, map->map_len, + new_len - map->map_len); } map->nbits = b + 1; diff --git a/src/util/vircommand.c b/src/util/vircommand.c index 99bb8b6506..d647463f35 100644 --- a/src/util/vircommand.c +++ b/src/util/vircommand.c @@ -1358,7 +1358,7 @@ virCommandAddEnv(virCommandPtr cmd, } /* Arg plus trailing NULL. */ - ignore_value(VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 1 + 1)); + VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 1 + 1); cmd->env[cmd->nenv++] = g_steal_pointer(&env); } @@ -1459,7 +1459,7 @@ virCommandAddEnvPassCommon(virCommandPtr cmd) if (virCommandHasError(cmd)) return; - ignore_value(VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 9)); + VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 9); virCommandAddEnvPair(cmd, "LC_ALL", "C"); @@ -1479,7 +1479,7 @@ virCommandAddEnvXDG(virCommandPtr cmd, const char *baseDir) if (virCommandHasError(cmd)) return; - ignore_value(VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 3)); + VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 3); virCommandAddEnvFormat(cmd, "XDG_DATA_HOME=%s/%s", baseDir, ".local/share"); @@ -1509,7 +1509,7 @@ virCommandAddArg(virCommandPtr cmd, const char *val) } /* Arg plus trailing NULL. */ - ignore_value(VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, 1 + 1)); + VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, 1 + 1); cmd->args[cmd->nargs++] = g_strdup(val); } @@ -1535,7 +1535,7 @@ virCommandAddArgBuffer(virCommandPtr cmd, virBufferPtr buf) str = g_strdup(""); /* Arg plus trailing NULL. */ - ignore_value(VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, 1 + 1)); + VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, 1 + 1); cmd->args[cmd->nargs] = g_steal_pointer(&str); cmd->nargs++; @@ -1564,7 +1564,7 @@ virCommandAddArgFormat(virCommandPtr cmd, const char *format, ...) va_end(list); /* Arg plus trailing NULL. */ - ignore_value(VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, 1 + 1)); + VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, 1 + 1); cmd->args[cmd->nargs++] = arg; } @@ -1611,7 +1611,7 @@ virCommandAddArgSet(virCommandPtr cmd, const char *const*vals) narg++; /* narg plus trailing NULL. */ - ignore_value(VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, narg + 1)); + VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, narg + 1); narg = 0; while (vals[narg] != NULL) { @@ -1644,7 +1644,7 @@ virCommandAddArgList(virCommandPtr cmd, ...) va_end(list); /* narg plus trailing NULL. */ - ignore_value(VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, narg + 1)); + VIR_RESIZE_N(cmd->args, cmd->maxargs, cmd->nargs, narg + 1); va_start(list, cmd); while (1) { diff --git a/src/util/virfile.c b/src/util/virfile.c index 180f2af405..56d6e0605e 100644 --- a/src/util/virfile.c +++ b/src/util/virfile.c @@ -3123,8 +3123,7 @@ virFileOpenTty(int *ttyprimary, char **ttyName, int rawmode) name = g_new0(char, len); while ((rc = ttyname_r(secondary, name, len)) == ERANGE) { - if (VIR_RESIZE_N(name, len, len, len) < 0) - goto cleanup; + VIR_RESIZE_N(name, len, len, len); } if (rc != 0) { errno = rc; diff --git a/src/util/virfirewall.c b/src/util/virfirewall.c index 0dc0cecd53..dd3e104dbe 100644 --- a/src/util/virfirewall.c +++ b/src/util/virfirewall.c @@ -262,10 +262,7 @@ void virFirewallFree(virFirewallPtr firewall) #define ADD_ARG(rule, str) \ do { \ - ignore_value(VIR_RESIZE_N(rule->args, \ - rule->argsAlloc, \ - rule->argsLen, 1)); \ - \ + VIR_RESIZE_N(rule->args, rule->argsAlloc, rule->argsLen, 1); \ rule->args[rule->argsLen++] = g_strdup(str); \ } while (0) diff --git a/src/util/virnetlink.c b/src/util/virnetlink.c index cfb86550a5..c26c03de32 100644 --- a/src/util/virnetlink.c +++ b/src/util/virnetlink.c @@ -1146,9 +1146,8 @@ virNetlinkEventAddClient(virNetlinkEventHandleCallback handleCB, if (srv->handlesCount == srv->handlesAlloc) { VIR_DEBUG("Used %zu handle slots, adding at least %d more", srv->handlesAlloc, NETLINK_EVENT_ALLOC_EXTENT); - if (VIR_RESIZE_N(srv->handles, srv->handlesAlloc, - srv->handlesCount, NETLINK_EVENT_ALLOC_EXTENT) < 0) - goto error; + VIR_RESIZE_N(srv->handles, srv->handlesAlloc, + srv->handlesCount, NETLINK_EVENT_ALLOC_EXTENT); } r = srv->handlesCount++; @@ -1167,7 +1166,7 @@ virNetlinkEventAddClient(virNetlinkEventHandleCallback handleCB, VIR_DEBUG("added client to loop slot: %d. with macaddr ptr=%p", r, macaddr); ret = nextWatch++; - error: + virNetlinkEventServerUnlock(srv); return ret; } diff --git a/src/util/virtypedparam-public.c b/src/util/virtypedparam-public.c index 91603cfe1c..4cb54f7ab1 100644 --- a/src/util/virtypedparam-public.c +++ b/src/util/virtypedparam-public.c @@ -446,8 +446,7 @@ virTypedParamsAddInt(virTypedParameterPtr *params, virResetLastError(); - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams = max; if (virTypedParameterAssign(*params + n, name, @@ -493,8 +492,7 @@ virTypedParamsAddUInt(virTypedParameterPtr *params, virResetLastError(); - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams = max; if (virTypedParameterAssign(*params + n, name, @@ -540,8 +538,7 @@ virTypedParamsAddLLong(virTypedParameterPtr *params, virResetLastError(); - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams = max; if (virTypedParameterAssign(*params + n, name, @@ -587,8 +584,7 @@ virTypedParamsAddULLong(virTypedParameterPtr *params, virResetLastError(); - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams = max; if (virTypedParameterAssign(*params + n, name, @@ -634,8 +630,7 @@ virTypedParamsAddDouble(virTypedParameterPtr *params, virResetLastError(); - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams = max; if (virTypedParameterAssign(*params + n, name, @@ -681,8 +676,7 @@ virTypedParamsAddBoolean(virTypedParameterPtr *params, virResetLastError(); - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams = max; if (virTypedParameterAssign(*params + n, name, @@ -731,8 +725,7 @@ virTypedParamsAddString(virTypedParameterPtr *params, virResetLastError(); - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams = max; str = g_strdup(value); @@ -823,8 +816,7 @@ virTypedParamsAddFromString(virTypedParameterPtr *params, virResetLastError(); - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams = max; if (virTypedParameterAssignFromStr(*params + n, name, type, value) < 0) diff --git a/src/util/virtypedparam.c b/src/util/virtypedparam.c index c4c26a1183..1b89233012 100644 --- a/src/util/virtypedparam.c +++ b/src/util/virtypedparam.c @@ -77,8 +77,7 @@ virTypedParamsValidate(virTypedParameterPtr params, int nparams, ...) name = va_arg(ap, const char *); while (name) { type = va_arg(ap, int); - if (VIR_RESIZE_N(keys, nkeysalloc, nkeys, 1) < 0) - goto cleanup; + VIR_RESIZE_N(keys, nkeysalloc, nkeys, 1); if (virStrcpyStatic(keys[nkeys].field, name) < 0) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -758,8 +757,7 @@ virTypedParamSetNameVPrintf(virTypedParameterPtr par, static virTypedParameterPtr virTypedParamListExtend(virTypedParamListPtr list) { - if (VIR_RESIZE_N(list->par, list->par_alloc, list->npar, 1) < 0) - return NULL; + VIR_RESIZE_N(list->par, list->par_alloc, list->npar, 1); list->npar++; diff --git a/src/util/viruri.c b/src/util/viruri.c index 1e8808ddc6..90594fce5e 100644 --- a/src/util/viruri.c +++ b/src/util/viruri.c @@ -45,8 +45,7 @@ virURIParamAppend(virURIPtr uri, pname = g_strdup(name); pvalue = g_strdup(value); - if (VIR_RESIZE_N(uri->params, uri->paramsAlloc, uri->paramsCount, 1) < 0) - goto error; + VIR_RESIZE_N(uri->params, uri->paramsAlloc, uri->paramsCount, 1); uri->params[uri->paramsCount].name = pname; uri->params[uri->paramsCount].value = pvalue; @@ -54,11 +53,6 @@ virURIParamAppend(virURIPtr uri, uri->paramsCount++; return 0; - - error: - VIR_FREE(pname); - VIR_FREE(pvalue); - return -1; } diff --git a/src/util/virutil.c b/src/util/virutil.c index 118ceec0db..3f49a469e5 100644 --- a/src/util/virutil.c +++ b/src/util/virutil.c @@ -618,8 +618,7 @@ virGetUserEnt(uid_t uid, char **name, gid_t *group, char **dir, char **shell, bo * The given name or uid was not found. */ while ((rc = getpwuid_r(uid, &pwbuf, strbuf, strbuflen, &pw)) == ERANGE) { - if (VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen) < 0) - goto cleanup; + VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen); } if (rc != 0) { @@ -687,10 +686,7 @@ static char *virGetGroupEnt(gid_t gid) * The given name or gid was not found. */ while ((rc = getgrgid_r(gid, &grbuf, strbuf, strbuflen, &gr)) == ERANGE) { - if (VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen) < 0) { - VIR_FREE(strbuf); - return NULL; - } + VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen); } if (rc != 0 || gr == NULL) { if (rc != 0) { @@ -765,8 +761,7 @@ virGetUserIDByName(const char *name, uid_t *uid, bool missing_ok) strbuf = g_new0(char, strbuflen); while ((rc = getpwnam_r(name, &pwbuf, strbuf, strbuflen, &pw)) == ERANGE) { - if (VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen) < 0) - goto cleanup; + VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen); } if (!pw) { @@ -846,8 +841,7 @@ virGetGroupIDByName(const char *name, gid_t *gid, bool missing_ok) strbuf = g_new0(char, strbuflen); while ((rc = getgrnam_r(name, &grbuf, strbuf, strbuflen, &gr)) == ERANGE) { - if (VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen) < 0) - goto cleanup; + VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen); } if (!gr) { diff --git a/tests/viralloctest.c b/tests/viralloctest.c index 0867be8ce1..4794fd7f93 100644 --- a/tests/viralloctest.c +++ b/tests/viralloctest.c @@ -214,8 +214,7 @@ testResizeArray(const void *opaque G_GNUC_UNUSED) t[i].b = 20; } - if (VIR_RESIZE_N(t, at, nt, 8) < 0) - goto cleanup; + VIR_RESIZE_N(t, at, nt, 8); if (testCheckNonNull(t) < 0) goto cleanup;