diff options
author | Jiri Slaby <jirislaby@gmail.com> | 2009-07-13 17:25:54 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2009-08-22 18:51:55 -0400 |
commit | e34ccdfe0e08a6acb8c5e649fef1e94e6cd637f9 (patch) | |
tree | 037fc5ca114556aefc9734d6034ba3ad1dfbc162 /drivers/scsi/lpfc/lpfc_init.c | |
parent | 0fd30f77693f4fef32d30d4801cd21dcd487c2f0 (diff) |
[SCSI] lpfc: don't dereference NULL
When kzalloc fails in lpfc_hba_alloc, don't dereference the NULL by
lpfc_printf_log. Use dev_err instead.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com>
Acked-By: James Smart <james.smart@emulex.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/lpfc/lpfc_init.c')
-rw-r--r-- | drivers/scsi/lpfc/lpfc_init.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/lpfc/lpfc_init.c b/drivers/scsi/lpfc/lpfc_init.c index fc67cc65c63b..2452dc9c9014 100644 --- a/drivers/scsi/lpfc/lpfc_init.c +++ b/drivers/scsi/lpfc/lpfc_init.c | |||
@@ -4130,8 +4130,7 @@ lpfc_hba_alloc(struct pci_dev *pdev) | |||
4130 | /* Allocate memory for HBA structure */ | 4130 | /* Allocate memory for HBA structure */ |
4131 | phba = kzalloc(sizeof(struct lpfc_hba), GFP_KERNEL); | 4131 | phba = kzalloc(sizeof(struct lpfc_hba), GFP_KERNEL); |
4132 | if (!phba) { | 4132 | if (!phba) { |
4133 | lpfc_printf_log(phba, KERN_ERR, LOG_INIT, | 4133 | dev_err(&pdev->dev, "failed to allocate hba struct\n"); |
4134 | "1417 Failed to allocate hba struct.\n"); | ||
4135 | return NULL; | 4134 | return NULL; |
4136 | } | 4135 | } |
4137 | 4136 | ||