Hi, 2017-12-22 11:48 GMT+01:00 Szymon Janc <szymon.janc@xxxxxxxxxxx>: > Hi Grzegorz, > > On Thursday, 21 December 2017 17:47:17 CET Grzegorz Kolodziejczyk wrote: >> Advertising manager proxy is required to handle advertise related >> operations in adapter context. >> --- >> tools/btpclient.c | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) >> >> diff --git a/tools/btpclient.c b/tools/btpclient.c >> index 27e5a498e..095c5ea3e 100644 >> --- a/tools/btpclient.c >> +++ b/tools/btpclient.c >> @@ -36,6 +36,7 @@ >> >> struct btp_adapter { >> struct l_dbus_proxy *proxy; >> + struct l_dbus_proxy *ad_proxy; >> uint8_t index; >> uint32_t supported_settings; >> uint32_t current_settings; >> @@ -659,6 +660,17 @@ static void proxy_added(struct l_dbus_proxy *proxy, >> void *user_data) device->proxy = proxy; >> >> l_queue_push_tail(adapter->devices, device); >> + >> + return; >> + } >> + >> + if (!strcmp(interface, "org.bluez.LEAdvertisingManager1")) { >> + struct btp_adapter *adapter; >> + >> + adapter = find_adapter_by_path(path); > > This may return NULL if we failed to add proxy of adapter or due to invalid > message. Please check that. OK > >> + >> + adapter->ad_proxy = proxy; >> + >> return; >> } >> } > > > -- > pozdrawiam > Szymon Janc pozdrawiam Grzegorz Kołodziejczyk -- 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