From 0a51aa7bfc88d55b98fdde0d497b072e6911457d Mon Sep 17 00:00:00 2001 From: Garrett D'Amore Date: Sat, 24 Jun 2017 14:11:35 -0700 Subject: Protocols keep their own reference counts. --- src/protocol/pubsub/sub.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'src/protocol/pubsub/sub.c') diff --git a/src/protocol/pubsub/sub.c b/src/protocol/pubsub/sub.c index 8340da77..cab745b5 100644 --- a/src/protocol/pubsub/sub.c +++ b/src/protocol/pubsub/sub.c @@ -117,18 +117,16 @@ nni_sub_pipe_start(void *arg) { nni_sub_pipe *sp = arg; - nni_pipe_hold(sp->pipe); nni_pipe_aio_recv(sp->pipe, &sp->aio_recv); return (0); } static void -nni_sub_pipe_stop(void *arg) +nni_sub_pipe_stop(nni_sub_pipe *sp) { - nni_sub_pipe *sp = arg; - nni_msgq_aio_cancel(sp->sub->urq, &sp->aio_putq); + nni_pipe_remove(sp->pipe); } @@ -141,8 +139,7 @@ nni_sub_recv_cb(void *arg) nni_msg *msg; if (nni_aio_result(&sp->aio_recv) != 0) { - nni_pipe_close(sp->pipe); - nni_pipe_rele(sp->pipe); + nni_sub_pipe_stop(sp); return; } @@ -160,8 +157,7 @@ nni_sub_putq_cb(void *arg) if (nni_aio_result(&sp->aio_putq) != 0) { nni_msg_free(sp->aio_putq.a_msg); sp->aio_putq.a_msg = NULL; - nni_pipe_close(sp->pipe); - nni_pipe_rele(sp->pipe); + nni_sub_pipe_stop(sp); return; } @@ -339,7 +335,6 @@ static nni_proto_pipe_ops nni_sub_pipe_ops = { .pipe_init = nni_sub_pipe_init, .pipe_fini = nni_sub_pipe_fini, .pipe_start = nni_sub_pipe_start, - .pipe_stop = nni_sub_pipe_stop, }; static nni_proto_sock_ops nni_sub_sock_ops = { -- cgit v1.2.3-70-g09d2