RDMA/core: Fix return error value in _ib_modify_qp() to negative
authorLi Heng <liheng40@huawei.com>
Sat, 25 Jul 2020 02:56:27 +0000 (10:56 +0800)
committerJason Gunthorpe <jgg@nvidia.com>
Mon, 27 Jul 2020 15:13:34 +0000 (12:13 -0300)
The error codes in _ib_modify_qp() are supposed to be negative errno.

Fixes: 7a5c938b9ed0 ("IB/core: Check for rdma_protocol_ib only after validating port_num")
Link: https://lore.kernel.org/r/1595645787-20375-1-git-send-email-liheng40@huawei.com
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Li Heng <liheng40@huawei.com>
Reviewed-by: Parav Pandit <parav@mellanox.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/core/verbs.c

index 10f650a17ae73bbb851ca3d577595ff0faf3f0d1..3096e73797b775f706c4d257fd140024c9031ebe 100644 (file)
@@ -1710,7 +1710,7 @@ static int _ib_modify_qp(struct ib_qp *qp, struct ib_qp_attr *attr,
                if (!(rdma_protocol_ib(qp->device,
                                       attr->alt_ah_attr.port_num) &&
                      rdma_protocol_ib(qp->device, port))) {
-                       ret = EINVAL;
+                       ret = -EINVAL;
                        goto out;
                }
        }