Fix Metaproxy stops logging after check config failed MP-590

[metaproxy-moved-to-github.git] / src / pipe.cpp

diff --git a/src/pipe.cpp b/src/pipe.cpp
index 227f31d..309f51b 100644 (file)
--- a/src/pipe.cpp
+++ b/src/pipe.cpp
@@ -1,15 +1,28 @@
+/* This file is part of Metaproxy.
+   Copyright (C) Index Data
 
-/* $Id: pipe.cpp,v 1.3 2005-11-07 22:43:17 adam Exp $
-   Copyright (c) 2005, Index Data.
+Metaproxy is free software; you can redistribute it and/or modify it under
+the terms of the GNU General Public License as published by the Free
+Software Foundation; either version 2, or (at your option) any later
+version.
+
+Metaproxy is distributed in the hope that it will be useful, but WITHOUT ANY
+WARRANTY; without even the implied warranty of MERCHANTABILITY or
+FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+for more details.
+
+You should have received a copy of the GNU General Public License
+along with this program; if not, write to the Free Software
+Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+*/
 
-%LICENSE%
- */
 #include "config.hpp"
 
 #if HAVE_UNISTD_H
 #include <unistd.h>
 #endif
 
+#include <signal.h>
 #include <errno.h>
 #ifdef WIN32
 #include <winsock.h>
@@ -34,15 +47,18 @@
 #include <boost/thread/condition.hpp>
 
 #include <stdio.h>
+#include <string.h>
 
 #include <deque>
 
-#include <yaz++/socket-observer.h>
+#include <yazpp/socket-observer.h>
 #include <yaz/log.h>
 
 #include "pipe.hpp"
 
-namespace yp2 {
+namespace mp = metaproxy_1;
+
+namespace metaproxy_1 {
     class Pipe::Rep : public boost::noncopyable {
         friend class Pipe;
         Rep();
@@ -53,7 +69,7 @@ namespace yp2 {
     };
 }
 
-using namespace yp2;
+using namespace mp;
 
 void Pipe::Rep::close(int &fd)
 {
@@ -91,6 +107,14 @@ bool Pipe::Rep::nonblock(int s)
 
 Pipe::Pipe(int port_to_use) : m_p(new Rep)
 {
+#ifdef WIN32
+    WSADATA wsaData;
+    WORD wVersionRequested = MAKEWORD(2, 0);
+    if (WSAStartup( wVersionRequested, &wsaData ))
+        throw Pipe::Error("WSAStartup failed");
+#else
+    port_to_use = 0;  // we'll just use pipe on Unix
+#endif
     if (port_to_use)
     {
         // create server socket
@@ -99,7 +123,7 @@ Pipe::Pipe(int port_to_use) : m_p(new Rep)
             throw Pipe::Error("could not create socket");
 #ifndef WIN32
         unsigned long one = 1;
-        if (setsockopt(m_p->m_socket, SOL_SOCKET, SO_REUSEADDR, (char*) 
+        if (setsockopt(m_p->m_socket, SOL_SOCKET, SO_REUSEADDR, (char*)
                        &one, sizeof(one)) < 0)
             throw Pipe::Error("setsockopt error");
 #endif
@@ -109,10 +133,10 @@ Pipe::Pipe(int port_to_use) : m_p(new Rep)
         add.sin_port = htons(port_to_use);
         add.sin_addr.s_addr = INADDR_ANY;
         struct sockaddr *addr = ( struct sockaddr *) &add;
-      
+
         if (bind(m_p->m_socket, addr, sizeof(struct sockaddr_in)))
             throw Pipe::Error("could not bind on socket");
-        
+
         if (listen(m_p->m_socket, 3) < 0)
             throw Pipe::Error("could not listen on socket");
 
@@ -123,11 +147,11 @@ Pipe::Pipe(int port_to_use) : m_p(new Rep)
             memcpy(&add.sin_addr.s_addr, &tmpadd, sizeof(struct in_addr));
         else
             throw Pipe::Error("inet_addr failed");
-            
+
         m_p->m_fd[1] = socket(AF_INET, SOCK_STREAM, 0);
         if (m_p->m_fd[1] < 0)
             throw Pipe::Error("could not create socket");
-        
+
         m_p->nonblock(m_p->m_fd[1]);
 
         if (connect(m_p->m_fd[1], addr, sizeof(*addr)) < 0)
@@ -165,7 +189,13 @@ Pipe::Pipe(int port_to_use) : m_p(new Rep)
     else
     {
 #ifndef WIN32
-        pipe(m_p->m_fd);
+        if (pipe(m_p->m_fd))
+            throw Pipe::Error("pipe failed");
+        else
+        {
+            assert(m_p->m_fd[0] >= 0);
+            assert(m_p->m_fd[1] >= 0);
+        }
 #endif
     }
 }
@@ -175,6 +205,9 @@ Pipe::~Pipe()
     m_p->close(m_p->m_fd[0]);
     m_p->close(m_p->m_fd[1]);
     m_p->close(m_p->m_socket);
+#ifdef WIN32
+    WSACleanup();
+#endif
 }
 
 int &Pipe::read_fd() const
@@ -190,8 +223,8 @@ int &Pipe::write_fd() const
 /*
  * Local variables:
  * c-basic-offset: 4
+ * c-file-style: "Stroustrup"
  * indent-tabs-mode: nil
- * c-file-style: "stroustrup"
  * End:
  * vim: shiftwidth=4 tabstop=8 expandtab
  */