rping: Fixes race, where ibv context was getting freed before memory was deregistered

While running rping as a client without server on the other end,
rping_test_client fails and the ibv context was getting freed
before memory was deregistered. This patch fixes it.

Signed-off-by: Hariprasad Shenai <hariprasad@chelsio.com>
Signed-off-by: Sean Hefty <sean.hefty@intel.com>

diff --git a/examples/rping.c b/examples/rping.c
index 949cbe673070591f7dce681045ec75a2f68322b5..f0414dec7e4ae6771b9dcecf46eb8fbe5d40c075 100644 (file)
--- a/examples/rping.c
+++ b/examples/rping.c
@@ -1055,18 +1055,19 @@ static int rping_run_client(struct rping_cb *cb)
        ret = rping_connect_client(cb);
        if (ret) {
                fprintf(stderr, "connect error %d\n", ret);
-               goto err2;
+               goto err3;
        }
 
        ret = rping_test_client(cb);
        if (ret) {
                fprintf(stderr, "rping client failed: %d\n", ret);
-               goto err3;
+               goto err4;
        }
 
        ret = 0;
-err3:
+err4:
        rdma_disconnect(cb->cm_id);
+err3:
        pthread_join(cb->cqthread, NULL);
 err2:
        rping_free_buffers(cb);