diff options
| -rw-r--r-- | TODO | 9 | ||||
| -rw-r--r-- | replay.c | 4 |
2 files changed, 12 insertions, 1 deletions
@@ -40,7 +40,14 @@ Debug log ("checking client socket <i> out of <fdmax>") gets busy after fdmax ge BLABOUNCER/stdin functions to add: - List connected/authenticated clients -KICK not handled. +KICK not handled (temporary work in kick-support-20230409.diff) +KICK: +Sun Apr 9 03:35:31 2023: BOUNCER-SERVER RECEIVED: ':bbnick!bbuser@2a06:5844:e:551:0:0:0:1d KICK #bla12 bbounce2 :bbnick +Sun Apr 9 03:35:31 2023: Server KICK: nick is NOT ours ('bbnick' vs 'bbounce2'). +PART: +Sun Apr 9 04:16:09 2023: BOUNCER-SERVER RECEIVED: ':bbounce2!l_bratch@2a06:5844:e:551:0:0:0:1d PART #bla12 :Leaving +Sun Apr 9 04:16:09 2023: Server PART: nick is ours ('bbounce2' vs 'bbounce2'). +^^Reminder, joining channels (and getting the nick list) seems to be broken right now (TG reminder). Crash when requesting 30 hour replay. @@ -461,6 +461,7 @@ long lastspokelinenumber(char *nick, char *basedir) { // Try to split if ((token = strsep(&strcopy, " ")) == NULL) { debugprint(DEBUG_CRIT, "lastspokelinenumber(): error splitting string on iteration '%d', returning -1!\n", i); + fclose(fp); return -1; } // Copy into the token array (strlen + 1 to get the NULL terminator) @@ -472,12 +473,14 @@ long lastspokelinenumber(char *nick, char *basedir) { // Make sure there were at least three tokens if (counter < 3) { debugprint(DEBUG_CRIT, "lastspokelinenumber(): not enough tokens on line, only '%d', returning -1!\n", counter); + fclose(fp); return -1; } // Make sure it started with a valid timestamp int timestamp = gettimestamp(tokens[0]); if (timestamp < 0) { + fclose(fp); debugprint(DEBUG_CRIT, "lastspokelinenumber(): line didn't start with a timestamp, returning -1!\n", counter); } @@ -622,6 +625,7 @@ int doreplaylastspoke(int sourcefd, long linenumber, struct client *clients, str snprintf(outgoingmsg, MAXDATASIZE, "NOTICE %s :Log replay complete.", ircdstate->ircnick); sendtoclient(sourcefd, outgoingmsg, clients, settings, 0); + fclose(fp); return 1; } |