On Fri, 2020-04-03 at 17:15 +0200, Rafael Fonseca wrote:
--- a/src/conf/virstorageobj.c
+++ b/src/conf/virstorageobj.c
@@ -1052,7 +1052,13 @@
virStoragePoolObjVolumeListExport(virConnectPtr conn,
return data.nvols;
error:
- virObjectListFree(data.vols);
+ if (data.vols) {
+ while (data.nvols--) {
+ if (data.vols[data.nvols])
+ g_object_unref(data.vols[data.nvols]);
+ }
+ VIR_FREE(data.vols);
+ }
return -1;
}
This pattern has been repeated several times now. Is it worth adding a
helper function that's equivalent to virObjectListFree but for
GObjects?