On Wed, 2017-09-27 at 11:12 +0200, Szymon Janc wrote: > Hi Bastien, > > On Monday, 4 September 2017 20:12:06 CEST Bastien Nocera wrote: > > <snip> > > + char device_addr[18]; > > If this is needed only for debug then just convert address to string > in place > where it is printed. The string was in string format already when we get it from the device, but I've switched that to pass around a bdaddr_t instead. <snip> > > + closure = g_try_new0(struct authentication_closure, 1); > > Just use new0() instead. Fixed. I've also added a btd_device_set_trusted() as well, which was bizarrely missing from this patch. I guess it got lost when I started splitting up the original patches. -- 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