On Wed, 2016-04-13 at 22:53 +0200, Thomas Haller wrote:
> On Wed, 2016-04-13 at 21:43 +0300, Mikhail Efremov wrote:
> >
> > A bus name is not an object path.
> > Fix crash introduced by commit
> > 17ae85788987ef1f7c92a251c535312163144c33.
> > ---
> > src/nm-pptp-pppd-plugin.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/nm-pptp-pppd-plugin.c b/src/nm-pptp-pppd-plugin.c
> > index 2b8c819..4567fcc 100644
> > --- a/src/nm-pptp-pppd-plugin.c
> > +++ b/src/nm-pptp-pppd-plugin.c
> > @@ -318,8 +318,8 @@ plugin_init (void)
> > proxy = g_dbus_proxy_new_sync (bus,
> > G_DBUS_PROXY_FLAGS_DO_NOT_L
> > OA
> > D_PROPERTIES,
> > NULL,
> > - NM_DBUS_SERVICE_PPTP_PPP,
> > bus_name,
> > + NM_DBUS_PATH_PPTP_PPP,
> > NM_DBUS_INTERFACE_PPTP_PPP,
> > NULL, &err);
> > g_object_unref (bus);
>
Thank you, applied.
> Hi Mikhail,
>
>
>
> Thank you. A few lines before there is also:
>
> bus_name = getenv ("NM_DBUS_SERVICE_PPTP");
> if (!bus_name)
> bus_name = NM_DBUS_SERVICE_PPTP;
>
>
> thus, this should be changed too.
That one looks correct to me?
>
>
> [[ I leave this to Lubomir (CC'ed) ]]
>
>
> Thomas
Lubo
_______________________________________________
networkmanager-list mailing list
[email protected]
https://mail.gnome.org/mailman/listinfo/networkmanager-list
