| Message ID | 3e184b93-083a-bfe6-2d11-9c81203ec935@omprussia.ru |
|---|---|
| State | Superseded |
| Headers | show |
| Series | scsi: sun3x_esp: add IRQ check | expand |
Index: scsi/drivers/scsi/sun3x_esp.c =================================================================== --- scsi.orig/drivers/scsi/sun3x_esp.c +++ scsi/drivers/scsi/sun3x_esp.c @@ -206,7 +206,9 @@ static int esp_sun3x_probe(struct platfo if (!esp->command_block) goto fail_unmap_regs_dma; - host->irq = platform_get_irq(dev, 0); + host->irq = err = platform_get_irq(dev, 0); + if (err < 0) + goto fail_unmap_command_block; err = request_irq(host->irq, scsi_esp_intr, IRQF_SHARED, "SUN3X ESP", esp); if (err < 0)
The driver neglects to check the result of platform_get_irq()'s call and blithely passes the negative error codes to request_irq() (which takes *unsigned* IRQ #), causing it to fail with -EINVAL, masking the actual error code. Propagate the negative IRQ error codes upstream instead. Fixes: 0bb67f181834 ("[SCSI] sun3x_esp: convert to esp_scsi") Signed-off-by: Sergey Shtylyov <s.shtylyov@omprussia.ru> --- drivers/scsi/sun3x_esp.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)