From 61c1117f08f4ed6f7e5cfa59cc9e02cf19c8f92d Mon Sep 17 00:00:00 2001 From: Hans de Goede Date: Tue, 31 May 2011 11:35:20 +0200 Subject: [PATCH] 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 --- usb-linux.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/usb-linux.c b/usb-linux.c index ed1d56add7..feb4d36d0b 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];