From nobody Sun Feb 8 04:52:30 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 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=1571227764; cv=none; d=zoho.com; s=zohoarc; b=KSV+3fMeyk8nj/3hBL/dWA63+sGkB0jWdFLIYSVGaz1wJfIc7cCNgTONxo6puZAYriZJRErb39IkHp+5hLgIh3VyBRsU/irDhbDDuHQ0xRLY5YWUUhZ2esuUd9yXJnStgcqMAaCHmcY+z3+gqiArkqOAOScOL+MC8NGoz38Oy8M= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1571227764; 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=dFIeyoJP4Uwd9MDA/JQ6n4WHAP8jJb6Xj9NMS/U5OP4=; b=JIf9D2FK6uF1DTwyk22sX6FcCqBmgkZJ4GpJVS6Dtl9PfMdC+f6L2pQQ89n7KE0mbiTLGCghH202CbY/UzNHQLvUeMhf8qNljbACjDbqp4jERMQsY18zp5rb/tfUqIm3lUNPu4y5opV/5B03sO/guERSMiFN8F8Tj6xD+ht3Zt8= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1571227764224953.0208362885598; Wed, 16 Oct 2019 05:09:24 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 436573003A49; Wed, 16 Oct 2019 12:09:22 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id F153D5D9CD; Wed, 16 Oct 2019 12:09:21 +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 ABAAC180B536; Wed, 16 Oct 2019 12:09:21 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id x9GC9JMP020939 for ; Wed, 16 Oct 2019 08:09:19 -0400 Received: by smtp.corp.redhat.com (Postfix) id 9263760166; Wed, 16 Oct 2019 12:09:19 +0000 (UTC) Received: from icr.brq.redhat.com (unknown [10.43.2.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id E6C0C60470 for ; Wed, 16 Oct 2019 12:09:18 +0000 (UTC) From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Date: Wed, 16 Oct 2019 14:09:05 +0200 Message-Id: <8a97cfa9c5fdab4d16ff102a74b2f102e2eedb80.1571227709.git.jtomko@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-loop: libvir-list@redhat.com Subject: [libvirt] [PATCH 4/9] conf: use g_steal_pointer instead of VIR_STEAL_PTR 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: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.42]); Wed, 16 Oct 2019 12:09:22 +0000 (UTC) Signed-off-by: J=C3=A1n Tomko --- src/conf/checkpoint_conf.c | 2 +- src/conf/cpu_conf.c | 18 ++-- src/conf/domain_addr.c | 4 +- src/conf/domain_capabilities.c | 4 +- src/conf/domain_conf.c | 138 +++++++++++++++---------------- src/conf/node_device_conf.c | 12 +-- src/conf/snapshot_conf.c | 8 +- src/conf/storage_conf.c | 8 +- src/conf/virnetworkobj.c | 2 +- src/conf/virnwfilterbindingobj.c | 2 +- src/conf/virsecretobj.c | 2 +- src/conf/virstorageobj.c | 4 +- 12 files changed, 102 insertions(+), 102 deletions(-) diff --git a/src/conf/checkpoint_conf.c b/src/conf/checkpoint_conf.c index e7b204a4d2..b254dce7fd 100644 --- a/src/conf/checkpoint_conf.c +++ b/src/conf/checkpoint_conf.c @@ -197,7 +197,7 @@ virDomainCheckpointDefParse(xmlXPathContextPtr ctxt, return NULL; } =20 - VIR_STEAL_PTR(ret, def); + ret =3D g_steal_pointer(&def); return ret; } =20 diff --git a/src/conf/cpu_conf.c b/src/conf/cpu_conf.c index a6bb9ea263..80b7687b86 100644 --- a/src/conf/cpu_conf.c +++ b/src/conf/cpu_conf.c @@ -185,14 +185,14 @@ virCPUDefStealModel(virCPUDefPtr dst, char *vendor_id =3D NULL; =20 if (keepVendor) { - VIR_STEAL_PTR(vendor, dst->vendor); - VIR_STEAL_PTR(vendor_id, dst->vendor_id); + vendor =3D g_steal_pointer(&dst->vendor); + vendor_id =3D g_steal_pointer(&dst->vendor_id); } =20 virCPUDefFreeModel(dst); =20 - VIR_STEAL_PTR(dst->model, src->model); - VIR_STEAL_PTR(dst->features, src->features); + dst->model =3D g_steal_pointer(&src->model); + dst->features =3D g_steal_pointer(&src->features); dst->microcodeVersion =3D src->microcodeVersion; dst->nfeatures_max =3D src->nfeatures_max; src->nfeatures_max =3D 0; @@ -203,8 +203,8 @@ virCPUDefStealModel(virCPUDefPtr dst, dst->vendor =3D vendor; dst->vendor_id =3D vendor_id; } else { - VIR_STEAL_PTR(dst->vendor, src->vendor); - VIR_STEAL_PTR(dst->vendor_id, src->vendor_id); + dst->vendor =3D g_steal_pointer(&src->vendor); + dst->vendor_id =3D g_steal_pointer(&src->vendor_id); } } =20 @@ -465,7 +465,7 @@ virCPUDefParseXML(xmlXPathContextPtr ctxt, tsc->scaling =3D scaling; } =20 - VIR_STEAL_PTR(def->tsc, tsc); + def->tsc =3D g_steal_pointer(&tsc); } } =20 @@ -644,7 +644,7 @@ virCPUDefParseXML(xmlXPathContextPtr ctxt, def->cache->mode =3D mode; } =20 - VIR_STEAL_PTR(*cpu, def); + *cpu =3D g_steal_pointer(&def); ret =3D 0; =20 cleanup: @@ -987,7 +987,7 @@ virCPUDefCheckFeatures(virCPUDefPtr cpu, } } =20 - VIR_STEAL_PTR(*features, list); + *features =3D g_steal_pointer(&list); return n; } =20 diff --git a/src/conf/domain_addr.c b/src/conf/domain_addr.c index fc612ee6a2..d0026942aa 100644 --- a/src/conf/domain_addr.c +++ b/src/conf/domain_addr.c @@ -1757,7 +1757,7 @@ virDomainVirtioSerialAddrSetCreateFromDomain(virDomai= nDefPtr def) addrs) < 0) goto cleanup; =20 - VIR_STEAL_PTR(ret, addrs); + ret =3D g_steal_pointer(&addrs); cleanup: virDomainVirtioSerialAddrSetFree(addrs); return ret; @@ -2094,7 +2094,7 @@ virDomainUSBAddressHubNew(size_t nports) goto cleanup; hub->nports =3D nports; =20 - VIR_STEAL_PTR(ret, hub); + ret =3D g_steal_pointer(&hub); cleanup: virDomainUSBAddressHubFree(hub); return ret; diff --git a/src/conf/domain_capabilities.c b/src/conf/domain_capabilities.c index b0fdd15d6c..43a778a505 100644 --- a/src/conf/domain_capabilities.c +++ b/src/conf/domain_capabilities.c @@ -229,10 +229,10 @@ virDomainCapsCPUModelsAddSteal(virDomainCapsCPUModels= Ptr cpuModels, return -1; =20 cpuModels->models[cpuModels->nmodels].usable =3D usable; - VIR_STEAL_PTR(cpuModels->models[cpuModels->nmodels].name, *name); + cpuModels->models[cpuModels->nmodels].name =3D g_steal_pointer(&*name); =20 if (blockers) - VIR_STEAL_PTR(cpuModels->models[cpuModels->nmodels].blockers, *blo= ckers); + cpuModels->models[cpuModels->nmodels].blockers =3D g_steal_pointer= (&*blockers); =20 cpuModels->nmodels++; return 0; diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 8b56ff3458..69464a3345 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -1379,7 +1379,7 @@ virDomainKeyWrapDefParseXML(virDomainDefPtr def, xmlX= PathContextPtr ctxt) } =20 if (keywrap->aes || keywrap->dea) - VIR_STEAL_PTR(def->keywrap, keywrap); + def->keywrap =3D g_steal_pointer(&keywrap); =20 return 0; } @@ -1794,7 +1794,7 @@ virDomainVcpuDefNew(virDomainXMLOptionPtr xmlopt) if (VIR_ALLOC(ret) < 0) return NULL; =20 - VIR_STEAL_PTR(ret->privateData, priv); + ret->privateData =3D g_steal_pointer(&priv); =20 return ret; } @@ -2362,7 +2362,7 @@ virDomainVsockDefNew(virDomainXMLOptionPtr xmlopt) !(vsock->privateData =3D xmlopt->privateData.vsockNew())) goto cleanup; =20 - VIR_STEAL_PTR(ret, vsock); + ret =3D g_steal_pointer(&vsock); cleanup: virDomainVsockDefFree(vsock); return ret; @@ -4938,7 +4938,7 @@ virDomainPostParseCheckISCSIPath(char **srcpath) if (virAsprintf(&path, "%s/0", *srcpath) < 0) return -1; VIR_FREE(*srcpath); - VIR_STEAL_PTR(*srcpath, path); + *srcpath =3D g_steal_pointer(&path); return 0; } =20 @@ -5660,10 +5660,10 @@ virDomainDefPostParseVideo(virDomainDefPtr def, /* we don't want to format any values we automatically fill in for * videos into the XML, so clear them, but retain any user-assigned * alias */ - VIR_STEAL_PTR(alias, def->videos[0]->info.alias); + alias =3D g_steal_pointer(&def->videos[0]->info.alias); virDomainVideoDefClear(def->videos[0]); def->videos[0]->type =3D VIR_DOMAIN_VIDEO_TYPE_NONE; - VIR_STEAL_PTR(def->videos[0]->info.alias, alias); + def->videos[0]->info.alias =3D g_steal_pointer(&alias); } else { virDomainDeviceDef device =3D { .type =3D VIR_DOMAIN_DEVICE_VIDEO, @@ -6527,7 +6527,7 @@ virDomainDefValidateAliases(const virDomainDef *def, goto cleanup; =20 if (aliases) - VIR_STEAL_PTR(*aliases, data.aliases); + *aliases =3D g_steal_pointer(&data.aliases); =20 ret =3D 0; cleanup: @@ -7511,7 +7511,7 @@ virDomainDeviceInfoParseXML(virDomainXMLOptionPtr xml= opt G_GNUC_UNUSED, (xmlopt->config.features & VIR_DOMAIN_DEF_FEATURE_USER_ALIAS && virDomainDeviceAliasIsUserAlias(aliasStr) && strspn(aliasStr, USER_ALIAS_CHARS) =3D=3D strlen(aliasStr))) - VIR_STEAL_PTR(info->alias, aliasStr); + info->alias =3D g_steal_pointer(&aliasStr); } =20 if (master) { @@ -8030,7 +8030,7 @@ virDomainHostdevSubsysSCSIiSCSIDefParseXML(xmlNodePtr= sourcenode, authdef->secrettype); return -1; } - VIR_STEAL_PTR(iscsisrc->src->auth, authdef); + iscsisrc->src->auth =3D g_steal_pointer(&authdef); } cur =3D cur->next; } @@ -8096,7 +8096,7 @@ virDomainHostdevSubsysSCSIVHostDefParseXML(xmlNodePtr= sourcenode, virReportError(VIR_ERR_XML_ERROR, "%s", _("malformed 'wwpn' va= lue")); return -1; } - VIR_STEAL_PTR(hostsrc->wwpn, wwpn); + hostsrc->wwpn =3D g_steal_pointer(&wwpn); break; case VIR_DOMAIN_HOSTDEV_SUBSYS_SCSI_HOST_PROTOCOL_TYPE_NONE: case VIR_DOMAIN_HOSTDEV_SUBSYS_SCSI_HOST_PROTOCOL_TYPE_LAST: @@ -8389,7 +8389,7 @@ virDomainNetIPParseXML(xmlNodePtr node) return NULL; } =20 - VIR_STEAL_PTR(ret, ip); + ret =3D g_steal_pointer(&ip); return ret; } =20 @@ -8803,7 +8803,7 @@ virSecurityLabelDefParseXML(xmlXPathContextPtr ctxt, goto error; } =20 - VIR_STEAL_PTR(seclabel->label, p); + seclabel->label =3D g_steal_pointer(&p); } =20 /* Only parse imagelabel, if requested live XML with relabeling */ @@ -8817,14 +8817,14 @@ virSecurityLabelDefParseXML(xmlXPathContextPtr ctxt, "%s", _("security imagelabel is missing")); goto error; } - VIR_STEAL_PTR(seclabel->imagelabel, p); + seclabel->imagelabel =3D g_steal_pointer(&p); } =20 /* Only parse baselabel for dynamic label type */ if (seclabel->type =3D=3D VIR_DOMAIN_SECLABEL_DYNAMIC) { p =3D virXPathStringLimit("string(./baselabel[1])", VIR_SECURITY_LABEL_BUFLEN-1, ctxt); - VIR_STEAL_PTR(seclabel->baselabel, p); + seclabel->baselabel =3D g_steal_pointer(&p); } =20 return seclabel; @@ -9105,9 +9105,9 @@ virDomainLeaseDefParseXML(xmlNodePtr node) goto error; } =20 - VIR_STEAL_PTR(def->key, key); - VIR_STEAL_PTR(def->lockspace, lockspace); - VIR_STEAL_PTR(def->path, path); + def->key =3D g_steal_pointer(&key); + def->lockspace =3D g_steal_pointer(&lockspace); + def->path =3D g_steal_pointer(&path); =20 cleanup: return def; @@ -9156,7 +9156,7 @@ virDomainDiskSourcePoolDefParse(xmlNodePtr node, goto cleanup; } =20 - VIR_STEAL_PTR(*srcpool, source); + *srcpool =3D g_steal_pointer(&source); ret =3D 0; =20 cleanup: @@ -9305,7 +9305,7 @@ virDomainStorageSourceParseBase(const char *type, return NULL; } =20 - VIR_STEAL_PTR(ret, src); + ret =3D g_steal_pointer(&src); return ret; } =20 @@ -9442,7 +9442,7 @@ virDomainDiskBackingStoreParse(xmlXPathContextPtr ctx= t, virDomainDiskBackingStoreParse(ctxt, backingStore, flags, xmlopt) = < 0) return -1; =20 - VIR_STEAL_PTR(src->backingStore, backingStore); + src->backingStore =3D g_steal_pointer(&backingStore); =20 return 0; } @@ -10343,16 +10343,16 @@ virDomainDiskDefParseXML(virDomainXMLOptionPtr xm= lopt, def->startupPolicy =3D val; } =20 - VIR_STEAL_PTR(def->dst, target); + def->dst =3D g_steal_pointer(&target); if (authdef) - VIR_STEAL_PTR(def->src->auth, authdef); + def->src->auth =3D g_steal_pointer(&authdef); if (encryption) - VIR_STEAL_PTR(def->src->encryption, encryption); - VIR_STEAL_PTR(def->domain_name, domain_name); - VIR_STEAL_PTR(def->serial, serial); - VIR_STEAL_PTR(def->wwn, wwn); - VIR_STEAL_PTR(def->vendor, vendor); - VIR_STEAL_PTR(def->product, product); + def->src->encryption =3D g_steal_pointer(&encryption); + def->domain_name =3D g_steal_pointer(&domain_name); + def->serial =3D g_steal_pointer(&serial); + def->wwn =3D g_steal_pointer(&wwn); + def->vendor =3D g_steal_pointer(&vendor); + def->product =3D g_steal_pointer(&product); =20 if (virDomainDiskBackingStoreParse(ctxt, def->src, flags, xmlopt) < 0) goto error; @@ -11179,8 +11179,8 @@ virDomainFSDefParseXML(virDomainXMLOptionPtr xmlopt, goto error; } =20 - VIR_STEAL_PTR(def->src->path, source); - VIR_STEAL_PTR(def->dst, target); + def->src->path =3D g_steal_pointer(&source); + def->dst =3D g_steal_pointer(&target); =20 if (virDomainDeviceInfoParseXML(xmlopt, node, &def->info, flags) < 0) goto error; @@ -11358,7 +11358,7 @@ virDomainActualNetDefParseXML(xmlNodePtr node, if (vlanNode && virNetDevVlanParse(vlanNode, ctxt, &actual->vlan) < 0) goto error; =20 - VIR_STEAL_PTR(*def, actual); + *def =3D g_steal_pointer(&actual); ret =3D 0; error: virDomainActualNetDefFree(actual); @@ -11762,9 +11762,9 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, goto error; } =20 - VIR_STEAL_PTR(def->data.network.name, network); - VIR_STEAL_PTR(def->data.network.portgroup, portgroup); - VIR_STEAL_PTR(def->data.network.actual, actual); + def->data.network.name =3D g_steal_pointer(&network); + def->data.network.portgroup =3D g_steal_pointer(&portgroup); + def->data.network.actual =3D g_steal_pointer(&actual); break; =20 case VIR_DOMAIN_NET_TYPE_VHOSTUSER: @@ -11810,7 +11810,7 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, goto error; =20 def->data.vhostuser->type =3D VIR_DOMAIN_CHR_TYPE_UNIX; - VIR_STEAL_PTR(def->data.vhostuser->data.nix.path, vhostuser_path); + def->data.vhostuser->data.nix.path =3D g_steal_pointer(&vhostuser_= path); =20 if (STREQ(vhostuser_mode, "server")) { def->data.vhostuser->data.nix.listen =3D true; @@ -11839,7 +11839,7 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, "specified with "= )); goto error; } - VIR_STEAL_PTR(def->data.bridge.brname, bridge); + def->data.bridge.brname =3D g_steal_pointer(&bridge); break; =20 case VIR_DOMAIN_NET_TYPE_CLIENT: @@ -11869,7 +11869,7 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, goto error; } } else { - VIR_STEAL_PTR(def->data.socket.address, address); + def->data.socket.address =3D g_steal_pointer(&address); } =20 if (def->type !=3D VIR_DOMAIN_NET_TYPE_UDP) @@ -11894,7 +11894,7 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, "specified with socket interface")); goto error; } else { - VIR_STEAL_PTR(def->data.socket.localaddr, localaddr); + def->data.socket.localaddr =3D g_steal_pointer(&localaddr); } break; =20 @@ -11905,7 +11905,7 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, "with ")); goto error; } - VIR_STEAL_PTR(def->data.internal.name, internal); + def->data.internal.name =3D g_steal_pointer(&internal); break; =20 case VIR_DOMAIN_NET_TYPE_DIRECT: @@ -11927,7 +11927,7 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, def->data.direct.mode =3D VIR_NETDEV_MACVLAN_MODE_VEPA; } =20 - VIR_STEAL_PTR(def->data.direct.linkdev, dev); + def->data.direct.linkdev =3D g_steal_pointer(&dev); =20 if (ifname && flags & VIR_DOMAIN_DEF_PARSE_INACTIVE && @@ -11990,15 +11990,15 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xml= opt, } =20 if (script !=3D NULL) - VIR_STEAL_PTR(def->script, script); + def->script =3D g_steal_pointer(&script); if (domain_name !=3D NULL) - VIR_STEAL_PTR(def->domain_name, domain_name); + def->domain_name =3D g_steal_pointer(&domain_name); if (ifname !=3D NULL) - VIR_STEAL_PTR(def->ifname, ifname); + def->ifname =3D g_steal_pointer(&ifname); if (ifname_guest !=3D NULL) - VIR_STEAL_PTR(def->ifname_guest, ifname_guest); + def->ifname_guest =3D g_steal_pointer(&ifname_guest); if (ifname_guest_actual !=3D NULL) - VIR_STEAL_PTR(def->ifname_guest_actual, ifname_guest_actual); + def->ifname_guest_actual =3D g_steal_pointer(&ifname_guest_actual); =20 if (def->type !=3D VIR_DOMAIN_NET_TYPE_HOSTDEV && virDomainNetIsVirtioModel(def)) { @@ -12198,7 +12198,7 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, def->driver.virtio.guest.ufo =3D val; } } - VIR_STEAL_PTR(def->backend.vhost, vhost_path); + def->backend.vhost =3D g_steal_pointer(&vhost_path); } =20 def->linkstate =3D VIR_DOMAIN_NET_INTERFACE_LINK_STATE_DEFAULT; @@ -12216,8 +12216,8 @@ virDomainNetDefParseXML(virDomainXMLOptionPtr xmlop= t, case VIR_DOMAIN_NET_TYPE_ETHERNET: case VIR_DOMAIN_NET_TYPE_NETWORK: case VIR_DOMAIN_NET_TYPE_BRIDGE: - VIR_STEAL_PTR(def->filter, filter); - VIR_STEAL_PTR(def->filterparams, filterparams); + def->filter =3D g_steal_pointer(&filter); + def->filterparams =3D g_steal_pointer(&filterparams); break; case VIR_DOMAIN_NET_TYPE_USER: case VIR_DOMAIN_NET_TYPE_VHOSTUSER: @@ -13213,7 +13213,7 @@ virDomainTPMDefParseXML(virDomainXMLOptionPtr xmlop= t, path =3D virXPathString("string(./backend/device/@path)", ctxt); if (!path && VIR_STRDUP(path, VIR_DOMAIN_TPM_DEFAULT_DEVICE) < 0) goto error; - VIR_STEAL_PTR(def->data.passthrough.source.data.file.path, path); + def->data.passthrough.source.data.file.path =3D g_steal_pointer(&p= ath); def->data.passthrough.source.type =3D VIR_DOMAIN_CHR_TYPE_DEV; break; case VIR_DOMAIN_TPM_TYPE_EMULATOR: @@ -13751,7 +13751,7 @@ virDomainGraphicsListenDefParseXML(virDomainGraphic= sListenDefPtr def, } =20 if (!address) - VIR_STEAL_PTR(address, addressCompat); + address =3D g_steal_pointer(&addressCompat); } =20 if (def->type =3D=3D VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_SOCKET) { @@ -13764,14 +13764,14 @@ virDomainGraphicsListenDefParseXML(virDomainGraph= icsListenDefPtr def, } =20 if (!socketPath) - VIR_STEAL_PTR(socketPath, socketCompat); + socketPath =3D g_steal_pointer(&socketCompat); } =20 if (address && address[0] && (def->type =3D=3D VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_ADDRESS || (def->type =3D=3D VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_NETWORK && !(flags & VIR_DOMAIN_DEF_PARSE_INACTIVE)))) { - VIR_STEAL_PTR(def->address, address); + def->address =3D g_steal_pointer(&address); } =20 if (network && network[0]) { @@ -13781,7 +13781,7 @@ virDomainGraphicsListenDefParseXML(virDomainGraphic= sListenDefPtr def, "type 'network'")); goto error; } - VIR_STEAL_PTR(def->network, network); + def->network =3D g_steal_pointer(&network); } =20 if (socketPath && socketPath[0]) { @@ -13791,7 +13791,7 @@ virDomainGraphicsListenDefParseXML(virDomainGraphic= sListenDefPtr def, "type 'socket'")); goto error; } - VIR_STEAL_PTR(def->socket, socketPath); + def->socket =3D g_steal_pointer(&socketPath); } =20 if (fromConfig && @@ -13870,7 +13870,7 @@ virDomainGraphicsListensParseXML(virDomainGraphicsD= efPtr def, =20 if (socketPath) { newListen.type =3D VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_SOCKET; - VIR_STEAL_PTR(newListen.socket, socketPath); + newListen.socket =3D g_steal_pointer(&socketPath); } else { newListen.type =3D VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_ADDRESS; newListen.address =3D virXMLPropString(node, "listen"); @@ -14348,7 +14348,7 @@ virDomainGraphicsDefParseXMLSpice(virDomainGraphics= DefPtr def, } =20 def->data.spice.gl =3D enableVal; - VIR_STEAL_PTR(def->data.spice.rendernode, rendernode); + def->data.spice.rendernode =3D g_steal_pointer(&rendernode= ); =20 } else if (virXMLNodeNameEqual(cur, "mouse")) { int modeVal; @@ -14921,7 +14921,7 @@ virDomainShmemDefParseXML(virDomainXMLOptionPtr xml= opt, goto cleanup; =20 =20 - VIR_STEAL_PTR(ret, def); + ret =3D g_steal_pointer(&def); cleanup: virDomainShmemDefFree(def); return ret; @@ -14981,7 +14981,7 @@ virSysinfoBIOSParseXML(xmlNodePtr node, def =3D NULL; } =20 - VIR_STEAL_PTR(*bios, def); + *bios =3D g_steal_pointer(&def); ret =3D 0; cleanup: virSysinfoBIOSDefFree(def); @@ -15057,7 +15057,7 @@ virSysinfoSystemParseXML(xmlNodePtr node, def =3D NULL; } =20 - VIR_STEAL_PTR(*sysdef, def); + *sysdef =3D g_steal_pointer(&def); ret =3D 0; cleanup: virSysinfoSystemDefFree(def); @@ -15108,7 +15108,7 @@ virSysinfoBaseBoardParseXML(xmlXPathContextPtr ctxt, } } =20 - VIR_STEAL_PTR(*baseBoard, boards); + *baseBoard =3D g_steal_pointer(&boards); *nbaseBoard =3D nboards; ret =3D 0; cleanup: @@ -15146,7 +15146,7 @@ virSysinfoOEMStringsParseXML(xmlNodePtr node, for (i =3D 0; i < nstrings; i++) def->values[i] =3D virXMLNodeContentString(strings[i]); =20 - VIR_STEAL_PTR(*oem, def); + *oem =3D g_steal_pointer(&def); ret =3D 0; cleanup: virSysinfoOEMStringsDefFree(def); @@ -15191,7 +15191,7 @@ virSysinfoChassisParseXML(xmlNodePtr node, def =3D NULL; } =20 - VIR_STEAL_PTR(*chassisdef, def); + *chassisdef =3D g_steal_pointer(&def); ret =3D 0; cleanup: virSysinfoChassisDefFree(def); @@ -16315,7 +16315,7 @@ virDomainIOMMUDefParseXML(xmlNodePtr node, } } =20 - VIR_STEAL_PTR(ret, iommu); + ret =3D g_steal_pointer(&iommu); =20 cleanup: return ret; @@ -16378,7 +16378,7 @@ virDomainVsockDefParseXML(virDomainXMLOptionPtr xml= opt, if (virDomainDeviceInfoParseXML(xmlopt, node, &vsock->info, flags) < 0) goto cleanup; =20 - VIR_STEAL_PTR(ret, vsock); + ret =3D g_steal_pointer(&vsock); =20 cleanup: return ret; @@ -18372,7 +18372,7 @@ virDomainIOThreadPinDefParseXML(xmlNodePtr node, return -1; } =20 - VIR_STEAL_PTR(iothrid->cpumask, cpumask); + iothrid->cpumask =3D g_steal_pointer(&cpumask); return 0; } =20 @@ -18403,7 +18403,7 @@ virDomainEmulatorPinDefParseXML(xmlNodePtr node) return NULL; } =20 - VIR_STEAL_PTR(ret, def); + ret =3D g_steal_pointer(&def); return ret; } =20 @@ -18738,7 +18738,7 @@ virDomainEmulatorSchedParse(xmlNodePtr node, &sched->priority) < 0) return -1; =20 - VIR_STEAL_PTR(def->cputune.emulatorsched, sched); + def->cputune.emulatorsched =3D g_steal_pointer(&sched); return 0; } =20 @@ -19506,7 +19506,7 @@ virDomainResctrlNew(xmlNodePtr node, =20 resctrl->alloc =3D virObjectRef(alloc); =20 - VIR_STEAL_PTR(ret, resctrl); + ret =3D g_steal_pointer(&resctrl); cleanup: virDomainResctrlDefFree(resctrl); return ret; @@ -30529,7 +30529,7 @@ virDomainDiskSetBlockIOTune(virDomainDiskDefPtr dis= k, =20 VIR_FREE(disk->blkdeviotune.group_name); disk->blkdeviotune =3D *info; - VIR_STEAL_PTR(disk->blkdeviotune.group_name, tmp_group); + disk->blkdeviotune.group_name =3D g_steal_pointer(&tmp_group); =20 return 0; } diff --git a/src/conf/node_device_conf.c b/src/conf/node_device_conf.c index d33f6736ab..5b4bb34a5a 100644 --- a/src/conf/node_device_conf.c +++ b/src/conf/node_device_conf.c @@ -2354,7 +2354,7 @@ virNodeDeviceCapsListExport(virNodeDeviceDefPtr def, #undef MAYBE_ADD_CAP =20 if (want_list) - VIR_STEAL_PTR(*list, tmp); + *list =3D g_steal_pointer(&tmp); ret =3D ncaps; cleanup: VIR_FREE(tmp); @@ -2386,18 +2386,18 @@ virNodeDeviceGetSCSIHostCaps(virNodeDevCapSCSIHostP= tr scsi_host) goto cleanup; } VIR_FREE(scsi_host->wwpn); - VIR_STEAL_PTR(scsi_host->wwpn, tmp); + scsi_host->wwpn =3D g_steal_pointer(&tmp); =20 if (!(tmp =3D virVHBAGetConfig(NULL, scsi_host->host, "node_name")= )) { VIR_WARN("Failed to read WWNN for host%d", scsi_host->host); goto cleanup; } VIR_FREE(scsi_host->wwnn); - VIR_STEAL_PTR(scsi_host->wwnn, tmp); + scsi_host->wwnn =3D g_steal_pointer(&tmp); =20 if ((tmp =3D virVHBAGetConfig(NULL, scsi_host->host, "fabric_name"= ))) { VIR_FREE(scsi_host->fabric_wwn); - VIR_STEAL_PTR(scsi_host->fabric_wwn, tmp); + scsi_host->fabric_wwn =3D g_steal_pointer(&tmp); } } =20 @@ -2466,7 +2466,7 @@ virNodeDeviceGetSCSITargetCaps(const char *sysfsPath, goto cleanup; =20 VIR_FREE(scsi_target->rport); - VIR_STEAL_PTR(scsi_target->rport, rport); + scsi_target->rport =3D g_steal_pointer(&rport); =20 if (virFCReadRportValue(scsi_target->rport, "port_name", &scsi_target->wwpn) < 0) { @@ -2591,7 +2591,7 @@ virNodeDeviceGetPCIMdevTypesCaps(const char *sysfspat= h, if (rc <=3D 0) return rc; =20 - VIR_STEAL_PTR(pci_dev->mdev_types, types); + pci_dev->mdev_types =3D g_steal_pointer(&types); pci_dev->nmdev_types =3D rc; pci_dev->flags |=3D VIR_NODE_DEV_CAP_FLAG_PCI_MDEV; =20 diff --git a/src/conf/snapshot_conf.c b/src/conf/snapshot_conf.c index 9628a8df62..0e9d307321 100644 --- a/src/conf/snapshot_conf.c +++ b/src/conf/snapshot_conf.c @@ -362,7 +362,7 @@ virDomainSnapshotDefParse(xmlXPathContextPtr ctxt, "disk-only snapshot")); goto cleanup; } - VIR_STEAL_PTR(def->file, memoryFile); + def->file =3D g_steal_pointer(&memoryFile); =20 /* verify that memory path is absolute */ if (def->file && def->file[0] !=3D '/') { @@ -407,7 +407,7 @@ virDomainSnapshotDefParse(xmlXPathContextPtr ctxt, if (!offline && virSaveCookieParse(ctxt, &def->cookie, saveCookie) < 0) goto cleanup; =20 - VIR_STEAL_PTR(ret, def); + ret =3D g_steal_pointer(&def); =20 cleanup: VIR_FREE(creation); @@ -543,7 +543,7 @@ virDomainSnapshotRedefineValidate(virDomainSnapshotDefP= tr def, return -1; } else { /* Transfer the domain def */ - VIR_STEAL_PTR(def->parent.dom, otherdef->parent.dom); + def->parent.dom =3D g_steal_pointer(&otherdef->parent.dom); } } } @@ -1008,7 +1008,7 @@ virDomainSnapshotRedefinePrep(virDomainObjPtr vm, flags) < 0) { /* revert any stealing of the snapshot domain definition */ if (check_if_stolen && def->parent.dom && !otherdef->parent.dom) - VIR_STEAL_PTR(otherdef->parent.dom, def->parent.dom); + otherdef->parent.dom =3D g_steal_pointer(&def->parent.dom); return -1; } if (other) { diff --git a/src/conf/storage_conf.c b/src/conf/storage_conf.c index bddede0934..9e4978b560 100644 --- a/src/conf/storage_conf.c +++ b/src/conf/storage_conf.c @@ -659,7 +659,7 @@ virStoragePoolDefParseSource(xmlXPathContextPtr ctxt, goto cleanup; } =20 - VIR_STEAL_PTR(source->auth, authdef); + source->auth =3D g_steal_pointer(&authdef); } =20 /* Option protocol version string (NFSvN) */ @@ -822,7 +822,7 @@ virStoragePoolDefRefreshParse(xmlXPathContextPtr ctxt, return -1; =20 refresh->volume.allocation =3D tmp; - VIR_STEAL_PTR(def->refresh, refresh); + def->refresh =3D g_steal_pointer(&refresh); return 0; } =20 @@ -998,7 +998,7 @@ virStoragePoolDefParseXML(xmlXPathContextPtr ctxt) return NULL; } =20 - VIR_STEAL_PTR(ret, def); + ret =3D g_steal_pointer(&def); return ret; } =20 @@ -1433,7 +1433,7 @@ virStorageVolDefParseXML(virStoragePoolDefPtr pool, VIR_FREE(nodes); } =20 - VIR_STEAL_PTR(ret, def); + ret =3D g_steal_pointer(&def); return ret; } =20 diff --git a/src/conf/virnetworkobj.c b/src/conf/virnetworkobj.c index 560839ea7d..435f3611e9 100644 --- a/src/conf/virnetworkobj.c +++ b/src/conf/virnetworkobj.c @@ -620,7 +620,7 @@ virNetworkObjAssignDefLocked(virNetworkObjListPtr nets, obj->persistent =3D !(flags & VIR_NETWORK_OBJ_LIST_ADD_LIVE); } =20 - VIR_STEAL_PTR(ret, obj); + ret =3D g_steal_pointer(&obj); =20 cleanup: virNetworkObjEndAPI(&obj); diff --git a/src/conf/virnwfilterbindingobj.c b/src/conf/virnwfilterbinding= obj.c index 06dd66a5d1..48d53fafb1 100644 --- a/src/conf/virnwfilterbindingobj.c +++ b/src/conf/virnwfilterbindingobj.c @@ -93,7 +93,7 @@ virNWFilterBindingObjStealDef(virNWFilterBindingObjPtr ob= j) { virNWFilterBindingDefPtr def; =20 - VIR_STEAL_PTR(def, obj->def); + def =3D g_steal_pointer(&obj->def); return def; } =20 diff --git a/src/conf/virsecretobj.c b/src/conf/virsecretobj.c index f9f222dcd3..ebf6ca7680 100644 --- a/src/conf/virsecretobj.c +++ b/src/conf/virsecretobj.c @@ -393,7 +393,7 @@ virSecretObjListAdd(virSecretObjListPtr secrets, virObjectRef(obj); } =20 - VIR_STEAL_PTR(ret, obj); + ret =3D g_steal_pointer(&obj); =20 cleanup: virSecretObjEndAPI(&obj); diff --git a/src/conf/virstorageobj.c b/src/conf/virstorageobj.c index b9717b2fa2..e5ef9c8a68 100644 --- a/src/conf/virstorageobj.c +++ b/src/conf/virstorageobj.c @@ -1115,7 +1115,7 @@ virStoragePoolObjIsDuplicate(virStoragePoolObjListPtr= pools, } } =20 - VIR_STEAL_PTR(*objRet, obj); + *objRet =3D g_steal_pointer(&obj); ret =3D 1; } else { /* UUID does not match, but if a name matches, refuse it */ @@ -1540,7 +1540,7 @@ virStoragePoolObjAssignDef(virStoragePoolObjPtr obj, } else { if (!obj->newDef && flags & VIR_STORAGE_POOL_OBJ_LIST_ADD_LIVE) - VIR_STEAL_PTR(obj->newDef, obj->def); + obj->newDef =3D g_steal_pointer(&obj->def); =20 virStoragePoolDefFree(obj->def); obj->def =3D def; --=20 2.19.2 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list