Skip to contentSkip to conversation

drivers/usb/image/* - compile warning

PATCH In review Stephen Biggs 1 participant 1 message Feb 14, 2005

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 15/21 drivers/usb/image/* - compile warning

Description: compile warning cleanup - handle error return from scsi_add_host

drivers/usb/image/microtek.c +6 -2
diff -Nurdp -X dontdiff-osdl linux-2.6.11-rc3-mm2-original/drivers/usb/image/microtek.c linux-2.6.11-rc3-mm2/drivers/usb/image/microtek.c
@@ -703,6 +703,7 @@ static int mts_usb_probe(struct usb_inte
 	int ep_in_set[3]; /* this will break if we have more than three endpoints
 			   which is why we check */
 	int *ep_in_current = ep_in_set;
+	int err_retval = -ENOMEM;
 
 	struct mts_desc * new_desc;
 	struct vendor_product const* p;
@@ -809,7 +810,10 @@ static int mts_usb_probe(struct usb_inte
 		goto out_free_urb;
 
 	new_desc->host->hostdata[0] = (unsigned long)new_desc;
-	scsi_add_host(new_desc->host, NULL); /* XXX handle failure */
+	if (scsi_add_host(new_desc->host, NULL)) {
+		err_retval = -EIO;
+		goto out_free_urb;
+	}
 	scsi_scan_host(new_desc->host);
 
 	usb_set_intfdata(intf, new_desc);
@@ -820,7 +824,7 @@ static int mts_usb_probe(struct usb_inte
  out_kfree:
 	kfree(new_desc);
  out:
-	return -ENOMEM;
+	return err_retval;
 }
 
 static void mts_usb_disconnect (struct usb_interface *intf)

_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors