iw_cxgb4: remove the stid on listen create failure
authorSteve Wise <swise@opengridcomputing.com>
Tue, 5 Sep 2017 18:52:34 +0000 (11:52 -0700)
committerDoug Ledford <dledford@redhat.com>
Fri, 22 Sep 2017 16:59:42 +0000 (12:59 -0400)
If a listen create fails, then the server tid (stid) is incorrectly left
in the stid idr table, which can cause a touch-after-free if the stid
is looked up and the already freed endpoint is touched.  So make sure
and remove it in the error path.

Cc: stable@vger.kernel.org
Signed-off-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/cxgb4/cm.c

index 19297e40882022271557e833d0685e359c20a7d6..daf7a56e5d7ebe4a7b6dffd93245d5a1655b3805 100644 (file)
@@ -3462,7 +3462,7 @@ int c4iw_create_listen(struct iw_cm_id *cm_id, int backlog)
                cm_id->provider_data = ep;
                goto out;
        }
-
+       remove_handle(ep->com.dev, &ep->com.dev->stid_idr, ep->stid);
        cxgb4_free_stid(ep->com.dev->rdev.lldi.tids, ep->stid,
                        ep->com.local_addr.ss_family);
 fail2: