diff options
author | Wei Yongjun <weiyongjun1@huawei.com> | 2016-10-31 15:04:10 +0000 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2016-10-31 10:28:08 -0600 |
commit | 18eddaedc940a49425364df98abda218ce1e771c (patch) | |
tree | 224fc5a5e32e8b6f749fa44218786a76883a48c6 /drivers/scsi/mvsas | |
parent | 0ce57f8af1782fd12d3a81872a4ab97244989802 (diff) | |
download | op-kernel-dev-18eddaedc940a49425364df98abda218ce1e771c.zip op-kernel-dev-18eddaedc940a49425364df98abda218ce1e771c.tar.gz |
mvsas: fix error return code in mvs_task_prep()
Fix to return error code -ENOMEM from the error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'drivers/scsi/mvsas')
-rw-r--r-- | drivers/scsi/mvsas/mv_sas.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c index 86eb199..c7cc803 100644 --- a/drivers/scsi/mvsas/mv_sas.c +++ b/drivers/scsi/mvsas/mv_sas.c @@ -791,8 +791,10 @@ static int mvs_task_prep(struct sas_task *task, struct mvs_info *mvi, int is_tmf slot->slot_tag = tag; slot->buf = pci_pool_alloc(mvi->dma_pool, GFP_ATOMIC, &slot->buf_dma); - if (!slot->buf) + if (!slot->buf) { + rc = -ENOMEM; goto err_out_tag; + } memset(slot->buf, 0, MVS_SLOT_BUF_SZ); tei.task = task; |