ShuriZma
/
xemu
mirror of https://github.com/xemu-project/xemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

usb: assert on calling usb_attach(port, NULL) on a port without a dev 

with the "usb-ehci: cleanup port reset handling" patch in place no callers
are calling usb_attach(port, NULL) for a port where port->dev is NULL.

Doing that makes no sense as that causes the port detach op to get called
for a port with nothing attached. Add an assert that port->dev != NULL when
dev == NULL, and remove the check for not having a port->dev in the dev == NULL
case.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
This commit is contained in:
Hans de Goede 2011-06-24 14:26:18 +02:00 committed by Gerd Hoffmann
parent fbf9db6457
commit 45b9fd3480
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hw/usb.c
View File

@ -40,13 +40,12 @@ void usb_attach(USBPort *port, USBDevice *dev)
} else { } else {
/* detach */ /* detach */
dev = port->dev; dev = port->dev;
assert(dev);
port->ops->detach(port); port->ops->detach(port);
if (dev) {
usb_send_msg(dev, USB_MSG_DETACH); usb_send_msg(dev, USB_MSG_DETACH);
dev->port = NULL; dev->port = NULL;
port->dev = NULL; port->dev = NULL;
} }
}
} }
void usb_wakeup(USBDevice *dev) void usb_wakeup(USBDevice *dev)