Author: fadams
Date: Sat Apr 26 16:12:29 2014
New Revision: 1590244

URL: http://svn.apache.org/r1590244
Log:
PROTON-569: Initialise pipe file descriptors to -1 in messenger prior to 
calling pn_pipe

Modified:
    qpid/proton/trunk/proton-c/src/messenger/messenger.c

Modified: qpid/proton/trunk/proton-c/src/messenger/messenger.c
URL: 
http://svn.apache.org/viewvc/qpid/proton/trunk/proton-c/src/messenger/messenger.c?rev=1590244&r1=1590243&r2=1590244&view=diff
==============================================================================
--- qpid/proton/trunk/proton-c/src/messenger/messenger.c (original)
+++ qpid/proton/trunk/proton-c/src/messenger/messenger.c Sat Apr 26 16:12:29 
2014
@@ -602,6 +602,10 @@ pn_messenger_t *pn_messenger(const char 
        pni_interruptor_finalize);
     pn_list_add(m->pending, m->interruptor);
     m->interrupted = false;
+    // Explicitly initialise pipe file descriptors to invalid values in case 
pipe
+    // fails, if we don't do this m->ctrl[0] could default to 0 - which is 
stdin.
+    m->ctrl[0] = -1;
+    m->ctrl[1] = -1;
     pn_pipe(m->io, m->ctrl);
     pni_selectable_set_fd(m->interruptor, m->ctrl[0]);
     pni_selectable_set_context(m->interruptor, m);



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to