diff options
| author | Alan Stern <stern@rowland.harvard.edu> | 2005-07-26 10:30:40 -0400 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.(none)> | 2005-09-10 11:21:02 -0500 |
| commit | b70d37bf61f278f9d9adf17c52af6b2d0ae7800c (patch) | |
| tree | 2b1e207cf93ac43e0787204764cf1b231fda3c93 /drivers/scsi/scsi_scan.c | |
| parent | f631b4be76355dc3bf49563c706a9fb938993bde (diff) | |
| download | linux-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/scsi/scsi_scan.c')
| -rw-r--r-- | drivers/scsi/scsi_scan.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index 76577fae60fa..a0975c78b968 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) |