Re: [PATCH] nm-pppd-plugin: fix crash



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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]