i40iw: Fixes for static checker warnings
authorShiraz Saleem <shiraz.saleem@intel.com>
Wed, 19 Jul 2017 18:55:26 +0000 (13:55 -0500)
committerDoug Ledford <dledford@redhat.com>
Fri, 18 Aug 2017 18:01:09 +0000 (14:01 -0400)
Remove NULL check for cm_node->listener in i40iw_accept
as listener is always present at this point.

Remove the check for cm_node->accept_pend and related code
in i40iw_cm_event_connected as the cm_node in this context
is only pertinent to active node and cm_node->accept_pend
is always 0.

This fixes the following smatch warnings,

drivers/infiniband/hw/i40iw/i40iw_cm.c:3691 i40iw_accept()
error: we previously assumed 'cm_node->listener' could be null

drivers/infiniband/hw/i40iw/i40iw_cm.c:4061 i40iw_cm_event_connected()
error: we previously assumed 'cm_node->listener' could be null

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Shiraz Saleem <shiraz.saleem@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/i40iw/i40iw_cm.c

index 5a2fa743676caa3aded77b9a5be3670928454caf..a2b135039e5ab859e76c3ddec0ae813f350bf7cc 100644 (file)
@@ -3687,8 +3687,6 @@ int i40iw_accept(struct iw_cm_id *cm_id, struct iw_cm_conn_param *conn_param)
 
        cm_node->accelerated = 1;
        if (cm_node->accept_pend) {
-               if (!cm_node->listener)
-                       i40iw_pr_err("cm_node->listener NULL for passive node\n");
                atomic_dec(&cm_node->listener->pend_accepts_cnt);
                cm_node->accept_pend = 0;
        }
@@ -4056,12 +4054,7 @@ static void i40iw_cm_event_connected(struct i40iw_cm_event *event)
        i40iw_modify_qp(&iwqp->ibqp, &attr, IB_QP_STATE, NULL);
 
        cm_node->accelerated = 1;
-       if (cm_node->accept_pend) {
-               if (!cm_node->listener)
-                       i40iw_pr_err("listener is null for passive node\n");
-               atomic_dec(&cm_node->listener->pend_accepts_cnt);
-               cm_node->accept_pend = 0;
-       }
+
        return;
 
 error: