diff options
| author | Gerd Hoffmann <kraxel@redhat.com> | 2012-03-30 13:20:21 +0200 |
|---|---|---|
| committer | Gerd Hoffmann <kraxel@redhat.com> | 2012-04-17 10:23:29 +0200 |
| commit | c7020c974073ba9c0110d45361720a29ff6b2f59 (patch) | |
| tree | 85d5c114ef0ef21c7f728af7dc1da0554cb3d585 /hw | |
| parent | 714f9db06c209fd42d67e6dffd4f7fd932b51b65 (diff) | |
| download | qemu-c7020c974073ba9c0110d45361720a29ff6b2f59.tar.gz qemu-c7020c974073ba9c0110d45361720a29ff6b2f59.tar.bz2 qemu-c7020c974073ba9c0110d45361720a29ff6b2f59.zip | |
usb-ehci: drop assert()
Not sure what the purpose of the assert() was, in any case it is bogous.
We can arrive there if transfer descriptors passed to us from the guest
failed to pass sanity checks, i.e. it is guest-triggerable. We deal
with that case by resetting the host controller. Everything is ok, no
need to throw a core dump here.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Diffstat (limited to 'hw')
| -rw-r--r-- | hw/usb/hcd-ehci.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c index e12f09870c..23631a47c9 100644 --- a/hw/usb/hcd-ehci.c +++ b/hw/usb/hcd-ehci.c @@ -2009,7 +2009,6 @@ static void ehci_advance_state(EHCIState *ehci, fprintf(stderr, "processing error - resetting ehci HC\n"); ehci_reset(ehci); again = 0; - assert(0); } } while (again); |