* Fix stack overflow in bot linking due to bot continually looping forever between autolink_cycle() via failed_link()

src/dcc.c

@@ -363,7 +363,7 @@ failed_link(int idx)
   strlcpy(nick, dcc[idx].nick, sizeof(nick));
   lostdcc(idx);
   if (conf.bot->hub || conf.bot->localhub)
-    autolink_cycle(nick);            /* Check for more auto-connections */
+    strlcpy(autolink_failed, nick, HANDLEN + 1);
 }
 
 static void

src/main.c

@@ -524,7 +524,7 @@ static void core_secondly()
   ++cnt;
 
   if (((conf.bot->localhub || conf.bot->hub) && (cnt % 30) == 0) || (cnt % 5) == 0) {
-    autolink_cycle(NULL);         /* attempt autolinks */
+    autolink_cycle();         /* attempt autolinks */
     cnt = 0;
   }
 
@@ -878,7 +878,7 @@ int main(int argc, char **argv)
   }
 
   online_since = now;
-  autolink_cycle(NULL);		/* Hurry and connect to tandem bots */
+  autolink_cycle();		/* Hurry and connect to tandem bots */
   timer_create_secs(1, STR("core_secondly"), (Function) core_secondly);
   timer_create_secs(10, STR("check_expired_dcc"), (Function) check_expired_dcc);
   timer_create_secs(10, STR("core_10secondly"), (Function) core_10secondly);

src/users.c

@@ -57,6 +57,7 @@
 #include "EncryptedStream.h"
 
 char userfile[121] = "";	/* where the user records are stored */
+char autolink_failed[HANDLEN + 1] = "";
 interval_t ignore_time = 10;		/* how many minutes will ignores last? */
 bool	dont_restructure = 0;		/* set when we botlink() to a hub with +U, only stops bot from restructuring */
 
@@ -1195,8 +1196,15 @@ void autolink_cycle_leaf(char *start)
 }
 
 
-void autolink_cycle(char *start)
+void autolink_cycle()
 {
+  char start[HANDLEN + 1] = "";
+
+  if (autolink_failed[0]) {
+    strlcpy(start, autolink_failed, HANDLEN + 1);
+    autolink_failed[0] = 0;
+  }
+
   if (conf.bot->hub)
     autolink_cycle_hub(start);
   else if (conf.bot->localhub)

src/users.h

@@ -169,7 +169,7 @@ char *delignore(char *);
 void tell_ignores(int, char *);
 bool match_ignore(char *);
 void check_expired_ignores();
-void autolink_cycle(char *);
+void autolink_cycle();
 void tell_file_stats(int, char *);
 void tell_user_ident(int, char *);
 void tell_users_match(int, char *, int, int, char *, int);
@@ -179,7 +179,7 @@ void check_pmode();
 void link_pref_val(struct userrec *u, char *lval);
 void check_stale_dcc_users();
 
-extern char			userfile[];
+extern char			userfile[], autolink_failed[];
 extern interval_t			ignore_time;
 extern bool			dont_restructure;