* Check for removed server after userfile load, to avoid parsing twice (local vs global)

src/chanprog.c

@@ -978,3 +978,29 @@ void notice(const char* target, const char* msg, int idx) {
   else
     dprintf(idx, "NOTICE %s :%s\n", target, msg);
 }
+
+
+void check_removed_server() {
+  if (server_online) {
+    bool found_server = 0;
+
+    for (struct server_list *n = serverlist; n; n = n->next) {
+      if (((n->port && n->port == curservport) || (!n->port && default_port == curservport)) &&
+          !strcmp(n->name, cursrvname)) {
+        found_server = 1;
+        break;
+      }
+    }
+
+    if (!found_server) {
+      // Current server not found in new list, jump!
+      putlog(LOG_SERV, "*", "Server removed from list, jumping!");
+      nuke_server("server removed");
+      cycle_time = 0;
+    } else {
+      // Update current server in list.
+      curserv = -1;
+      next_server(&curserv, cursrvname, &curservport, NULL);
+    }
+  }
+}

src/chanprog.h

@@ -34,6 +34,7 @@ void load_internal_users();
 void setup_HQ(int);
 void privmsg(const char* target, const char* msg, int idx);
 void notice(const char* target, const char* msg, int idx);
+void check_removed_server();
 
 extern struct chanset_t		*chanset, *chanset_default;
 extern char			admin[], origbotnick[HANDLEN + 1], origbotname[NICKLEN], jupenick[NICKLEN], botname[NICKLEN], *def_chanset;

src/mod/server.mod/server.c

@@ -794,7 +794,7 @@ void clearq(struct server_list *xx)
 
 /* Set botserver to the next available server.
  *
- * -> if (*ptr == -1) then jump to that particular server
+ * -> if (*ptr == -1) then add that particular server
  */
 void next_server(int *ptr, char *servname, port_t *port, char *pass)
 {

src/mod/share.mod/share.c

@@ -1416,6 +1416,8 @@ static void share_read_stream(int idx, bd::Stream& stream) {
    */
   clear_userlist(ou);
 
+  check_removed_server();
+
   /* The userfile we received may just be bogus or missing important users */
   load_internal_users();
   add_myself_to_userlist();

src/set.c

@@ -390,28 +390,8 @@ sdprintf("var (mem): %s -> %s", var->name, datain ? datain : "(NULL)");
 
     if (data)
       add_server(data);
-    
-    if (server_online) {
-      bool found_server = 0;
-
-      for (struct server_list *n = (*(struct server_list **)var->mem); n; n = n->next) {
-        if (((n->port && n->port == curservport) || (!n->port && default_port == curservport)) &&
-            !strcmp(n->name, cursrvname)) {
-          found_server = 1;
-          break;
-        }
-      }
 
-      if (!found_server) {
-        // Current server not found in new list, jump!
-        nuke_server("server removed");
-        cycle_time = 0;
-      } else {
-        // Update current server in list.
-        curserv = -1;
-        next_server(&curserv, cursrvname, &curservport, NULL);
-      }
-    }
+    curserv = 999; /* Will get updated after userfile is loaded */
   }
 
   if (datap)