summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHans de Goede <hdegoede@redhat.com>2011-05-31 11:35:20 +0200
committerGerd Hoffmann <kraxel@redhat.com>2011-06-14 12:56:49 +0200
commit61c1117f08f4ed6f7e5cfa59cc9e02cf19c8f92d (patch)
tree5d2e0b260fe150cedbc5cd1e761b39837eb6f7fc
parentf264cfbf4a010e1be879bf2e62d323443989d72f (diff)
downloadhqemu-61c1117f08f4ed6f7e5cfa59cc9e02cf19c8f92d.zip
hqemu-61c1117f08f4ed6f7e5cfa59cc9e02cf19c8f92d.tar.gz
usb-linux: Don't do perror when errno is not set
Note that "op" also is not set, so before this change these error paths would feed NULL to perror. Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
-rw-r--r--usb-linux.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/usb-linux.c b/usb-linux.c
index ed1d56a..feb4d36 100644
--- a/usb-linux.c
+++ b/usb-linux.c
@@ -376,7 +376,8 @@ static int usb_host_claim_interfaces(USBHostDevice *dev, int configuration)
i = 0;
dev_descr_len = dev->descr[0];
if (dev_descr_len > dev->descr_len) {
- goto fail;
+ fprintf(stderr, "husb: update iface failed. descr too short\n");
+ return 0;
}
i += dev_descr_len;
@@ -404,7 +405,7 @@ static int usb_host_claim_interfaces(USBHostDevice *dev, int configuration)
if (i >= dev->descr_len) {
fprintf(stderr,
"husb: update iface failed. no matching configuration\n");
- goto fail;
+ return 0;
}
nb_interfaces = dev->descr[i + 4];
OpenPOWER on IntegriCloud