linux-kernel - Re: usb: dwc3: ep0: return early on NULL requests

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20141110203229.GA31249@redhat.com>
Date:	Mon, 10 Nov 2014 15:32:29 -0500
From:	Dave Jones <davej@...hat.com>
To:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Cc:	balbi@...com
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@...com>
 > AuthorDate: Tue Sep 30 11:43:20 2014 -0500
 > Committer:  Felipe Balbi <balbi@...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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/