summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorAlan Stern <stern@rowland.harvard.edu>2005-07-26 10:30:40 -0400
committerJames Bottomley <jejb@mulgrave.(none)>2005-09-10 11:21:02 -0500
commitb70d37bf61f278f9d9adf17c52af6b2d0ae7800c (patch)
tree2b1e207cf93ac43e0787204764cf1b231fda3c93 /drivers
parentf631b4be76355dc3bf49563c706a9fb938993bde (diff)
downloadop-kernel-dev-b70d37bf61f278f9d9adf17c52af6b2d0ae7800c.zip
op-kernel-dev-b70d37bf61f278f9d9adf17c52af6b2d0ae7800c.tar.gz
[SCSI] Fix module removal/device add race
This patch (as546) fixes an oops-causing failure to check the return code from scsi_device_get. The call can return an error if the LLD is being unloaded from memory. Signed-off-by: Alan Stern <stern@rowland.harvard.edu> Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/scsi/scsi_scan.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c
index 76577fa..a0975c7 100644
--- a/drivers/scsi/scsi_scan.c
+++ b/drivers/scsi/scsi_scan.c
@@ -870,8 +870,12 @@ static int scsi_probe_and_add_lun(struct scsi_target *starget,
out_free_sdev:
if (res == SCSI_SCAN_LUN_PRESENT) {
if (sdevp) {
- scsi_device_get(sdev);
- *sdevp = sdev;
+ if (scsi_device_get(sdev) == 0) {
+ *sdevp = sdev;
+ } else {
+ __scsi_remove_device(sdev);
+ res = SCSI_SCAN_NO_RESPONSE;
+ }
}
} else {
if (sdev->host->hostt->slave_destroy)
OpenPOWER on IntegriCloud