Author: ZenShadow
[ircu2.10.12-pk.git] / ircd / ircd.c
index 0f5270dba08d033dc58148ffe30250dd84dab927..ededf241ae1b8d78a60b5a5dc1411b48cf93e05b 100644 (file)
  *
  * $Id$
  */
-#include "ircd.h"
+
+
+/*----------------------------------------------------------------------------
+ * Platform Includes
+ *--------------------------------------------------------------------------*/
+#include <assert.h>
+#include <errno.h>
+#include <fcntl.h>
+#include <pwd.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <sys/socket.h>
+#include <sys/stat.h>
+#include <unistd.h>
+#include <netdb.h>
+
+
+/*----------------------------------------------------------------------------
+ * ircu Includes
+ *--------------------------------------------------------------------------*/
+#include "config.h"
+
+#include "IPcheck.h"
 #include "class.h"
 #include "client.h"
 #include "crule.h"
 #include "hash.h"
-#include "ircd_alloc.h" /* set_nomem_handler */
+#include "ircd.h"
+#include "ircd_alloc.h"
 #include "ircd_log.h"
+#include "ircd_reply.h"
 #include "ircd_signal.h"
 #include "ircd_string.h"
 #include "jupe.h"
 #include "list.h"
-#include "listener.h"
 #include "match.h"
+#include "msg.h"
 #include "numeric.h"
 #include "numnicks.h"
 #include "parse.h"
 #include "s_debug.h"
 #include "s_misc.h"
 #include "send.h"
-#include "struct.h"
 #include "sys.h"
 #include "uping.h"
 #include "userload.h"
 #include "version.h"
 #include "whowas.h"
-#include "msg.h"
 
-#include <assert.h>
-#include <errno.h>
-#include <fcntl.h>
-#include <pwd.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-#include <sys/socket.h>
-#include <sys/stat.h>
-#include <unistd.h>
-#include <netdb.h>
 
+/*----------------------------------------------------------------------------
+ * External stuff
+ *--------------------------------------------------------------------------*/
 extern void init_counters(void);
 
-struct Client  me;                      /* That's me */
-struct Client* GlobalClientList = &me;  /* Pointer to beginning of Client list */
-time_t         TSoffset = 0;      /* Offset of timestamps to system clock */
-int            GlobalRehashFlag = 0;    /* do a rehash if set */
-int            GlobalRestartFlag = 0;   /* do a restart if set */
-time_t         CurrentTime;       /* Updated every time we leave select() */
-
-char **myargv;
-char *configfile = CPATH;       /* Server configuration file */
-int debuglevel = -1;            /* Server debug level */
-unsigned int bootopt = 0;       /* Server boot option flags */
-char *debugmode = "";           /*  -"-    -"-   -"-  */
-static char *dpath = DPATH;
-
-time_t nextconnect = 1;         /* time for next try_connections call */
-time_t nextping = 1;            /* same as above for check_pings() */
-time_t nextdnscheck = 0;        /* next time to poll dns to force timeouts */
-time_t nextexpire = 1;          /* next expire run on the dns cache */
-
-#ifdef PROFIL
-extern etext(void);
-#endif
+/*----------------------------------------------------------------------------
+ * Constants / Enums
+ *--------------------------------------------------------------------------*/
+enum {
+  BOOT_DEBUG = 1,
+  BOOT_TTY   = 2
+};
 
-static void server_reboot(const char* message)
-{
-  int i;
 
-  sendto_ops("Restarting server: %s", message);
-  Debug((DEBUG_NOTICE, "Restarting server..."));
-  flush_connections(0);
-  /*
-   * fd 0 must be 'preserved' if either the -d or -i options have
-   * been passed to us before restarting.
-   */
-  close_log();
+/*----------------------------------------------------------------------------
+ * Global data (YUCK!)
+ *--------------------------------------------------------------------------*/
+struct Client  me;                      // That's me
+struct Client *GlobalClientList  = &me; // Pointer to beginning of Client list
+time_t         TSoffset          = 0;   // Offset of timestamps to system clock
+int            GlobalRehashFlag  = 0;   // do a rehash if set
+int            GlobalRestartFlag = 0;   // do a restart if set
+time_t         CurrentTime;             // Updated every time we leave select()
 
-  for (i = 3; i < MAXCONNECTIONS; i++)
-    close(i);
-  if (!(bootopt & (BOOT_TTY | BOOT_DEBUG)))
-    close(2);
-  close(1);
-  close(0);
+char          *configfile        = CPATH; // Server configuration file
+int            debuglevel        = -1;    // Server debug level 
+char          *debugmode         = "";    // Server debug level
+static char   *dpath             = DPATH;
 
-  execv(SPATH, myargv);
+time_t         nextconnect       = 1; // time for next try_connections call
+time_t         nextping          = 1; // same as above for check_pings()
+
+static struct Daemon thisServer  = { 0 };     // server process info 
 
-  /* Have to reopen since it has been closed above */
-  open_log(myargv[0]);
-  ircd_log(L_CRIT, "execv(%s,%s) failed: %m\n", SPATH, myargv[0]);
 
-  Debug((DEBUG_FATAL, "Couldn't restart server \"%s\": %s",
-         SPATH, (strerror(errno)) ? strerror(errno) : ""));
-  exit(2);
-}
 
-void server_die(const char* message)
-{
+/*----------------------------------------------------------------------------
+ * API: server_die
+ *--------------------------------------------------------------------------*/
+void server_die(const char* message) {
   ircd_log(L_CRIT, "Server terminating: %s", message);
-  sendto_ops("Server terminating: %s", message);
+  sendto_opmask_butone(0, SNO_OLDSNO, "Server terminating: %s", message);
   flush_connections(0);
-  exit(2);
+  close_connections(1);
+  thisServer.running = 0;
 }
 
-void server_restart(const char* message)
-{
+
+/*----------------------------------------------------------------------------
+ * API: server_restart
+ *--------------------------------------------------------------------------*/
+void server_restart(const char* message) {
   static int restarting = 0;
 
   ircd_log(L_WARNING, "Restarting Server: %s", message);
-  if (restarting == 0) {
-    restarting = 1;
-    server_reboot(message);
-  }
+  if (restarting)
+    return;
+
+  sendto_opmask_butone(0, SNO_OLDSNO, "Restarting server: %s", message);
+  Debug((DEBUG_NOTICE, "Restarting server..."));
+  flush_connections(0);
+
+  close_log();
+  close_connections(!(thisServer.bootopt & (BOOT_TTY | BOOT_DEBUG)));
+
+  execv(SPATH, thisServer.argv);
+
+  /* Have to reopen since it has been closed above */
+  open_log(*thisServer.argv);
+  ircd_log(L_CRIT, "execv(%s,%s) failed: %m\n", SPATH, *thisServer.argv);
+
+  Debug((DEBUG_FATAL, "Couldn't restart server \"%s\": %s",
+         SPATH, (strerror(errno)) ? strerror(errno) : ""));
+  exit(8);
 }
 
-static void outofmemory(void)
-{
+
+/*----------------------------------------------------------------------------
+ * outofmemory:  Handler for out of memory conditions...
+ *--------------------------------------------------------------------------*/
+static void outofmemory(void) {
   Debug((DEBUG_FATAL, "Out of memory: restarting server..."));
   server_restart("Out of Memory");
 } 
 
-static void write_pidfile(void)
-{
-#ifdef PPATH
-  int fd;
-  char buff[20];
-  if ((fd = open(PPATH, O_CREAT | O_WRONLY, 0600)) == -1) {
-    Debug((DEBUG_NOTICE, "Error opening pid file \"%s\": %s",
-           PPATH, strerror(errno)));
+
+/*----------------------------------------------------------------------------
+ * write_pidfile
+ *--------------------------------------------------------------------------*/
+static void write_pidfile(void) {
+  FILE *pidf;
+
+  if (!(pidf = fopen(PPATH, "w+"))) {
+    Debug((DEBUG_NOTICE, 
+          "Error opening pid file \"%s\": %s", PPATH, strerror(errno)));
     return;
   }
-  memset(buff, 0, sizeof(buff));
-  sprintf(buff, "%5d\n", getpid());
-  if (write(fd, buff, strlen(buff)) == -1)
+    
+  if (fprintf(pidf, "%5d\n", getpid()) < 5)
     Debug((DEBUG_NOTICE, "Error writing to pid file %s", PPATH));
-  close(fd);
-#endif
+
+  fclose(pidf);
 }
 
-/*
+
+/*----------------------------------------------------------------------------
  * try_connections
  *
  * Scan through configuration and try new connections.
@@ -169,25 +187,23 @@ static void write_pidfile(void)
  * Returns the calendar time when the next call to this
  * function should be made latest. (No harm done if this
  * is called earlier or later...)
- */
-static time_t try_connections(void)
-{
-  struct ConfItem*  aconf;
-  struct Client*    cptr;
-  struct ConfItem** pconf;
-  int               connecting;
-  int               confrq;
-  time_t            next = 0;
-  struct ConfClass* cltmp;
-  struct ConfItem*  cconf;
-  strcut ConfItem*  con_conf = NULL;
-  struct Jupe*      ajupe;
-  unsigned int      con_class = 0;
+ *--------------------------------------------------------------------------*/
+static time_t try_connections(void) {
+  struct ConfItem   *aconf;
+  struct Client     *cptr;
+  struct ConfItem  **pconf;
+  int                connecting;
+  int                confrq;
+  time_t             next        = 0;
+  struct ConfClass  *cltmp;
+  struct ConfItem   *cconf;
+  struct ConfItem   *con_conf    = NULL;
+  struct Jupe       *ajupe;
+  unsigned int       con_class   = 0;
 
   connecting = FALSE;
   Debug((DEBUG_NOTICE, "Connection check at   : %s", myctime(CurrentTime)));
-  for (aconf = GlobalConfList; aconf; aconf = aconf->next)
-  {
+  for (aconf = GlobalConfList; aconf; aconf = aconf->next) {
     /* Also when already connecting! (update holdtimes) --SRB */
     if (!(aconf->status & CONF_SERVER) || aconf->port == 0)
       continue;
@@ -196,41 +212,34 @@ static time_t try_connections(void)
     if ((ajupe = jupe_find(aconf->name)) && JupeIsActive(ajupe))
       continue;
 
-    cltmp = aconf->confClass;
-    /*
-     * Skip this entry if the use of it is still on hold until
-     * future. Otherwise handle this entry (and set it on hold
-     * until next time). Will reset only hold times, if already
-     * made one successfull connection... [this algorithm is
-     * a bit fuzzy... -- msa >;) ]
+    /* Skip this entry if the use of it is still on hold until
+     * future. Otherwise handle this entry (and set it on hold until next
+     * time). Will reset only hold times, if already made one successfull
+     * connection... [this algorithm is a bit fuzzy... -- msa >;) ]
      */
-
-    if ((aconf->hold > CurrentTime))
-    {
-      if ((next > aconf->hold) || (next == 0))
-        next = aconf->hold;
+    if (aconf->hold > CurrentTime && (next > aconf->hold || next == 0)) {
+      next = aconf->hold;
       continue;
     }
 
+    cltmp = aconf->confClass;
     confrq = get_con_freq(cltmp);
     aconf->hold = CurrentTime + confrq;
-    /*
-     * Found a CONNECT config with port specified, scan clients
-     * and see if this server is already connected?
+
+    /* Found a CONNECT config with port specified, scan clients and see if
+     * this server is already connected?
      */
     cptr = FindServer(aconf->name);
 
     if (!cptr && (Links(cltmp) < MaxLinks(cltmp)) &&
-        (!connecting || (ConClass(cltmp) > con_class)))
-    {
+        (!connecting || (ConClass(cltmp) > con_class))) {
       /* Check connect rules to see if we're allowed to try */
-      for (cconf = GlobalConfList; cconf; cconf = cconf->next)
-        if ((cconf->status & CONF_CRULE) &&
-            (match(cconf->host, aconf->name) == 0))
+      for (cconf = GlobalConfList; cconf; cconf = cconf->next) {
+        if ((cconf->status & CONF_CRULE) && !match(cconf->host, aconf->name))
           if (crule_eval(cconf->passwd))
             break;
-      if (!cconf)
-      {
+      }
+      if (!cconf) {
         con_class = ConClass(cltmp);
         con_conf = aconf;
         /* We connect only one at time... */
@@ -240,501 +249,464 @@ static time_t try_connections(void)
     if ((next > aconf->hold) || (next == 0))
       next = aconf->hold;
   }
-  if (connecting)
-  {
-    if (con_conf->next)         /* are we already last? */
-    {
+  if (connecting) {
+    if (con_conf->next) { /* are we already last? */
       /* Put the current one at the end and make sure we try all connections */
       for (pconf = &GlobalConfList; (aconf = *pconf); pconf = &(aconf->next))
         if (aconf == con_conf)
           *pconf = aconf->next;
       (*pconf = con_conf)->next = 0;
     }
+
     if (connect_server(con_conf, 0, 0))
-      sendto_ops("Connection to %s activated.", con_conf->name);
+      sendto_opmask_butone(0, SNO_OLDSNO, "Connection to %s activated.",
+                          con_conf->name);
   }
+
   Debug((DEBUG_NOTICE, "Next connection check : %s", myctime(next)));
-  return (next);
+  return(next);
 }
 
-static time_t check_pings(void)
-{
-  struct Client *cptr;
-  int ping = 0;
-  int i;
-  time_t oldest = CurrentTime + PINGFREQUENCY;
-  time_t timeout;
 
-  for (i = 0; i <= HighestFd; i++) {
-    if (!(cptr = LocalClientArray[i]))
+/*----------------------------------------------------------------------------
+ * check_pings
+ *
+ * TODO: This should be moved out of ircd.c.  It's protocol-specific when you
+ *       get right down to it.  Can't really be done until the server is more
+ *       modular, however...
+ *--------------------------------------------------------------------------*/
+static time_t check_pings(void) {
+  int expire     = 0; 
+  int next_check = CurrentTime + PINGFREQUENCY;
+  int max_ping   = 0;
+  int i;
+  
+  /* Scan through the client table */
+  for (i=0; i <= HighestFd; i++) {
+    struct Client *cptr = LocalClientArray[i];
+   
+    if (!cptr)
       continue;
-    /*
-     * me is never in the local client array
-     */
-    assert(cptr != &me);
-    /*
-     * Note: No need to notify opers here.
-     * It's already done when "FLAGS_DEADSOCKET" is set.
-     */
+     
+    assert(&me != cptr);  /* I should never be in the local client array! */
+   
+
+    /* Remove dead clients. */
     if (IsDead(cptr)) {
       exit_client(cptr, cptr, &me, cptr->info);
       continue;
     }
 
-    ping = IsRegistered(cptr) ? get_client_ping(cptr) : CONNECTTIMEOUT;
-    Debug((DEBUG_DEBUG, "c(%s)=%d p %d a %d",
-          cptr->name, cptr->status, ping, 
-          (int)(CurrentTime - cptr->lasttime)));
-    /*
-     * Ok, so goto's are ugly and can be avoided here but this code
-     * is already indented enough so I think its justified. -avalon
-     */
-    if (IsRegistered(cptr) && (ping >= CurrentTime - cptr->lasttime))
-      goto ping_timeout;
-    /*
-     * If the server hasnt talked to us in 2 * ping seconds
-     * and it has a ping time, then close its connection.
-     * If the client is a user and a KILL line was found
-     * to be active, close this connection too.
+    max_ping = IsRegistered(cptr) ? get_client_ping(cptr) : CONNECTTIMEOUT;
+   
+    Debug((DEBUG_DEBUG, "check_pings(%s)=status:%s limit: %d current: %d",
+          cptr->name, (cptr->flags & FLAGS_PINGSENT) ? "[Ping Sent]" : "[]", 
+          max_ping, (int)(CurrentTime - cptr->lasttime)));
+          
+
+    /* Ok, the thing that will happen most frequently, is that someone will
+     * have sent something recently.  Cover this first for speed.
      */
-    if (((CurrentTime - cptr->lasttime) >= (2 * ping) && (cptr->flags & FLAGS_PINGSENT)) ||
-        (!IsRegistered(cptr) && !IsHandshake(cptr) && (CurrentTime - cptr->firsttime) >= ping))
-    {
+    if (CurrentTime-cptr->lasttime < max_ping) {
+      expire = cptr->lasttime + max_ping;
+      if (expire < next_check) 
+       next_check = expire;
+      continue;
+    }
+
+    /* Quit the client after max_ping*2 - they should have answered by now */
+    if (CurrentTime-cptr->lasttime >= (max_ping*2) ) {
+      /* If it was a server, then tell ops about it. */
       if (IsServer(cptr) || IsConnecting(cptr) || IsHandshake(cptr))
-      {
-        sendto_ops("No response from %s, closing link", cptr->name);
-        exit_client(cptr, cptr, &me, "Ping timeout");
-        continue;
-      }
-      else {
-        if (!IsRegistered(cptr) && *cptr->name && *cptr->user->username) {
-          sendto_one(cptr,
-              ":%s %d %s :Your client may not be compatible with this server.",
-              me.name, ERR_BADPING, cptr->name);
-          sendto_one(cptr,
-              ":%s %d %s :Compatible clients are available at "
-              "ftp://ftp.undernet.org/pub/irc/clients",
-              me.name, ERR_BADPING, cptr->name);
-        }
-        exit_client_msg(cptr, cptr, &me, "Ping timeout for %s",
-                        get_client_name(cptr, HIDE_IP));
-      }
+       sendto_opmask_butone(0, SNO_OLDSNO,
+                            "No response from %s, closing link", cptr->name);
+      exit_client_msg(cptr, cptr, &me, "Ping timeout");
       continue;
     }
-    else if (IsRegistered(cptr) && 0 == (cptr->flags & FLAGS_PINGSENT)) {
-      /*
-       * If we havent PINGed the connection and we havent
-       * heard from it in a while, PING it to make sure
-       * it is still alive.
+    
+    /* Unregistered clients pingout after max_ping seconds, they don't
+     * get given a second chance - if they were then people could not quite
+     * finish registration and hold resources without being subject to k/g
+     * lines
+     */
+    if (!IsRegistered(cptr)) {
+      /* Display message if they have sent a NICK and a USER but no
+       * nospoof PONG.
        */
-      cptr->flags |= FLAGS_PINGSENT;
-      /*
-       * not nice but does the job
+      if (*cptr->name && cptr->user && *cptr->user->username) {
+       send_reply(cptr, SND_EXPLICIT | ERR_BADPING,
+                  ":Your client may not be compatible with this server.");
+       send_reply(cptr, SND_EXPLICIT | ERR_BADPING,
+                  ":Compatible clients are available at "
+                  "ftp://ftp.undernet.org/pub/irc/clients");
+      }    
+      exit_client_msg(cptr,cptr,&me, "Ping Timeout");
+      continue;
+    }
+    
+    if (!(cptr->flags & FLAGS_PINGSENT)) {
+      /* If we havent PINGed the connection and we havent heard from it in a
+       * while, PING it to make sure it is still alive.
        */
-      cptr->lasttime = CurrentTime - ping;
+      cptr->flags |= FLAGS_PINGSENT;
+
+      /* If we're late in noticing don't hold it against them :) */
+      cptr->lasttime = CurrentTime - max_ping;
+      
       if (IsUser(cptr))
-        sendto_one(cptr, "PING :%s", me.name);
+       sendrawto_one(cptr, MSG_PING " :%s", me.name);
       else
-        sendto_one(cptr, "%s " TOK_PING " :%s", NumServ(&me), me.name);
+       sendcmdto_one(&me, CMD_PING, cptr, ":%s", me.name);
     }
-ping_timeout:
-    timeout = cptr->lasttime + ping;
-    while (timeout <= CurrentTime)
-      timeout += ping;
-    if (timeout < oldest)
-      oldest = timeout;
+    
+    expire = cptr->lasttime + max_ping * 2;
+    if (expire < next_check)
+      next_check=expire;
   }
-  if (oldest < CurrentTime)
-    oldest = CurrentTime + PINGFREQUENCY;
-  Debug((DEBUG_NOTICE,
-        "Next check_ping() call at: %s, %d " TIME_T_FMT " " TIME_T_FMT,
-        myctime(oldest), ping, oldest, CurrentTime));
+  
+  assert(next_check >= CurrentTime);
+  
+  Debug((DEBUG_DEBUG, "[%i] check_pings() again in %is",
+        CurrentTime, next_check-CurrentTime));
+  
+  return next_check;
+}
+
+
+/*----------------------------------------------------------------------------
+ * parse_command_line
+ *--------------------------------------------------------------------------*/
+static void parse_command_line(int argc, char** argv) {
+  const char *options = "d:f:h:ntvx:";
+  int opt;
+
+  if (thisServer.euid != thisServer.uid)
+    setuid(thisServer.uid);
 
-  return (oldest);
+  /* Do we really need to santiy check the non-NULLness of optarg?  That's
+   * getopt()'s job...  Removing those... -zs
+   */
+  while ((opt = getopt(argc, argv, options)) != EOF)
+    switch (opt) {
+    case 'n':
+    case 't':  thisServer.bootopt |= BOOT_TTY;         break;
+    case 'd':  dpath      = optarg;                    break;
+    case 'f':  configfile = optarg;                    break;
+    case 'h':  ircd_strncpy(me.name, optarg, HOSTLEN); break;
+    case 'v':  printf("ircd %s\n", version);           exit(0);
+      
+    case 'x':
+      debuglevel = atoi(optarg);
+      if (debuglevel < 0)
+       debuglevel = 0;
+      debugmode = optarg;
+      thisServer.bootopt |= BOOT_DEBUG;
+      break;
+      
+    default:
+      printf("Usage: ircd [-f config] [-h servername] [-x loglevel] [-ntv]\n");
+      printf("\n -n -t\t Don't detach\n -v\t display version\n\n");
+      printf("Server not started.\n");
+      exit(1);
+    }
 }
 
-/*
- * bad_command
- *
- * This is called when the commandline is not acceptable.
- * Give error message and exit without starting anything.
- */
-static int bad_command(void)
-{
-  printf("Usage: ircd %s[-h servername] [-x loglevel] [-t]\n",
-#ifdef CMDLINE_CONFIG
-      "[-f config] "
-#else
-      ""
+
+/*----------------------------------------------------------------------------
+ * daemon_init
+ *--------------------------------------------------------------------------*/
+static void daemon_init(int no_fork) {
+  if (!init_connection_limits())
+    exit(9);
+
+  close_connections(!(thisServer.bootopt & (BOOT_DEBUG | BOOT_TTY)));
+
+  if (no_fork)
+    return;
+
+  if (fork())
+    exit(0);
+
+#ifdef TIOCNOTTY
+  {
+    int fd;
+    if ((fd = open("/dev/tty", O_RDWR)) > -1) {
+      ioctl(fd, TIOCNOTTY, 0);
+      close(fd);
+    }
+  }
 #endif
-      );
-  printf("Server not started\n\n");
-  return (-1);
+
+  setsid();
 }
 
-int main(int argc, char *argv[])
-{
-  uid_t uid;
-  uid_t euid;
-  time_t delay = 0;
-#if defined(HAVE_SETRLIMIT) && defined(RLIMIT_CORE)
-  struct rlimit corelim;
-#endif
 
-  CurrentTime = time(NULL);
+/*----------------------------------------------------------------------------
+ * event_loop
+ *--------------------------------------------------------------------------*/
+static void event_loop(void) {
+  time_t nextdnscheck = 0;
+  time_t delay        = 0;
 
-  /*
-   * sanity check
-   */
-  if (MAXCONNECTIONS < 64 || MAXCONNECTIONS > 256000) {
-    fprintf(stderr, "%s: MAXCONNECTIONS insane: %d\n", *argv, MAXCONNECTIONS);
-    return 2;
-  }
+  thisServer.running = 1;
+  while (thisServer.running) {
+    /* We only want to connect if a connection is due, not every time through.
+     * Note, if there are no active C lines, this call to Tryconnections is
+     * made once only; it will return 0. - avalon
+     */
+    if (nextconnect && CurrentTime >= nextconnect)
+      nextconnect = try_connections();
+
+    /* DNS checks. One to timeout queries, one for cache expiries. */
+    nextdnscheck = timeout_resolver(CurrentTime);
+
+    /* Take the smaller of the two 'timed' event times as the time of next
+     * event (stops us being late :) - avalon
+     * WARNING - nextconnect can return 0!
+     */
+    if (nextconnect)
+      delay = IRCD_MIN(nextping, nextconnect);
+    else
+      delay = nextping;
+
+    delay = IRCD_MIN(nextdnscheck, delay) - CurrentTime;
+
+    /* Adjust delay to something reasonable [ad hoc values] (one might think
+     * something more clever here... --msa) We don't really need to check that
+     * often and as long as we don't delay too long, everything should be ok.
+     * waiting too long can cause things to timeout...  i.e. PINGS -> a
+     * disconnection :( - avalon
+     */
+    if (delay < 1)
+      read_message(1);
+    else
+      read_message(IRCD_MIN(delay, TIMESEC));
+
+    Debug((DEBUG_DEBUG, "Got message(s)"));
+
+    /* ...perhaps should not do these loops every time, but only if there is
+     * some chance of something happening (but, note that conf->hold times may
+     * be changed elsewhere--so precomputed next event time might be too far
+     * away... (similarly with ping times) --msa
+     */
+    if (CurrentTime >= nextping)
+      nextping = check_pings();
     
-  uid = getuid();
-  euid = geteuid();
-#ifdef PROFIL
-  monstartup(0, etext);
-  moncontrol(1);
-  signal(SIGUSR1, s_monitor);
-#endif
+    /* timeout pending queries that haven't been responded to */
+    timeout_auth_queries(CurrentTime);
 
-#ifdef CHROOTDIR
-  if (chdir(DPATH))
-  {
-    fprintf(stderr, "Fail: Cannot chdir(%s): %s\n", DPATH, (strerror(errno)) ? strerror(errno) : "");
-    exit(2);
-  }
-  if (chroot(DPATH))
-  {
-    fprintf(stderr, "Fail: Cannot chroot(%s): %s\n", DPATH, (strerror(errno)) ? strerror(errno) : "");
-    exit(5);
+    ip_registry_expire();
+
+    if (GlobalRehashFlag) {
+      rehash(&me, 1);
+      GlobalRehashFlag = 0;
+    }
+
+    if (GlobalRestartFlag)
+      server_restart("caught signal: SIGINT");
   }
-  dpath = "/";
-#endif /*CHROOTDIR */
+}
 
-  myargv = argv;
-  umask(077);                   /* better safe than sorry --SRB */
-  memset(&me, 0, sizeof(me));
-  me.fd = -1;
 
-#if 0
-#ifdef VIRTUAL_HOST
-  memset(&vserv, 0, sizeof(vserv));
-#endif
+/*----------------------------------------------------------------------------
+ * check_file_access:  random helper function to make sure that a file is
+ *                     accessible in a certain way, and complain if not.
+ *--------------------------------------------------------------------------*/
+static char check_file_access(const char *path, char which, int mode) {
+  if (!access(path, mode))
+    return 1;
+
+  fprintf(stderr, 
+         "Check on %cPATH (%s) failed: %s\n"
+         "Please create file and/or rerun `make config' and "
+         "recompile to correct this.\n",
+         which, path, strerror(errno));
+
+#ifdef CHROOTDIR
+  fprintf(stderr, "Keep in mind that paths are relative to CHROOTDIR.\n");
 #endif
 
-  setup_signals();
-  initload();
+  return 0;
+}
 
+
+/*----------------------------------------------------------------------------
+ * set_core_limit
+ *--------------------------------------------------------------------------*/
 #if defined(HAVE_SETRLIMIT) && defined(RLIMIT_CORE)
-  if (getrlimit(RLIMIT_CORE, &corelim))
-  {
-    fprintf(stderr, "Read of rlimit core size failed: %s\n", (strerror(errno) ? strerror(errno) : "");
+static void set_core_limit(void) {
+  struct rlimit corelim;
+
+  if (getrlimit(RLIMIT_CORE, &corelim)) {
+    fprintf(stderr, "Read of rlimit core size failed: %s\n", strerror(errno));
     corelim.rlim_max = RLIM_INFINITY;   /* Try to recover */
   }
+
   corelim.rlim_cur = corelim.rlim_max;
   if (setrlimit(RLIMIT_CORE, &corelim))
-    fprintf(stderr, "Setting rlimit core size failed: %s\n", (strerror(errno) ? strerror(errno) : "");
+    fprintf(stderr, "Setting rlimit core size failed: %s\n", strerror(errno));
+}
 #endif
 
-  /*
-   * All command line parameters have the syntax "-fstring"
-   * or "-f string" (e.g. the space is optional). String may
-   * be empty. Flag characters cannot be concatenated (like
-   * "-fxyz"), it would conflict with the form "-fstring".
-   */
-  while (--argc > 0 && (*++argv)[0] == '-')
-  {
-    char *p = argv[0] + 1;
-    int flag = *p++;
-
-    if (flag == '\0' || *p == '\0')
-    {
-      if (argc > 1 && argv[1][0] != '-')
-      {
-        p = *++argv;
-        argc -= 1;
-      }
-      else
-        p = "";
-    }
 
-    switch (flag)
-    {
-      case 'q':
-        bootopt |= BOOT_QUICK;
-        break;
-      case 'd':
-        if (euid != uid)
-          setuid((uid_t) uid);
-        dpath = p;
-        break;
-#ifdef CMDLINE_CONFIG
-      case 'f':
-        if (euid != uid)
-          setuid((uid_t) uid);
-        configfile = p;
-        break;
-#endif
-      case 'h':
-        ircd_strncpy(me.name, p, HOSTLEN);
-        break;
-      case 't':
-        if (euid != uid)
-          setuid((uid_t) uid);
-        bootopt |= BOOT_TTY;
-        break;
-      case 'v':
-        printf("ircd %s\n", version);
-        exit(0);
-#if 0
-#ifdef VIRTUAL_HOST
-      case 'w':
-      {
-        struct hostent *hep;
-        if (!(hep = gethostbyname(p)))
-        {
-          fprintf(stderr, "%s: Error creating virtual host \"%s\": %d",
-              argv[0], p, h_errno);
-          return 2;
-        }
-        if (hep->h_addrtype == AF_INET && hep->h_addr_list[0] &&
-            !hep->h_addr_list[1])
-        {
-          memcpy(&vserv.sin_addr, hep->h_addr_list[0], sizeof(struct in_addr));
-          vserv.sin_family = AF_INET;
-        }
-        else
-        {
-          fprintf(stderr, "%s: Error creating virtual host \"%s\": "
-              "Use -w <IP-number of interface>\n", argv[0], p);
-          return 2;
-        }
-        break;
-      }
-#endif
-#endif
-      case 'x':
-#ifdef  DEBUGMODE
-        if (euid != uid)
-          setuid((uid_t) uid);
-        debuglevel = atoi(p);
-        debugmode = *p ? p : "0";
-        bootopt |= BOOT_DEBUG;
-        break;
-#else
-        fprintf(stderr, "%s: DEBUGMODE must be defined for -x y\n", myargv[0]);
-        exit(0);
-#endif
-      default:
-        bad_command();
-        break;
-    }
-  }
 
-  if (chdir(dpath))
-  {
-    fprintf(stderr, "Fail: Cannot chdir(%s): %s\n", dpath, (strerror(errno)) ? strerror(errno) : "");
-    exit(2);
+/*----------------------------------------------------------------------------
+ * set_chroot_environment
+ *--------------------------------------------------------------------------*/
+#ifdef CHROOTDIR
+static char set_chroot_environment(void) {
+  /* Must be root to chroot! Silly if you ask me... */
+  if (geteuid())
+    seteuid(0);
+
+  if (chdir(DPATH)) {
+    fprintf(stderr, "Fail: Cannot chdir(%s): %s\n", DPATH, strerror(errno));
+    return 0;
   }
-
-#ifndef IRC_UID
-  if ((uid != euid) && !euid)
-  {
-    fprintf(stderr,
-        "ERROR: do not run ircd setuid root. Make it setuid a normal user.\n");
-    exit(2);
+  if (chroot(DPATH)) {
+    fprintf(stderr, "Fail: Cannot chroot(%s): %s\n", DPATH, strerror(errno));
+    return 0;
   }
+  dpath = "/";
+  return 1;
+}
 #endif
 
-#if !defined(CHROOTDIR) || (defined(IRC_UID) && defined(IRC_GID))
-  if (euid != uid)
-  {
-    setuid(uid);
-    setuid(euid);
-  }
 
-  if (0 == getuid())
-  {
-#if defined(IRC_UID) && defined(IRC_GID)
-
-    /* run as a specified user */
-    fprintf(stderr, "WARNING: running ircd with uid = %d\n", IRC_UID);
-    fprintf(stderr, "         changing to gid %d.\n", IRC_GID);
-    setuid(IRC_UID);
-    setgid(IRC_GID);
-#else
-    /* check for setuid root as usual */
-    fprintf(stderr,
-        "ERROR: do not run ircd setuid root. Make it setuid a normal user.\n");
-    exit(2);
+/*----------------------------------------------------------------------------
+ * set_userid_if_needed()
+ *--------------------------------------------------------------------------*/
+static int set_userid_if_needed(void) {
+  /* TODO: Drop privs correctly! */
+#if defined(IRC_GID) && defined(IRC_UID)
+  setgid (IRC_GID);
+  setegid(IRC_GID);
+  setuid (IRC_UID);
+  seteuid(IRC_UID);
 #endif
+
+  if (getuid() == 0 || geteuid() == 0 ||
+      getgid() == 0 || getegid() == 0) {
+    fprintf(stderr, "ERROR:  This server will not run as superuser.\n");
+    return 0;
   }
-#endif /*CHROOTDIR/UID/GID */
 
-  if (argc > 0)
-    return bad_command();       /* This should exit out */
+  return 1;
+}
+
+
+/*----------------------------------------------------------------------------
+ * main - entrypoint
+ *
+ * TODO:  This should set the basic environment up and start the main loop.
+ *        we're doing waaaaaaaaay too much server initialization here.  I hate
+ *        long and ugly control paths...  -smd
+ *--------------------------------------------------------------------------*/
+int main(int argc, char **argv) {
+  CurrentTime = time(NULL);
+
+  thisServer.argc = argc;
+  thisServer.argv = argv;
+  thisServer.uid  = getuid();
+  thisServer.euid = geteuid();
 
-  /* Sanity checks */
-  {
-    char c;
-    char *path;
-
-    c = 'S';
-    path = SPATH;
-    if (access(path, X_OK) == 0) {
-      c = 'C';
-      path = CPATH;
-      if (access(path, R_OK) == 0) {
-        c = 'M';
-        path = MPATH;
-        if (access(path, R_OK) == 0) {
-          c = 'R';
-          path = RPATH;
-          if (access(path, R_OK) == 0) {
-#ifndef DEBUG
-            c = 0;
-#else
-            c = 'L';
-            path = LPATH;
-            if (access(path, W_OK) == 0)
-              c = 0;
-#endif
-          }
-        }
-      }
-    }
-    if (c)
-    {
-      fprintf(stderr, "Check on %cPATH (%s) failed: %s\n", 
-              c, path, (strerror(errno)) ? strerror(errno) : "");
-      fprintf(stderr,
-          "Please create file and/or rerun `make config' and recompile to correct this.\n");
 #ifdef CHROOTDIR
-      fprintf(stderr,
-          "Keep in mind that all paths are relative to CHROOTDIR.\n");
+  if (!set_chroot_environment())
+    exit(1);
 #endif
-      exit(2);
-    }
+
+#if defined(HAVE_SETRLIMIT) && defined(RLIMIT_CORE)
+  set_core_limit();
+#endif
+
+  umask(077);                   /* better safe than sorry --SRB */
+  memset(&me, 0, sizeof(me));
+  me.fd = -1;
+
+  setup_signals();
+  initload();
+
+  parse_command_line(argc, argv);
+
+  if (chdir(dpath)) {
+    fprintf(stderr, "Fail: Cannot chdir(%s): %s\n", dpath, strerror(errno));
+    exit(2);
   }
 
+  if (!set_userid_if_needed())
+    exit(3);
+
+  /* Check paths for accessibility */
+  if (!check_file_access(SPATH, 'S', X_OK) ||
+      !check_file_access(CPATH, 'C', R_OK) ||
+      !check_file_access(MPATH, 'M', R_OK) ||
+      !check_file_access(RPATH, 'R', R_OK))
+    exit(4);
+      
+#ifdef DEBUG
+  if (!check_file_access(LPATH, 'L', W_OK))
+    exit(5);
+#endif
+
   init_list();
   hash_init();
   initclass();
   initwhowas();
   initmsgtree();
   initstats();
-  open_debugfile();
-  init_sys();
-  set_nomem_handler(outofmemory);
 
-  me.fd = -1;
+  debug_init(thisServer.bootopt & BOOT_TTY);
+  daemon_init(thisServer.bootopt & BOOT_TTY);
+
+  set_nomem_handler(outofmemory);
+  init_resolver();
 
-  open_log(myargv[0]);
+  open_log(*argv);
 
-  if (initconf(bootopt) == -1) {
-    Debug((DEBUG_FATAL, "Failed in reading configuration file %s", configfile));
+  if (!conf_init()) {
+    Debug((DEBUG_FATAL, "Failed to read configuration file %s", configfile));
     printf("Couldn't open configuration file %s\n", configfile);
-    exit(2);
+    exit(6);
   }
+
   if (!init_server_identity()) {
     Debug((DEBUG_FATAL, "Failed to initialize server identity"));
-    exit(2);
+    exit(7);
   }
+
   uping_init();
   read_tlines();
-  rmotd = read_motd(RPATH);
-  motd = read_motd(MPATH);
+
+  rmotd       = read_motd(RPATH);
+  motd        = read_motd(MPATH);
   CurrentTime = time(NULL);
-  me.from = &me;
+  me.from     = &me;
+
   SetMe(&me);
   make_server(&me);
-  /*
-   * Abuse own link timestamp as start timestamp:
-   */
-  me.serv->timestamp = TStime();
-  me.serv->prot = atoi(MAJOR_PROTOCOL);
-  me.serv->up = &me;
-  me.serv->down = NULL;
-  me.handler = SERVER_HANDLER;
+
+  me.serv->timestamp = TStime();  /* Abuse own link timestamp as start TS */
+  me.serv->prot      = atoi(MAJOR_PROTOCOL);
+  me.serv->up        = &me;
+  me.serv->down      = NULL;
+  me.handler         = SERVER_HANDLER;
 
   SetYXXCapacity(&me, MAXCLIENTS);
 
   me.lasttime = me.since = me.firsttime = CurrentTime;
+
   hAddClient(&me);
 
   check_class();
   write_pidfile();
-
   init_counters();
 
   Debug((DEBUG_NOTICE, "Server ready..."));
   ircd_log(L_NOTICE, "Server Ready");
 
-  for (;;)
-  {
-    /*
-     * We only want to connect if a connection is due,
-     * not every time through.   Note, if there are no
-     * active C lines, this call to Tryconnections is
-     * made once only; it will return 0. - avalon
-     */
-    if (nextconnect && CurrentTime >= nextconnect)
-      nextconnect = try_connections();
-    /*
-     * DNS checks. One to timeout queries, one for cache expiries.
-     */
-    nextdnscheck = timeout_resolver(CurrentTime);
-    /*
-     * Take the smaller of the two 'timed' event times as
-     * the time of next event (stops us being late :) - avalon
-     * WARNING - nextconnect can return 0!
-     */
-    if (nextconnect)
-      delay = IRCD_MIN(nextping, nextconnect);
-    else
-      delay = nextping;
-    delay = IRCD_MIN(nextdnscheck, delay);
-    delay -= CurrentTime;
-    /*
-     * Adjust delay to something reasonable [ad hoc values]
-     * (one might think something more clever here... --msa)
-     * We don't really need to check that often and as long
-     * as we don't delay too long, everything should be ok.
-     * waiting too long can cause things to timeout...
-     * i.e. PINGS -> a disconnection :(
-     * - avalon
-     */
-    if (delay < 1)
-      delay = 1;
-    else
-      delay = IRCD_MIN(delay, TIMESEC);
-    read_message(delay);
+  event_loop();
 
-    Debug((DEBUG_DEBUG, "Got message(s)"));
+  return 0;
+}
 
-    /*
-     * ...perhaps should not do these loops every time,
-     * but only if there is some chance of something
-     * happening (but, note that conf->hold times may
-     * be changed elsewhere--so precomputed next event
-     * time might be too far away... (similarly with
-     * ping times) --msa
-     */
-    if (CurrentTime >= nextping)
-      nextping = check_pings();
-    
-    /*
-     * timeout pending queries that haven't been responded to
-     */
-    timeout_auth_queries(CurrentTime);
 
-    if (GlobalRehashFlag) {
-      rehash(&me, 1);
-      GlobalRehashFlag = 0;
-    }
-    if (GlobalRestartFlag)
-      server_restart("caught signal: SIGINT");
-  }
-}