diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-03-10 17:24:47 -0500 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-03-12 10:46:11 -0500 |
| commit | a6becb084d4beb0b0c83651541f5b53f6da01e36 (patch) | |
| tree | df88f32a2874731289c094c556ea19257eea65d7 /drivers/scsi/ibmmca.c | |
| parent | c3c026ba5a16867af47578ab2030ef51f9080084 (diff) | |
[SCSI] ibmmca: fix a NULL pointer dereference
The variable was dereferenced only if it was NULL (sic)...
Spotted by the Coverity checker.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/ibmmca.c')
| -rw-r--r-- | drivers/scsi/ibmmca.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/ibmmca.c b/drivers/scsi/ibmmca.c index b60c1b9270f1..3a8462e8d063 100644 --- a/drivers/scsi/ibmmca.c +++ b/drivers/scsi/ibmmca.c | |||
| @@ -2412,8 +2412,7 @@ static int ibmmca_proc_info(struct Scsi_Host *shpnt, char *buffer, char **start, | |||
| 2412 | spin_lock_irqsave(hosts[i]->host_lock, flags); /* Check it */ | 2412 | spin_lock_irqsave(hosts[i]->host_lock, flags); /* Check it */ |
| 2413 | host_index = i; | 2413 | host_index = i; |
| 2414 | if (!shpnt) { | 2414 | if (!shpnt) { |
| 2415 | len += sprintf(buffer + len, "\nIBM MCA SCSI: Can't find adapter for host number %d\n", | 2415 | len += sprintf(buffer + len, "\nIBM MCA SCSI: Can't find adapter"); |
| 2416 | shpnt->host_no); | ||
| 2417 | return len; | 2416 | return len; |
| 2418 | } | 2417 | } |
| 2419 | max_pun = subsystem_maxid(host_index); | 2418 | max_pun = subsystem_maxid(host_index); |