Signed-off-by: Rafael Fonseca <r4f4rfs@xxxxxxxxx>
---
src/conf/domain_conf.c | 6 +--
src/conf/virnetworkobj.c | 5 +--
src/datatypes.c | 66 ++++++++++++++++++++---------
src/datatypes.h | 15 ++++---
src/libvirt-network.c | 7 ++-
src/libvirt_private.syms | 2 +-
src/remote/remote_daemon_dispatch.c | 7 +--
src/remote/remote_driver.c | 4 +-
src/rpc/gendispatch.pl | 1 +
9 files changed, 64 insertions(+), 49 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 57a2c09b91..fa87fb0590 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -31572,7 +31572,7 @@ virDomainNetReleaseActualDevice(virConnectPtr conn,
virDomainNetDefPtr iface)
{
g_autoptr(virNetwork) net = NULL;
- virNetworkPortPtr port = NULL;
+ g_autoptr(virNetworkPort) port = NULL;
int ret = -1;
/* Port might not exist if a failure occurred during VM startup */
@@ -31591,7 +31591,6 @@ virDomainNetReleaseActualDevice(virConnectPtr conn,
goto cleanup;
cleanup:
- virObjectUnref(port);
return ret;
}
@@ -31645,7 +31644,7 @@ virDomainNetBandwidthUpdate(virDomainNetDefPtr iface,
virNetDevBandwidthPtr newBandwidth)
{
g_autoptr(virNetwork) net = NULL;
- virNetworkPortPtr port = NULL;
+ g_autoptr(virNetworkPort) port = NULL;
virTypedParameterPtr params = NULL;
int nparams = 0;
virConnectPtr conn = NULL;
@@ -31670,7 +31669,6 @@ virDomainNetBandwidthUpdate(virDomainNetDefPtr iface,
cleanup:
virObjectUnref(conn);
virTypedParamsFree(params, nparams);
- virObjectUnref(port);
return ret;
}
diff --git a/src/conf/virnetworkobj.c b/src/conf/virnetworkobj.c
index 2d375deb47..946008cb5e 100644
--- a/src/conf/virnetworkobj.c
+++ b/src/conf/virnetworkobj.c
@@ -1821,10 +1821,7 @@ virNetworkObjPortListExport(virNetworkPtr net,
ret = data.nports;
cleanup:
- while (data.ports && data.nports)
- virObjectUnref(data.ports[--data.nports]);
-
- VIR_FREE(data.ports);
+ virGObjectListFreeCount(data.ports, data.nports);
return ret;
}
diff --git a/src/datatypes.c b/src/datatypes.c
index cb4c5705b2..f06438c1c3 100644
--- a/src/datatypes.c
+++ b/src/datatypes.c
@@ -37,7 +37,6 @@ virClassPtr virConnectClass;
virClassPtr virConnectCloseCallbackDataClass;
virClassPtr virDomainClass;
virClassPtr virInterfaceClass;
-virClassPtr virNetworkPortClass;
virClassPtr virNodeDeviceClass;
virClassPtr virSecretClass;
virClassPtr virStreamClass;
@@ -48,7 +47,6 @@ static void virConnectDispose(void *obj);
static void virConnectCloseCallbackDataDispose(void *obj);
static void virDomainDispose(void *obj);
static void virInterfaceDispose(void *obj);
-static void virNetworkPortDispose(void *obj);
static void virNodeDeviceDispose(void *obj);
static void virSecretDispose(void *obj);
static void virStreamDispose(void *obj);
@@ -109,6 +107,24 @@ vir_network_class_init(virNetworkClass *klass)
obj->finalize = virNetworkFinalize;
}
+G_DEFINE_TYPE(virNetworkPort, vir_network_port, G_TYPE_OBJECT);
+static void virNetworkPortDispose(GObject *obj);
+static void virNetworkPortFinalize(GObject *obj);
+
+static void
+vir_network_port_init(virNetworkPort *np G_GNUC_UNUSED)
+{
+}
+
+static void
+vir_network_port_class_init(virNetworkPortClass *klass)
+{
+ GObjectClass *obj = G_OBJECT_CLASS(klass);
+
+ obj->dispose = virNetworkPortDispose;
+ obj->finalize = virNetworkPortFinalize;
+}
+
G_DEFINE_TYPE(virNWFilter, vir_nw_filter, G_TYPE_OBJECT);
static void virNWFilterDispose(GObject *obj);
static void virNWFilterFinalize(GObject *obj);
@@ -200,7 +216,6 @@ virDataTypesOnceInit(void)
DECLARE_CLASS_LOCKABLE(virConnectCloseCallbackData);
DECLARE_CLASS(virDomain);
DECLARE_CLASS(virInterface);
- DECLARE_CLASS(virNetworkPort);
DECLARE_CLASS(virNodeDevice);
DECLARE_CLASS(virSecret);
DECLARE_CLASS(virStream);
@@ -508,54 +523,63 @@ virNetworkFinalize(GObject *obj)
* @uuid: pointer to the uuid
*
* Allocates a new network port object. When the object is no longer needed,
- * virObjectUnref() must be called in order to not leak data.
+ * g_object_unref() must be called in order to not leak data.
*
* Returns a pointer to the network port object, or NULL on error.
*/
virNetworkPortPtr
virGetNetworkPort(virNetworkPtr net, const unsigned char *uuid)
{
- virNetworkPortPtr ret = NULL;
+ g_autoptr(virNetworkPort) ret = NULL;
if (virDataTypesInitialize() < 0)
return NULL;
- virCheckNetworkGoto(net, error);
- virCheckNonNullArgGoto(uuid, error);
+ virCheckNetworkReturn(net, NULL);
+ virCheckNonNullArgReturn(uuid, NULL);
- if (!(ret = virObjectNew(virNetworkPortClass)))
- goto error;
+ ret = VIR_NETWORK_PORT(g_object_new(VIR_TYPE_NETWORK_PORT, NULL));
ret->net = g_object_ref(net);
memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN);
- return ret;
-
- error:
- virObjectUnref(ret);
- return NULL;
+ return g_steal_pointer(&ret);
}
/**
* virNetworkPortDispose:
* @obj: the network port to release
*
- * Unconditionally release all memory associated with a network port.
- * The network port object must not be used once this method returns.
+ * Unreferences the associated network object, which may also be
+ * released if its ref count hits zero.
+ */
+static void
+virNetworkPortDispose(GObject *obj)
+{
+ virNetworkPortPtr port = VIR_NETWORK_PORT(obj);
+
+ g_clear_object(&port->net);
+
+ G_OBJECT_CLASS(vir_network_port_parent_class)->dispose(obj);
+}
+
+/**
+ * virNetworkPortFinalize:
+ * @obj: the network port to release
*
- * It will also unreference the associated network object,
- * which may also be released if its ref count hits zero.
+ * Unconditionally releases all memory associated with a network port.
+ * The network port object must not be used once this method returns.
*/
static void
-virNetworkPortDispose(void *obj)
+virNetworkPortFinalize(GObject *obj)
{
- virNetworkPortPtr port = obj;
+ virNetworkPortPtr port = VIR_NETWORK_PORT(obj);
char uuidstr[VIR_UUID_STRING_BUFLEN];
virUUIDFormat(port->uuid, uuidstr);
VIR_DEBUG("release network port %p %s", port, uuidstr);
- g_object_unref(port->net);
+ G_OBJECT_CLASS(vir_network_port_parent_class)->finalize(obj);
}
diff --git a/src/datatypes.h b/src/datatypes.h
index 706cec340f..41f366d21a 100644
--- a/src/datatypes.h
+++ b/src/datatypes.h
@@ -33,7 +33,6 @@
extern virClassPtr virConnectClass;
extern virClassPtr virDomainClass;
extern virClassPtr virInterfaceClass;
-extern virClassPtr virNetworkPortClass;
extern virClassPtr virNodeDeviceClass;
extern virClassPtr virSecretClass;
extern virClassPtr virStreamClass;
@@ -57,6 +56,9 @@ G_DECLARE_FINAL_TYPE(virDomainSnapshot,
#define VIR_TYPE_NETWORK vir_network_get_type()
G_DECLARE_FINAL_TYPE(virNetwork, vir_network, VIR, NETWORK, GObject);
+#define VIR_TYPE_NETWORK_PORT vir_network_port_get_type()
+G_DECLARE_FINAL_TYPE(virNetworkPort, vir_network_port, VIR, NETWORK_PORT, GObject);
+
#define VIR_TYPE_NW_FILTER vir_nw_filter_get_type()
G_DECLARE_FINAL_TYPE(virNWFilter, vir_nw_filter, VIR, NW_FILTER, GObject);
@@ -146,8 +148,8 @@ G_DECLARE_FINAL_TYPE(virAdmClient, vir_adm_client, VIR, ADM_CLIENT, GObject);
#define virCheckNetworkPortReturn(obj, retval) \
do { \
- virNetworkPortPtr _port = (obj); \
- if (!virObjectIsClass(_port, virNetworkPortClass) || \
+ virNetworkPortPtr _port = VIR_NETWORK_PORT(obj); \
+ if (_port == NULL || \
!VIR_IS_NETWORK(_port->net)) { \
virReportErrorHelper(VIR_FROM_NETWORK, \
VIR_ERR_INVALID_NETWORK_PORT, \
@@ -160,8 +162,8 @@ G_DECLARE_FINAL_TYPE(virAdmClient, vir_adm_client, VIR, ADM_CLIENT, GObject);
#define virCheckNetworkPortGoto(obj, label) \
do { \
- virNetworkPortPtr _port = (obj); \
- if (!virObjectIsClass(_port, virNetworkPortClass) || \
+ virNetworkPortPtr _port = VIR_NETWORK_PORT(obj); \
+ if (_port == NULL || \
!VIR_IS_NETWORK(_port->net)) { \
virReportErrorHelper(VIR_FROM_NETWORK, \
VIR_ERR_INVALID_NETWORK_PORT, \
@@ -654,12 +656,11 @@ struct _virNetwork {
* Internal structure associated to a network port
*/
struct _virNetworkPort {
- virObject parent;
+ GObject parent;
virNetworkPtr net; /* pointer back to the connection */
unsigned char uuid[VIR_UUID_BUFLEN]; /* the network unique identifier */
};
-G_DEFINE_AUTOPTR_CLEANUP_FUNC(virNetworkPort, virObjectUnref);
/**
diff --git a/src/libvirt-network.c b/src/libvirt-network.c
index 3847617d4d..1ab499fb18 100644
--- a/src/libvirt-network.c
+++ b/src/libvirt-network.c
@@ -1688,7 +1688,7 @@ virNetworkPortFree(virNetworkPortPtr port)
virCheckNetworkPortReturn(port, -1);
- virObjectUnref(port);
+ g_object_unref(port);
return 0;
}
@@ -1713,13 +1713,12 @@ virNetworkPortFree(virNetworkPortPtr port)
int
virNetworkPortRef(virNetworkPortPtr port)
{
- VIR_DEBUG("port=%p refs=%d", port,
- port ? port->parent.u.s.refs : 0);
+ VIR_DEBUG("port=%p", port);
virResetLastError();
virCheckNetworkPortReturn(port, -1);
- virObjectRef(port);
+ g_object_ref(port);
return 0;
}
diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index b41eee4e59..69134087c1 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -1349,6 +1349,7 @@ virCPUx86FeatureFilterSelectMSR;
vir_domain_checkpoint_get_type;
vir_domain_snapshot_get_type;
vir_network_get_type;
+vir_network_port_get_type;
vir_nw_filter_binding_get_type;
vir_nw_filter_get_type;
virConnectClass;
@@ -1373,7 +1374,6 @@ virGetStoragePool;
virGetStorageVol;
virGetStream;
virInterfaceClass;
-virNetworkPortClass;
virNewConnectCloseCallbackData;
virNodeDeviceClass;
virSecretClass;
diff --git a/src/remote/remote_daemon_dispatch.c b/src/remote/remote_daemon_dispatch.c
index f0f2a075fb..2e4e1a65db 100644
--- a/src/remote/remote_daemon_dispatch.c
+++ b/src/remote/remote_daemon_dispatch.c
@@ -7170,7 +7170,7 @@ remoteDispatchNetworkPortGetParameters(virNetServerPtr server G_GNUC_UNUSED,
remote_network_port_get_parameters_ret *ret)
{
int rv = -1;
- virNetworkPortPtr port = NULL;
+ g_autoptr(virNetworkPort) port = NULL;
virTypedParameterPtr params = NULL;
int nparams = 0;
virConnectPtr conn = remoteGetNetworkConn(client);
@@ -7196,7 +7196,6 @@ remoteDispatchNetworkPortGetParameters(virNetServerPtr server G_GNUC_UNUSED,
cleanup:
if (rv < 0)
virNetMessageSaveError(rerr);
- virObjectUnref(port);
virTypedParamsFree(params, nparams);
return rv;
}
@@ -7225,13 +7224,11 @@ get_nonnull_network(virConnectPtr conn, remote_nonnull_network network)
static virNetworkPortPtr
get_nonnull_network_port(virConnectPtr conn, remote_nonnull_network_port port)
{
- virNetworkPortPtr ret;
g_autoptr(virNetwork) net =
virGetNetwork(conn, port.net.name, BAD_CAST port.net.uuid);
if (!net)
return NULL;
- ret = virGetNetworkPort(net, BAD_CAST port.uuid);
- return ret;
+ return virGetNetworkPort(net, BAD_CAST port.uuid);
}
static virInterfacePtr
diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index d479c3a9df..9373ba78b7 100644
--- a/src/remote/remote_driver.c
+++ b/src/remote/remote_driver.c
@@ -8268,13 +8268,11 @@ get_nonnull_network(virConnectPtr conn, remote_nonnull_network network)
static virNetworkPortPtr
get_nonnull_network_port(virConnectPtr conn, remote_nonnull_network_port port)
{
- virNetworkPortPtr ret;
g_autoptr(virNetwork) net =
virGetNetwork(conn, port.net.name, BAD_CAST port.net.uuid);
if (!net)
return NULL;
- ret = virGetNetworkPort(net, BAD_CAST port.uuid);
- return ret;
+ return virGetNetworkPort(net, BAD_CAST port.uuid);
}
static virInterfacePtr
diff --git a/src/rpc/gendispatch.pl b/src/rpc/gendispatch.pl
index 2c101fab6d..5c14038428 100755
--- a/src/rpc/gendispatch.pl
+++ b/src/rpc/gendispatch.pl
@@ -199,6 +199,7 @@ my %gobject_impl = (
virDomainCheckpoint => 1,
virDomainSnapshot => 1,
virNetwork => 1,
+ virNetworkPort => 1,
virNWFilter => 1,
virNWFilterBinding => 1,
);
--
2.25.3
