-
src/sbbs3/main.cpp
From
rswindell@VERT to
CVS commit on Wednesday, January 08, 2014 02:17:54
src/sbbs3 main.cpp 1.600 1.601
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv23230
Modified Files:
main.cpp
Log Message:
Terminate event_thread() more immediately (don't continue to interate through timed events, QWK call-out nodes, etc.) when the server has been terminated.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, March 07, 2014 18:06:33
src/sbbs3 main.cpp 1.602 1.603
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv29273
Modified Files:
main.cpp
Log Message:
Eliminate uses of sprintf() with string arguments.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, March 13, 2014 01:15:05
src/sbbs3 main.cpp 1.604 1.605
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv8852
Modified Files:
main.cpp
Log Message:
Fix the log output for Telnet environment options.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
deuce@VERT to
CVS commit on Thursday, April 24, 2014 12:54:36
src/sbbs3 main.cpp 1.605 1.606
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv21016
Modified Files:
main.cpp
Log Message:
Disable SBBS_TELNET_ENVIRON_SUPPORT for now. If the environment is too large for the static 64-byte sbbs->telnet_cmd buffer, it will bork the session.
(Seen with a long user-name and DISPLAY when using the OS X telnet client).
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, September 01, 2014 00:01:04
src/sbbs3 main.cpp 1.607 1.608
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv14050
Modified Files:
main.cpp
Log Message:
input_thread_mutex protection (don't destroy unless successfully created).
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
sbbs@VERT to
CVS commit on Friday, April 10, 2015 15:50:49
src/sbbs3 main.cpp 1.608 1.609
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Don't block on read of /dev/random since on some systems (with no entropy?), this read() call will block indefinitely. Fall-through to pseudo-random seeding if read() fails. Fix for FozzTexx-reported problem on reddit.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
deuce@VERT to
CVS commit on Friday, April 24, 2015 23:12:41
src/sbbs3 main.cpp 1.609 1.610
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv26794
Modified Files:
main.cpp
Log Message:
It is an error in Clang to cast a pointer to a smaller size. Since a thread
ID is a pointer on at least FreeBSD, cast to uintmax_t instead of DWORD.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, August 05, 2015 17:43:19
src/sbbs3 main.cpp 1.611 1.612
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv31176
Modified Files:
main.cpp
Log Message:
Fix: clear input and output buffers after accepting a new terminal client connection, just incase there's any stale data in the input or output buffers. Added a comment regarding the rioctl() function to decree that it's crazy minimalistic API be left alone in tribute to Steve Deppe.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, August 15, 2015 02:55:25
src/sbbs3 main.cpp 1.612 1.613
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv6283
Modified Files:
main.cpp
Log Message:
Added comment regarding observed crash.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, August 17, 2015 00:05:13
src/sbbs3 main.cpp 1.613 1.614
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv11893
Modified Files:
main.cpp
Log Message:
Initialize cursubnum and curdirnum members of sbbs_t to INVALID_SUB (previously, uninitialized values).
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
deuce@VERT to
CVS commit on Saturday, August 22, 2015 00:11:56
src/sbbs3 main.cpp 1.618 1.619
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv31538
Modified Files:
main.cpp
Log Message:
Fix some more warnings... still a couple left.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, August 24, 2015 23:13:13
src/sbbs3 main.cpp 1.620 1.621
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv28650
Modified Files:
main.cpp
Log Message:
Re-parenthesize the if() expression for QWKnet call-out event to address clang warning: warning: '&&' within '||' [-Wlogical-op-parentheses]
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
deuce@VERT to
CVS commit on Saturday, August 29, 2015 04:10:23
src/sbbs3 main.cpp 1.621 1.622
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv31553
Modified Files:
main.cpp
Log Message:
If zero bytes were read (such as a CTRL-C after avail was set), continue
rather than send the zero bytes.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, September 27, 2015 05:07:11
src/sbbs3 main.cpp 1.623 1.624
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv3726
Modified Files:
main.cpp
Log Message:
Handle the case correclty where no argument is passed to prompt()
(display no string), just like it is documented and implemented in jsexec.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, November 24, 2015 18:36:36
src/sbbs3 main.cpp 1.626 1.627
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv2126
Modified Files:
main.cpp
Log Message:
Log a warning when the configured time zone does not match the detected local time zone.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, December 04, 2015 13:33:20
src/sbbs3 main.cpp 1.627 1.628
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv29780
Modified Files:
main.cpp
Log Message:
More complete time zone misconfiguration warning message (include string version of configured time zone, e.g. "PST").
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, March 09, 2016 03:01:23
src/sbbs3 main.cpp 1.630 1.631
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv6738
Modified Files:
main.cpp
Log Message:
Better SSH-related initialization failure error logging (e.g. failure to load cryptlib.key).
Log timed event return values as an error (when non-zero).
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, June 08, 2017 19:20:20
src/sbbs3 main.cpp 1.647 1.648
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv1517
Modified Files:
main.cpp
Log Message:
More verbose log messages with regards to max_concurrent_connections being reached (connections with a successful login aren't counted).
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, August 14, 2017 00:41:19
src/sbbs3 main.cpp 1.648 1.649
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv13822
Modified Files:
main.cpp
Log Message:
Fix bug (regression) reported by Uatu13:
badip.msg wasn't be sent (display to) the IP-blocked user before disconnecting. This regression was introduced in rev 1.632 of this file when the temp IP-ban feature was implemented.
---
Synchronet Vertrauen Home of Synchronet
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, November 14, 2017 02:08:43
src/sbbs3 main.cpp 1.650 1.651
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Fix for SSH:
Active connections (sessions) would be interrupted if another SSH
connetion came in and failed (e.g. IP was blocked or failed negotiation).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
KK4QBN@VERT/KK4QBN to
rswindell on Tuesday, November 14, 2017 06:46:51
Re: src/sbbs3/main.cpp
By: rswindell to CVS commit on Tue Nov 14 2017 02:08:43
Modified Files:
main.cpp
Log Message:
Fix for SSH:
Active connections (sessions) would be interrupted if another SSH connetion came in and failed (e.g. IP was blocked or failed negotiation).
Beautiful... I'll try to build a new build tonight and use GDB and reactivate my mail server too, I've had SSH off for some time for this reason (and I've seen it's been known for a while) I really never had no one use ssh.. or at least ones who don't mind using telnet anyway.. so I really just saw it as wasted resources.. especially when it was kicking sessions out when bots would fail to gain access..
I'll start reading up on the GDB process again and maybe update the src tree later today.
--
Tim Smith (KK4QBN)
KK4QBN BBS
---
* Synchronet * KK4QBN BBS - kk4qbn.com - kk4qbn.synchro.net - Chatsworth GA USA
-
From
KenDB3@VERT/KD3NET to
rswindell on Tuesday, November 14, 2017 14:57:06
src/sbbs3 main.cpp 1.650 1.651
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Fix for SSH:
Active connections (sessions) would be interrupted if another SSH
connetion came in and failed (e.g. IP was blocked or failed negotiation).
Ohhhhh.... That's what was happening. I just thought SSH was finicky.
~KenDB3
---
Synchronet KD3net-Rhode Island's only BBS about nothing.
http://bbs.kd3.us
-
From
Digital Man@VERT to
KenDB3 on Tuesday, November 14, 2017 13:39:31
Re: Re: src/sbbs3/main.cpp
By: KenDB3 to rswindell on Tue Nov 14 2017 02:57 pm
src/sbbs3 main.cpp 1.650 1.651
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Fix for SSH:
Active connections (sessions) would be interrupted if another SSH connetion came in and failed (e.g. IP was blocked or failed negotiation).
Ohhhhh.... That's what was happening. I just thought SSH was finicky.
So did I, so I really didn't use it (SSH) much. It wasn't until I was logged in via SSH right in front of the BBS that I noticed a new SSH connection failure immediately coincided with my loss of SSH connection and little light bulb went on in my head. Luckily (?) that SSH client kept hammering away on my terminal server so it made it easy to reproduce, find and fix.
I didn't write the SSH stuff so I stear clear of it usually, but this one was pretty easy to identify.
digital man
Synchronet "Real Fact" #22:
The second ever Synchronet BBS was the Mid-Nite Hacker BBS (sysop: The Zapper). Norco, CA WX: 77.9F, 30.0% humidity, 3 mph E wind, 0.00 inches rain/24hrs
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Accession@VERT/PHARCYDE to
Digital Man on Tuesday, November 14, 2017 17:02:10
Hello Digital,
On Tue Nov 14 2017 13:39:30, Digital Man wrote to KenDB3:
I didn't write the SSH stuff so I stear clear of it usually, but this
one was pretty easy to identify.
While you're digging into it, if you could add better support for the lastest versions of openssh (not requiring changing new(er) standard options in openssh
itself - although I forget exactly what it was) would be awesomenomnom! ;)
Regards,
Nick
... "Не знаю. Я здесь только работаю."
--- GoldED+/LNX 1.1.5-b20170303
* Origin: thePharcyde_ distribution system (Wisconsin) (723:1/1)
Synchronet thePharcyde_
telnet://bbs.pharcyde.org (Wisconsin)
-
From
Digital Man@VERT to
Accession on Tuesday, November 14, 2017 18:22:48
Re: src/sbbs3/main.cpp
By: Accession to Digital Man on Tue Nov 14 2017 05:02 pm
Hello Digital,
On Tue Nov 14 2017 13:39:30, Digital Man wrote to KenDB3:
I didn't write the SSH stuff so I stear clear of it usually, but this one was pretty easy to identify.
While you're digging into it, if you could add better support for the lastest versions of openssh (not requiring changing new(er) standard options in openssh
itself - although I forget exactly what it was) would be awesomenomnom! ;)
I'm pretty sure that'd require an update of the Cryptlib library we're using. We're currently at v3.4.2 and the current release is v3.4.3.1 (with v3.4.4 in beta). It looks like v3.4.3.1 was release in Jan-2017, so it's probably time to update and what's been broke...er uh... fixed I mean. :-)
digital man
Synchronet "Real Fact" #94:
Synchronet v3.15b was released in October of 2011 (5 years after v3.14a). Norco, CA WX: 67.6F, 57.0% humidity, 5 mph E wind, 0.00 inches rain/24hrs
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, November 14, 2017 20:27:48
src/sbbs3 main.cpp 1.651 1.652
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Clear the ssh_session a little later to stop receive errors during SSH connect. Fixed warning: converting to non-pointer type ‘CRYPT_SESSION {aka int}’ from NULL.
Fix the race-condition crash Deuce reported:
<Deuce> #0 0x0000000802478d51 in pthread_mutex_destroy () from /lib/libthr.so.3
<Deuce> #1 0x0000000800d4e4fa in RingBufFree ()
<Deuce> #2 0x0000000800d27148 in passthru_input_thread ()
The output RingBuf can't be destroyed while the passthru_input_thread is still running.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
KenDB3@VERT/KD3NET to
Digital Man on Wednesday, November 15, 2017 09:51:11
Re: Re: src/sbbs3/main.cpp
By: KenDB3 to rswindell on Tue Nov 14 2017 02:57 pm
src/sbbs3 main.cpp 1.650 1.651
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Fix for SSH:
Active connections (sessions) would be interrupted if another SSH connetion came in and failed (e.g. IP was blocked or failed negotiation).
Ohhhhh.... That's what was happening. I just thought SSH was finicky.
So did I, so I really didn't use it (SSH) much. It wasn't until I was logged in via SSH right in front of the BBS that I noticed a new SSH connection failure immediately coincided with my loss of SSH connection and little light bulb went on in my head. Luckily (?) that SSH client kept hammering away on my terminal server so it made it easy to reproduce, find and fix.
I didn't write the SSH stuff so I stear clear of it usually, but this one was pretty easy to identify.
digital man
Well done man! It never clicked for me that it was related.
~KenDB3
---
Synchronet KD3net-Rhode Island's only BBS about nothing.
http://bbs.kd3.us
-
From
Accession@VERT/PHARCYDE to
Digital Man on Wednesday, November 15, 2017 16:53:16
Hello Digital,
On Tue Nov 14 2017 18:22:48, Digital Man wrote to Accession:
While you're digging into it, if you could add better support for
the lastest versions of openssh (not requiring changing new(er)
standard options in openssh itself - although I forget exactly what
it was) would be awesomenomnom! ;)
I'm pretty sure that'd require an update of the Cryptlib library we're using. We're currently at v3.4.2 and the current release is v3.4.3.1
(with v3.4.4 in beta). It looks like v3.4.3.1 was release in Jan-2017,
so it's probably time to update and what's been broke...er uh... fixed
I mean. :-)
No worries. I just figured I'd remind you about whatever little issue there was
since you were messing with it at the time. ;)
Regards,
Nick
... "Не знаю. Я здесь только работаю."
--- GoldED+/LNX 1.1.5-b20170303
* Origin: thePharcyde_ distribution system (Wisconsin) (723:1/1)
Synchronet thePharcyde_
telnet://bbs.pharcyde.org (Wisconsin)
-
From
rswindell@VERT to
CVS commit on Saturday, November 25, 2017 17:40:31
src/sbbs3 main.cpp 1.656 1.657
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv21125
Modified Files:
main.cpp
Log Message:
Include the protocol (e.g. Telnet, RLogin, or SSH) in the initial
"connection from" message sent to the remote terminal. This could be potentially helpful debug output for a file corruption sighting
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, November 28, 2017 01:34:08
src/sbbs3 main.cpp 1.657 1.658
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Got rid of unused array: uspy_listen_socket[], apparently left over from
before the IPv6 changes made in rev1.615. This also left the localspy#.sock files in the temp directory after sbbs was shutdown, so that's now fixed.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, January 19, 2018 20:16:37
src/sbbs3 main.cpp 1.660 1.661
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv7413
Modified Files:
main.cpp
Log Message:
Fixed problem where alert() calls in JS scripts executed as events were
not going to the event log (going nowhere, in fact).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, January 29, 2018 12:13:32
src/sbbs3 main.cpp 1.662 1.663
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv26471
Modified Files:
main.cpp
Log Message:
Clear thread ownership of SSH session after authentication.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, January 29, 2018 14:46:22
src/sbbs3 main.cpp 1.663 1.664
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv10511
Modified Files:
main.cpp
Log Message:
Set default read timeout to 1.
Previous versions of cryptlib defaulted to zero, but that's no longer the
case, and zero won't work.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, February 19, 2018 13:25:43
src/sbbs3 main.cpp 1.666 1.667
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv15334
Modified Files:
main.cpp
Log Message:
Having fixed all the other bugs, fix a typo in a random comment.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, February 22, 2018 01:57:14
src/sbbs3 main.cpp 1.668 1.669
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv6422
Modified Files:
main.cpp
Log Message:
eprintf() and eputs() won't print/log blank/empty strings now.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, March 05, 2018 23:33:49
src/sbbs3 main.cpp 1.672 1.673
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv32100
Modified Files:
main.cpp
Log Message:
A little more detail in the "Packing completed" log line.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, March 07, 2018 14:30:36
src/sbbs3 main.cpp 1.673 1.674
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv6287
Modified Files:
main.cpp
Log Message:
Fix socket-counter bug introduced in rev 1.615: after accept()ing a connection, the startup->socket_open() callback wasn't being called.
I'm not sure if this is correct with the *nix Spy socket stuff however.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Wednesday, March 07, 2018 14:39:57
src/sbbs3 main.cpp 1.674 1.675
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv7307
Modified Files:
main.cpp
Log Message:
Don't count non-client sockets since they don't call the close callback.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, March 09, 2018 18:19:08
src/sbbs3 main.cpp 1.675 1.676
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20332
Modified Files:
main.cpp
Log Message:
Resolve clang scan-build warning about JS write_raw() method potentially calling putcom() with a len value of 0 (which triggers an auto-string-length grab via strlen() which should be 0 - but.. meh).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, March 09, 2018 20:29:37
src/sbbs3 main.cpp 1.676 1.677
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv3700
Modified Files:
main.cpp
Log Message:
Elminated the redundnat "XXX Server listening" log messages
(the multisock stuff already logs this for each interface and port).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Friday, March 09, 2018 20:49:08
src/sbbs3 main.cpp 1.677 1.678
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv6231
Modified Files:
main.cpp
Log Message:
Use new cryptlib log stuff for SSH.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Friday, March 09, 2018 22:16:31
src/sbbs3 main.cpp 1.679 1.680
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv17238
Modified Files:
main.cpp
Log Message:
Fix warnings.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Friday, March 09, 2018 22:30:59
src/sbbs3 main.cpp 1.680 1.681
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv19303
Modified Files:
main.cpp
Log Message:
Add "SSH" to all the cryptlib SSH logging.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, March 12, 2018 11:07:38
src/sbbs3 main.cpp 1.685 1.686
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv18443
Modified Files:
main.cpp
Log Message:
Now that we've fixed the zero timeout on rx issue, we can set the rx timeout
to zero again. Unfortunately, there appears to be a related bug where the
TX timeout is sometimes used for RX... that's OK, we'll just set them both
to zero.
Also, deal with secondary channels (ie: forwarded ports). For now, we just close these channels any time we get data on them. In the future, we can
do scp/sftp pretty easily now... though neither is likely to ever work with
the sysop account due to the system password requirement.
This change ABSOLUTELY REQUIRES an updated cryptlib. Windows guys don't
update until DigitalMan commits cl.lib or whatever it's called.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, March 12, 2018 11:26:18
src/sbbs3 main.cpp 1.686 1.687
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20966
Modified Files:
main.cpp
Log Message:
Remove unused variables.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, March 12, 2018 18:23:08
src/sbbs3 main.cpp 1.687 1.688
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv13705
Modified Files:
main.cpp
Log Message:
Set the TX channel before reading the primary channel type. Also add
missing "SSH" when logging channel use errors.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, March 12, 2018 18:58:03
src/sbbs3 main.cpp 1.688 1.689
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv23055
Modified Files:
main.cpp
Log Message:
No matter the reason we don't get the channel type, still disconnect
the SSH session. Also, do an INFO level log entry to indicate what
happened to your socket.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, March 12, 2018 22:10:22
src/sbbs3 main.cpp 1.689 1.690
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv13637
Modified Files:
main.cpp
Log Message:
Upper-case the timed event internal code and display it in more of the
log messages.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Tuesday, March 13, 2018 22:37:48
src/sbbs3 main.cpp 1.690 1.691
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv19732
Modified Files:
main.cpp
Log Message:
Toss invalid session connections into badlogin().
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Tuesday, March 13, 2018 22:39:47
src/sbbs3 main.cpp 1.691 1.692
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20025
Modified Files:
main.cpp
Log Message:
Actually, anything except failure to send credentials should count as a
bad login.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Thursday, March 15, 2018 00:05:39
src/sbbs3 main.cpp 1.692 1.693
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv9870
Modified Files:
main.cpp
Log Message:
badlogin() can't be used for SSH sessions that are terminated early, this
code merely introduced a 5 second delay in the Node 0 thread.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Thursday, March 15, 2018 00:28:07
src/sbbs3 main.cpp 1.693 1.694
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv26008
Modified Files:
main.cpp
Log Message:
Connections without a "session" channel get treated as a dupe-free failed attempt.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Thursday, March 15, 2018 00:58:18
src/sbbs3 main.cpp 1.694 1.695
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv31114
Modified Files:
main.cpp
Log Message:
More hacks to work around a known bug in Cryptlib SSH...
"Sometimes" the write timeout value is used for read timeouts. Since we
use a read timeout of zero, and a mutex, this can cause some serious delays
in SSH processing. As a workaround, we set the write timeout to zero.
However, a flush failure has historically been fatal.
So, we now change the write timeout to one second just before a flush, and
have a flush timeout not be fatal anymore... we set it back to zero after
the flush.
The fiddling with the write timouts will be removed once the bug is fixed,
but the flush timout not being fatal may stay forever... add some comments
to remind us to remove the hack in the future.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Thursday, March 15, 2018 01:13:07
src/sbbs3 main.cpp 1.695 1.696
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv2153
Modified Files:
main.cpp
Log Message:
We certainly can't re-use i for error return values when it's how many
bytes have been sent!
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Thursday, March 15, 2018 14:42:13
src/sbbs3 main.cpp 1.696 1.697
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv25838
Modified Files:
main.cpp
Log Message:
Fix unterminated string usage.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, March 15, 2018 23:06:52
src/sbbs3 main.cpp 1.697 1.698
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv21340
Modified Files:
main.cpp
Log Message:
Adjusted log output:
Added "Node # SSH" to a lot of SSH-related log messages.
Lowered the severity of the SSH "attempt to use channel" msg from ERROR
to WARNING.
Removed "last:" from temp-ban message when there is no last-used username.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, March 19, 2018 17:37:16
src/sbbs3 main.cpp 1.698 1.699
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv1168
Modified Files:
main.cpp
Log Message:
Some four spaces to tab conversion at the beginnings of lines.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, March 23, 2018 16:57:46
src/sbbs3 main.cpp 1.700 1.701
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv27599
Modified Files:
main.cpp
Log Message:
Log a generic "SSH session establishment failed" message (with NOTICE
severity) whenever there is an SSH session establishment failure, because
some of the SSH failure errors are DEBUG level severity and won't be visible with "Normal" (informational) log output filtering. This resolves the issue
of seing "SSH connection accepted from <ip-address> port <port>" messages
with no follow-up messages as to what happened to that connection.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Sunday, March 25, 2018 21:40:49
src/sbbs3 main.cpp 1.702 1.703
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv27858
Modified Files:
main.cpp
Log Message:
Since much more than the number of bytes we're reading can be sent on a
channel we ignore/close, we would get a separate warning for each chunk of
data on that channel until we read past the close. Instead, if we're closing
a channel, we note the channel ID, and simply discard the information immediately on the most recently closed channel.
A remote could still cause a bunch of log output by multiplexing two or more writes to invalid channels, but nobody seems to do that, so this should be
good for now.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Saturday, March 31, 2018 01:14:49
src/sbbs3 main.cpp 1.703 1.704
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv28048
Modified Files:
main.cpp
Log Message:
Tabs are stupid.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, March 31, 2018 01:40:31
src/sbbs3 main.cpp 1.704 1.705
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv30934
Modified Files:
main.cpp
Log Message:
Check the terminate_output_thread flag after acquiring the ssh_muex.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, April 01, 2018 13:48:02
src/sbbs3 main.cpp 1.705 1.706
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Added function names to SSH error messages so we can better tell where
these errors are coming from.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, April 02, 2018 16:45:42
src/sbbs3 main.cpp 1.706 1.707
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv4421
Modified Files:
main.cpp
Log Message:
Create yet-another GCES macro (GCESSTR) to log the output_thread instance description string ('node') in SSH-related log messages (e.g. errors) from the output_thread: No more "Node 0" log entries from here.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, April 12, 2018 22:25:07
src/sbbs3 main.cpp 1.709 1.710
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Apparent fix for SSH cryptlib session leak (leading to eventual
'Resource/space exhausted' errors in TLS and SSH:
The wrong ssh_session is being passed to cryptDestroySession() in sbbs_t::hangup() (introduced in rev 1.615), this call to cryptDestorySession() fails (returns -1, bad param) thus leaving the actual
ssh_session never destroyed (leaked).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, April 16, 2018 15:39:20
src/sbbs3 main.cpp 1.711 1.712
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Fix typo in previous commit.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, April 17, 2018 23:46:24
src/sbbs3 main.cpp 1.712 1.713
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv4929
Modified Files:
main.cpp
Log Message:
Some log output updates, mainly adding errno string-representation where missing, but also now logging when renaming (or failing to rename) bad
received .REP packet files.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, April 20, 2018 01:18:18
src/sbbs3 main.cpp 1.713 1.714
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv21015
Modified Files:
main.cpp
Log Message:
Fix race-condition causing SSH errors:
Terminal Server SSH ERROR 'Bad argument, parameter 1' (-1)
... from output_thread
The bbs_thread() sets the global/server sbbs ssh_mode to false and ssh_session to 0 (the "parameter 1" value used in the cryptlib function calls in output_thread) but was doing this without owning the ssh_mutex, so the output_thread had a race condition where it would check ssh_mode=true and
then use grab the ssh_mutex and use ssh_session in a few cryptlib function calls. The fix for the bbs_thread() to grab the ssh_mutex before setting ssh_mode to false and ssh_session to 0 and have the output_thread() re-check the ssh_mode after grabbing the ssh_mutex and not call any cryptlib
functions if ssh_mode was set to false while waiting for the mutex.
The cause would have been more obvious if the various cryptlib error/log messages contained the cryptlib session ID value (which was 0 in this case).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, April 23, 2018 18:43:26
src/sbbs3 main.cpp 1.714 1.715
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Fix more race conditions which cause Terminal Server SSH ERROR 'Bad argument, parameter 1' (-1) ... from output_thread
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, April 23, 2018 19:12:45
src/sbbs3 main.cpp 1.715 1.716
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Fix yet another race condition that could cause:
Terminal Server SSH ERROR 'Bad argument, parameter 1' (-1) ...
from output_thread
Don't set ssh_mode=true until the ssh_session has been created successfully.
Also added some more debug output with regards to Cryptlib session number.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, April 23, 2018 19:23:17
src/sbbs3 main.cpp 1.716 1.717
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv26275
Modified Files:
main.cpp
Log Message:
Added protocol (e.g. Telnet, Rlogin, or SSH) to more log messages
(e.g. warnings and errors).
Also, stripped trailing white-space from many lines (editor setting).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, June 29, 2018 18:11:08
src/sbbs3 main.cpp 1.719 1.720
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv6643
Modified Files:
main.cpp
Log Message:
Fix problem with socket-I/O external programs executed while the client
using SSH: the output from the program (file transfer driver, door game)
would reach the SSH client, but input from the client (or user) would not
reach the external program:
passthru_output_thread() - responsible for SSH input from the client/user
to be sent to a "raw" pass-through socket used by the external program,
had a reversed logic bug added around rev 1.686.
Problem was introduced as part of the cryptlib update/SSH overhaul of
early 2018. Thanks to Thumper @ THEWASTE for the bug report!
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, July 24, 2018 21:00:55
src/sbbs3 main.cpp 1.721 1.722
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv19028
Modified Files:
main.cpp
Log Message:
More eprintf() to sbbs->lputs/lprintf() replacements.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, July 24, 2018 23:12:15
src/sbbs3 main.cpp 1.722 1.723
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv10420
Modified Files:
main.cpp
Log Message:
Make sure the current user number is 0 when terminating the event thread
or else we may end up with user names in log msgs that don't apply.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, July 24, 2018 23:23:34
src/sbbs3 main.cpp 1.723 1.724
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv12956
Modified Files:
main.cpp
Log Message:
More useron number clearing after timed event segments.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, July 24, 2018 23:28:27
src/sbbs3 main.cpp 1.724 1.725
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv14954
Modified Files:
main.cpp
Log Message:
Strip control codes from strings logged with sbbs_t::lputs().
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, July 25, 2018 01:40:48
src/sbbs3 main.cpp 1.725 1.726
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv2227
Modified Files:
main.cpp
Log Message:
More eprintf() to sbbs->lprintf() conversion.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, August 02, 2018 23:27:06
src/sbbs3 main.cpp 1.726 1.727
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv8199
Modified Files:
main.cpp
Log Message:
Fix conditions where event log messages could still be sent to the terminal server log output/window.
Also added comment about an observed exception upon shutdown with regards to the node_threads_running counter (protected int) being accessed after destruction.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, October 01, 2018 16:01:03
src/sbbs3 main.cpp 1.727 1.728
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv4756
Modified Files:
main.cpp
Log Message:
Log the execution result (error-level/return-value) of more events and
QWKnet call-outs. Hopefully help debug Mortifis' problem with QWKnet
call-outs.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, October 01, 2018 16:21:09
src/sbbs3 main.cpp 1.728 1.729
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv7009
Modified Files:
main.cpp
Log Message:
Log errors when failing to delete semaphore (e.g. *.now) and other files.
Fixed some funny typos in comments.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, October 04, 2018 23:25:36
src/sbbs3 main.cpp 1.729 1.730
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv13955
Modified Files:
main.cpp
Log Message:
Enclose user's name/alias in < > in log output, this makes the name clearly separate from the log text and is consistent with the mail server log
output.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, October 06, 2018 15:36:04
src/sbbs3 main.cpp 1.730 1.731
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv14540
Modified Files:
main.cpp
Log Message:
Zero-initialize the subscan array.
Lock the mail base while backing it up as part of the daily maintenance.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, October 17, 2018 11:59:47
src/sbbs3 main.cpp 1.732 1.733
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Fixed some gcc warnings:
format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has
type ‘unsigned int’
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, October 17, 2018 12:09:18
src/sbbs3 main.cpp 1.733 1.734
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Deal with gcc warnings that crop up because sizeof() is an unsigned long sometimes and sometimes it's an unsigned int.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, October 22, 2018 00:38:07
src/sbbs3 main.cpp 1.735 1.736
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv19254
Modified Files:
main.cpp
Log Message:
When logging event-thread error messages, include the "evnt " prefix before
the string (which starts with the event internal code).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, November 23, 2018 09:18:32
src/sbbs3 main.cpp 1.738 1.739
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv26989
Modified Files:
main.cpp
Log Message:
Cosmetic fixes to timed event logging (the event code part of the log messages)
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, December 29, 2018 21:33:33
src/sbbs3 main.cpp 1.739 1.740
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv9095
Modified Files:
main.cpp
Log Message:
Change the "unpack REP" log messgae prefix to "unpackREP" instead of "unQWK".
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, February 14, 2019 21:09:33
src/sbbs3 main.cpp 1.741 1.742
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv3304
Modified Files:
main.cpp
Log Message:
Added new semfile to clear the failed login attempt list: "ctrl/clear"
So now if you need to clear the failed-login/temp-ban list, just
'touch /sbbs/ctrl/clear'
Other changes to the semfiles:
The terminal server is now identified as "term" rather than "telnet" in the (optional) server/protocol specific semfile naming.
Removed support for the old telnet.rec semfile (long deprecated).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
deuce@VERT to
CVS commit on Monday, June 03, 2019 13:55:17
src/sbbs3 main.cpp 1.749 1.750
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv16111
Modified Files:
main.cpp
Log Message:
New cryptlib uses 345 not 3450.
*sigh*
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, August 12, 2019 12:03:49
src/sbbs3 main.cpp 1.757 1.758
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv22730
Modified Files:
main.cpp
Log Message:
Accept and store the "Internet host number" portion of the telnet
TERMINAL LOCATION NUMBER (RFC 946) in sbbs_t::telnet_location.
Some (older?) versions of fTelnet request to send this option to the BBS to share the real IP address of the client. Accept this option (when using
"format 0") and store the first 32-bits (an IPv4 address) in the telnet location storage / caller-ID information. The same place the telnet
SEND LOCATION value is stored if received.
Now, we're not requesting this option (e.g. with a Telnet "DO" command), but fTelnet doesn't need that: It offers to send it, so we accept.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, August 13, 2019 17:10:20
src/sbbs3 main.cpp 1.758 1.759
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv17377
Modified Files:
main.cpp
Log Message:
Log the length of received telnet sub-negotiation commands (in bytes).
Only retain the last (most recent) 10 data/<id>.qwk.*.bad files and data/file/<user-num>.rep.*.bad files.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, August 22, 2019 16:49:15
src/sbbs3 main.cpp 1.760 1.761
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv12091
Modified Files:
main.cpp
Log Message:
We need to expand telnet IACs contained in passthru socket data received from external programs (when in Telnet mode).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, August 24, 2019 22:29:55
src/sbbs3 main.cpp 1.763 1.764
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
A couple of passthrough socket fixes:
1. when de-activating the passthru socket, give the passthru_thread some
cycles to copy any remaining socket data into the outbuf before we let
the node_thread continue on and spew its own data into the outbuf.
This fixes the problem of final messages of a file transfer protocol
being intermixed with BBS data (prompts, menus, and such) and causing
file download finalization issues.
2. Don't read from the passthru socket until there is enough room in the
outbuf for the maximum possible telnet-expanded read size. This fixes
streaming download protocol (ZMODEM and YMODEM-G) errors.
Also, for good measure, check the passthru_socket for writability before send()ing on it and log a warning if it's ever not writable.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, August 25, 2019 22:44:36
src/sbbs3 main.cpp 1.765 1.766
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv27823
Modified Files:
main.cpp
Log Message:
Re-enable socket optoins and blocking configuration for the client_socket_dup when de-activating the passthru_socket.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, August 26, 2019 17:08:34
src/sbbs3 main.cpp 1.766 1.767
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv4564
Modified Files:
main.cpp
Log Message:
passthru_socket():
Always call recv() when select() says the passthru_socket is readable, even
if the outbuf is full. This resolves an issue where the disconnection of the passthru_socket would not be detected if the outbuf was full (e.g. the output_thread stopped running).
Also, check if RingBufWrite() returns a short-write count and log the values (actual versus expected write byte count).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, August 26, 2019 18:17:25
src/sbbs3 main.cpp 1.767 1.768
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv14985
Modified Files:
main.cpp
Log Message:
Log value of 'writable' in "could not write to passthru socket" log msg.
May help determine if the socket is disconnected or just not writable.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, August 26, 2019 18:19:25
src/sbbs3 main.cpp 1.768 1.769
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
main.cpp
Log Message:
Resolve a couple harmless GCC warnings.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, August 27, 2019 00:48:00
src/sbbs3 main.cpp 1.769 1.770
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv28680
Modified Files:
main.cpp
Log Message:
When the passthru_thread terminates, set passthru_socket_active to false.
This resolves the issue where if an external program closes the client_socket_dup, the passthru_thread can't write to the passthru_socket,
so it terminates itself - by definition, deactivating the passthru_socket, but it didn't clear passthru_socket_active, so the input_thread would keep trying to write to it rather than sending the input from the user to the BBS.
Also, terminate the passthru_thread (without any special log message) when sbbs->online is non-zero.
Also overhauled a lot of the passthru-related log messages. Passthru ain't just for SSH anymore.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, August 27, 2019 03:36:42
src/sbbs3 main.cpp 1.770 1.771
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv21353
Modified Files:
main.cpp
Log Message:
Leave room for telnet IAC expansion (as much as 100%) in the outbuf.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, August 27, 2019 09:24:23
src/sbbs3 main.cpp 1.771 1.772
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv12576
Modified Files:
main.cpp
Log Message:
Eliminate call to inet_ntop() which is apparently not present in WinXP:
The procedure entry point inet_ntop could not be located in dynamic link library ws2_32.dll
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, August 28, 2019 19:23:41
src/sbbs3 main.cpp 1.772 1.773
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20280
Modified Files:
main.cpp
Log Message:
Don't try to remove a bad .QWK packet if it's been renamed to *.bad.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Ragnarok@VERT/DOCKSUD to
rswindell on Thursday, August 29, 2019 11:07:23
El 28/8/19 a las 23:23, rswindell escribi:
src/sbbs3 main.cpp 1.772 1.773
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20280
Modified Files:
main.cpp
Log Message:
Don't try to remove a bad .QWK packet if it's been renamed to *.bad.
---
i notice that have several *.bad packets at my data dir.
-rw------- 1 root root 6807 ago 16 03:47 VERT.qwk.5d5a512f.bad
-rw------- 1 root root 7898 ago 13 19:47 VERT.qwk.5d5a5944.bad
-rw------- 1 root root 7245 ago 14 14:47 VERT.qwk.5d5a59bc.bad
i rename to vert.qwk to reprocess it, and say that all message are dupe.
Then, i think that was process and import messages fine.
is this commit related to this issue?
Aug 29 10:24:45 localhost synchronet: evnt Inbound QWK Packet detected: /sbbs/data/VERT.qwk
Aug 29 10:24:45 localhost synchronet: evnt QNET Executing external:
unzip -o -j /sbbs/data/VERT.qwk * -d /sbbs/temp/event/
Aug 29 10:24:45 localhost synchronet: evnt QNET Reading /sbbs/temp/event/HEADERS.DAT
Aug 29 10:24:45 localhost synchronet: evnt QNET Reading /sbbs/temp/event/VOTING.DAT
Aug 29 10:24:45 localhost synchronet: evnt Importing QWK Network Packet: /sbbs/data/VERT.qwk
Aug 29 10:24:45 localhost synchronet: evnt Importing QWK messages from
VERT into DOVE-Net Synchronet Discussion
Aug 29 10:24:45 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D564377.6017.sync@vert.synchro.net> found in message #36400
Aug 29 10:24:45 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D563EDE.40572.dove_sync@digitaldistortionbbs.com> found in message #36401
Aug 29 10:24:45 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D563EDE.40572.dove_sync@digitaldistortionbbs.com> found in message #36401
Aug 29 10:24:45 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D564039.40573.dove_sync@digitaldistortionbbs.com> found in message #36402
Aug 29 10:24:45 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D5633E8.3542.dove-syncdisc@bbs.leenooks.net> found in
message #36403
Aug 29 10:24:45 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D564ECC.6022.sync@vert.synchro.net> found in message #36404
Aug 29 10:24:45 localhost synchronet: evnt Importing QWK messages from
VERT into DOVE-Net Synchronet Programming (C/C++ and CVS)
Aug 29 10:24:45 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <5D56516D.39346.syncprog@cvsuser> found in message #21047
Aug 29 10:24:45 localhost synchronet: evnt Importing QWK messages from
VERT into DOVE-Net Synchronet Data
Aug 29 10:24:45 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D564BFC.177089.syncdata@vert.synchro.net> found in message
#13539
Aug 29 10:24:45 localhost synchronet: evnt Finished Importing QWK
Network Packet from VERT: (0 msgs) in 1 seconds (0 msgs/sec), 1 errors,
8 dupes
Aug 29 10:24:45 localhost synchronet: evnt QNET /sbbs/data/VERT.qwk
renamed to /sbbs/data/VERT.qwk.5d67d21d.bad
Aug 29 10:24:45 localhost synchronet: evnt QNET !ERROR 2 (No such file
or directory) in main.cpp line 2876 (event_thread) removing "/sbbs/data/VERT.qwk" access=0
thanks!!
---
Synchronet Dock Sud BBS TLD 24 HS -
http://bbs.docksud.com.ar -
telnet://bbs.docksud.com.ar
-
From
Digital Man@VERT to
Ragnarok on Thursday, August 29, 2019 14:35:36
Re: Re: src/sbbs3/main.cpp
By: Ragnarok to rswindell on Thu Aug 29 2019 11:07 am
El 28/8/19 a las 23:23, rswindell escribi:
src/sbbs3 main.cpp 1.772 1.773
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20280
Modified Files:
main.cpp
Log Message:
Don't try to remove a bad .QWK packet if it's been renamed to *.bad.
---
i notice that have several *.bad packets at my data dir.
-rw------- 1 root root 6807 ago 16 03:47 VERT.qwk.5d5a512f.bad -rw------- 1 root root 7898 ago 13 19:47 VERT.qwk.5d5a5944.bad -rw------- 1 root root 7245 ago 14 14:47 VERT.qwk.5d5a59bc.bad
i rename to vert.qwk to reprocess it, and say that all message are dupe. Then, i think that was process and import messages fine.
is this commit related to this issue?
The commit fixes this error:
Aug 29 10:24:45 localhost synchronet: evnt QNET !ERROR 2 (No such file
or directory) in main.cpp line 2876 (event_thread) removing "/sbbs/data/VERT.qwk" access=0
digital man
This Is Spinal Tap quote #37:
David St. Hubbins: We are Spinal Tap from the UK - you must be the USA!
Norco, CA WX: 92.1F, 42.0% humidity, 4 mph NE wind, 0.00 inches rain/24hrs
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Ragnarok@VERT/DOCKSUD to
Digital Man on Thursday, August 29, 2019 23:57:38
i enable debug:, the "QWK vote failure, offset 6144 in
/sbbs/data/VERT.qwk" appear:
Aug 29 23:54:29 localhost synchronet: evnt Inbound QWK Packet detected: /sbbs/data/VERT.qwk
Aug 29 23:54:29 localhost synchronet: evnt QNET Executing external:
unzip -o -j /sbbs/data/VERT.qwk * -d /sbbs/temp/event/
Aug 29 23:54:29 localhost synchronet: evnt QNET Reading /sbbs/temp/event/HEADERS.DAT
Aug 29 23:54:29 localhost synchronet: evnt QNET Reading /sbbs/temp/event/VOTING.DAT
Aug 29 23:54:29 localhost synchronet: evnt Importing QWK Network Packet: /sbbs/data/VERT.qwk
Aug 29 23:54:29 localhost synchronet: evnt Importing QWK messages from
VERT into DOVE-Net Synchronet Discussion
Aug 29 23:54:29 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D564377.6017.sync@vert.synchro.net> found in message #36400
Aug 29 23:54:29 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D563EDE.40572.dove_sync@digitaldistortionbbs.com> found in message
#36401
Aug 29 23:54:29 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D563EDE.40572.dove_sync@digitaldistortionbbs.com> found in message
#36401
Aug 29 23:54:29 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D564039.40573.dove_sync@digitaldistortionbbs.com> found in message
#36402
Aug 29 23:54:29 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D5633E8.3542.dove-syncdisc@bbs.leenooks.net> found in
message #364
03
Aug 29 23:54:29 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D564ECC.6022.sync@vert.synchro.net> found in message #36404
Aug 29 23:54:29 localhost synchronet: evnt QNET Duplicate vote-msg from VERT Aug 29 23:54:29 localhost synchronet: evnt QNET QWK vote failure, offset
6144 in /sbbs/data/VERT.qwk
Aug 29 23:54:29 localhost synchronet: evnt Importing QWK messages from
VERT into DOVE-Net Synchronet Programming (C/C++ and CVS)
Aug 29 23:54:29 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <5D56516D.39346.syncprog@cvsuser> found in message #21047
Aug 29 23:54:29 localhost synchronet: evnt Importing QWK messages from
VERT into DOVE-Net Synchronet Data
Aug 29 23:54:29 localhost synchronet: evnt QNET !smb_addmsg duplicate Message-ID: <
5D564BFC.177089.syncdata@vert.synchro.net> found in message
#13539
Aug 29 23:54:29 localhost synchronet: evnt Finished Importing QWK
Network Packet from VERT: (0 msgs) in 1 seconds (0 msgs/sec), 1 errors,
8 dupes
Aug 29 23:54:29 localhost synchronet: evnt QNET /sbbs/data/VERT.qwk
renamed to /sbbs/data/VERT.qwk.5d688fe5.bad
---
Synchronet Dock Sud BBS TLD 24 HS -
http://bbs.docksud.com.ar -
telnet://bbs.docksud.com.ar
-
From
rswindell@VERT to
CVS commit on Friday, August 30, 2019 19:05:01
src/sbbs3 main.cpp 1.773 1.774
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv22798
Modified Files:
main.cpp
Log Message:
Log (debug-level) interesting sizeof() values during initialization, debug-builds only.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, September 01, 2019 02:08:40
src/sbbs3 main.cpp 1.774 1.775
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv15235
Modified Files:
main.cpp
Log Message:
Received-telnet command improvements:
- If a telnet command was received in multiple packets, the memcpy optimization
in telnet_interpret() would skip/drop all bytes in the subsequent pkt payload
before an IAC char. Don't optimize when in the middle of a telnet command.
- If a received telnet command exceeds the telnet_cmd buffer, reset the
received telnet_cmdlen and log a warning-level message
- If a telnet sub-negotiation END command is received as the beginning of a
new telnet command, log a warning-level message and reset the telnet_cmdlen.
This fixes the occasional problem observed when using fTelnet and its sending the "SEND-LOCATION" sub-neg command split between 2 TCP packets. Only
part of the first packet would be used as the location data and the rest processed as input from the users (e.g. as the Login: prompt). Thanks, Ree!
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, September 22, 2019 19:18:21
src/sbbs3 main.cpp 1.775 1.776
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv18524
Modified Files:
main.cpp
Log Message:
This change goes with the previous commit to scfglib2.c (with no commit msg): Address problem reported by Mark Lewis:
If a timed event is configured to both run "exclusively" and on "Any" node, then any nodes in use at the time of the event would get their status changed to waiting-for-connection (WFC) and it could actually become possible for 2 clients to be using the same node number.
So, when loading the configuration, turn off the "exclusive" flag if the event's node is set to "Any" (0).
Just for completeness, fix the actual bug (introduced in main.cpp 1.744) too.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, December 20, 2019 00:09:34
src/sbbs3 main.cpp 1.777 1.778
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv10703
Modified Files:
main.cpp
Log Message:
Fix feature added in rev 1.759 (wrong path argument to delfiles):
Only retain the last (most recent) 10 data/<id>.qwk.*.bad files and data/file/<user-num>.rep.*.bad files.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, February 01, 2020 18:55:14
src/sbbs3 main.cpp 1.778 1.779
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv12537
Modified Files:
main.cpp
Log Message:
The log msg "passthru socket listening on port n" was always wrong
(said port 0) since the port had not been bound yet.
So fix the display order of that msg (after listen() is called)
and lower the log level to debug.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, April 16, 2020 22:38:31
src/sbbs3 main.cpp 1.782 1.783
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv1099
Modified Files:
main.cpp
Log Message:
Clear the current attribute after the user stirng input in the JS prompt() function (don't leave it high-intensity white on accident).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, April 27, 2020 14:00:32
src/sbbs3 main.cpp 1.783 1.784
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20792
Modified Files:
main.cpp
Log Message:
The "all nodes in use" message specifically said "telnet nodes". Change to "terminal nodes".
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, May 23, 2020 12:11:07
src/sbbs3 main.cpp 1.789 1.790
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv533
Modified Files:
main.cpp
Log Message:
Address problem reported by wkitty42:
May 21 00:00:15 southeaststar synchronet: evnt BBS Events Executing external: ?logonlist -m
May 21 00:00:15 southeaststar synchronet: evnt BBS Events !JavaScript warning /sbbs/exec/load/nodedefs.js line 167: Disconnected
... by setting the online mode to ON_LOCAL while executing the sys_daily event.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, July 19, 2020 21:12:22
src/sbbs3 main.cpp 1.790 1.791
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv13583
Modified Files:
main.cpp
Log Message:
Don't display full path (just the filename) of errored JS file to user.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to sbbs/master on Sunday, August 23, 2020 17:50:06
-
From
Rob Swindell@VERT to
Git commit to sbbs/master on Thursday, September 03, 2020 22:29:28
-
From
Rob Swindell@VERT to
Git commit to sbbs/master on Saturday, September 19, 2020 22:33:42
https://gitlab.synchro.net/sbbs/sbbs/-/commit/9bf4b7bba19d951f3ca48c96
Modified Files:
src/sbbs3/main.cpp
Log Message:
Don't send telnet commands to spy sockets
send_telnet_cmd() now sends telnet commands/replies directly to the client socket. This avoid the entire output_thread() teeing of sent data to spy sockets/buffers (which may not be Telnet at all).
There are some concerns with this change:
- if client_socket isn't writable, will sendsocket() block?
- if output_thread is sending on the same socket, are our 2/3-byte telnet
commands going to have other output data interleaved within? (seems very
unlikely)
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to sbbs/master on Saturday, September 19, 2020 23:04:07
-
From
Rob Swindell@VERT to
Git commit to sbbs/master on Friday, October 23, 2020 01:51:57
https://gitlab.synchro.net/sbbs/sbbs/-/commit/ec6d7e3edf557779be7e54cd
Modified Files:
src/sbbs3/main.cpp
Log Message:
Don't count guest logins as authenticated connections...
When enforcing the MaxConcurrentConnections limit, don't count connections for user's with blank passwords (i.e. Guest) as authenticated.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to sbbs/master on Wednesday, November 04, 2020 23:28:33
-
From
Rob Swindell@VERT to
Git commit to sbbs/master on Friday, November 13, 2020 00:24:17
https://gitlab.synchro.net/sbbs/sbbs/-/commit/bd78d5c5725843afc6659822
Modified Files:
src/sbbs3/main.cpp
Log Message:
Add timestamp of node.log file to crash.log
When a preexisting node.log file is found, a terminal server crash is suspected. Include the timestamp of the node.log in the message appended to the system log (data/logs/*.log) to help identify the likely time of the crash.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Deon George@VERT to
Git commit to sbbs/master on Sunday, November 22, 2020 00:35:59
-
From
Deon George@VERT to
Git commit to sbbs/master on Sunday, November 22, 2020 00:35:59
-
From
Digital Man@VERT to
Deon George on Sunday, November 22, 2020 13:23:45
Re: src/sbbs3/main.cpp
By: Deon George to Git commit to sbbs/master on Sun Nov 22 2020 12:35 am
https://gitlab.synchro.net/sbbs/sbbs/-/commit/6d0ec492cd2a5db51e416548 Modified Files:
src/sbbs3/main.cpp
Log Message:
Ooops, forgot to make btox static
Oops, I forgot to squash this giant merge. <grimmace>
--
digital man
Rush quote #64:
He's cleaning up his systems to keep his nature pure .. New World Man
Norco, CA WX: 75.9F, 29.0% humidity, 6 mph NNE wind, 0.00 inches rain/24hrs
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, November 29, 2020 00:31:28
https://gitlab.synchro.net/main/sbbs/-/commit/82d3ae6e61d8876d35444bdf
Modified Files:
src/sbbs3/main.cpp
Log Message:
Load/keep/refresh each node's configuration in memory.
Previously, all nodes shared the same copy of the configuration in memory. This prevented any node from seeing an updated configuration until all nodes were offline or waiting for connection and could be recycled. Now, no recycling of the server is needed for a node to load a new config. A node in-use *still* cannot reload configuration until the user disconnects, but all you have to do is logon to another unused node and you (or that user) will get an updated configuration. Bots or server recycling problems will no longer prevent the sysop or user from getting a current configuration when connecting to any node.
This is one of those instances of giving the sysop what he wants rather than what he asked for. I should've done this a long time ago, but the idea just occurred to me. You're welcome. :-)
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, November 29, 2020 11:40:10
https://gitlab.synchro.net/main/sbbs/-/commit/827063bb1d4343e145a85ffa
Modified Files:
src/sbbs3/main.cpp
Log Message:
Copy event last-run info from global config into node config
Fixes issue introduced in last commit to this file (the each-node-has-its-own-config-in-memory enhancement) that would prevent users from logging in because their time was reduced for an upcoming event that had already run. The change assumes the event thread is running (its the thing that reads the event last-run info from time.dab). Thanks to mlong for the report.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, November 29, 2020 12:28:05
https://gitlab.synchro.net/main/sbbs/-/commit/07bd3442362ada96e99a4294
Modified Files:
src/sbbs3/main.cpp
Log Message:
Fallback to the "first node" configuration
If a node's configuration can't be loaded (e.g. nodeX/node.cnf is missing), fall-back and load the "first node" (e.g. Node1) configuration. On recent *nix installs (since the migration to Git), the node2+/node.cnf files were not copied/created, so let's solve that soon-to-be-FAQ now.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, November 29, 2020 13:10:58
-
From
Bob Roberts@VERT/HOVAL to
Rob Swindell on Sunday, November 29, 2020 12:45:15
Re: src/sbbs3/main.cpp
By: Rob Swindell to Git commit to main/sbbs/master on Sun Nov 29 2020 12:31 am
another unused node and you (or that user) will get an updated configuration. Bots or server recycling
problems will no longer prevent the sysop or user from getting a current configuration when connecting
to any node.
This is one of those instances of giving the sysop what he wants rather than what he asked for. I
should've done this a long time ago, but the idea just occurred to me. You're welcome. :-)
This is freaking awesome. Thanks DM!
|01bobbobbobbob|09bob|03bob|11bob|03bob|09bob|01bobbobbob |01robrobrobrob|09rob|03rob|11rob|03rob|09rob|01robrobrob
|07
---
Synchronet Halls of Valhalla =San=Francisco= Happy Holidays
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, December 06, 2020 20:21:13
https://gitlab.synchro.net/main/sbbs/-/commit/d9ea54258ab2a39b3e71bdcf
Modified Files:
src/sbbs3/main.cpp
Log Message:
Don't create a JS "client" object for timed events
There is no actual TCP-client when a timed event is executing, so don't create one in the JS context when the socket is invalid.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, December 13, 2020 02:00:30
https://gitlab.synchro.net/main/sbbs/-/commit/0d8ea19e309c6afd80a587c8
Modified Files:
src/sbbs3/main.cpp
Log Message:
Create node#/client.ini for each node client connection
Paves the way for utilities like umonitor to show client details for node connections (e.g. IP address, hostname, connection duration, etc.)
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Saturday, December 19, 2020 16:09:15
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, February 21, 2021 19:38:20
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Monday, February 22, 2021 00:27:34
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Friday, March 05, 2021 20:41:25
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Monday, March 15, 2021 00:16:56
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Monday, March 15, 2021 00:52:25
https://gitlab.synchro.net/main/sbbs/-/commit/ffb202237f4fde4d79c407a2
Modified Files:
src/sbbs3/main.cpp
Log Message:
Changing timeout didn't help. Use PopData of zero bytes to rekey...
This issue may actually be a rekey issue, to clear this, we need
to call cryptPopData(), but we can pop zero bytes.
In this case, re-try the flush.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Monday, March 15, 2021 01:11:15
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Wednesday, March 31, 2021 16:51:17
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, April 04, 2021 15:13:57
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Monday, April 05, 2021 01:13:34
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Monday, April 05, 2021 01:15:12
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Monday, April 05, 2021 01:25:04
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Wednesday, May 12, 2021 22:53:45
https://gitlab.synchro.net/main/sbbs/-/commit/3202b477e52faa34e6fa5666
Modified Files:
src/sbbs3/main.cpp
Log Message:
Attempt to resolve "!ERROR 11 receiving from socket" reported by plt
<plt> May 13 00:39:40 sbbs synchronet: term Node 1 <plt> append key into keybuf: 4E (N)
<plt> May 13 00:39:40 sbbs synchronet: term Node 1 <plt> Executing external: /sbbs/exec/sexyz 14 -Telnet sz /mnt/disk1/dl/COCO/AGI/agi-xmas.zip
<plt> May 13 00:39:40 sbbs synchronet: term Node 1 <plt> sexyz: Synchronet External X/Y/ZMODEM v3.0 master/23b741a1c Copyright Rob Swindell
<plt> May 13 00:39:40 sbbs synchronet: term Node 1 <plt> sexyz: !File skipped by receiver
<plt> May 13 00:39:40 sbbs synchronet: term Node 1 <plt> sexyz: !File Skipped <plt> May 13 00:39:40 sbbs synchronet: term Node 1 <plt> sexyz: !0 Aborting Transfer (Sending ZABORT)
<plt> May 13 00:39:40 sbbs synchronet: term Node 1 !ERROR 11 receiving from socket 14
<plt> May 13 00:39:40 sbbs synchronet: term Node 1 input thread terminated (received 365 bytes in 35 blocks)
Not sure why this isn't happening for other sysops - perhaps plt was spying on the node?
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, May 16, 2021 16:47:09
https://gitlab.synchro.net/main/sbbs/-/commit/4a705a4667fea6e699d7965a
Modified Files:
src/sbbs3/main.cpp
Log Message:
Log local address in passthru connection failure
Altere reported via IRC:
!ERROR -1 (113) connecting to passthru socket
It might be helpful to know the address the passthru socket is bound to and the connection is being attempted on, so log that with the error.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, May 16, 2021 18:13:51
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Tuesday, June 08, 2021 21:59:20
https://gitlab.synchro.net/main/sbbs/-/commit/31149a208abdd6cf12317cb4
Modified Files:
src/sbbs3/main.cpp
Log Message:
Always use rename() on *nix when mv() is called with copy=FALSE
This should fix issue #272 reported by Ragnarok. Not the libarchive error part, but the second (mv) CopyFile error part.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Friday, June 11, 2021 11:29:25
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Wednesday, January 19, 2022 01:31:29
https://gitlab.synchro.net/main/sbbs/-/commit/26bc17de8f51f3e34977bd82
Modified Files:
src/sbbs3/main.cpp
Log Message:
Add run-time check for non-terminating/unsafe snprintf() function
Only in debug builds.
If this check fails, then we may be producing/using potentially un-terminated strings and need to do something about that. By default, xpdev/genwrap.h redefines snprintf() as safe_snprintf(), so this check *should* always succeed.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, February 06, 2022 02:34:55
https://gitlab.synchro.net/main/sbbs/-/commit/dbf0a87ad95f01a4f88a324d
Modified Files:
src/sbbs3/main.cpp
Log Message:
Clear mouse hot spots (if any) before hanging up.
<nelgin> Ah, here's a good one. If I ssh to my bbs server, then ssh to my bbs, do whatever then /O to logout, when I left click in the putty window I get "0;98;20M0;98;20m and stuff like that. I have to run reset to get it working properly.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Monday, February 21, 2022 15:59:31
https://gitlab.synchro.net/main/sbbs/-/commit/b269b65ad5d09814abbbff51
Modified Files:
src/sbbs3/main.cpp
Log Message:
Refactor timed-event and QWKnet call-out scheduling
Reduced probably the biggest if() conditionals in sbbs to a single line by breaking the "time to run" logic into separate functions and sharing those functions between both QWKnet and timed-event scheduling. There was no actual problem with this code/logic, it was just very difficult to read and understand and step-through with a debugger and understand why or why not an event might run under different configurations and circumstances.
Also removed the PostLink network call-out logic. pnet.dab is no longer read and written-to and if you happened to have any PostLink hubs configured (how?!?), they'll no longer be "polled". This is the only functional change unless I did something wrong in the process.
One thing I noticed and contemplated, the current time is not queried between consecutive timed-event scheduling/execution. It's possible that an executed event can take a long time and impact the criteria for the next timed event. The events are checked for scheduling every few seconds, so I can't really think of a big down-side to the current design (apparently intended to reduce unnecessary querying of the current date/time), so I didn't do anything to change that. Just something I noticed.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Wednesday, February 23, 2022 23:25:45
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, February 27, 2022 21:50:18
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Tuesday, March 01, 2022 22:43:07
https://gitlab.synchro.net/main/sbbs/-/commit/b9e78bdf93e4d9b3e7608e4d
Modified Files:
src/sbbs3/main.cpp
Log Message:
Batch download queue not cleared upon packed QWK packet
Before the "new filebase" merge of 2021, a user's batch download queue would be cleared when a QWK packet was successfully packed for them via the event thread (i.e. triggered via semaphore file). That was lost in the new filebase change (the batch download queue would not be cleared) - caught by Coverity (CID 350336).
It appears pre-packed QWK packets never cleared the batch download queue (does anyone actually use this feature in this day and age?) - fixed.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, March 03, 2022 16:45:56
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, March 13, 2022 00:26:38
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, March 31, 2022 00:11:44
https://gitlab.synchro.net/main/sbbs/-/commit/141112e104ee22552140d82f
Modified Files:
src/sbbs3/main.cpp
Log Message:
Lower level of new "Logoff time <= logon time" log message
... from ERROR to WARNING, but log the human-readable logon time too.
This condition can easily happen if the system clock is corrected (e.g. a few seconds via NTP adjustment) during a quick connect/disconnect.
Don't call logoffstats() unless a user actually successfully logged-in. I don't think sysops actually want all the time bots sit at the logon prompt counted as "time online" in their stats anyway.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, March 31, 2022 18:59:38
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, April 14, 2022 14:59:08
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Friday, April 15, 2022 16:09:33
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, July 10, 2022 14:26:22
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Friday, September 16, 2022 16:46:29
https://gitlab.synchro.net/main/sbbs/-/commit/b91acb61c474ed71a1fa3a2e
Modified Files:
src/sbbs3/main.cpp
Log Message:
Fix crash when cleanup() was called twice in a row, e.g. load_cfg failure
Need to nullify the xpms after destroying/freeing (as is done in other
servers already).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Saturday, November 19, 2022 13:20:40
https://gitlab.synchro.net/main/sbbs/-/commit/d1f4767a9597b659d7996e83
Modified Files:
src/sbbs3/main.cpp
Log Message:
Correct node number *always* before clearning NODE_RRUN [R] status flag
If nodeX/node.cnf has the wrong node number, the [R] flag would be cleared from the wrong node upon automatic configuration reload. This likely fixes the issue that Keyop is reporting in #synchronet
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Thursday, December 29, 2022 13:54:23
https://gitlab.synchro.net/main/sbbs/-/commit/56d1e7fe4b3c6bf6176e4406
Modified Files:
src/sbbs3/main.cpp
Log Message:
Output snappiness improvements.
When the system checks for an incoming byte, if there's any pending
output, send it immediately, bypassing the outbuf highwater mark.
This allows the final non-full packet to be sent without waiting
for the OutbufDrainTimeout (default 10ms), and makes a big difference
when doing a large number of ANSI queries (send an ANSI code, wait
for a response). There's a small but noticable placebo effect as
well that makes everything feel smoother.
This commit also cleans up the output thread where it pulls from
the ring buffer into the linear buffer to take advantage of the new
event-based ring buffers. Much easier to read now.
Speaking of easier to read, this also includes some whitespace
fixups.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Thursday, December 29, 2022 17:14:12
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Thursday, December 29, 2022 18:49:37
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, December 29, 2022 19:36:44
https://gitlab.synchro.net/main/sbbs/-/commit/dfc4e9320299020943ea640c
Modified Files:
src/sbbs3/main.cpp
Log Message:
Eliminate the global 'sbbs' here, was masking bugs
I'm not sure what purpose this global variable once had, but it wasn't needed now and it was masking copy/paste issues in some of the sbbs_t methods in this file. Just make an 'sbbs' local to bbs_thread(), since there's where it's legit-used.
FYI: All the Cryptlib-SSH macros assume there's an 'sbbs' in scope.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, December 29, 2022 19:36:44
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, December 29, 2022 19:38:24
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Friday, December 30, 2022 01:16:32
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Friday, December 30, 2022 03:37:24
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, January 01, 2023 20:28:10
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Tuesday, January 03, 2023 21:41:33
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Saturday, January 21, 2023 13:15:16
https://gitlab.synchro.net/main/sbbs/-/commit/6426954c87ae1bd5c04691e4
Modified Files:
src/sbbs3/main.cpp
Log Message:
Don't attempt to remove a non-existing qwk-pack semfile
Reduce occurrences of subsequent errors (unsure of root-cause):
!ERROR 2 (No such file or directory) in main.cpp line 2778 (event_thread) removing "/sbbs/data/pack0341.now"
This does seem to be the only location that deleted pack*.now files, so weird that they'd disappear sometime after calling glob(), but I do understand that its theoretically possible.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Tuesday, January 24, 2023 21:30:09
https://gitlab.synchro.net/main/sbbs/-/commit/5f7f1104db341b7c52e4e209
Modified Files:
src/sbbs3/main.cpp
Log Message:
Don't try to upgrade */csts.dab and */dsts.dab files that don't exist
In a fresh v3.20 install, the legacy stats files don't exist, so don't try
to convert/upgrade them and log errors about it.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Thursday, February 02, 2023 13:30:13
https://gitlab.synchro.net/main/sbbs/-/commit/d60262aa81e4b14405e6fd57
Modified Files:
src/sbbs3/main.cpp
Log Message:
Lower severity of "Spy socket ... connected" log message from ERR (?) to INFO
Looks like just a confluence of search/replace errors over the past 19 years, starting with commit 9ef382e50c6.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Debian Linux)@VERT to
Git commit to main/sbbs/master on Friday, February 10, 2023 21:52:17
-
From
Rob Swindell (in GitKraken)@VERT to
Git commit to main/sbbs/master on Sunday, March 12, 2023 18:21:16
https://gitlab.synchro.net/main/sbbs/-/commit/9eeb09f19514e1ccf10d85eb
Modified Files:
src/sbbs3/main.cpp
Log Message:
Fix PETSCII 40/80 column port connections for IPv6
The connected TCP port detection method only worked for IPv4, so automatically detecting a CBM/PETSCII connection over IPv6 didn't work. Thanks to Deuce's xp_sockaddr and helper functions, this was an easy change.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Debian Linux)@VERT to
Git commit to main/sbbs/master on Sunday, May 07, 2023 13:45:08
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Friday, June 02, 2023 17:49:52
https://gitlab.synchro.net/main/sbbs/-/commit/ca7ab040466b030281a9aaca
Modified Files:
src/sbbs3/main.cpp
Log Message:
Add a 60-second timeout to sbbs_t::passthru_socket_activate()
Keyop reported an issue via irc whereby a user that failed to download a file would leave the node "hung" in "downloading via telnet" node status even
though the user had long since disconnected and the log reflected that the terminal server was aware of this:
term Node 4 <user> sexyz: !1152 zmodem_recv_raw TIMEOUT (10 seconds)
term Node 4 <user> sexyz: !zmodem_recv_header TIMEOUT
term Node 4 <user> external Timeout waiting for output buffer to empty
<minutes later>
term Node 4 connection reset by peer on send
term Node 4 !ERROR 32 sending on socket 102
term Node 4 !ERROR 32 sending on socket 102
term Node 4 !ERROR 32 sending on socket 102
term Node 4 !ERROR 32 sending on socket 102
term Node 4 !ERROR 32 sending on socket 102
term Node 4 disconnected
term Node 4 !ERROR 32 sending on socket 102
and
term Node 3 <user> sexyz: !1152 zmodem_recv_raw TIMEOUT (10 seconds)
term Node 3 <user> sexyz: !zmodem_recv_header TIMEOUT
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !Receive timeout (1 seconds)
term Node 3 <user> sexyz: !1152 zmodem_recv_raw TIMEOUT (10 seconds)
term Node 3 <user> sexyz: !zmodem_recv_header TIMEOUT
term Node 3 <user> external Timeout waiting for output buffer to empty
<minutes later>
term Node 3 connection reset by peer on receive
term Node 3 !ERROR 32 sending on socket 96
These nodes were then locked up in call to passthru_socket_activate(false)
as reported by gdb, e.g.
Looking at passthru_socket_activate(), the deactivation path (called at the
end of external() in this case), it was clear that this could be an infinite loop in the case the user had disconnected:
do { // Allow time for the passthru_thread to move any pending socket data to the outbuf
SLEEP(100); // Before the node_thread starts sending its own data to the outbuf
} while(RingBufFull(&outbuf));
These flush/purge loops aren't strictly needed if the user has disconnected, but as can be seen by the above logs, the terminal server may not know that (the socket may not indicate disconnect) before passthru_socket_activate()
is called by external().
So... worst case, just do the activation and deactivation buffer flushes
and purges for 60 seconds.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Wednesday, September 13, 2023 18:56:11
https://gitlab.synchro.net/main/sbbs/-/commit/23cd780b3dc51f1d55398748
Modified Files:
src/sbbs3/main.cpp
Log Message:
Remove redundant timestamp from "End of preexisting log entry" log message
Most logs sysops are looking at already have a date/timestamp for each
message, so this one ended up with 3 date/timestamps in a single line.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Tuesday, September 19, 2023 18:37:16
https://gitlab.synchro.net/main/sbbs/-/commit/e5f64faf337040f12844c4cc
Modified Files:
src/sbbs3/main.cpp
Log Message:
Log a critical error message if a node is WFC, but its socket is still in use
This should avoid/work-around issue #630
Nelgin's terminal server log indicated that the node's configuration was being reloaded for a new client connection (which doesn't normally/always happen, only when there's been a config file change or a node "rerun" requested),
while there was still a user online using that node. This would explain the segfault (configuration fields being freed and reallocated).
The node_socket[] array indicates when a node thread is actively servicing
a client, so it should contain INVALID_SOCKET for the any node's that WFC.
If it doesn't, that means something/someone changed the node's status value
to 0 (WFC) in the node.dab file while the node was still in-use. Let's not crash in this case and rather just log the message and give the client the
next available node (if any).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Sunday, September 24, 2023 01:32:43
https://gitlab.synchro.net/main/sbbs/-/commit/229cca918d31e14add4b2ef7
Modified Files:
src/sbbs3/main.cpp
Log Message:
Close node socket after waiting one hour to go inactive to run exclusive event
After 90 minutes of waiting, we'll do the same abort wait (and run the event anyway), but closing the node's socket should be enough to get the node_thread to terminate and set the node status back to NODE_WFC.
Apparently some sysops like to leave their terminals idling (e.g. running MRC) and never disconnect and since they're T-exempt, the BBS won't limit their
time online to allow events to run. Exclusive events will wait for all nodes
to become inactive, but give up after 90 minutes of waiting and run the event anyway and set node status to WFC at the end. If the node was actually still connected/in-use, this could lead to the (new) critical error messages logged "!Node X status is WFC, but the node socket (N) and thread are still in use!" and other chaos (NODE STATUS FIXUP and the like).
This should prevent all that by just abruptly disconnecting the node after waiting 60 minutes for the sysop to gracefully disconnect. The log message
when this happens:
"!TIRED of waiting for node N to become inactive (status=X), closing socket Y"
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Monday, September 25, 2023 18:07:33
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Friday, December 29, 2023 10:17:20
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Thursday, January 04, 2024 19:19:36
https://gitlab.synchro.net/main/sbbs/-/commit/7df6956da2b4246a9ca3940e
Modified Files:
src/sbbs3/main.cpp
Log Message:
Don't log an error if there's QWKnet sub pointer value to be read
This partially reverts commit 44c3350f where a bunch of read() and write() return value checking was added.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Debian Linux)@VERT to
Git commit to main/sbbs/master on Thursday, January 11, 2024 00:33:48
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Wednesday, January 17, 2024 13:31:24
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Saturday, January 20, 2024 20:25:21
https://gitlab.synchro.net/main/sbbs/-/commit/e53f173153fdaf0341cca95c
Modified Files:
src/sbbs3/main.cpp
Log Message:
Remove unnecessary call to DuplicateHandle() in Win32 builds of sbbs_t::init()
The client_socket_dup is overwritten after with the return value of accept(), so this was likely a socket/handle leak (since commit 74470573, 17 years ago).
Noticed this only after logging a new error in an SSH connection when this
call to DuplicateHandle() failed for some reason (WinError 6), but it seems like it's been redundant code for a long time now. So just ace it and possibly fix a resource leak.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Monday, January 22, 2024 17:43:44
https://gitlab.synchro.net/main/sbbs/-/commit/346a2d304c7d921d656489ad
Modified Files:
src/sbbs3/main.cpp
Log Message:
Log the socket descriptor and protocol when getsockname() fails
This is a new error that just started cropping up. I suspect it's related to recent SSH changes. This might help a little with context, when it happens again.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Tuesday, February 06, 2024 12:35:50
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Wednesday, February 07, 2024 12:23:42
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Wednesday, February 14, 2024 15:11:56
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Saturday, February 24, 2024 14:54:24
-
From
Rob Swindell (on Debian Linux)@VERT to
Git commit to main/sbbs/master on Saturday, February 24, 2024 20:31:04
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Monday, February 26, 2024 22:41:59
https://gitlab.synchro.net/main/sbbs/-/commit/a474017fb7c51f91ef9c4095
Modified Files:
src/sbbs3/main.cpp
Log Message:
sbbs_t::incom() should return NOINP immediately if/when socket disconnected
JS console.inkey() and friends use this as the underlying function to get
input from the TCP socket.
This might address issue #725
Hoping Nelgin will let us know.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Tuesday, February 27, 2024 20:20:48
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Wednesday, February 28, 2024 15:29:55
https://gitlab.synchro.net/main/sbbs/-/commit/e4adbcb541c6f7139e63d2ca
Modified Files:
src/sbbs3/main.cpp
Log Message:
Prevent the input thread from closing channel that has a session,
but does not yet have a session type.
Also, fix what appears to be a fairly blatant memory leak.
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Thursday, February 29, 2024 11:55:23
https://gitlab.synchro.net/main/sbbs/-/commit/346849ac253a08374b9bfca0
Modified Files:
src/sbbs3/main.cpp
Log Message:
Unlock ssh_mutex before calling sftps_recv()
This should fix the sftp hangs...
While FreeBSD will return EDEADLOCK when a mutex would deadlock,
Linux will do what I ask and deadlock.
Since we're not checking the return value of pthread_mutex_lock(),
this would go completely unnoticed on FreeBSD, resulting in the
mutex being unlocked early and all sorts of unprotected accesses
would happen, potentially doing crazy things. On Linux it just
deadlocks, and taking a peek at the thread backtraces makes the
problem obvious.
So, point to Linux for making life more correct for people who
do incorrect things (locking without checking the return value).
I'd also like to thank nelgin, for sticking with me on this issue
and being my gdb puppet.
And really, this commit should be shared by all the contributors
who...
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Thursday, March 21, 2024 23:34:07
-
From
Deuc@VERT to
Git commit to main/sbbs/master on Thursday, March 21, 2024 23:38:26
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Friday, March 22, 2024 21:28:05
https://gitlab.synchro.net/main/sbbs/-/commit/be8603b11bd6cf12c6f74256
Modified Files:
src/sbbs3/main.cpp
Log Message:
Replace (and auto-upgrade) ctrl/time.dab with time.ini
This will allow us to change scfg_t.event[].last to a time_t value (from time32_t) whenever we abandon Borland C/C++ in the future.
Now it's trival for sysops to view/edit when their timed events 'last run' values. And there's no more endianness concerns. Bad .dab files, bad, go away.
After I wrote this change, I found the original conversion code in v4upgrade.c which is almost exactly the same (except I was going to write to different filename). Finally actually implemented this. Next up: qnet.dab
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Friday, March 22, 2024 21:28:05
https://gitlab.synchro.net/main/sbbs/-/commit/03be991a6f30b98ae9c0320b
Modified Files:
src/sbbs3/main.cpp
Log Message:
Convert (and auto-upgrade) ctrl/qnet.dab to ctrl/time.ini as well
I suppose this should've been part of the previous commit.
Again, this is almost verbatim what v4upgrade.c did (though never actually used).
---
Synchronet Vertrauen Home of Synchronet [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Debian Linux)@VERT to
Git commit to main/sbbs/master on Friday, March 22, 2024 22:50:36
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Monday, April 08, 2024 16:32:54
-
From
Rob Swindell (on Debian Linux)@VERT to
Git commit to main/sbbs/master on Wednesday, April 10, 2024 23:07:19