Hi, On 06/27/2013 06:22 PM, Chris Dickens wrote: > Hi, > > > On Thu, Jun 27, 2013 at 3:31 AM, Hans de Goede <hdego...@redhat.com > <mailto:hdego...@redhat.com>> wrote: > > Hi, > > +/* device will still be marked as attached if hotplug monitor thread > +* hasn't processed remove event yet */ > +if (handle->dev->attached) { > > > This check is racy, you need to take the hotplug lock before making this > check, also linux_device_disconnected expects the hotplug lock to be > taken when it is called, so simply but a check around the entire block. > > > Ok, patch revised. With the current design this race condition is benign, but > better to be safe than sorry if something changes in the future. > > Ok, that makes sense, I can live with having a check in 2 places, please > respin with the suggested > fixes and send with git send-email, or attach, so that your mail client > does not foobar the patch. > > > Patch is attached.
Thanks, I've just merged this patch and pushed it to master. Regards, Hans ------------------------------------------------------------------------------ Introducing Performance Central, a new site from SourceForge and AppDynamics. Performance Central is your source for news, insights, analysis and resources for efficient Application Performance Management. Visit us today! http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.clktrk _______________________________________________ libusbx-devel mailing list libusbx-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/libusbx-devel
