[libvirt] [dbus PATCH 01/10] Implement BridgeName property for Network interface.
Katerina Koukiou
kkoukiou at redhat.com
Thu Apr 5 13:52:17 UTC 2018
On Thu, 2018-04-05 at 15:44 +0200, Pavel Hrdina wrote:
> On Thu, Apr 05, 2018 at 03:29:25PM +0200, Katerina Koukiou wrote:
> > Signed-off-by: Katerina Koukiou <kkoukiou at redhat.com>
> > ---
> > data/org.libvirt.Network.xml | 4 ++++
> > src/network.c | 23 +++++++++++++++++++++++
> > test/test_network.py | 1 +
> > 3 files changed, 28 insertions(+)
> >
> > diff --git a/data/org.libvirt.Network.xml
> > b/data/org.libvirt.Network.xml
> > index 1215ac3..83a6b9e 100644
> > --- a/data/org.libvirt.Network.xml
> > +++ b/data/org.libvirt.Network.xml
> > @@ -3,6 +3,10 @@
> >
> > <node name="/org/libvirt/network">
> > <interface name="org.libvirt.Network">
> > + <property name="BridgeName" type="s" access="read">
> > + <annotation name="org.gtk.GDBus.DocString"
> > + value="See https://libvirt.org/html/libvirt-libvirt-networ
> > k.html#virNetworkGetBridgeName"/>;
> > + </property>
> > <property name="Name" type="s" access="read">
> > <annotation name="org.gtk.GDBus.DocString"
> > value="See https://libvirt.org/html/libvirt-libvirt-networ
> > k.html#virNetworkGetName"/>;
> > diff --git a/src/network.c b/src/network.c
> > index 56cbb41..fab017c 100644
> > --- a/src/network.c
> > +++ b/src/network.c
> > @@ -24,6 +24,28 @@ virtDBusNetworkGetVirNetwork(virtDBusConnect
> > *connect,
> > return network;
> > }
> >
> > +static void
> > +virtDBusNetworkGetBridgeName(const gchar *objectPath,
> > + gpointer userData,
> > + GVariant **value,
> > + GError **error)
> > +{
> > + virtDBusConnect *connect = userData;
> > + g_autoptr(virNetwork) network = NULL;
> > + const gchar *bridge;
>
> This needs to be g_autofree gchar *bridge = NULL;
>
> virtDBusNetworkGetVirNetwork returns a string that needs to be freed.
You mean virNetworkGetBridgeName. Nice catch, thanks.
>
> > +
> > + network = virtDBusNetworkGetVirNetwork(connect, objectPath,
> > error);
> > + if (!network)
> > + return;
> > +
> > + bridge = virNetworkGetBridgeName(network);
> > +
> > + if (!bridge)
> > + return virtDBusUtilSetLastVirtError(error);
> > +
> > + *value = g_variant_new("s", bridge);
> > +}
>
> Reviewed-by: Pavel Hrdina <phrdina at redhat.com>
More information about the libvir-list
mailing list