Use g_steal_pointer where possible

Via coccinelle (not the handbag!)
spatches used:
@ rule1 @
identifier a, b;
symbol NULL;
@@

- b = a;
  ... when != a
- a = NULL;
+ b = g_steal_pointer(&a);

@@

- *b = a;
  ... when != a
- a = NULL;
+ *b = g_steal_pointer(&a);

Signed-off-by: Kristina Hanicova <khanicov@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
Signed-off-by: Ján Tomko <jtomko@redhat.com>

src/admin/admin_server.c

@@ -50,8 +50,7 @@ adminConnectListServers(virNetDaemonPtr dmn,
         goto cleanup;
 
     if (servers) {
-        *servers = srvs;
+        *servers = g_steal_pointer(&srvs);
-        srvs = NULL;
     }
  cleanup:
     if (ret > 0)
@@ -179,8 +178,7 @@ adminServerListClients(virNetServerPtr srv,
         return -1;
 
     if (clients) {
-        *clients = clts;
+        *clients = g_steal_pointer(&clts);
-        clts = NULL;
     }
 
     virObjectListFreeCount(clts, ret);

src/conf/capabilities.c

@@ -1501,9 +1501,8 @@ virCapabilitiesGetNUMASiblingInfo(int node,
     if (VIR_REALLOC_N(tmp, tmp_size) < 0)
         goto cleanup;
 
-    *siblings = tmp;
     *nsiblings = tmp_size;
-    tmp = NULL;
+    *siblings = g_steal_pointer(&tmp);
     tmp_size = 0;
     ret = 0;
  cleanup:

src/conf/netdev_bandwidth_conf.c

@@ -205,8 +205,7 @@ virNetDevBandwidthParse(virNetDevBandwidthPtr *bandwidth,
     if (!def->in && !def->out)
         VIR_FREE(def);
 
-    *bandwidth = def;
+    *bandwidth = g_steal_pointer(&def);
-    def = NULL;
     ret = 0;
 
  cleanup:

src/conf/virdomainobjlist.c

@@ -1083,8 +1083,7 @@ virDomainObjListExport(virDomainObjListPtr domlist,
                 goto cleanup;
         }
 
-        *domains = doms;
+        *domains = g_steal_pointer(&doms);
-        doms = NULL;
     }
 
     ret = nvms;

src/conf/virnwfilterobj.c

@@ -464,8 +464,7 @@ virNWFilterObjListExport(virConnectPtr conn,
         virNWFilterObjUnlock(obj);
     }
 
-    *filters = tmp_filters;
+    *filters = g_steal_pointer(&tmp_filters);
-    tmp_filters = NULL;
     ret = nfilters;
 
  cleanup:

src/interface/interface_backend_netcf.c

@@ -715,8 +715,7 @@ netcfConnectListAllInterfaces(virConnectPtr conn,
     if (tmp_iface_objs) {
         /* trim the array to the final size */
         ignore_value(VIR_REALLOC_N(tmp_iface_objs, niface_objs + 1));
-        *ifaces = tmp_iface_objs;
+        *ifaces = g_steal_pointer(&tmp_iface_objs);
-        tmp_iface_objs = NULL;
     }
 
     ret = niface_objs;

src/interface/interface_backend_udev.c

@@ -394,8 +394,7 @@ udevConnectListAllInterfaces(virConnectPtr conn,
     /* Trim the array to its final size */
     if (ifaces) {
         ignore_value(VIR_REALLOC_N(ifaces_list, count + 1));
-        *ifaces = ifaces_list;
+        *ifaces = g_steal_pointer(&ifaces_list);
-        ifaces_list = NULL;
     }
 
     return count;

src/qemu/qemu_driver.c

@@ -5097,8 +5097,7 @@ qemuDomainGetIOThreadsConfig(virDomainDefPtr targetDef,
         bitmap = NULL;
     }
 
-    *info = info_ret;
+    *info = g_steal_pointer(&info_ret);
-    info_ret = NULL;
     ret = targetDef->niothreadids;
 
  cleanup:
@@ -9133,9 +9132,8 @@ qemuDomainGetPerfEvents(virDomainPtr dom,
             goto endjob;
     }
 
-    *params = par;
     *nparams = npar;
-    par = NULL;
+    *params = g_steal_pointer(&par);
     npar = 0;
     ret = 0;
 
@@ -18729,8 +18727,7 @@ qemuConnectGetAllDomainStats(virConnectPtr conn,
         virObjectUnlock(vm);
     }
 
-    *retStats = tmpstats;
+    *retStats = g_steal_pointer(&tmpstats);
-    tmpstats = NULL;
 
     ret = nstats;
 
@@ -19259,9 +19256,8 @@ qemuDomainGetGuestVcpusParams(virTypedParameterPtr *params,
 
 #undef ADD_BITMAP
 
-    *params = par;
     *nparams = npar;
-    par = NULL;
+    *params = g_steal_pointer(&par);
     ret = 0;
 
  cleanup:

src/qemu/qemu_monitor_json.c

@@ -5822,8 +5822,7 @@ int qemuMonitorJSONGetMachines(qemuMonitorPtr mon,
     }
 
     ret = n;
-    *machines = infolist;
+    *machines = g_steal_pointer(&infolist);
-    infolist = NULL;
 
  cleanup:
     if (infolist) {
@@ -6270,8 +6269,7 @@ int qemuMonitorJSONGetCommands(qemuMonitorPtr mon,
     }
 
     ret = n;
-    *commands = commandlist;
+    *commands = g_steal_pointer(&commandlist);
-    commandlist = NULL;
 
 
  cleanup:
@@ -6329,8 +6327,7 @@ int qemuMonitorJSONGetEvents(qemuMonitorPtr mon,
     }
 
     ret = n;
-    *events = eventlist;
+    *events = g_steal_pointer(&eventlist);
-    eventlist = NULL;
 
  cleanup:
     g_strfreev(eventlist);
@@ -6474,8 +6471,7 @@ int qemuMonitorJSONGetObjectTypes(qemuMonitorPtr mon,
     }
 
     ret = n;
-    *types = typelist;
+    *types = g_steal_pointer(&typelist);
-    typelist = NULL;
 
  cleanup:
     g_strfreev(typelist);
@@ -6544,8 +6540,7 @@ int qemuMonitorJSONGetObjectListPaths(qemuMonitorPtr mon,
     }
 
     ret = n;
-    *paths = pathlist;
+    *paths = g_steal_pointer(&pathlist);
-    pathlist = NULL;
 
  cleanup:
     if (pathlist) {
@@ -6781,8 +6776,7 @@ qemuMonitorJSONParsePropsList(virJSONValuePtr cmd,
     }
 
     ret = count;
-    *props = proplist;
+    *props = g_steal_pointer(&proplist);
-    proplist = NULL;
 
  cleanup:
     g_strfreev(proplist);
@@ -6964,8 +6958,7 @@ qemuMonitorJSONGetMigrationCapabilities(qemuMonitorPtr mon,
     }
 
     ret = n;
-    *capabilities = list;
+    *capabilities = g_steal_pointer(&list);
-    list = NULL;
 
  cleanup:
     g_strfreev(list);
@@ -7093,8 +7086,7 @@ qemuMonitorJSONGetGICCapabilities(qemuMonitorPtr mon,
     }
 
     ret = n;
-    *capabilities = list;
+    *capabilities = g_steal_pointer(&list);
-    list = NULL;
 
  cleanup:
     VIR_FREE(list);
@@ -8155,8 +8147,7 @@ qemuMonitorJSONGetIOThreads(qemuMonitorPtr mon,
     }
 
     *niothreads = n;
-    *iothreads = infolist;
+    *iothreads = g_steal_pointer(&infolist);
-    infolist = NULL;
     ret = 0;
 
  cleanup:

src/remote/remote_driver.c

@@ -6299,8 +6299,7 @@ remoteConnectGetCPUModelNames(virConnectPtr conn,
             retmodels[i] = ret.models.models_val[i];
             ret.models.models_val[i] = NULL;
         }
-        *models = retmodels;
+        *models = g_steal_pointer(&retmodels);
-        retmodels = NULL;
     }
 
     rv = ret.ret;
@@ -7464,8 +7463,7 @@ remoteNetworkGetDHCPLeases(virNetworkPtr net,
                 goto cleanup;
         }
 
-        *leases = leases_ret;
+        *leases = g_steal_pointer(&leases_ret);
-        leases_ret = NULL;
     }
 
     rv = ret.ret;
@@ -7555,8 +7553,7 @@ remoteConnectGetAllDomainStats(virConnectPtr conn,
         elem = NULL;
     }
 
-    *retStats = tmpret;
+    *retStats = g_steal_pointer(&tmpret);
-    tmpret = NULL;
     rv = ret.retStats.retStats_len;
 
  cleanup:
@@ -7774,8 +7771,7 @@ remoteDomainInterfaceAddresses(virDomainPtr dom,
             }
         }
     }
-    *ifaces = ifaces_ret;
+    *ifaces = g_steal_pointer(&ifaces_ret);
-    ifaces_ret = NULL;
 
     rv = ret.ifaces.ifaces_len;
 

src/rpc/virnetserver.c

@@ -1111,8 +1111,7 @@ virNetServerGetClients(virNetServerPtr srv,
         }
     }
 
-    *clts = list;
+    *clts = g_steal_pointer(&list);
-    list = NULL;
     ret = nclients;
 
  cleanup:

src/util/virfile.c

@@ -840,8 +840,7 @@ int virFileLoopDeviceAssociate(const char *file,
     }
 
     VIR_DEBUG("Attached loop device  %s %d to %s", file, lofd, loname);
-    *dev = loname;
+    *dev = g_steal_pointer(&loname);
-    loname = NULL;
 
     ret = 0;
 
@@ -3200,8 +3199,7 @@ virFileOpenTty(int *ttyprimary, char **ttyName, int rawmode)
             errno = rc;
             goto cleanup;
         }
-        *ttyName = name;
+        *ttyName = g_steal_pointer(&name);
-        name = NULL;
     }
 
     ret = 0;
@@ -3637,9 +3635,8 @@ virFileFindHugeTLBFS(virHugeTLBFSPtr *ret_fs,
         tmp->deflt = tmp->size == default_hugepagesz;
     }
 
-    *ret_fs = fs;
     *ret_nfs = nfs;
-    fs = NULL;
+    *ret_fs = g_steal_pointer(&fs);
     nfs = 0;
     ret = 0;
 

src/util/virlog.c

@@ -1658,8 +1658,7 @@ virLogParseOutputs(const char *src, virLogOutputPtr **outputs)
     }
 
     ret = noutputs;
-    *outputs = list;
+    *outputs = g_steal_pointer(&list);
-    list = NULL;
  cleanup:
     g_strfreev(strings);
     return ret;
@@ -1708,8 +1707,7 @@ virLogParseFilters(const char *src, virLogFilterPtr **filters)
     }
 
     ret = nfilters;
-    *filters = list;
+    *filters = g_steal_pointer(&list);
-    list = NULL;
  cleanup:
     g_strfreev(strings);
     return ret;

src/util/virmdev.c

@@ -95,8 +95,7 @@ virMediatedDeviceGetSysfsDeviceAPI(virMediatedDevicePtr dev,
     if ((tmp = strchr(buf, '\n')))
         *tmp = '\0';
 
-    *device_api = buf;
+    *device_api = g_steal_pointer(&buf);
-    buf = NULL;
 
     return 0;
 }

src/util/virsysinfo.c

@@ -258,8 +258,7 @@ virSysinfoParsePPCSystem(const char *base, virSysinfoSystemDefPtr *sysdef)
         def = NULL;
     }
 
-    *sysdef = def;
+    *sysdef = g_steal_pointer(&def);
-    def = NULL;
     ret = 0;
     virSysinfoSystemDefFree(def);
     return ret;
@@ -381,8 +380,7 @@ virSysinfoParseARMSystem(const char *base, virSysinfoSystemDefPtr *sysdef)
         def = NULL;
     }
 
-    *sysdef = def;
+    *sysdef = g_steal_pointer(&def);
-    def = NULL;
     ret = 0;
     virSysinfoSystemDefFree(def);
     return ret;
@@ -521,8 +519,7 @@ virSysinfoParseS390System(const char *base, virSysinfoSystemDefPtr *sysdef)
         def = NULL;
     }
 
-    *sysdef = def;
+    *sysdef = g_steal_pointer(&def);
-    def = NULL;
     ret = 0;
  cleanup:
     virSysinfoSystemDefFree(def);
@@ -686,8 +683,7 @@ virSysinfoParseBIOS(const char *base, virSysinfoBIOSDefPtr *bios)
         def = NULL;
     }
 
-    *bios = def;
+    *bios = g_steal_pointer(&def);
-    def = NULL;
     ret = 0;
     virSysinfoBIOSDefFree(def);
     return ret;
@@ -763,8 +759,7 @@ virSysinfoParseX86System(const char *base, virSysinfoSystemDefPtr *sysdef)
         def = NULL;
     }
 
-    *sysdef = def;
+    *sysdef = g_steal_pointer(&def);
-    def = NULL;
     ret = 0;
     virSysinfoSystemDefFree(def);
     return ret;
@@ -845,9 +840,8 @@ virSysinfoParseX86BaseBoard(const char *base,
         ignore_value(VIR_REALLOC_N(boards, nboards));
     }
 
-    *baseBoard = boards;
     *nbaseBoard = nboards;
-    boards = NULL;
+    *baseBoard = g_steal_pointer(&boards);
     nboards = 0;
     ret = 0;
  cleanup:
@@ -915,8 +909,7 @@ virSysinfoParseX86Chassis(const char *base,
         def = NULL;
     }
 
-    *chassisdef = def;
+    *chassisdef = g_steal_pointer(&def);
-    def = NULL;
     ret = 0;
     virSysinfoChassisDefFree(def);
     return ret;

src/util/virtypedparam.c

@@ -706,9 +706,8 @@ virTypedParamsSerialize(virTypedParameterPtr params,
         j++;
     }
 
-    *remote_params_val = params_val;
     *remote_params_len = params_len;
-    params_val = NULL;
+    *remote_params_val = g_steal_pointer(&params_val);
     rv = 0;
 
  cleanup:

src/vbox/vbox_common.c

@@ -7583,8 +7583,7 @@ vboxConnectListAllDomains(virConnectPtr conn,
         /* safe to ignore, new size will be equal or less than
          * previous allocation */
         ignore_value(VIR_REALLOC_N(doms, count + 1));
-        *domains = doms;
+        *domains = g_steal_pointer(&doms);
-        doms = NULL;
     }
 
     ret = count;

src/vz/vz_driver.c

@@ -3817,8 +3817,7 @@ vzConnectGetAllDomainStats(virConnectPtr conn,
         tmpstats[nstats++] = tmp;
     }
 
-    *retStats = tmpstats;
+    *retStats = g_steal_pointer(&tmpstats);
-    tmpstats = NULL;
     ret = nstats;
 
  cleanup:

tests/commandhelper.c

@@ -116,8 +116,7 @@ static struct Arguments *parseArguments(int argc, char** argv)
         }
     }
 
-    ret = args;
+    ret = g_steal_pointer(&args);
-    args = NULL;
     return ret;
 }
 

tools/virsh-checkpoint.c

@@ -633,8 +633,7 @@ virshCheckpointListCollect(vshControl *ctl,
         qsort(checkpointlist->chks, checkpointlist->nchks,
               sizeof(*checkpointlist->chks), virshChkSorter);
 
-    ret = checkpointlist;
+    ret = g_steal_pointer(&checkpointlist);
-    checkpointlist = NULL;
 
  cleanup:
     virshCheckpointListFree(checkpointlist);

tools/virsh-domain.c

@@ -5187,8 +5187,7 @@ cmdSchedInfoUpdate(vshControl *ctl, const vshCmd *cmd,
         goto cleanup;
 
     ret = nparams;
-    *update_params = params;
+    *update_params = g_steal_pointer(&params);
-    params = NULL;
 
  cleanup:
     VIR_FREE(set_field);