RDMA/rtrs-srv: Do not pass a valid pointer to PTR_ERR()
authorJack Wang <jinpu.wang@cloud.ionos.com>
Tue, 16 Feb 2021 14:38:07 +0000 (15:38 +0100)
committerJason Gunthorpe <jgg@nvidia.com>
Wed, 17 Feb 2021 00:01:56 +0000 (20:01 -0400)
commited408529679737a9a7ad816c8de5d59ba104bb11
treeee39bb3934f28b0fafb4151c8909cce4ae0bb0b3
parent2b5715fc17386a6223490d5b8f08d031999b0c0b
RDMA/rtrs-srv: Do not pass a valid pointer to PTR_ERR()

smatch gives the warning:

  drivers/infiniband/ulp/rtrs/rtrs-srv.c:1805 rtrs_rdma_connect() warn: passing zero to 'PTR_ERR'

Which is trying to say smatch has shown that srv is not an error pointer
and thus cannot be passed to PTR_ERR.

The solution is to move the list_add() down after full initilization of
rtrs_srv. To avoid holding the srv_mutex too long, only hold it during the
list operation as suggested by Leon.

Fixes: 03e9b33a0fd6 ("RDMA/rtrs: Only allow addition of path to an already established session")
Link: https://lore.kernel.org/r/20210216143807.65923-1-jinpu.wang@cloud.ionos.com
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jack Wang <jinpu.wang@cloud.ionos.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/ulp/rtrs/rtrs-srv.c