Remove all usage of VIR_RETURN_PTR

Prefer:
    return g_steal_pointer(&ptr);

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

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 8616f820fccc1cf61c151a1955c2f880f593eba0..8b56ff3458d886e48af28ee53561da0d6ae50eb4 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -16560,7 +16560,7 @@ virDomainDeviceDefParse(const char *xmlStr,
     if (virDomainDeviceDefValidate(dev, def, flags, xmlopt) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(dev);
+    return g_steal_pointer(&dev);
 }
 
 
@@ -21681,7 +21681,7 @@ virDomainDefParseNode(xmlDocPtr xml,
     if (virDomainDefValidate(def, caps, flags, xmlopt) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(def);
+    return g_steal_pointer(&def);
 }
 
 
@@ -30672,7 +30672,7 @@ virDomainNetDefToNetworkPort(virDomainDefPtr dom,
 
     port->trustGuestRxFilters = iface->trustGuestRxFilters;
 
-    VIR_RETURN_PTR(port);
+    return g_steal_pointer(&port);
 }
 
 int
@@ -30925,7 +30925,7 @@ virDomainNetDefActualToNetworkPort(virDomainDefPtr dom,
     port->class_id = actual->class_id;
     port->trustGuestRxFilters = actual->trustGuestRxFilters;
 
-    VIR_RETURN_PTR(port);
+    return g_steal_pointer(&port);
 }
 
 

diff --git a/src/conf/secret_conf.c b/src/conf/secret_conf.c
index 6ee93159337da021cf95cd9cfc7963d2990a6422..3716d5731f01f9bcd02107ac4926923b57aa700f 100644 (file)
--- a/src/conf/secret_conf.c
+++ b/src/conf/secret_conf.c
@@ -193,7 +193,7 @@ secretXMLParseNode(xmlDocPtr xml, xmlNodePtr root)
         && virSecretDefParseUsage(ctxt, def) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(def);
+    return g_steal_pointer(&def);
 }
 
 static virSecretDefPtr

diff --git a/src/conf/storage_conf.c b/src/conf/storage_conf.c
index 4c7e7b3f4d09308da931bfce11852d8feec7c639..bddede0934509143ac85db5da0726ea1c3660de1 100644 (file)
--- a/src/conf/storage_conf.c
+++ b/src/conf/storage_conf.c
@@ -718,7 +718,7 @@ virStoragePoolDefParseSourceString(const char *srcSpec,
                                      node) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(def);
+    return g_steal_pointer(&def);
 }
 
 

diff --git a/src/conf/virnetworkportdef.c b/src/conf/virnetworkportdef.c
index 32834e41d4210b8b9ed4f6a8302e8ac264be74fe..459384611c063411432ef4f8925099f0621b555d 100644 (file)
--- a/src/conf/virnetworkportdef.c
+++ b/src/conf/virnetworkportdef.c
@@ -253,7 +253,7 @@ virNetworkPortDefParseXML(xmlXPathContextPtr ctxt)
         return NULL;
     }
 
-    VIR_RETURN_PTR(def);
+    return g_steal_pointer(&def);
 }
 
 

diff --git a/src/libxl/xen_xl.c b/src/libxl/xen_xl.c
index 8ae0dbbbd5b70326a5dab26c49bddd0914ee5976..cbd3364fe9b826f3a7606fcd5b584120f66b96cc 100644 (file)
--- a/src/libxl/xen_xl.c
+++ b/src/libxl/xen_xl.c
@@ -2247,5 +2247,5 @@ xenFormatXL(virDomainDefPtr def, virConnectPtr conn)
     if (xenFormatXLDomainChannels(conf, def) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(conf);
+    return g_steal_pointer(&conf);
 }

diff --git a/src/libxl/xen_xm.c b/src/libxl/xen_xm.c
index 65882b7c0b484f768f5b5512a0f21faa770d43bc..e368482cacfee961fd5ea0fffea0a35d518a88e3 100644 (file)
--- a/src/libxl/xen_xm.c
+++ b/src/libxl/xen_xm.c
@@ -616,5 +616,5 @@ xenFormatXM(virConnectPtr conn,
     if (xenFormatXMInputDevs(conf, def) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(conf);
+    return g_steal_pointer(&conf);
 }

diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c
index 46584f2f1b14a21a2b01b16643028fb70d5e3576..8841006c9659b54d1c70b6be2d2fe8ea937db532 100644 (file)
--- a/src/qemu/qemu_block.c
+++ b/src/qemu/qemu_block.c
@@ -255,7 +255,7 @@ qemuBlockNodeNameGetBackingChain(virJSONValuePtr namednodes,
                                       &data) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(disks);
+    return g_steal_pointer(&disks);
 }
 
 
@@ -380,7 +380,7 @@ qemuBlockGetNodeData(virJSONValuePtr data)
                                       qemuBlockNamedNodesArrayToHash, nodedata) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(nodedata);
+    return g_steal_pointer(&nodedata);
 }
 
 
@@ -449,7 +449,7 @@ qemuBlockStorageSourceGetURI(virStorageSourcePtr src)
     if (VIR_STRDUP(uri->server, src->hosts->name) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(uri);
+    return g_steal_pointer(&uri);
 }
 
 
@@ -514,7 +514,7 @@ qemuBlockStorageSourceBuildJSONSocketAddress(virStorageNetHostDefPtr host,
         return NULL;
     }
 
-    VIR_RETURN_PTR(server);
+    return g_steal_pointer(&server);
 }
 
 
@@ -550,7 +550,7 @@ qemuBlockStorageSourceBuildHostsJSONSocketAddress(virStorageSourcePtr src,
         server = NULL;
     }
 
-    VIR_RETURN_PTR(servers);
+    return g_steal_pointer(&servers);
 }
 
 
@@ -617,7 +617,7 @@ qemuBlockStorageSourceBuildHostsJSONInetSocketAddress(virStorageSourcePtr src)
         server = NULL;
     }
 
-    VIR_RETURN_PTR(servers);
+    return g_steal_pointer(&servers);
 }
 
 
@@ -649,7 +649,7 @@ qemuBlockStorageSourceGetGlusterProps(virStorageSourcePtr src,
         virJSONValueObjectAdd(props, "u:debug", src->debugLevel, NULL) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(props);
+    return g_steal_pointer(&props);
 }
 
 
@@ -989,7 +989,7 @@ qemuBlockStorageSourceGetVvfatProps(virStorageSourcePtr src,
         virJSONValueObjectAdd(ret, "b:rw", !src->readonly, NULL) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(ret);
+    return g_steal_pointer(&ret);
 }
 
 
@@ -1162,7 +1162,7 @@ qemuBlockStorageSourceGetBackendProps(virStorageSourcePtr src,
         }
     }
 
-    VIR_RETURN_PTR(fileprops);
+    return g_steal_pointer(&fileprops);
 }
 
 
@@ -1315,7 +1315,7 @@ qemuBlockStorageSourceGetBlockdevFormatCommonProps(virStorageSourcePtr src)
     if (qemuBlockStorageSourceGetBlockdevGetCacheProps(src, props) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(props);
+    return g_steal_pointer(&props);
 }
 
 
@@ -1387,7 +1387,7 @@ qemuBlockStorageSourceGetBlockdevFormatProps(virStorageSourcePtr src)
         virJSONValueObjectAdd(props, "s:driver", driver, NULL) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(props);
+    return g_steal_pointer(&props);
 }
 
 
@@ -1435,7 +1435,7 @@ qemuBlockStorageSourceGetBlockdevProps(virStorageSourcePtr src,
         }
     }
 
-    VIR_RETURN_PTR(props);
+    return g_steal_pointer(&props);
 }
 
 
@@ -1496,7 +1496,7 @@ qemuBlockStorageSourceAttachPrepareBlockdev(virStorageSourcePtr src,
     data->storageNodeName = src->nodestorage;
     data->formatNodeName = src->nodeformat;
 
-    VIR_RETURN_PTR(data);
+    return g_steal_pointer(&data);
 }
 
 
@@ -1753,7 +1753,7 @@ qemuBlockStorageSourceChainDetachPrepareBlockdev(virStorageSourcePtr src)
             return NULL;
     }
 
-    VIR_RETURN_PTR(data);
+    return g_steal_pointer(&data);
 }
 
 
@@ -1781,7 +1781,7 @@ qemuBlockStorageSourceChainDetachPrepareDrive(virStorageSourcePtr src,
     if (VIR_APPEND_ELEMENT(data->srcdata, data->nsrcdata, backend) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(data);
+    return g_steal_pointer(&data);
 }
 
 

diff --git a/src/qemu/qemu_blockjob.c b/src/qemu/qemu_blockjob.c
index cda31f8719a0d6d3082cb574c4695f9ae87af0f0..6b2c370d9f4811f1ccf045ea50af157fc5701a8f 100644 (file)
--- a/src/qemu/qemu_blockjob.c
+++ b/src/qemu/qemu_blockjob.c
@@ -118,7 +118,7 @@ qemuBlockJobDataNew(qemuBlockJobType type,
     job->newstate = -1;
     job->type = type;
 
-    VIR_RETURN_PTR(job);
+    return g_steal_pointer(&job);
 }
 
 
@@ -214,7 +214,7 @@ qemuBlockJobDiskNew(virDomainObjPtr vm,
     if (qemuBlockJobRegister(job, vm, disk, true) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(job);
+    return g_steal_pointer(&job);
 }
 
 
@@ -243,7 +243,7 @@ qemuBlockJobDiskNewPull(virDomainObjPtr vm,
     if (qemuBlockJobRegister(job, vm, disk, true) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(job);
+    return g_steal_pointer(&job);
 }
 
 
@@ -280,7 +280,7 @@ qemuBlockJobDiskNewCommit(virDomainObjPtr vm,
     if (qemuBlockJobRegister(job, vm, disk, true) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(job);
+    return g_steal_pointer(&job);
 }
 
 
@@ -311,7 +311,7 @@ qemuBlockJobNewCreate(virDomainObjPtr vm,
     if (qemuBlockJobRegister(job, vm, NULL, true) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(job);
+    return g_steal_pointer(&job);
 }
 
 
@@ -345,7 +345,7 @@ qemuBlockJobDiskNewCopy(virDomainObjPtr vm,
     if (qemuBlockJobRegister(job, vm, disk, true) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(job);
+    return g_steal_pointer(&job);
 }
 
 

diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index d5f29ba5d479f1ac9335571f2a73be841a35d202..3991ea3234c22b60502c4b9e05d56122e42ad306 100644 (file)
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -10535,7 +10535,7 @@ qemuBuildCommandLine(virQEMUDriverPtr driver,
         cfg->logTimestamp)
         virCommandAddArgList(cmd, "-msg", "timestamp=on", NULL);
 
-    VIR_RETURN_PTR(cmd);
+    return g_steal_pointer(&cmd);
 }
 
 
@@ -10724,7 +10724,7 @@ qemuBuildHotpluggableCPUProps(const virDomainVcpuDef *vcpu)
         virJSONValueObjectPrependString(ret, "driver", vcpupriv->type) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(ret);
+    return g_steal_pointer(&ret);
 }
 
 
@@ -10824,7 +10824,7 @@ qemuBuildStorageSourceChainAttachPrepareDrive(virDomainDiskDefPtr disk,
     if (VIR_APPEND_ELEMENT(data->srcdata, data->nsrcdata, elem) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(data);
+    return g_steal_pointer(&data);
 }
 
 
@@ -10874,7 +10874,7 @@ qemuBuildStorageSourceChainAttachPrepareBlockdev(virStorageSourcePtr top,
             return NULL;
     }
 
-    VIR_RETURN_PTR(data);
+    return g_steal_pointer(&data);
 }
 
 
@@ -10901,5 +10901,5 @@ qemuBuildStorageSourceChainAttachPrepareBlockdevTop(virStorageSourcePtr top,
                                                             qemuCaps) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(data);
+    return g_steal_pointer(&data);
 }

diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
index 10d06d7e1c6eb2b13e6d45393d886c01a9d181b6..01ac6faed8b0de70db25b67746af1eaa75937aec 100644 (file)
--- a/src/qemu/qemu_conf.c
+++ b/src/qemu/qemu_conf.c
@@ -302,7 +302,7 @@ virQEMUDriverConfigPtr virQEMUDriverConfigNew(bool privileged)
                              &cfg->nfirmwares) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(cfg);
+    return g_steal_pointer(&cfg);
 }
 
 
@@ -1292,7 +1292,7 @@ virCapsPtr virQEMUDriverCreateCapabilities(virQEMUDriverPtr driver)
                   "DOI \"%s\"", model, doi);
     }
 
-    VIR_RETURN_PTR(caps);
+    return g_steal_pointer(&caps);
 }
 
 
@@ -1411,7 +1411,7 @@ virQEMUDriverGetDomainCapabilities(virQEMUDriverPtr driver,
     }
 
     virObjectRef(domCaps);
-    VIR_RETURN_PTR(domCaps);
+    return g_steal_pointer(&domCaps);
 }
 
 

diff --git a/src/qemu/qemu_dbus.c b/src/qemu/qemu_dbus.c
index d0f25dd0d410b710f85e37ba18d77e149678204e..f4205a74bb0109c1e4a973653d106f7c386c9942 100644 (file)
--- a/src/qemu/qemu_dbus.c
+++ b/src/qemu/qemu_dbus.c
@@ -48,7 +48,7 @@ qemuDBusVMStateNew(const char *id, const char *addr)
     if (VIR_STRDUP(self->addr, addr) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(self);
+    return g_steal_pointer(&self);
 }
 
 

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 1bda2f5159ee563815c5e8614867746899b1a01b..e8f8749b9469f3c7e07ef1a0896385003ada70d9 100644 (file)
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -3039,7 +3039,7 @@ qemuDomainObjPrivateXMLParseBlockjobChain(xmlNodePtr node,
     if (virDomainDiskBackingStoreParse(ctxt, src, xmlflags, xmlopt) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(src);
+    return g_steal_pointer(&src);
 }
 
 

diff --git a/src/qemu/qemu_interface.c b/src/qemu/qemu_interface.c
index 11d21837eaa91391fa01f0ee289ecd6388d461ab..6e3e634b309b3093b50d32793a9e77071257719e 100644 (file)
--- a/src/qemu/qemu_interface.c
+++ b/src/qemu/qemu_interface.c
@@ -662,7 +662,7 @@ qemuInterfacePrepareSlirp(virQEMUDriverPtr driver,
             return NULL;
     }
 
-    VIR_RETURN_PTR(slirp);
+    return g_steal_pointer(&slirp);
 }
 
 

diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index 5ca7c592dcf89ce96f9798332e415734161bf3cb..70edd0477344b2836108c2c55769cbc15b45e5f3 100644 (file)
--- a/src/qemu/qemu_monitor_json.c
+++ b/src/qemu/qemu_monitor_json.c
@@ -2568,7 +2568,7 @@ qemuMonitorJSONBlockStatsCollectData(virJSONValuePtr dev,
             bstats->wr_highest_offset_valid = true;
     }
 
-    VIR_RETURN_PTR(bstats);
+    return g_steal_pointer(&bstats);
 }
 
 

diff --git a/src/qemu/qemu_qapi.c b/src/qemu/qemu_qapi.c
index 41179f72ecad7406621deac57ee6ff83e9071e3c..484f274c6334752379c14e855973420fdbd867b1 100644 (file)
--- a/src/qemu/qemu_qapi.c
+++ b/src/qemu/qemu_qapi.c
@@ -512,5 +512,5 @@ virQEMUQAPISchemaConvert(virJSONValuePtr schemareply)
                                       schema) < 0)
         return NULL;
 
-    VIR_RETURN_PTR(schema);
+    return g_steal_pointer(&schema);
 }

diff --git a/src/qemu/qemu_slirp.c b/src/qemu/qemu_slirp.c
index 6e6393f49dde07b8f6adabe4e11c8b6549a42b64..0b45315a0b1545e1ef2e45263e9b9e3bdf8f3075 100644 (file)
--- a/src/qemu/qemu_slirp.c
+++ b/src/qemu/qemu_slirp.c
@@ -90,7 +90,7 @@ qemuSlirpNew(void)
     slirp->pid = (pid_t)-1;
     slirp->fd[0] = slirp->fd[1] = -1;
 
-    VIR_RETURN_PTR(slirp);
+    return g_steal_pointer(&slirp);
 }
 
 
@@ -141,7 +141,7 @@ qemuSlirpNewForHelper(const char *helper)
         qemuSlirpSetFeature(slirp, tmp);
     }
 
-    VIR_RETURN_PTR(slirp);
+    return g_steal_pointer(&slirp);
 }
 
 

diff --git a/src/util/virhostdev.c b/src/util/virhostdev.c
index 5ceadc60260c9d85a567514c240ecb97c4477393..2a4a09534756055b4ebd99d197f1dc38c549a7ca 100644 (file)
--- a/src/util/virhostdev.c
+++ b/src/util/virhostdev.c
@@ -200,7 +200,7 @@ virHostdevManagerNew(void)
         umask(old_umask);
     }
 
-    VIR_RETURN_PTR(hostdevMgr);
+    return g_steal_pointer(&hostdevMgr);
 }
 
 virHostdevManagerPtr
@@ -285,7 +285,7 @@ virHostdevGetPCIHostDeviceList(virDomainHostdevDefPtr *hostdevs, int nhostdevs)
         pci = NULL;
     }
 
-    VIR_RETURN_PTR(pcidevs);
+    return g_steal_pointer(&pcidevs);
 }
 
 static int

diff --git a/src/util/virpci.c b/src/util/virpci.c
index fa1c67b4cbf3425ad47ca05c19ea9ab4f0cf43f8..d173b29476184040e3fb6b2c11e67d705dca697b 100644 (file)
--- a/src/util/virpci.c
+++ b/src/util/virpci.c
@@ -1419,7 +1419,7 @@ virPCIDeviceNew(unsigned int domain,
 
     VIR_DEBUG("%s %s: initialized", dev->id, dev->name);
 
-    VIR_RETURN_PTR(dev);
+    return g_steal_pointer(&dev);
 }
 
 

diff --git a/tools/virsh-completer.c b/tools/virsh-completer.c
index b7d347b389c5df26b96113ea7ac19a65e333dd7e..dd03c98a1654e847c06af9aa374aa16a38a2f70f 100644 (file)
--- a/tools/virsh-completer.c
+++ b/tools/virsh-completer.c
@@ -126,5 +126,5 @@ virshCommaStringListComplete(const char *input,
         nret++;
     }
 
-    VIR_RETURN_PTR(ret);
+    return g_steal_pointer(&ret);
 }