On Llu, 2005-01-10 at 10:42, Matthias Urlichs wrote: > hcd = container_of (bus, struct usb_hcd, self); > + if(hcd->rh_timer.data) > + del_timer_sync (&hcd->rh_timer); > kfree(hcd);
If init_timer has ever been called then you can just use del_timer_sync and it does the right thing on any time (it has to since the timer could expire as you delete it). If not then timer.data isn't safe either. I think you can thus lose that "if" ------------------------------------------------------- The SF.Net email is sponsored by: Beat the post-holiday blues Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek. It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel