Hi Szymon, On Thu, Feb 07, 2013, Szymon Janc wrote: > --- > plugins/neard.c | 23 ++++++++++++++--------- > 1 file changed, 14 insertions(+), 9 deletions(-) Patches 1-8 have been applied but this one had something that imo needs fixing: > -static gboolean agent_registered = FALSE; > +static char *neard_path = NULL; > static gboolean agent_register_postpone = FALSE; > > /* For NFC mimetype limits max OOB EIR size */ > @@ -115,7 +115,7 @@ static void register_agent_cb(DBusPendingCall *call, void *user_data) > } > > dbus_message_unref(reply); > - agent_registered = TRUE; > + neard_path = g_strdup(dbus_message_get_sender(reply)); If you're going to store the neard D-Bus name in this variable then please don't call it "path". That's just confusing. Use neard_name or neard_service instead. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
