Configuration fixes. Repair automatic outbound connects.
[ircu2.10.12-pk.git] / ircd / ircd.c
index 01757c15b04479f61ac573fb4ca14f796b56bb61..d43f2b33062d8cb12d845f194705b6d84063fe39 100644 (file)
@@ -17,7 +17,7 @@
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
-/* @file
+/** @file
  * @brief Entry point and other initialization functions for the daemon.
  * @version $Id$
  */
@@ -61,7 +61,7 @@
 #include "version.h"
 #include "whowas.h"
 
-#include <assert.h>
+/* #include <assert.h> -- Now using assert in ircd_log.h */
 #include <errno.h>
 #include <fcntl.h>
 #include <netdb.h>
@@ -244,79 +244,62 @@ static int check_pid(void)
  */
 static void try_connections(struct Event* ev) {
   struct ConfItem*  aconf;
-  struct Client*    cptr;
   struct ConfItem** pconf;
-  int               connecting;
-  int               confrq;
   time_t            next        = 0;
   struct ConnectionClass* cltmp;
-  struct ConfItem*  con_conf    = 0;
   struct Jupe*      ajupe;
-  const char*       con_class   = NULL;
+  int hold;
 
   assert(ET_EXPIRE == ev_type(ev));
   assert(0 != ev_timer(ev));
 
-  connecting = FALSE;
   Debug((DEBUG_NOTICE, "Connection check at   : %s", myctime(CurrentTime)));
   for (aconf = GlobalConfList; aconf; aconf = aconf->next) {
-    /* Also when already connecting! (update holdtimes) --SRB */
-    if (!(aconf->status & CONF_SERVER) || aconf->address.port == 0 || aconf->hold == 0)
-      continue;
-
-    /* Also skip juped servers */
-    if ((ajupe = jupe_find(aconf->name)) && JupeIsActive(ajupe))
-      continue;
-
-    /* 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 >;) ]
+    /* Only consider server items with non-zero port and non-zero
+     * connect times that are not actively juped.
      */
-    if (aconf->hold > CurrentTime && (next > aconf->hold || next == 0)) {
-      next = aconf->hold;
+    if (!(aconf->status & CONF_SERVER)
+        || aconf->address.port == 0
+        || ((ajupe = jupe_find(aconf->name)) && JupeIsActive(ajupe)))
       continue;
-    }
 
+    /* Update next possible connection check time. */
+    if (next > aconf->hold || next == 0)
+        next = aconf->hold;
+
+    /* Update the next time we can consider this entry. */
     cltmp = aconf->conn_class;
-    confrq = get_con_freq(cltmp);
-    if(confrq == 0)
-      aconf->hold = next = 0;
-    else
-      aconf->hold = CurrentTime + confrq;
-
-    /* Found a CONNECT config with port specified, scan clients and see if
-     * this server is already connected?
+    hold = aconf->hold > CurrentTime; /* before we update aconf->hold */
+    aconf->hold = ConFreq(cltmp) ? CurrentTime + ConFreq(cltmp) : 0;
+
+    /* Do not try to connect if its use is still on hold until future,
+     * too many links in its connection class, it is already linked,
+     * or if connect rules forbid a link now.
      */
-    cptr = FindServer(aconf->name);
+    if (hold
+        || (Links(cltmp) > MaxLinks(cltmp))
+        || FindServer(aconf->name)
+        || conf_eval_crule(aconf->name, CRULE_MASK))
+      continue;
 
-    if (!cptr && (Links(cltmp) < MaxLinks(cltmp)) &&
-        (!connecting /*|| (ConClass(cltmp) > con_class)*/)) {
-      /*
-       * Check connect rules to see if we're allowed to try
-       */
-      if (0 == conf_eval_crule(aconf->name, CRULE_MASK)) {
-        con_class = ConClass(cltmp);
-        con_conf = aconf;
-        /* We connect only one at time... */
-        connecting = TRUE;
-      }
-    }
-    if ((next > aconf->hold) || (next == 0))
-      next = aconf->hold;
-  }
-  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)
+    /* Ensure it is at the end of the list for future checks. */
+    if (aconf->next) {
+      /* Find aconf's location in the list and splice it out. */
+      for (pconf = &GlobalConfList; *pconf; pconf = &(*pconf)->next)
+        if (*pconf == aconf)
           *pconf = aconf->next;
-      (*pconf = con_conf)->next = 0;
+      /* Reinsert it at the end of the list (where pconf is now). */
+      *pconf = aconf;
+      aconf->next = 0;
     }
 
-    if (connect_server(con_conf, 0))
+    /* Activate the connection itself. */
+    if (connect_server(aconf, 0))
       sendto_opmask_butone(0, SNO_OLDSNO, "Connection to %s activated.",
-                          con_conf->name);
+                          aconf->name);
+
+    /* And stop looking for further candidates. */
+    break;
   }
 
   if (next == 0)
@@ -479,7 +462,7 @@ static void parse_command_line(int argc, char** argv) {
    */
   while ((opt = getopt(argc, argv, options)) != EOF)
     switch (opt) {
-    case 'k':  thisServer.bootopt |= BOOT_CHKCONF;     break;
+    case 'k':  thisServer.bootopt |= BOOT_CHKCONF | BOOT_TTY; break;
     case 'n':
     case 't':  thisServer.bootopt |= BOOT_TTY;         break;
     case 'd':  dpath      = optarg;                    break;
@@ -654,6 +637,11 @@ int main(int argc, char **argv) {
 
   close_connections(!(thisServer.bootopt & (BOOT_DEBUG | BOOT_TTY | BOOT_CHKCONF)));
 
+  /* daemon_init() must be before event_init() because kqueue() FDs
+   * are, perversely, not inherited across fork().
+   */
+  daemon_init(thisServer.bootopt & BOOT_TTY);
+
   event_init(MAXCONNECTIONS);
 
   setup_signals();
@@ -674,8 +662,6 @@ int main(int argc, char **argv) {
   initmsgtree();
   initstats();
 
-  init_resolver();
-
   /* we need this for now, when we're modular this 
      should be removed -- hikari */
   ircd_crypt_init();
@@ -694,7 +680,6 @@ int main(int argc, char **argv) {
   }
 
   debug_init(thisServer.bootopt & BOOT_TTY);
-  daemon_init(thisServer.bootopt & BOOT_TTY);
   if (check_pid()) {
     Debug((DEBUG_FATAL, "Failed to acquire PID file lock after fork"));
     exit(2);