IB/mthca: Check validity of output parameter pointer
authorLeon Romanovsky <leon@kernel.org>
Sat, 15 Apr 2017 15:47:25 +0000 (18:47 +0300)
committerDoug Ledford <dledford@redhat.com>
Fri, 21 Apr 2017 16:29:31 +0000 (12:29 -0400)
The mthca driver didn't check supplied pointer to functions
mthca_cmd_poll() and mthca_cmd_wait(). This caused to the following
smatch errors:

drivers/infiniband/hw/mthca/mthca_cmd.c:371 mthca_cmd_poll() error: we previously assumed 'out_param' could be null (see line 353)
drivers/infiniband/hw/mthca/mthca_cmd.c:454 mthca_cmd_wait() error: we previously assumed 'out_param' could be null (see line 432)

In reality all callers of these functions are setting out_is_imm
flag are providing pointer too. However it is better to check
again to remove smatch errors to achieve warning free subsystem.

Signed-off-by: Leon Romanovsky <leon@kernel.org>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/mthca/mthca_cmd.c

index c7f49bbb0c72b6d6af86eb2bb42984c5e3effb17..9d83a53c0c67df1ce035fba618d20257725cd486 100644 (file)
@@ -367,12 +367,16 @@ static int mthca_cmd_poll(struct mthca_dev *dev,
                goto out;
        }
 
-       if (out_is_imm)
+       if (out_is_imm && out_param) {
                *out_param =
                        (u64) be32_to_cpu((__force __be32)
                                          __raw_readl(dev->hcr + HCR_OUT_PARAM_OFFSET)) << 32 |
                        (u64) be32_to_cpu((__force __be32)
                                          __raw_readl(dev->hcr + HCR_OUT_PARAM_OFFSET + 4));
+       } else if (out_is_imm) {
+               err = -EINVAL;
+               goto out;
+       }
 
        status = be32_to_cpu((__force __be32) __raw_readl(dev->hcr + HCR_STATUS_OFFSET)) >> 24;
        if (status) {
@@ -450,8 +454,12 @@ static int mthca_cmd_wait(struct mthca_dev *dev,
                err = mthca_status_to_errno(context->status);
        }
 
-       if (out_is_imm)
+       if (out_is_imm && out_param) {
                *out_param = context->out_param;
+       } else if (out_is_imm) {
+               err = -EINVAL;
+               goto out;
+       }
 
 out:
        spin_lock(&dev->cmd.context_lock);