* Fixed rembot() using an already free`d pointer to check for duplicate bots connecting

svn: 1195

doc/UPDATES

@@ -23,6 +23,7 @@ This is a summary of ChangeLog basically.
 * Moved config settings (bad-cookie, op-time-slack, manop, mdop, mop) to 'chanset'
 * Renamed 'op-time-slack' to 'cookie-time-slack'
 * SIGHUP now restarts bot.
+* Fixed possible SEGFAULT on bot timeout.
 
 1.1.9
 

src/botnet.c

@@ -268,10 +268,11 @@ void partyaway(char *bot, int sock, char *msg)
 
 /* Remove a tandem bot from the chain list
  */
-void rembot(const char *who)
+void rembot(const char *whoin)
 {
   tand_t **ptr = &tandbot, *ptr2 = NULL;
   struct userrec *u = NULL;
+  char *who = strdup(whoin);
 
   while (*ptr) {
     if (!egg_strcasecmp((*ptr)->bot, who))
@@ -292,6 +293,7 @@ void rembot(const char *who)
   tands--;
 
   dupwait_notify(who);
+  free(who);
 }
 
 void remparty(char *bot, int sock)