Hi, On Thursday 16 of January 2014 15:50:32 Johan Hedberg wrote: > Hi Szymon, > > On Thu, Jan 16, 2014, Szymon Janc wrote: > > @@ -983,12 +994,15 @@ static void update_found_device(const bdaddr_t *bdaddr, uint8_t bdaddr_type, > > eir_parse(&eir, data, data_len); > > > > dev = find_device(bdaddr); > > - if (!dev) { > > + if (!dev || !dev->found) { > > I think this would still deserve a code comment here explaining why > dev->found is important. Other than that the patch is fine, so feel free > to push it yourself once you've added the comment. Pushed. -- Best regards, Szymon Janc -- 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