Messages in this thread Patch in this message | | | Date | Mon, 10 Nov 2014 15:32:29 -0500 | From | Dave Jones <> | Subject | Re: usb: dwc3: ep0: return early on NULL requests |
| |
On Sat, Nov 01, 2014 at 10:20:47PM +0000, Linux Kernel wrote: > Gitweb: http://git.kernel.org/linus/;a=commit;h=6856d30c6c0038dc0648009853533af3af6c5ba8 > Commit: 6856d30c6c0038dc0648009853533af3af6c5ba8 > Parent: 7d643664ea559b36188cae264047ce3c9bfec3a2 > Refname: refs/heads/master > Author: Felipe Balbi <balbi@ti.com> > AuthorDate: Tue Sep 30 11:43:20 2014 -0500 > Committer: Felipe Balbi <balbi@ti.com> > CommitDate: Mon Oct 20 15:58:49 2014 -0500 > > usb: dwc3: ep0: return early on NULL requests > > if our list of requests is empty, return early.
....
> diff --git a/drivers/usb/dwc3/ep0.c b/drivers/usb/dwc3/ep0.c > index ae6b575..a47cc1e 100644 > --- a/drivers/usb/dwc3/ep0.c > +++ b/drivers/usb/dwc3/ep0.c > @@ -789,9 +789,6 @@ static void dwc3_ep0_complete_data(struct dwc3 *dwc, > > dwc->ep0_next_event = DWC3_EP0_NRDY_STATUS; > > - r = next_request(&ep0->request_list); > - ur = &r->request; > - > trb = dwc->ep0_trb;
Since the deletion of this assignment to 'r', we're now doing a comparison on it, resulting in dead code that will never be reached..
** CID 1251381: Logically dead code (DEADCODE) /drivers/usb/dwc3/ep0.c: 799 in dwc3_ep0_complete_data()
________________________________________________________________________________________________________ *** CID 1251381: Logically dead code (DEADCODE) /drivers/usb/dwc3/ep0.c: 799 in dwc3_ep0_complete_data() 793 794 status = DWC3_TRB_SIZE_TRBSTS(trb->size); 795 if (status == DWC3_TRBSTS_SETUP_PENDING) { 796 dwc3_trace(trace_dwc3_ep0, "Setup Pending received"); 797 798 if (r) >>> CID 1251381: Logically dead code (DEADCODE) >>> Execution cannot reach this statement: "dwc3_gadget_giveback(ep0, r...". 799 dwc3_gadget_giveback(ep0, r, -ECONNRESET); 800 801 return; 802 } 803 804 r = next_request(&ep0->request_list); Picked up by Coverity scan.
Dave
| |