From nobody Sun Feb 8 19:30:37 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 63.128.21.124 as permitted sender) client-ip=63.128.21.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 63.128.21.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1616197059; cv=none; d=zohomail.com; s=zohoarc; b=Y6Nkp4CVg5G30ZMmIbJZhsvZUlJ6I329VJaoFs/0MFQ+gnSoQ51z8BIgdvVvRK+IUbzO7lQE0Eil+rcHmC94LXSxRfwGecWwhWt7nlL1qnuQzALtZDwi05BZyS1ZlomFvkYVggaNRfX83HhtioSqjLcIB9kyKbFVs4VapysUoyQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616197059; h=Content-Type:Content-Transfer-Encoding:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=Zu3MXYv2D+gHiJxl+bKhfMkNFXDIF9EB+v+mbmCJK1w=; b=k92O6LhYNOL3N6wHLjBvLuM0kpXCBO0RtPY6nxR28f8xVM6U6ObyxohLy9rBIH/smVGS2kaJVxGWtBKHoRoIRELQplPR9nVT5zEAXwExOiukdLay2O/x3G4H5RrfVnF4c0DShSymbsol+Dxma3tLU5UbsK0CYitZf3ZM7yqxxis= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 63.128.21.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [63.128.21.124]) by mx.zohomail.com with SMTPS id 1616197059898144.4176770658072; Fri, 19 Mar 2021 16:37:39 -0700 (PDT) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-522-D-4QLNzbNJC_DSF7fPCoLw-1; Fri, 19 Mar 2021 19:37:36 -0400 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 137DB1034B34; Fri, 19 Mar 2021 23:37:31 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id DCD3619D9D; Fri, 19 Mar 2021 23:37:30 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id A027C4A7CD; Fri, 19 Mar 2021 23:37:30 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 12JNbGeV029466 for ; Fri, 19 Mar 2021 19:37:16 -0400 Received: by smtp.corp.redhat.com (Postfix) id 872A260C04; Fri, 19 Mar 2021 23:37:16 +0000 (UTC) Received: from virval.usersys.redhat.com (unknown [10.40.194.173]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 0A2D060C17 for ; Fri, 19 Mar 2021 23:37:08 +0000 (UTC) Received: by virval.usersys.redhat.com (Postfix, from userid 500) id 6D095244031; Sat, 20 Mar 2021 00:37:07 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1616197058; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=Zu3MXYv2D+gHiJxl+bKhfMkNFXDIF9EB+v+mbmCJK1w=; b=Plv8zD4so/UeAfVFg4l5xGliswdVPPxdDO2lc2ZCl4ur+ysOHuupEd+TANKGiOEqvO2FeD zzlJZX8uaEtc+DSKsAfTPTy5z6uS2rteYDytMcyqf1JgWX8EHa17T/fiCiit5MKd36Ey5F McuDQdAEz4sET/B+ipGR3Cm+i8uXU/s= X-MC-Unique: D-4QLNzbNJC_DSF7fPCoLw-1 From: Jiri Denemark To: libvir-list@redhat.com Subject: [libvirt PATCH 2/7] Do not check return value of VIR_RESIZE_N Date: Sat, 20 Mar 2021 00:37:01 +0100 Message-Id: In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) Content-Type: text/plain; charset="utf-8" Signed-off-by: Jiri Denemark --- src/bhyve/bhyve_capabilities.c | 5 +---- src/bhyve/bhyve_parse_command.c | 11 ++-------- src/conf/capabilities.c | 36 ++++++++------------------------- src/conf/cpu_conf.c | 11 ++-------- src/conf/domain_capabilities.c | 5 ++--- src/esx/esx_driver.c | 3 +-- src/hyperv/hyperv_driver.c | 3 +-- src/qemu/qemu_domain.c | 13 +++++------- src/util/virbitmap.c | 5 ++--- src/util/vircommand.c | 16 +++++++-------- src/util/virfile.c | 3 +-- src/util/virfirewall.c | 5 +---- src/util/virnetlink.c | 7 +++---- src/util/virtypedparam-public.c | 24 ++++++++-------------- src/util/virtypedparam.c | 6 ++---- src/util/viruri.c | 8 +------- src/util/virutil.c | 14 ++++--------- tests/viralloctest.c | 3 +-- 18 files changed, 53 insertions(+), 125 deletions(-) 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, =20 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->nv= alues, 1); firmwares->values[firmwares->nvalues] =3D 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_comman= d.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 =3D g_strdup(curr); =20 - 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); =20 if (*line) lines[line_count++] =3D line; @@ -203,11 +200,7 @@ bhyveCommandLineToArgv(const char *nativeConfig, if (next && (*next =3D=3D '\'' || *next =3D=3D '"')) next++; =20 - 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++] =3D arg; arglist[args_count] =3D NULL; =20 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] =3D g_strdup(name); caps->host.nfeatures++; =20 @@ -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] =3D g_strdup(name); caps->host.nmigrateTrans++; =20 @@ -447,9 +443,7 @@ virCapabilitiesAddGuest(virCapsPtr caps, guest->arch.defaultInfo.emulator =3D g_strdup(emulator); guest->arch.defaultInfo.loader =3D g_strdup(loader); =20 - 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++] =3D guest; =20 if (nmachines) { @@ -458,10 +452,6 @@ virCapabilitiesAddGuest(virCapsPtr caps, } =20 return guest; - - error: - virCapabilitiesFreeGuest(guest); - return NULL; } =20 =20 @@ -493,9 +483,8 @@ virCapabilitiesAddGuestDomain(virCapsGuestPtr guest, dom->info.emulator =3D g_strdup(emulator); dom->info.loader =3D g_strdup(loader); =20 - 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] =3D dom; guest->arch.ndomains++; =20 @@ -505,10 +494,6 @@ virCapabilitiesAddGuestDomain(virCapsGuestPtr guest, } =20 return dom; - - error: - virCapabilitiesFreeGuestDomain(dom); - return NULL; } =20 =20 @@ -803,15 +788,10 @@ virCapabilitiesAddStoragePool(virCapsPtr caps, =20 pool->type =3D poolType; =20 - 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++] =3D pool; =20 return 0; - - error: - virCapabilitiesFreeStoragePool(pool); - return -1; } =20 =20 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; } =20 - 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 =3D n; } =20 @@ -898,12 +895,8 @@ virCPUDefAddFeatureInternal(virCPUDefPtr def, } } =20 - 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 =3D g_strdup(name); - def->features[def->nfeatures].policy =3D policy; def->nfeatures++; =20 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 cpu= Models, =20 nameCopy =3D g_strdup(name); =20 - 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); =20 cpu =3D 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; } =20 - if (VIR_RESIZE_N(doms, ndoms, count, 2) < 0) - goto cleanup; + VIR_RESIZE_N(doms, ndoms, count, 2); =20 /* Only running/suspended virtual machines have an ID !=3D -1 */ if (powerState =3D=3D 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; } =20 - if (VIR_RESIZE_N(doms, ndoms, count, 2) < 0) - goto cleanup; + VIR_RESIZE_N(doms, ndoms, count, 2); =20 domain =3D NULL; =20 diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index fff67481a7..3b54adc614 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -4287,10 +4287,8 @@ qemuDomainDefCPUPostParse(virDomainDefPtr def, sveFeature->policy =3D 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); =20 def->cpu->features[def->cpu->nfeatures].name =3D g_strdup("sve= "); def->cpu->features[def->cpu->nfeatures].policy =3D VIR_CPU_FEA= TURE_REQUIRE; @@ -7340,10 +7338,9 @@ qemuDomainCleanupAdd(virDomainObjPtr vm, return 0; } =20 - 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); =20 priv->cleanupCallbacks[priv->ncleanupCallbacks++] =3D 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, =20 /* 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); } =20 map->nbits =3D 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, } =20 /* 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); =20 cmd->env[cmd->nenv++] =3D g_steal_pointer(&env); } @@ -1459,7 +1459,7 @@ virCommandAddEnvPassCommon(virCommandPtr cmd) if (virCommandHasError(cmd)) return; =20 - ignore_value(VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 9)); + VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 9); =20 virCommandAddEnvPair(cmd, "LC_ALL", "C"); =20 @@ -1479,7 +1479,7 @@ virCommandAddEnvXDG(virCommandPtr cmd, const char *ba= seDir) if (virCommandHasError(cmd)) return; =20 - ignore_value(VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 3)); + VIR_RESIZE_N(cmd->env, cmd->maxenv, cmd->nenv, 3); =20 virCommandAddEnvFormat(cmd, "XDG_DATA_HOME=3D%s/%s", baseDir, ".local/share"); @@ -1509,7 +1509,7 @@ virCommandAddArg(virCommandPtr cmd, const char *val) } =20 /* 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); =20 cmd->args[cmd->nargs++] =3D g_strdup(val); } @@ -1535,7 +1535,7 @@ virCommandAddArgBuffer(virCommandPtr cmd, virBufferPt= r buf) str =3D g_strdup(""); =20 /* 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); =20 cmd->args[cmd->nargs] =3D g_steal_pointer(&str); cmd->nargs++; @@ -1564,7 +1564,7 @@ virCommandAddArgFormat(virCommandPtr cmd, const char = *format, ...) va_end(list); =20 /* 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); =20 cmd->args[cmd->nargs++] =3D arg; } @@ -1611,7 +1611,7 @@ virCommandAddArgSet(virCommandPtr cmd, const char *co= nst*vals) narg++; =20 /* 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); =20 narg =3D 0; while (vals[narg] !=3D NULL) { @@ -1644,7 +1644,7 @@ virCommandAddArgList(virCommandPtr cmd, ...) va_end(list); =20 /* 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); =20 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 r= awmode) name =3D g_new0(char, len); =20 while ((rc =3D ttyname_r(secondary, name, len)) =3D=3D ERANGE) { - if (VIR_RESIZE_N(name, len, len, len) < 0) - goto cleanup; + VIR_RESIZE_N(name, len, len, len); } if (rc !=3D 0) { errno =3D 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) =20 #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++] =3D g_strdup(str); \ } while (0) =20 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(virNetlinkEventHandleCallbac= k handleCB, if (srv->handlesCount =3D=3D 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 =3D srv->handlesCount++; =20 @@ -1167,7 +1166,7 @@ virNetlinkEventAddClient(virNetlinkEventHandleCallbac= k handleCB, VIR_DEBUG("added client to loop slot: %d. with macaddr ptr=3D%p", r, m= acaddr); =20 ret =3D nextWatch++; - error: + virNetlinkEventServerUnlock(srv); return ret; } diff --git a/src/util/virtypedparam-public.c b/src/util/virtypedparam-publi= c.c index 91603cfe1c..4cb54f7ab1 100644 --- a/src/util/virtypedparam-public.c +++ b/src/util/virtypedparam-public.c @@ -446,8 +446,7 @@ virTypedParamsAddInt(virTypedParameterPtr *params, =20 virResetLastError(); =20 - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams =3D max; =20 if (virTypedParameterAssign(*params + n, name, @@ -493,8 +492,7 @@ virTypedParamsAddUInt(virTypedParameterPtr *params, =20 virResetLastError(); =20 - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams =3D max; =20 if (virTypedParameterAssign(*params + n, name, @@ -540,8 +538,7 @@ virTypedParamsAddLLong(virTypedParameterPtr *params, =20 virResetLastError(); =20 - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams =3D max; =20 if (virTypedParameterAssign(*params + n, name, @@ -587,8 +584,7 @@ virTypedParamsAddULLong(virTypedParameterPtr *params, =20 virResetLastError(); =20 - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams =3D max; =20 if (virTypedParameterAssign(*params + n, name, @@ -634,8 +630,7 @@ virTypedParamsAddDouble(virTypedParameterPtr *params, =20 virResetLastError(); =20 - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams =3D max; =20 if (virTypedParameterAssign(*params + n, name, @@ -681,8 +676,7 @@ virTypedParamsAddBoolean(virTypedParameterPtr *params, =20 virResetLastError(); =20 - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams =3D max; =20 if (virTypedParameterAssign(*params + n, name, @@ -731,8 +725,7 @@ virTypedParamsAddString(virTypedParameterPtr *params, =20 virResetLastError(); =20 - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams =3D max; =20 str =3D g_strdup(value); @@ -823,8 +816,7 @@ virTypedParamsAddFromString(virTypedParameterPtr *param= s, =20 virResetLastError(); =20 - if (VIR_RESIZE_N(*params, max, n, 1) < 0) - goto error; + VIR_RESIZE_N(*params, max, n, 1); *maxparams =3D max; =20 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 n= params, ...) name =3D va_arg(ap, const char *); while (name) { type =3D va_arg(ap, int); - if (VIR_RESIZE_N(keys, nkeysalloc, nkeys, 1) < 0) - goto cleanup; + VIR_RESIZE_N(keys, nkeysalloc, nkeys, 1); =20 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); =20 list->npar++; =20 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 =3D g_strdup(name); pvalue =3D g_strdup(value); =20 - if (VIR_RESIZE_N(uri->params, uri->paramsAlloc, uri->paramsCount, 1) <= 0) - goto error; + VIR_RESIZE_N(uri->params, uri->paramsAlloc, uri->paramsCount, 1); =20 uri->params[uri->paramsCount].name =3D pname; uri->params[uri->paramsCount].value =3D pvalue; @@ -54,11 +53,6 @@ virURIParamAppend(virURIPtr uri, uri->paramsCount++; =20 return 0; - - error: - VIR_FREE(pname); - VIR_FREE(pvalue); - return -1; } =20 =20 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, cha= r **dir, char **shell, bo * The given name or uid was not found. */ while ((rc =3D getpwuid_r(uid, &pwbuf, strbuf, strbuflen, &pw)) =3D=3D= ERANGE) { - if (VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen) < 0) - goto cleanup; + VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen); } =20 if (rc !=3D 0) { @@ -687,10 +686,7 @@ static char *virGetGroupEnt(gid_t gid) * The given name or gid was not found. */ while ((rc =3D getgrgid_r(gid, &grbuf, strbuf, strbuflen, &gr)) =3D=3D= ERANGE) { - if (VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen) < 0) { - VIR_FREE(strbuf); - return NULL; - } + VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen); } if (rc !=3D 0 || gr =3D=3D NULL) { if (rc !=3D 0) { @@ -765,8 +761,7 @@ virGetUserIDByName(const char *name, uid_t *uid, bool m= issing_ok) strbuf =3D g_new0(char, strbuflen); =20 while ((rc =3D getpwnam_r(name, &pwbuf, strbuf, strbuflen, &pw)) =3D= =3D ERANGE) { - if (VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen) < 0) - goto cleanup; + VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen); } =20 if (!pw) { @@ -846,8 +841,7 @@ virGetGroupIDByName(const char *name, gid_t *gid, bool = missing_ok) strbuf =3D g_new0(char, strbuflen); =20 while ((rc =3D getgrnam_r(name, &grbuf, strbuf, strbuflen, &gr)) =3D= =3D ERANGE) { - if (VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen) < 0) - goto cleanup; + VIR_RESIZE_N(strbuf, strbuflen, strbuflen, strbuflen); } =20 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 =3D 20; } =20 - if (VIR_RESIZE_N(t, at, nt, 8) < 0) - goto cleanup; + VIR_RESIZE_N(t, at, nt, 8); =20 if (testCheckNonNull(t) < 0) goto cleanup; --=20 2.31.0