dev_io/dev_io_core.c: fix duplicate destroy on zactor PIPE

We don't need to destroy zactor PIPE sockets, as CZMQ zactor infrastructure already does this.

dev_io/dev_io_core.c: fix duplicate destroy on zactor PIPE
We don't need to destroy zactor PIPE sockets, as CZMQ zactor infrastructure already does this.
bb2f6119 · Lucas Russo · dfc97063 · bb2f6119
Commit bb2f6119 authored Apr 27, 2016 by Lucas Russo
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

dev_io_core.c src/dev_io/dev_io_core.c +5 -5

No files found.
--- a/src/dev_io/dev_io_core.c
+++ b/src/dev_io/dev_io_core.c
@@ -378,11 +378,11 @@ devio_err_e devio_destroy (devio_t **self_p)
        zsock_destroy (&self->pipe_backend);
        zsock_destroy (&self->pipe_frontend);

-        if (self->pipe) {
-            DBE_DEBUG (DBG_DEV_IO | DBG_LVL_INFO,
-                    "[dev_io_core:destroy] Destroying DEVIO pipe\n");
-            zsock_destroy(&self->pipe);
-        }
+        /* Do not destroy PIPE as CZMQ actor thread will do it.
+         * See github issue #116 (https://github.com/lnls-dig/bpm-sw/issues/116)
+         *
+         *  zsock_destroy(&self->pipe);
+         * */

        /* Destroy all remamining sockets if any */
        DBE_DEBUG (DBG_DEV_IO | DBG_LVL_INFO,