Commit 714bc5ef authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman

musb: potential use after free

We assign "urb->hcpriv = qh;" a few lines down.  I'm pretty sure we
want it "urb->hcpriv" to be NULL not a freed value.
Signed-off-by: default avatarDan Carpenter <error27@gmail.com>
Signed-off-by: default avatarFelipe Balbi <felipe.balbi@nokia.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 5e0e61af
...@@ -2042,6 +2042,7 @@ static int musb_urb_enqueue( ...@@ -2042,6 +2042,7 @@ static int musb_urb_enqueue(
* odd, rare, error prone, but legal. * odd, rare, error prone, but legal.
*/ */
kfree(qh); kfree(qh);
qh = NULL;
ret = 0; ret = 0;
} else } else
ret = musb_schedule(musb, qh, ret = musb_schedule(musb, qh,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment