On Tue, 2010-03-02 at 11:40 +0100, Oliver Neukum wrote: > Am Dienstag, 2. März 2010 11:28:45 schrieb Libin Yang: > > @@ -827,9 +864,18 @@ int xhci_urb_dequeue(struct usb_hcd *hcd, struct urb *urb, int status) > > ep_ring = ep->ring; > > xhci_dbg(xhci, "Endpoint ring:\n"); > > xhci_debug_ring(xhci, ep_ring); > > - td = (struct xhci_td *) urb->hcpriv; > > > > - list_add_tail(&td->cancelled_td_list, &ep->cancelled_td_list); > > + urb_priv = urb->hcpriv; > > + > > + if (urb_priv) { > > How can this be NULL? > > Regards > Oliver > Thanks for the comment. Will remove unnecessary if-clauses. -- Thanks, Andiry -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html