diff options
author | Fengguang Wu <fengguang.wu@intel.com> | 2012-07-30 16:19:07 -0400 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-08-16 20:33:28 -0400 |
commit | f0e0e9bba5eed2086f6001be14f21566a49bed10 (patch) | |
tree | 9b27b7e2162d1707e444267630cc77c95542e92c | |
parent | d9875690d9b89a866022ff49e3fcea892345ad92 (diff) |
tcm_vhost: Fix incorrect IS_ERR() usage in vhost_scsi_map_iov_to_sgl
Fix up a new coccinelle warnings reported by Fengguang Wu + Intel
0-DAY kernel build testing backend:
drivers/vhost/tcm_vhost.c:537:23-29: ERROR: allocation function on line
533 returns NULL not ERR_PTR on failure
vim +537 drivers/vhost/tcm_vhost.c
534 if (!sg)
535 return -ENOMEM;
536 pr_debug("%s sg %p sgl_count %u is_err %ld\n", __func__,
> 537 sg, sgl_count, IS_ERR(sg));
538 sg_init_table(sg, sgl_count);
539
540 tv_cmd->tvc_sgl = sg;
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
-rw-r--r-- | drivers/vhost/tcm_vhost.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/vhost/tcm_vhost.c b/drivers/vhost/tcm_vhost.c index fb366540ed54..481af88a5ff5 100644 --- a/drivers/vhost/tcm_vhost.c +++ b/drivers/vhost/tcm_vhost.c | |||
@@ -533,8 +533,8 @@ static int vhost_scsi_map_iov_to_sgl(struct tcm_vhost_cmd *tv_cmd, | |||
533 | sg = kmalloc(sizeof(tv_cmd->tvc_sgl[0]) * sgl_count, GFP_ATOMIC); | 533 | sg = kmalloc(sizeof(tv_cmd->tvc_sgl[0]) * sgl_count, GFP_ATOMIC); |
534 | if (!sg) | 534 | if (!sg) |
535 | return -ENOMEM; | 535 | return -ENOMEM; |
536 | pr_debug("%s sg %p sgl_count %u is_err %ld\n", __func__, | 536 | pr_debug("%s sg %p sgl_count %u is_err %d\n", __func__, |
537 | sg, sgl_count, IS_ERR(sg)); | 537 | sg, sgl_count, !sg); |
538 | sg_init_table(sg, sgl_count); | 538 | sg_init_table(sg, sgl_count); |
539 | 539 | ||
540 | tv_cmd->tvc_sgl = sg; | 540 | tv_cmd->tvc_sgl = sg; |