[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 4/9] ehci: check device is not NULL before callin
From: |
Liam Merwick |
Subject: |
[Qemu-devel] [PATCH v2 4/9] ehci: check device is not NULL before calling usb_ep_get() |
Date: |
Wed, 6 Feb 2019 13:36:51 +0000 |
In ehci_process_itd(), the call to ehci_find_device() can return NULL
if it doesn't find a device matching 'devaddr' so explicitly check
the return value before passing it to usb_ep_get().
Signed-off-by: Liam Merwick <address@hidden>
---
hw/usb/hcd-ehci.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
index 9b132cb0d392..62dab0592fa2 100644
--- a/hw/usb/hcd-ehci.c
+++ b/hw/usb/hcd-ehci.c
@@ -1439,9 +1439,12 @@ static int ehci_process_itd(EHCIState *ehci,
qemu_sglist_add(&ehci->isgl, ptr1 + off, len);
}
- pid = dir ? USB_TOKEN_IN : USB_TOKEN_OUT;
-
dev = ehci_find_device(ehci, devaddr);
+ if (dev == NULL) {
+ ehci_trace_guest_bug(ehci, "no device found");
+ return -1;
+ }
+ pid = dir ? USB_TOKEN_IN : USB_TOKEN_OUT;
ep = usb_ep_get(dev, pid, endp);
if (ep && ep->type == USB_ENDPOINT_XFER_ISOC) {
usb_packet_setup(&ehci->ipacket, pid, ep, 0, addr, false,
--
1.8.3.1
- [Qemu-devel] [PATCH v2 0/9] Dealing with Null devices in usb_ep_get(), Liam Merwick, 2019/02/06
- [Qemu-devel] [PATCH v2 1/9] usb: rearrange usb_ep_get(), Liam Merwick, 2019/02/06
- [Qemu-devel] [PATCH v2 5/9] ohci: check device is not NULL before calling usb_ep_get(), Liam Merwick, 2019/02/06
- [Qemu-devel] [PATCH v2 2/9] xhci: add asserts to help with static code analysis, Liam Merwick, 2019/02/06
- [Qemu-devel] [PATCH v2 3/9] xhci: check device is not NULL before calling usb_ep_get(), Liam Merwick, 2019/02/06
- [Qemu-devel] [PATCH v2 4/9] ehci: check device is not NULL before calling usb_ep_get(),
Liam Merwick <=
- [Qemu-devel] [PATCH v2 8/9] usb: add device checks before redirector calls to usb_ep_get(), Liam Merwick, 2019/02/06
- [Qemu-devel] [PATCH v2 6/9] uhci: check device is not NULL before calling usb_ep_get(), Liam Merwick, 2019/02/06
- [Qemu-devel] [PATCH v2 7/9] usb: check device is not NULL before calling usb_ep_get(), Liam Merwick, 2019/02/06
- [Qemu-devel] [PATCH v2 9/9] usb: remove unnecessary NULL device check from usb_ep_get(), Liam Merwick, 2019/02/06
- Re: [Qemu-devel] [PATCH v2 0/9] Dealing with Null devices in usb_ep_get(), Gerd Hoffmann, 2019/02/08