Fixed bug in inetd code. The server listened on tcp:@:9999 even
[yaz-moved-to-github.git] / server / statserv.c
index 03822ef..e9d7912 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1995-1997, Index Data
+ * Copyright (c) 1995-1998, Index Data
  * See the file LICENSE for details.
  * Sebastian Hammer, Adam Dickmeiss
  *
@@ -7,7 +7,35 @@
  *   Chas Woodfield, Fretwell Downing Datasystem.
  *
  * $Log: statserv.c,v $
- * Revision 1.39  1997-09-09 10:10:19  adam
+ * Revision 1.46  1998-01-30 15:24:57  adam
+ * Fixed bug in inetd code. The server listened on tcp:@:9999 even
+ * though it was started in inetd mode.
+ *
+ * Revision 1.45  1998/01/29 13:30:23  adam
+ * Better event handle system for NT/Unix.
+ *
+ * Revision 1.44  1997/11/07 13:31:52  adam
+ * Added NT Service name part of statserv_options_block. Moved NT
+ * service utility to server library.
+ *
+ * Revision 1.43  1997/10/31 12:20:09  adam
+ * Improved memory debugging for xmalloc/nmem.c. References to NMEM
+ * instead of ODR in n ESPEC-1 handling in source d1_espec.c.
+ * Bug fix: missing fclose in data1_read_espec1.
+ *
+ * Revision 1.42  1997/10/27 14:03:02  adam
+ * Added new member to statserver_options_block, pre_init, which
+ * specifies a callback to be invoked after command line parsing and
+ * before the server listens for the first time.
+ *
+ * Revision 1.41  1997/09/29 07:19:32  adam
+ * Server library uses nmem_init/nmem_exit. The log prefix no longer
+ * includes leading path on NT.
+ *
+ * Revision 1.40  1997/09/17 12:10:41  adam
+ * YAZ version 1.4.
+ *
+ * Revision 1.39  1997/09/09 10:10:19  adam
  * Another MSV5.0 port. Changed projects to include proper
  * library/include paths.
  * Server starts server in test-mode when no options are given.
 
 #include <yconfig.h>
 #include <stdio.h>
+#include <string.h>
 #ifdef WINDOWS
 #include <process.h>
 #include <winsock.h>
 #include <direct.h>
+#include "service.h"
 #else
 #include <unistd.h>
 #include <pwd.h>
 #include <errno.h>
 
 #include <options.h>
-#include "eventl.h"
-#include "session.h"
 #include <comstack.h>
 #include <tcpip.h>
 #ifdef USE_XTIMOSI
 #include <xmosi.h>
 #endif
 #include <log.h>
+#include "eventl.h"
+#include "session.h"
 #include <statserv.h>
 
-static IOCHAN pListener;
+static IOCHAN pListener = NULL;
 
 static char *me = "statserver";
 /*
@@ -173,7 +203,9 @@ static statserv_options_block control_block = {
     60,                         /* idle timeout (minutes) */
     1024*1024,                  /* maximum PDU size (approx.) to allow */
     "default-config",           /* configuration name to pass to backend */
-    ""                          /* set user id */
+    "",                         /* set user id */
+    NULL,                       /* pre init handler */
+    "Z39.50 Server"             /* NT Service Name */
 };
 
 /*
@@ -186,12 +218,12 @@ static statserv_options_block control_block = {
 
 typedef struct _ThreadList ThreadList;
 
-typedef struct _ThreadList
+struct _ThreadList
 {
     HANDLE hThread;
     IOCHAN pIOChannel;
     ThreadList *pNext;
-} ThreadList;
+};
 
 static ThreadList *pFirstThread;
 static CRITICAL_SECTION Thread_CritSect;
@@ -334,6 +366,11 @@ void statserv_closedown()
     }
 }
 
+int __stdcall event_loop_thread (IOCHAN iochan)
+{
+    return event_loop (&iochan);
+}
+
 static void listener(IOCHAN h, int event)
 {
     COMSTACK line = (COMSTACK) iochan_getdata(h);
@@ -367,7 +404,7 @@ static void listener(IOCHAN h, int event)
            iochan_setflags(h, EVENT_INPUT | EVENT_EXCEPT); /* reset listener */
            return;
        }
-       logf(LOG_DEBUG, "accept ok");
+       logf(LOG_DEBUG, "Accept ok");
 
        if (!(new_chan = iochan_create(cs_fileno(new_line), ir_session, EVENT_INPUT)))
        {
@@ -376,25 +413,29 @@ static void listener(IOCHAN h, int event)
             return;
        }
 
-       logf(LOG_DEBUG, "accept ok 2");
+       logf(LOG_DEBUG, "Creating association");
        if (!(newas = create_association(new_chan, new_line)))
        {
            logf(LOG_FATAL, "Failed to create new assoc.");
             iochan_destroy(h);
             return;
        }
-       logf(LOG_DEBUG, "accept ok 3");
+       logf(LOG_DEBUG, "Setting timeout %d", control_block.idle_timeout);
        iochan_setdata(new_chan, newas);
        iochan_settimeout(new_chan, control_block.idle_timeout * 60);
+#ifndef WINDOWS
+       logf(LOG_DEBUG, "Determining client address");
        a = cs_addrstr(new_line);
        logf(LOG_LOG, "Accepted connection from %s", a ? a : "[Unknown]");
+#endif
     /* Now what we need todo is create a new thread with this iochan as
        the parameter */
-    /* if (CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)event_loop, new_chan,
-           0, &ThreadId) == NULL) */
+    /* if (CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)event_loop_thread,
+                       new_chan, 0, &ThreadId) == NULL) */
     /* Somehow, somewhere we need to store this thread id, otherwise we won't be
        able to close cleanly */
-        NewHandle = (HANDLE)_beginthreadex(NULL, 0, event_loop, new_chan, 0, &ThreadId);
+        NewHandle = (HANDLE)_beginthreadex(NULL, 0, event_loop_thread,
+                                           new_chan, 0, &ThreadId);
         if (NewHandle == (HANDLE)-1)
        {
 
@@ -425,9 +466,9 @@ void statserv_remove(IOCHAN pIOChannel)
 
 void statserv_closedown()
 {
-    /* We don't need todoanything here - or do we */
-    if (pListener != NULL)
-        iochan_destroy(pListener);
+    IOCHAN p;
+    for (p = pListener; p; p = p->next)
+        iochan_destroy(p);
 }
 
 static void listener(IOCHAN h, int event)
@@ -463,7 +504,7 @@ static void listener(IOCHAN h, int event)
 
                close(hand[0]);
                child = 1;
-               for (pp = iochan_getchan(); pp; pp = iochan_getnext(pp))
+               for (pp = pListener; pp; pp = iochan_getnext(pp))
                {
                    if (pp != h)
                    {
@@ -526,7 +567,7 @@ static void listener(IOCHAN h, int event)
        {
            IOCHAN pp;
            /* close our half of the listener socket */
-           for (pp = iochan_getchan(); pp; pp = iochan_getnext(pp))
+           for (pp = pListener; pp; pp = iochan_getnext(pp))
            {
                COMSTACK l = iochan_getdata(pp);
                cs_close(l);
@@ -546,6 +587,8 @@ static void listener(IOCHAN h, int event)
             iochan_destroy(h);
             return;
        }
+        new_chan->next = pListener;
+        pListener = new_chan;
        if (!(newas = create_association(new_chan, new_line)))
        {
            logf(LOG_FATAL, "Failed to create new assoc.");
@@ -589,6 +632,8 @@ static void inetd_connection(int what)
             {
                logf(LOG_FATAL, "Failed to create association structure");
             }
+            chan->next = pListener;
+            pListener = chan;
         }
         else
         {
@@ -614,32 +659,24 @@ static void add_listener(char *where, int what)
 
     if (!where || sscanf(where, "%[^:]:%s", mode, addr) != 2)
     {
-       fprintf(stderr, "%s: Address format: ('tcp'|'osi')':'<address>.\n",
-           me);
+       logf (LOG_WARN, "%s: Address format: ('tcp'|'osi')':'<address>", me);
+       return;
     }
     if (!strcmp(mode, "tcp"))
-    {
-       if (!(ap = tcpip_strtoaddr(addr)))
-       {
-           fprintf(stderr, "Address resolution failed for TCP.\n");
-       }
        type = tcpip_type;
-    }
     else if (!strcmp(mode, "osi"))
     {
 #ifdef USE_XTIMOSI
-       if (!(ap = mosi_strtoaddr(addr)))
-       {
-           fprintf(stderr, "Address resolution failed for TCP.\n");
-       }
        type = mosi_type;
 #else
-       fprintf(stderr, "OSI Transport not allowed by configuration.\n");
+       logf (LOG_WARN, "OSI Transport not allowed by configuration.");
+       return;
 #endif
     }
     else
     {
-       fprintf(stderr, "You must specify either 'osi:' or 'tcp:'.\n");
+       logf (LOG_WARN, "You must specify either 'osi:' or 'tcp:'");
+       return;
     }
     logf(LOG_LOG, "Adding %s %s listener on %s",
         control_block.dynamic ? "dynamic" : "static",
@@ -648,6 +685,13 @@ static void add_listener(char *where, int what)
     {
        logf(LOG_FATAL|LOG_ERRNO, "Failed to create listener");
     }
+    ap = cs_straddr (l, addr);
+    if (!ap)
+    {
+       fprintf(stderr, "Address resolution failed.\n");
+       cs_close (l);
+       return;
+    }
     if (cs_bind(l, ap, CS_SERVER) < 0)
     {
        logf(LOG_FATAL|LOG_ERRNO, "Failed to bind to %s", where);
@@ -687,18 +731,26 @@ void statserv_setcontrol(statserv_options_block *block)
     memcpy(&control_block, block, sizeof(*block));
 }
 
-int statserv_main(int argc, char **argv)
+int statserv_start(int argc, char **argv)
 {
     int ret, listeners = 0, inetd = 0, r;
     char *arg;
     int protocol = control_block.default_proto;
 
+    nmem_init ();
 #ifdef WINDOWS
     /* We need to initialize the thread list */
     ThreadList_Initialize();
 #endif /* WINDOWS */
 
+#ifdef WINDOWS
+    if ((me = strrchr (argv[0], '\\')))
+       me++;
+    else
+       me = argv[0];
+#else
     me = argv[0];
+#endif
     while ((ret = options("a:iszSl:v:u:c:w:t:k:", argv, argc, &arg)) != -2)
     {
        switch (ret)
@@ -760,19 +812,13 @@ int statserv_main(int argc, char **argv)
                return(1);
             }
     }
-
-#if 0
-    log_init(control_block.loglevel, NULL, control_block.logfile);
-#endif /* WINDOWS */
-
-    if ((pListener == NULL) && *control_block.default_listen)
-           add_listener(control_block.default_listen, protocol);
-
 #ifndef WINDOWS
     if (inetd)
        inetd_connection(protocol);
     else
     {
+       if (control_block.pre_init)
+           (*control_block.pre_init)(&control_block);
        if (control_block.dynamic)
            signal(SIGCHLD, catchchld);
     }
@@ -793,10 +839,74 @@ int statserv_main(int argc, char **argv)
     }
 #endif /* WINDOWS */
 
+    if ((pListener == NULL) && *control_block.default_listen)
+       add_listener(control_block.default_listen, protocol);
+
     logf(LOG_LOG, "Entering event loop.");
        
     if (pListener == NULL)
-        return(1);
+       ret = 1;
     else
-        return event_loop(pListener);
+        ret = event_loop(&pListener);
+    nmem_exit ();
+    return ret;
+}
+
+#ifdef WINDOWS
+typedef struct _Args
+{
+    char **argv;
+    int argc;
+} Args; 
+
+static Args ArgDetails;
+
+/* name of the executable */
+#define SZAPPNAME            "server"
+
+/* list of service dependencies - "dep1\0dep2\0\0" */
+#define SZDEPENDENCIES       ""
+
+int statserv_main(int argc, char **argv)
+{
+    statserv_options_block *cb = statserv_getcontrol();
+
+        /* Lets setup the Arg structure */
+    ArgDetails.argc = argc;
+    ArgDetails.argv = argv;
+
+    /* Now setup the service with the service controller */
+    SetupService(argc, argv, &ArgDetails, SZAPPNAME,
+        cb->service_name, /* internal service name */
+        cb->service_name, /* displayed name of the service */
+        SZDEPENDENCIES);
+    return 0;
 }
+
+int StartAppService(void *pHandle, int argc, char **argv)
+{
+    /* Initializes the App */
+    return 1;
+}
+
+void RunAppService(void *pHandle)
+{
+    Args *pArgs = (Args *)pHandle;
+
+    /* Starts the app running */
+    statserv_start(pArgs->argc, pArgs->argv);
+}
+
+void StopAppService(void *pHandle)
+{
+    /* Stops the app */
+    statserv_closedown();
+}
+#else
+int statserv_main(int argc, char **argv)
+{
+    int ret = statserv_start (argc, argv);
+    statserv_closedown ();
+    return ret;
+}
+#endif