diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-05-17 03:13:40 -0400 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2012-05-22 06:52:20 -0400 |
commit | 841f97bffc11daaef9eb28d201be25b1509db97d (patch) | |
tree | 45b9a5843858874093b0cd6ddc2a8e602d265ece /drivers/scsi/qla2xxx/qla_iocb.c | |
parent | 83ff74e33849cd5592efa1ea583175600b6a485e (diff) |
[SCSI] qla2xxx: don't free pool that wasn't allocated
In the original code, if dma_pool_alloc() fails then we call
dma_pool_free(). It causes an error, possibly a NULL dereference.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/qla2xxx/qla_iocb.c')
-rw-r--r-- | drivers/scsi/qla2xxx/qla_iocb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/qla2xxx/qla_iocb.c b/drivers/scsi/qla2xxx/qla_iocb.c index 73cc96da9421..70dbf53d9e0f 100644 --- a/drivers/scsi/qla2xxx/qla_iocb.c +++ b/drivers/scsi/qla2xxx/qla_iocb.c | |||
@@ -2382,7 +2382,7 @@ sufficient_dsds: | |||
2382 | if (!ctx->fcp_cmnd) { | 2382 | if (!ctx->fcp_cmnd) { |
2383 | ql_log(ql_log_fatal, vha, 0x3011, | 2383 | ql_log(ql_log_fatal, vha, 0x3011, |
2384 | "Failed to allocate fcp_cmnd for cmd=%p.\n", cmd); | 2384 | "Failed to allocate fcp_cmnd for cmd=%p.\n", cmd); |
2385 | goto queuing_error_fcp_cmnd; | 2385 | goto queuing_error; |
2386 | } | 2386 | } |
2387 | 2387 | ||
2388 | /* Initialize the DSD list and dma handle */ | 2388 | /* Initialize the DSD list and dma handle */ |