mirror of
https://github.com/torvalds/linux.git
synced 2024-09-24 08:53:09 +00:00
staging: dwc2: check for null pointer before dereferencing it
We were testing qtd->urb pointer for null after we had already dereferenced it Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Paul Zimmerman <paulz@synopsys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
de9169a1e0
commit
7902c16b9a
|
@ -1377,13 +1377,12 @@ static void dwc2_hc_nyet_intr(struct dwc2_hsotg *hsotg,
|
||||||
hsotg->core_params->dma_enable > 0) {
|
hsotg->core_params->dma_enable > 0) {
|
||||||
qtd->complete_split = 0;
|
qtd->complete_split = 0;
|
||||||
qtd->isoc_split_offset = 0;
|
qtd->isoc_split_offset = 0;
|
||||||
if (++qtd->isoc_frame_index == qtd->urb->packet_count) {
|
if (qtd->urb &&
|
||||||
if (qtd->urb)
|
++qtd->isoc_frame_index == qtd->urb->packet_count) {
|
||||||
dwc2_host_complete(hsotg,
|
dwc2_host_complete(hsotg, qtd->urb->priv,
|
||||||
qtd->urb->priv,
|
qtd->urb, 0);
|
||||||
qtd->urb, 0);
|
|
||||||
dwc2_release_channel(hsotg, chan, qtd,
|
dwc2_release_channel(hsotg, chan, qtd,
|
||||||
DWC2_HC_XFER_URB_COMPLETE);
|
DWC2_HC_XFER_URB_COMPLETE);
|
||||||
} else {
|
} else {
|
||||||
dwc2_release_channel(hsotg, chan, qtd,
|
dwc2_release_channel(hsotg, chan, qtd,
|
||||||
DWC2_HC_XFER_NO_HALT_STATUS);
|
DWC2_HC_XFER_NO_HALT_STATUS);
|
||||||
|
|
Loading…
Reference in New Issue
Block a user