Synchronet Git Commit Log

Modified Files:


exec/load/dd_lightbar_menu.js
diff

Made a fix (kludge?) to properly write menu items with the check character
(multi-selected) in a borderless menu (vs. a menu with borders).

Modified Files:


exec/load/dd_lightbar_menu.js
diff

Merge branch 'ddLightBarMenuMultiSelectDrawFix' into 'master'

dd_lightbar_menu: Made a fix (kludge?) to properly write menu items with the
check character

See merge request main/sbbs!127

Modified Files:


3rdp/build/cl-poll-not-select.patch
diff

Remove check for fd <= FD_SETSIZE

This was preventing cryptlib from working with socket descriptors
over FD_SETSIZE despite being patched to use poll() to avoid the
issue it's protecting against.

May fix the various SSH/SSL internal error issues.

Modified Files:


src/sbbs3/sbbs3.sln
diff

Don't build sbbsexec.dll by default for debug builds

This build is touchy and requires an explicit path to a Win10 SDK
directory, so just disable by default for (for debug build). We still
want this built by default for CI, which are release builds.

Modified Files:


src/sbbs3/ftpsrvr.c
diff

Fix FTPS upload failure: !DATA ERROR 0 receiving on data socket

Don't treat CRYPT_ERROR_COMPLETE (-24) as a socket error during upload since
it's an indication that the remote closed the connection and is the normal "end
of file/transfer" indicator, not an error. 'rd' is already 0 in this case, so
no need to set at all (since recv() returns 0 upon disconnect and that's what
we're emulating here).

Fixes issue #309 reported by Jas Hud.

Modified Files:


src/sbbs3/sexyz.c
diff

src/sbbs3/xmodem.c
diff

Allow local abort/cancel to abort (more) retries/loops

While debugging a SyncTERM upload issue, I came across some scenarios where an
out of control file transfer could not be aborted via local key press (e.g.
Ctrl-C, Ctrl-Break).

We use zm.local_abort for all local abort indications (even for X/YMODEM).
<shrug>

Modified Files:


src/xpdev/genwrap.c
diff

Work around GetVersionEx deprecation warning/error

For some reason this deprecation warning is being treated as an error (and only
with newer Windows platform SDKs). Just disable this warning for the
GetVersionEx line.
See https://stackoverflow.com/questions/22303824/warning-c4996-getversionexw-wa
s-declared-deprecated for details.

Added Files:


src/sbbs3/sbbsexec.vcxproj
diff

Modified Files:


src/sbbs3/sbbs3.sln
diff

Add sbbsexec.dll project (again)

Regenerated for MSVC2019, the Synchronet virtual UART/FOSSIL driver (VDD) can
be built from Visual Studio again (the makevdd.bat can probably go away now). I
haven't had an MSVC project to build this DLL since the old MSVC 6.0
sbbsexec.dsp file.

A few oddities about this project:
1. xpdev files had to directly included/compiled in this project rather than
linking with the xpdev_mt.lib because __stdcall calling convention is required
for compatibility with ntvdm.lib and xpdev_mt.lib is built using __cdecl
calling convention (the default). Rather than re-introducing the calling
convention game-playing that I just recently removed (e.g. #define DLLCALL),
just build the required xpdev files directly as part of this project. Perhaps
there is a more elegant solution that I'll revisit later.

2. The absolute path of my install Windows 10 SDK directories had to be
specified (for include and library paths) to find vdd_svc.h and ntvdm.lib.
Weird thing is that I didn't have to do anything hacky like this with the
command-line/batch file build method. Again, there's probably a better solution
I'm not seeing. But as is, this project might not build on other systems.

So why use this rather than the brute-force batch file (makevdd.bat)? Well,
this should be a little easier to get into CI/nightly builds and the resulting
release build was 30KB (instead of 150KB), so that's good.

Before:
           1AA00 size of code
            AA00 size of initialized data
               0 size of uninitialized data
After:
            4C00 size of code
            2A00 size of initialized data
               0 size of uninitialized data

Those default DLL project build options include some good optimization flags
apparently.

Modified Files:


src/sbbs3/dosxtrn/dosxtrn.c
diff

src/sbbs3/sbbsdefs.h
diff

src/sbbs3/sbbsexec.c
diff

src/sbbs3/scfg/scfgxtrn.c
diff

src/sbbs3/vdd_func.h
diff

src/sbbs3/xtrn.cpp
diff

src/sbbs3/xtrn_sec.cpp
diff

Removed Files:



src/sbbs3/execvxd.h
diff

src/sbbs3/execvxd/c.bat
diff

src/sbbs3/execvxd/debugout.h
diff

src/sbbs3/execvxd/makefile
diff

src/sbbs3/execvxd/sbbsexec.cpp
diff

src/sbbs3/execvxd/sbbsexec.h
diff

src/sbbs3/execvxd/sbbsexec.vrc
diff

Remove the last remnants of Win9x/Me support

Allow "FOSSIL Only" as a configuration option for external programs in SCFG.
This is a much easier method of disabling the virtual UART support in the
Virtual UART/FOSSIL device driver (if/when necessary) than editing the
sbbsexec.ini file (which is still supported).

Modified Files:


src/sbbs3/sbbsexec.c
diff

The sem_wait() call was just a bad idea after-all

The sem_wait() call from vdd_read() that was fixed in the previous commit to
this file was a bad idea to begin with:
There is (often) not a 1:1 ratio of bytes in the receive ring buffer and the
semaphore count, so we'd often halt here waiting for the semaphore to be
signaled even though there were still characters in the receive buffer. This
caused (new) stalls/hangs in keyboard input in DOS door games. We didn't see
this previously because the sem_wait() call just didn't work (wrong semaphore
pointer value).

Now, call sem_trywait_block() in vdd_read(), but only if the RingBufRead()
returned 0 (no bytes read) and even then, only block/wait for a maximum of 30
seconds (same as X00 FOSSIL driver). This blocking behavior is specified for
FOSSIL function 02h (Get received character with wait) and would not have
worked previous to the previous "fix", so now we have that corrected behavior
too. The FTSC spec says the wait is indefinite, but I think a 30 second timeout
(ala X00) is more reasonable.

Log the git brach/hash instead of the old CVS revision.

GetMailSlotInfo() apparently always fails on Windows 7 with error 87 (even
using Microsoft's own sample code), so lower the log message severity of that
failure from ERR to DEBUG. Apparently this is pretty harmless anyway as nothing
really needs the write-mailslot status. <shrug>

Modified Files:


src/sbbs3/dosxtrn/dosxtrn.c
diff

src/sbbs3/fossdefs.h
diff

Fix some FOSSIL functions

Function 0Ch (peek) never worked correctly (character was not returned in AX);
I guess nothing uses this function (?).

Function 01h (transmit char w/wait) now will set the timeout flag (0x8000) in
the AX if the VDD_WRITE function fails (returns value other than 1).

Optimize Function 18h (read block) - no need for the extra VDD_STATUS call.

Use newly defined FOSSIL macros instead of magic numbers of port status bits.

Log the git branch/hash values to the Windows debug log (via sbbsexec.dll).

Modified Files:


src/sbbs3/chat.cpp
diff

Add a debug line when wrapping split-screen chat lines.

Is this the point when the lost chars problem happens?

Added Files:


src/sbbs3/dosxtrn/.gitignore
diff

Modified Files:


src/sbbs3/.gitignore
diff

Ignore sbbsexec.dll and dosxtrn.exe build artifacts.

Modified Files:


install/GNUmakefile
diff

Fix typo in condition (if linux) for setting the SETCAP/setcap target

As reported via irc by Nelgin, the setcap target was not being passed
to 'make -C src/sbbs3' by default (on Linux) due to a misplaced paren.

Modified Files:


src/sbbs3/ringbuf.h
diff

Finally get rid of RINGBUFCALL (finish what was started in April of this year)

Missing part of commit 4da964dd8ff83256df5755e525a0f3f42b1f30d3

Added Files:


src/sbbs3/makevdd.bat
diff

A batch file to build SBBSEXEC.DLL (may or may not require NTDDK?)

Using options (e.g. RINGBUF_MUTEX) from old sbbsexec.dsp

Modified Files:


src/sbbs3/dosxtrn/dosxtrn.c
diff

src/sbbs3/sbbsexec.c
diff

src/sbbs3/xtrn.cpp
diff

Add support for NTVDMx64

Yes, you can run 16-bit DOS doors on 64-bit (x64) Windows now.

Install NTVDMx64 (http://www.columbia.edu/~em36/ntvdmx64.html, it's not as
onerous as it sounds) and re-enable DOS program support in SBBS (i.e. make sure
"NO_DOS" is not in your sbbs.ini [bbs] Options value) and voila: DOS doors
work.

This change adds an empty init routine to sbbsexec.dll since older versions of
NTVDM (which NTVDMx64 is based on) required one. Also, the sbbsexec.dll should
be located in your Synchronet "exec" directory when using NTVDMx64 (in addition
to or instead of your Windows/System32 directory).

Modified Files:


src/sbbs3/sbbsexec.c
diff

Fix sem_wait() call in vdd_read()

Introduced in Revision 1.16 (CVS)
Wed May 10 08:52:11 2006 UTC (15 years, 6 months ago) by rswindell

This was supposed to wait (block) until there was a new received
byte (the ringbuf semaphore was signaled), but we were passing the sem_t
value rather than the pointer to the sem_t. Since sem_t (on Windows)
is a HANDLE (which is a void*), there was no compiler warning or
error. Type-safety in C sucks.
So, this just never worked (I'm assuming the call would just fail).

I discovered this bug while working on NTVDMx64 support and in that
case, this call would block/wait forever. Simple one character fix.

Modified Files:


xtrn/bullshit/bullshit-lib.js
diff

Don't crap out if [files] section empty or missing.

Modified Files:


exec/websocketservice.js
diff

FWebSocketHeader should just be a bare object.

Modified Files:


exec/websocketservice.js
diff

Case-insensitive search for headers in ShakeHands. Some proxies
like to mangle these unless you tell them not to.

Capture the value of the X-Forwarded-For header, if present, and
stash it in FWebSocketHeader['X-Forwarded-For'].

Modified Files:


exec/load/ftelnethelper.js
diff

Added UsingHAProxy() function to this here funclib dot js

Modified Files:


exec/websocketservice.js
diff

If BBS is using HAProxy ([BBS] Options line has HAPROXY_PROTO)
then throw an error if X-Forwarded-For header was not received,
or otherwise do the HAProxy thing upon connecting.

Modified Files:


exec/websocketservice.js
diff

Avoid spurious connections to the terminal server - and some
other unnecessary work - if the BBS is using HAProxy but we can't
do the thing with the stuff because we don't have the things.

Modified Files:


exec/websocketservice.js
diff

Added inet_pton function borrowed from the internet.
Added incomplete send of haproxy protocol header. Need to sort
out lookup of target (BBS) IP address.

Modified Files:


exec/websocketservice.js
diff

Hurr.

But also, this needs some revision since I'm supposed to send
the header all at once and probably shouldn't be doing a bunch
of individual sendBin calls.

Modified Files:


exec/websocketservice.js
diff

Build protocol header as string and send all in one big barforama.

Modified Files:


exec/websocketservice.js
diff

Use client address from header

Modified Files:


exec/websocketservice.js
diff

Address family must occupy bytes 15 & 16

Modified Files:


exec/presence-service.js
diff

Changed the format of some messages.

Modified Files:


src/sbbs3/dosxtrn/dosxtrn.c
diff

Implement FOSSIL function 0Dh (Keyboard read without wait)

As a work around for TradeWars 2 (v11.14 and v11.20) FOSSIL support:
This door game would never call the FOSSIL "get received character"
function (02h) . But I did notice that it was calling the "Keyboard
read" functions. I never implemented the "Keyboard read" functions
(figured the door game could read the keyboard itself if it wanted to),
but appparently TW2 won't ever read the COM port (using function 02h)
unless the Keyboard read function returns AX=0xFFFF (no keyboard data
available). So now, I just do that and TW2 works (with FOSSIL I/O).

Incremented the revision to 26.
Logging the Git branch/hash and other helpful FOSSIL-debug stuff now.

Modified Files:


src/sbbs3/xtrn_sec.cpp
diff

Continuation of previous commit (support for UART-only doors on Windows)

Needed the actually mapping of XTRN_UART -> EX_UART mode flag here.

Modified Files:


src/sbbs3/execvxd.h
diff

src/sbbs3/sbbsdefs.h
diff

src/sbbs3/scfg/scfg.h
diff

src/sbbs3/scfg/scfgchat.c
diff

src/sbbs3/scfg/scfgxtrn.c
diff

src/sbbs3/xtrn.cpp
diff

Allow FOSSIL mode of the Win32 virtual UART/FOSSIL driver to be disabled

When configuring an external program, a Win32 sysop can now choose to *only*
enable the virtual UART (and not the FOSSIL driver) feature of the Synchronet
virtual UART/FOSSIL driver (dosxtrn.exe/sbbsexec.dll). Some programs (e.g.
TradeWars 2) will always use the FOSSIL driver if one is detected and in order
to force the use of COM/UART driver for I/O (if that is desired), then one must
disable the FOSSIL driver. It's a rare use case, but I can see the potential
need.

Renamed the "Intercept I/O" option in SCFG to "I/O Method" and make it clear
that "Socket" (for native programs) and "FOSSIL or UART" (for 16-bit DOS
programs) is the default method (what was previously identified as "No" I/O
Interception).

If a sysop want to disable the virtual UART support (on Win32, e.g. so *only*
FOSSIL is available to one or all DOS programs), they do that via their
sbbsexec.ini file.

Modified Files:


src/xpdev/gen_defs.h
diff

MSVC 1.5c defines _MSDOS, not __MSDOS__

This change needed to build dosxtrn.exe (again).

Modified Files:


src/sbbs3/dosxtrn/make.bat
diff

Added helpful reminder remark to build this old beast

Modified Files:


src/sbbs3/sbbsecho.c
diff

Resolve gcc warning

warning: format ‘%lx’ expects argument of type ‘long unsigned int’

Modified Files:


ctrl/text.dat
diff

Add All and ListKey strings.

Modified Files:


src/sbbs3/atcodes.cpp
diff

src/sbbs3/bat_xfer.cpp
diff

src/sbbs3/data.cpp
diff

src/sbbs3/download.cpp
diff

src/sbbs3/email.cpp
diff

src/sbbs3/getkey.cpp
diff

src/sbbs3/getstr.cpp
diff

src/sbbs3/listfile.cpp
diff

src/sbbs3/netmail.cpp
diff

src/sbbs3/qwk.cpp
diff

src/sbbs3/sbbs.h
diff

src/sbbs3/text.h
diff

src/sbbs3/text_defaults.c
diff

src/sbbs3/upload.cpp
diff

src/sbbs3/useredit.cpp
diff

src/sbbs3/writemsg.cpp
diff

Add text.dat lines for "All" and the List key ('L'), used in quoting

The internal line editor's quoting feature add some hard-coded strings ("Done"
and "All") and the (L)ist key was hard-coded. Use the text.dat string (new and
pre-existing) for these now.

Also, use the new sbbs_t *_key() methods for referencing the configured key
bindings (via text.dat) for these common key-stroke commands.
Convert the text.dat strings for keys (e.g. YNQP) to uppercase always as well.

Modified Files:


ctrl/text.dat
diff

Part of previous commit: new text strings for msg/file scan config prompt/keys

Modified Files:


src/sbbs3/sbbs.h
diff

src/sbbs3/scandirs.cpp
diff

src/sbbs3/scansubs.cpp
diff

src/sbbs3/text.h
diff

src/sbbs3/text_defaults.c
diff

Make msg/file scan configuration prompts and keys configurable

For localization purposes, no more hard-coded text strings or characters in the
new-file and file scan configuration menus.

This change fixes issue 232.

A Hungarian sysop also requested this change via Facebook not long ago.

Modified Files:


src/sbbs3/scfgdefs.h
diff

src/sbbs3/ssl.c
diff

Don't use time_t in scfg_t definition

sbbsctrl.exe is built with Borland C++ still which only has a 32-bit time_t, so
to keep the scfg_t definition compatible between MSVC and Borland builds, use
time32_t (32-bit time_t) values only.

This fixes the reported sbbsctrl.exe error:
Error loading configuration
cfg->size (23944) != sizeof(scfg_t) (23952)

Introduced in commit e535aaac.
Reported via DOVE-Net by  Daryl Stout (TBOLT), confirmed by Nightfox (DIGDIST).

Modified Files:


src/sbbs3/mailsrvr.c
diff

Include twit-listed sender name in quotes in log message

Since mail server log messages have their white-space condensed, it was not
obvious why some sender's name would match a twitlist.cfg line that filters
names beginning with a space: "\ *" because the initial space of the sender's
name was condensed/combined with space before it in the log message.

Modified Files:


src/sbbs3/qwktomsg.cpp
diff

Initialize time zone variable (zone)

As found and reported by Talisam author, apam (HAPPYLND).

sscanf() won't initialize the buffer of the argument passed if the format
string is not matched, so when there was no hexadecimal/SMB timezone value, the
timezone specified in the ISO date/time string (if any) would be overwritten
with 0, converted from the uninitialized 'zone' variable (unlikely to be valid
hex-ASCII chars). Thanks apam!

Modified Files:


src/sbbs3/qwk.cpp
diff

src/sbbs3/sbbsecho.c
diff

Add support for "name@address" syntax in twitlist.cfg

where "address" is a QWKnet ID or path or a FidoNet originating address.
Wildcards can be used (e.g. "*@VERT").

For Trikester.

Modified Files:


src/sbbs3/filedat.c
diff

src/sbbs3/xtrn.cpp
diff

Add command-line specifier for user's real name (auto-quoted)

%+ will now expand to the current user's real name, automatically enclosed in
quotes if it contains any spaces.

Modified Files:


src/sbbs3/sbbsecho.c
diff

Generate a MSGID kludge for netmail messages created by SBBSecho

The "serialno" field is a bad/naive one (a time_t value), but this is really
just an experiment for Ragnarok to see if this fixes issue #306 (with hotdoged
receiving AreaFix responses/notices).

Modified Files:


src/sbbs3/upload.cpp
diff

Fix sbbsfile.nam contents: contained file's description instead of name

Found during review of previous commit.
Introduced as part of the new file base branch merge (commit fcf58640).

Modified Files:


src/sbbs3/sbbsecho.c
diff

Log an error and exit immediately when a local linked-node is configured

When a linked-node (in echocfg->Linked Nodes) contains one of the local
system's FTN addresses, display/log an error and exit immediately, e.g.:
Configuration ERROR: Linked node #20 is your own address: 1:103/705

I'd perform this check in EchoCfg, but:
1. echocfg doesn't read the BBS's configuration files (where the local FTN
addresses are configured), and
2. some sysops just directly edit their sbbsecho.ini file

Modified Files:


src/sbbs3/sbbs.h
diff

src/sbbs3/upload.cpp
diff

Don't write uploaded filename and description to env variables

Environment variables are shared among all threads of a process (e.g. all nodes
of a instance of sbbs), so this scheme of passing the uploaded filename and
description to an external file tester (upload processor) via environment
variables has always been fraught with the possibility of failure or flakiness
in Synchronet v3. Since I very much doubt that any upload processor is actually
using this scheme, just remove it. Upload processors can and should)use the
sbbsfile.nam and sbbsfile.des text files created in the node directory instead.

Modified Files:


src/sbbs3/scfgdefs.h
diff

src/sbbs3/ssl.c
diff

Keep track of certificate file timestamp and reload if it has changed

With the old method, it was possible for a certificate to remain used
eternally, and letsyncrypt.js can change it relatively often.

Modified Files:


src/encode/utf8.c
diff

src/xpdev/unicode_defs.h
diff

Re-synchronize utf8_getc/putc with OpenSSL 3.0

Now correctly rejects UTF-8 encoded Unicode surrogates and does not support
5 and 6 byte UTF-8 encodings.

For reference: https://github.com/openssl/openssl/commit/ba64e5a92a6f009e311ad1
c3565817820a1632a4

Modified Files:


exec/rlogin.js
diff

Allow the client-name, server-name, and term-type to be passed as arguments

Optional, for easier use with game servers that take one of the 3 rlogin
negotation parameters as the name/code of the door to execute. A telgate
mode flag value argument must be provided (use 0 for default behavior)
if you want to provide any of the other arguments to override the defaults
(the user's alias, real name, and current detected terminal type).

Modified Files:


src/sbbs3/bat_xfer.cpp
diff

Fix segfault after batch-upload when no "uploads" dir specified

When there's no "uploads" directory configured by the sysop, cfg.upload_dir
will be set to INVALID_DIR, which cannot be used as an index into cfg.dir[] to
determine if the time used for uploading the files should be "given back" to
the user.

In v3.18, we used the first file in the upload queue, if there was one, else
fell back to the "uploads" dir (which had to have been defined if there were no
files in the queue). So this illegal array indexing was a regression in v3.19.

Reported by Zoltán Gábor on Facebook

Modified Files:


3rdp/build/GNUmakefile
diff

Replace $(DIRSEP) with '/' to make this file easier to view/edit

We don't support platforms where '/' doesn't work as a path element separator.

Modified Files:


3rdp/build/GNUmakefile
diff

Replace some apparently errant double-slashes with single-slash

Added Files:


3rdp/build/cl-linux-yield.patch
diff

Modified Files:


3rdp/build/GNUmakefile
diff

Use shed_yield() instead of pthread_yield() on Linux

Per https://man7.org/linux/man-pages/man3/pthread_yield.3.html
	This call is nonstandard, but present on several other systems.
        Use the standardized sched_yield(2) instead.

Should fix #299.

Modified Files:


src/xpdev/genwrap.h
diff

Don't use the deprecated pthread_yield() function

Per https://man7.org/linux/man-pages/man3/pthread_yield.3.html
       This call is nonstandard, but present on several other systems.
       Use the standardized sched_yield(2) instead.

Should fix #299.

Modified Files:


exec/load/ircd/unregistered.js
diff

Don't count unregistered clients

Modified Files:


exec/ircd.js
diff

exec/load/ircd/core.js
diff

Move ircd 1.9 out of beta in anticipation of Synchronet 3.19

Modified Files:


exec/load/ircd/core.js
diff

Bump connect delay from 1s to 10s, thanks to keyop for testing!

Modified Files:


exec/ircd.js
diff

exec/load/ircd/core.js
diff

Only allow one outbound CONNECT at a time

Modified Files:


src/sbbs3/sbbsecho.c
diff

When packing netmail (stored messages), don't discard non-zero zones

As reported by Ragnarok (DOCKSUD) via DOVE-Net, when packing (packetizing)
netmail messages, the source and destination zones from the netmail header were
discarded (since they are defined as "optional" in FTS-1) and the zone values
were replaced with the system's primary/default zone number. If the netmail
message included an INTL kludge, the source and destination zone values would
be parsed from that kludge line and this issue would not be noticed. 

So the fix is: Only if the netmail message header contains a source or
destination zone number of 0, replace that zone with the system's
primary/default zone number. As before, the INTL kludge zone information can
override the header fields.

Also, there's no reason to force the point numbers from the header to zero,
since if they were not valid (e.g. "fill" in the original FTS-1 specification),
they would be already zero. So use the source and destination point numbers, as
is, from the netmail header too (with the FMPT and TOPT kludge override support
left in-tact).

Modified Files:


src/sbbs3/pktdump.c
diff

Print "All good" or "All bad" when trying to recover or split a packet

At Nelgin's suggestion, if all the messages in a packet are either good or bad,
say so (since no .good or .bad file will be generated in that case).

Modified Files:


text/menu/chat.msg
diff

text/menu/maincfg.msg
diff

text/menu/qwk.msg
diff

text/menu/xfercfg.msg
diff

Fix PETSCII display of this menu's title/tab

Modified Files:


text/menu/e-mail.msg
diff

Fix PETSCII display of this menu's title/tab

Added Files:


text/menu/mailread.msg
diff

text/menu/sysmailr.msg
diff

Removed Files:



text/menu/mailread.asc
diff

text/menu/sysmailr.asc
diff

Make 40-column happy versions of mail reading menu (and sysop sub-menu)

Renamed from .asc to .msg for PabloDraw convenience.

Modified Files:


src/sbbs3/atcodes.cpp
diff

New @-codes SEX and GENDERS

SEX is the user's sex/gender
GENDERS is the list of system-supported genders for new users to choose from.

Modified Files:


src/sbbs3/atcodes.cpp
diff

New @-code: !x to perform Ctrl-A style terminal/attribute control

Most Ctrl-A code terminal/attribute effects can now be invoked via @!x@ where x
is one or more Ctrl-A sequence operands (e.g. @!B@ to set the current attribute
to blue, @!P@ to pause the terminal). Multiple attribute codes may be specified
in a single @!x@ sequence. Both upper and lowercase attribute codes are
supported. The "^A@" sequence is not supported, but there's already an @-code
to perform the equivalent operation ("SHOW:20").

Rationale: when using PabloDraw to edit display (e.g. .msg) files, there was no
easy way to explicitly clear (set to "normal") the current attributes at the
end of a line of text. If that line of text contained a background color and
the line caused the terminal to scroll, that background color would fill the
next/new line resulting in some potential undesirable effects. There are 2
common solutions to this problem: either clear the screen before displaying the
file (not always desirable) or clear the attributes at the end of the line
(before the CR/LF). This can now be achieved by adding a @!N@ (the equivalent
of Ctrl-AN for "normal" attributes) to the end of a line. There's no way to
insert custom Ctrl-A sequences or other way to force PabloDraw to clear the
attributes at the end of a line, that I found.

I chose the '!' character because it's (on my keyboard anyway) the Shift-1 key
and Ctrl-A is ASCII 1. I considered a sequence such as @A:x@, but that's one
more character and since this sequence usually won't display anything, a
shorter sequence is likely preferred. For the shortest-possible sequence, use
Ctrl-A sequences instead. However, for many consecutive Ctrl-A sequences, the
@-code equivalent might actually end up being shorter(!). @-codes aren't
support in as many places in Synchronet where Ctrl-A codes are supported (for
security reasons), so it's not like this is going to replace the use of Ctrl-A
codes everywhere.

Modified Files:


src/sbbs3/fmsgdump.c
diff

src/sbbs3/pktdump.c
diff

Don't append the stale static string in fmsgattr_str()

When this function was called multiple times (e.g. multiple messages in a pkt),
it would keep appending to the current (static) string, eventually overflowing
the buffer and corrupting the stack. Although reported in pktdump, it could
have also happened with fmsgdump if passed multiple stored message (.msg)
files.

Fixes #295 reported by Nelgin

Modified Files:


text/menu/multchat.msg
diff

Remove the invisible (black on black) dots

I added them to solve the ANSI screen scroll background attribute fill-new
line issue as I didn't want this menu to clear the screen (there could be chat
lines of text after all).

If this file gets edited with PabloDraw again later, this will have to be fixed
again.

Added Files:


text/menu/multchat.msg
diff

40-column friendly version of the (now defunct) multchat.asc menu.

Removed Files:



text/menu/multchat.asc
diff

Removed in favor of the new multchat.msg

Added Files:


text/menu/main.40col.msg
diff

text/menu/transfer.40col.msg
diff

Rename the 40col menus from .asc to .msg so they appear for CBM/PETSCII terms

The menu file selection priority is like this (for 40col CBM/PETSCII
terminals):
main.40col.seq
main.seq
main.40col.msg
main.msg
main.40col.asc
main.asc

so the main.msg was being chosen before main.40col.asc.

Plus, these files contain ex-ASCII chars, so .msg is actually the more correct
file extension anyway.

Modified Files:


ctrl/text.dat
diff

EnterYourSex -> EnterYourGender

Uses new @code: GENDERS
parts of enhancement #291

Modified Files:


src/sbbs3/atcodes.cpp
diff

src/sbbs3/newuser.cpp
diff

src/sbbs3/scfg/scfgsys.c
diff

src/sbbs3/scfgdefs.h
diff

src/sbbs3/scfglib1.c
diff

src/sbbs3/scfgsave.c
diff

src/sbbs3/text.h
diff

src/sbbs3/text_defaults.c
diff

src/sbbs3/useredit.cpp
diff

Allow sysop to configure new user gender options (not just M/F)

Up to 40 characters can be configured by the sysop for gender options.
The default choice/configuration is now: "M/F/X" (not just "M/F")
New @-code: GENDERS
text.dat EnterYourSex -> EnterYourGender

Closes enhancement request #291 by Ragnarok

Modified Files:


src/sbbs3/email.cpp
diff

Call smb_updatethread() from sbbs_t::email() to perform Keyop "magic"

As discovered by Keyop and reported via IRC, when replying *to* a local mail
box and *not* using the hard-coded email menu (e.g. when when using
msglist.js), the original message's "Replied" attribute flag was not set. When
replying to a netmail address (at least, Internet or FidoNet, unsure about
QWKnetmail), the attribute would be set.

This discrepancy was because this email() function doesn't use the  "modern"
smb_addmsg() function (which also calls smb_updatethread()), but add the
message body text the old/manual way and never called smb_updatethread().
smb_updatethread() does the thread linkage stuff (which is usually unnoticed in
the mail base) *and* sets the "Replied" attribute of the original message, if
it's not already set.

This function should be overhauled to use smb_addmsg(), but this 2-line change
at least addresses this issue for now. There's probably other places (e.g.
importing mail replies via QWK/REP packets) where this thread-linkage and
"Replied" attribute flag setting is still not happening.

Modified Files:


exec/msglist.js
diff

Added "Replied" attribute to mail messages that are replied-to.

Reported by Keyop.

Modified Files:


src/smblib/smballoc.c
diff

src/smblib/smblib.c
diff

src/smblib/smblib.h
diff

src/xpdev/semfile.c
diff

Resolve some GCC warnings about uncheck return values from std C functions

write()
chsize()

Modified Files:


src/sbbs3/scfg/scfgmsg.c
diff

When importing FidoNet message area tags > 40 chars, use newsgroup name

Follow-up to previous commit for Keyop (extending max area tag length from 35
to 40/50 chars), when importing echolists/areas.bbs files, if the area tag is
longer than 40 chars, then import it into the newsgroup name field instead (63
chars max).

Modified Files:


src/sbbs3/fidodefs.h
diff

src/sbbs3/scfg/scfgxfr2.c
diff

src/sbbs3/scfgdefs.h
diff

src/sbbs3/scfglib1.c
diff

src/sbbs3/scfglib2.c
diff

src/sbbs3/scfgsave.c
diff

Increase FidoNet echo/area tag max length from 35 to 40/50 chars

Per FMail.txt:
Area name
          This is the name of the conference. It can be up to 50
          characters long. This name is often referred to as the
          area tag.

I did not locate a FidoNet spec to contradict that and support a 35 character
maximum area tag length. I think Mystic also has a 50 character area tag limit.
So 50 character area tags are fine I guess.

So up to 50 characters are now supported in SBBSecho and SCFG for area tags
(for message and file echoes). The exception is the (new to 3.19) optional area
tag stored in msgs.cnf/file.cnf is limited to 40 characters due to reserved
space limitations. If you need a tag stored per sub (most sysops do not) and it
needs to be longer than 40 chars, use the newsgroup name instead (limited to 63
chars).

Changed at the request of Keyop via IRC.

Modified Files:


exec/load/ircd/server.js
diff

Fix for 0 TS on server-to-server MODE

Modified Files:


src/sbbs3/userdat.c
diff

Ignore trailing non-alpha-numeric characters in matchusername()

A name ending in a symbol (e.g. "Erich B.") would cause problems with this
matching logic.

Modified Files:


src/sbbs3/ftpsrvr.c
diff

Handle FTP requests for paths that start with "/./"

Midnight Commander (mc) apparently sends requests like this for files
(e.g. aliases) in the virtual root directory.

Fixes another part of the reported issue #288.

Modified Files:


src/sbbs3/ftpsrvr.c
diff

Fix segfault reported with ftpalias.cfg support in some cases

ftpalias() can return true even when the directory is not set to a valid
directory index (i.e. set to -1), so using as an array index would definitely
segfault. Part of commit 8ad30b6c by Deuce 3 years ago.

I didn't test this as I'm not sure exactly the combination of ftpalias.cfg
content and FTP command received that would trigger this, but it's most
definitely a bug.

So should fix the segfault reported in issue #288.

Modified Files:


exec/load/ircd/core.js
diff

Don't use uninitialized variables in LIST

Modified Files:


exec/load/ircd/channel.js
diff

exec/load/ircd/server.js
diff

exec/load/ircd/user.js
diff

Clean up set_chanmode() a bit

Modified Files:


exec/load/ircd/server.js
diff

Don't ignore mode arguments in SJOIN

Modified Files:


ctrl/text.dat
diff

Change SearchExtendedQ to DisplayExtendedFileInfoQ

Relates to feature request #285.

Modified Files:


src/sbbs3/execfile.cpp
diff

src/sbbs3/listfile.cpp
diff

src/sbbs3/sbbsdefs.h
diff

src/sbbs3/scandirs.cpp
diff

src/sbbs3/text.h
diff

src/sbbs3/text_defaults.c
diff

When searching for text in file descriptions, always search ext desc

Since users can (and usually do) display extended descriptions in their normal
file listings anyway, it makes sense to go ahead and search them for the text
(e.g. with the 'F' command from the default command shell File menu/prompt). So
the "search/display extended info" prompt has been changed to just "Display
extended file info" since it no longer controls whether or not the extended
description is searched for the text (it always is). The FL_EXFIND was renamed
to FL_EXT and only controls whether the extended info is display (the FL_FIND
mode flag, always renamed, must also be specified for a find with extended
info).

This closes feature request #285 by Phil (@plt).

Modified Files:


exec/load/ircd/core.js
diff

Check for 'port zero' on the remote port as well

Modified Files:


exec/load/ircd/core.js
diff

Use correct object when logging socket errors

Modified Files:


exec/load/ircd/unregistered.js
diff

Drop DNS replies for closed connections

Modified Files:


src/sbbs3/scfg/scfgxtrn.c
diff

If hitting ESC at Editor Info File prompt, don't prompt for lowercase

Slight usability improvement.

Modified Files:


src/sbbs3/writemsg.cpp
diff

When writing/reading quotes.txt, ignore first 2 lines if WWIV-type

WWIV writes 2 lines of metadata to the beginning of its quotes.txt file created
for external message editors. Its not clear yet what these lines should
contain, but the first line apparently should contain a '#' character since
BREdit checks for this character before deciding to throw-away/ignore these
lines (don't use them for quoted text). Thanks mlong for this tip!

So if the external editor is set to "WWIV CHAIN.TXT" in SCFG (even if the door
doesn't need/use it), then the quotes.txt file created by Synchronet will
contain the extra 2 lines. We're just writing a # to the first line and the
second line is blank.

This fixes reported issue #188.

Modified Files:


src/sbbs3/atcodes.cpp
diff

New @-codes UDR (upload/download byte-ratio) and UDFR (file ratio)

Expands to a percentage, similar to PCR (not capped at 100).

Remove the failed-attempt to cap the percentage at 100 for the PCR atcode.

Modified Files:


exec/load/ircd/core.js
diff

Reject connections if they're from "port zero"

Modified Files:


exec/load/ircd/core.js
diff

exec/load/ircd/server.js
diff

exec/load/ircd/unregistered.js
diff

Make auto-connects more robust

Modified Files:


src/sbbs3/jsexec.c
diff

Don't fclose(errfp) if it's NULL.

Modified Files:


src/sbbs3/js_system.c
diff

Improve the systme.findstr() JSDOCS description text

Modified Files:


src/sbbs3/str_util.c
diff

src/sbbs3/str_util.h
diff

Allow '*' pattern character to be used mid-string (not just beg or end)

Previously, the '*' matching pattern character could only be used as the
beginning or ending character of a pattern string; for any other use, it was
treated just as any other exact-match character.

Now, you can have exact-match characters on both the left and right sides of
the '*', so pattern matching like the following is possible:

<*@gmail.com>
digital*man

Multiple '*'s are still not treated special (only the first/left-most '*" is),
so, for example, "*blah*" will not likely produce the desired matching effect
(use "blah~" instead for this case).

Modified Files:


exec/load/ircd/channel.js
diff

Don't use uninitialized variable

Modified Files:


exec/logon.js
diff

Allow text of all guest information prompts to be overridden in modopts.ini

By setting guest_name, guest_email, guest_location, or guest_referral to any
truthy value, a prompt will be display. By setting it to a string (instead of
true), that string will be displayed. I'm not sure how I feel about this kind
of tri-state setting (true/false/string), but it does reduce the number of
settings/options. The alternative is to have a separate boolean and string
setting for each prompt (which I first considered).

So, as of now:

guest_name = false <- no prompt for Guest's name
guest_name = "" <- no prompt for Guest's name
guest_name = true (the default) <- default prompt text for Guest's name
guest_name = "Enter your name, dude" <- replaces prompt text for Guest's name

For enter trailing white-space and ctrl-a codes, use string literals, e.g.
guest_name: "\x01hEnter your name, dude: "

Modified Files:


exec/logon.js
diff

Guest user prompts can be disabled via modopts.ini [logon] settings

guest_name = false will disable name prompt
guest_email = false will disable the email address prompt
guest_location = false will disable the location prompt
guest_referral = false will disable the "where'd you hear about this BBS"
prompt

Note, at minimum, I do recommend prompting for the guest's email address so
that if they send the sysop feedback (e.g. ask a question), they at least have
some hope of receiving a reply.

Modified Files:


src/sbbs3/mailsrvr.c
diff

When logging sender address forgeries, log the address being forged

Helpful for debugging issues with this forgery detection logic (if there are
any).

Modified Files:


src/sbbs3/useredit.cpp
diff

Stop beeping at the sysop when user-searches are successful.

If anything, I suppose would be beep if a search fails, but really, I think
beeps are kind of annoying these days. Not changing the currently
selected/viewed user is likely all that's really needed to indicate a search
failure.

Modified Files:


src/sbbs3/mailsrvr.c
diff

Don't allow unauthenticated DNS-blacklisted clients to post on subs

Eliminate some of the infrequent SPAM posts to the SYNCPROG conference. Maybe
make this behavior configurable?

Modified Files:


exec/load/ircd/core.js
diff

Don't auto-connect to an invalid port. Reported by nelgin, thanks!

Modified Files:


src/sbbs3/js_msgbase.c
diff

Ignore the PRIVATE message attribute for the "mail" base

When setting the value of a message's 'can_read' property, ignore the PRIVATE
message attribute (which is sometimes set in FTN netmail messages) since it's
assumed all messages in the mail base are private, no special destination (to)
name matching is needed here.

This only popped up recently via msglist.js because of the recent addition of
checking each messages's 'can_read' property.

As reported by <Diehard> via IRC PM.

Modified Files:


exec/load/ircd/config.js
diff

exec/load/ircd/core.js
diff

exec/load/ircd/server.js
diff

exec/load/ircd/unregistered.js
diff

Don't auto-connect if no port is defined in C:Line, minor Y:Line changes

Modified Files:


exec/newslink.js
diff

Merge branch 'master' of gitlab.synchro.net:main/sbbs

Modified Files:


exec/newslink.js
diff

Add support for optiona "approved" option to be specify moderator's email addr

Apparently required by some news servers.
As requested via IRC:
<poindexter> Hi all, I've got a weird request. I'd like to post to a newsgroup
that relies on automoderation. You need to add an approved: header to any
posting to have it be accepted - keeps the non-techie riffraff away. is there a
way to do that with newslink.js?

Modified Files:


exec/load/ircd/core.js
diff

exec/load/ircd/server.js
diff

Improve server connects (both auto and manual)

Modified Files:


exec/load/ircd/core.js
diff

Use servername when detecting if a server's already there

Modified Files:


exec/load/ircd/core.js
diff

exec/load/ircd/server.js
diff

exec/load/ircd/unregistered.js
diff

Deal with maxlinks on the Y:Line properly for servers

Modified Files:


exec/load/ircd/server.js
diff

Use integer instead of string in initial TOPIC burst

Modified Files:


exec/msglist.js
diff

Don't list messages that can't be read by user

Should fix unvalidated/moderated message readability bug reported by Phil

Modified Files:


src/sbbs3/websrvr.c
diff

Fix NULL pointer dereference in read_post_data()

What took down cvs/sbbs yesterday:
Program terminated with signal SIGSEGV, Segmentation fault.
6203                    session->req.post_data[session->req.post_len]=0;
[Current thread is 1 (Thread 0x7f2b989ff700 (LWP 17031))]
(gdb) print post_len
No symbol "post_len" in current context.
(gdb) print session->req.post_len
$1 = 0
(gdb) print session->req.post_data
$2 = 0x0

Modified Files:


src/sbbs3/scansubs.cpp
diff

Initialize local variable 'str' in scanallsubs().

Fixes issue #276

Modified Files:


exec/msgutil.js
diff

Don't load() 822header.js unless we're exporting messages

822header.js adds an enumerable method to the header object which ends up in
the output when using the -hdrs option/command.

This addresses issue #277

Note: this is not actually a supported or documented script.

Modified Files:


exec/load/ircd/server.js
diff

exec/load/ircd/unregistered.js
diff

exec/load/ircd/user.js
diff

Fix up STATS m and EVAL

Modified Files:


exec/load/ircd/core.js
diff

exec/load/ircd/user.js
diff

Clean up LIST

Modified Files:


exec/load/ircd/core.js
diff

exec/load/ircd/server.js
diff

exec/load/ircd/unregistered.js
diff

exec/load/ircd/user.js
diff

Make better use of format()

Modified Files:


exec/load/ircd/server.js
diff

Improve TS handling in SJOIN

Modified Files:


exec/load/ircd/server.js
diff

Fix up the server-to-server AKILL message

Modified Files:


exec/load/ircd/server.js
diff

Use correct formatting for printf in NICK

Modified Files:


exec/load/ircd/server.js
diff

Reject malformed NICK messages from servers and notify

Modified Files:


exec/load/ircd/core.js
diff

Fix STATS m to report proper command, found by nelgin, thanks!

Modified Files:


exec/load/ircd/server.js
diff

Use better source of nickname on SJOIN

Modified Files:


exec/load/ircd/user.js
diff

We don't use command anymore, fixes crash on /OPER. Found by nelgin, thanks!

Modified Files:


exec/load/ircd/user.js
diff

Use proper variable on KILL

Modified Files:


ctrl/ircbot.ini
diff

Use a better default hostname for the bot

Modified Files:


exec/ircd.js
diff

exec/load/ircd/core.js
diff

Fix up epoch use in strftime as well

Modified Files:


exec/ircd.js
diff

exec/load/ircd/channel.js
diff

exec/load/ircd/config.js
diff

exec/load/ircd/core.js
diff

exec/load/ircd/server.js
diff

exec/load/ircd/unregistered.js
diff

exec/load/ircd/user.js
diff

exec/load/irclib.js
diff

Deprecate time() in favour of new Date()

Modified Files:


exec/load/ircd/server.js
diff

Fix mode desync on server SJOIN. Thanks to Retros for finding where this was
coming from!

Modified Files:


src/sbbs3/file.cpp
diff

Fix memory leak (of extended description) when moving files

Modified Files:


exec/load/ircd/user.js
diff

Fix local KILL (bug reported by phil)

Modified Files:


src/sbbs3/main.cpp
diff

Fix newly-introduced (but likely harmless) bug in mv()

Thanks Coverity! (CID 332340)

Modified Files:


src/sbbs3/bat_xfer.cpp
diff

Call smb_freefilemem() instead of smb_msgmem() for files

Not that it really matters, but perhaps it'll matter in the future. And the
code reads better.

Modified Files:


src/sbbs3/main.cpp
diff

src/sbbs3/sbbs.h
diff

mv() simplified for the non-copy case

Constified function arguments. Source path is still case-insensitive, but
destination is case-sensitive (hopefully that's not an issue).

When a "move" (!copy) is requested, a rename() is always attempted first and if
successful, job done. Otherwise, we continue on with the file-copy (and then
remove).

This should resolve CID 332219 (DEADCODE).

Modified Files:


src/sbbs3/file.cpp
diff

src/sbbs3/filedat.c
diff

src/sbbs3/filedat.h
diff

src/sbbs3/ftpsrvr.c
diff

src/sbbs3/js_filebase.c
diff

src/sbbs3/listfile.cpp
diff

src/sbbs3/upload.cpp
diff

src/smblib/smbdefs.h
diff

src/smblib/smbfile.c
diff

src/smblib/smblib.h
diff

Standardize on "metadata" as the description of a file's "tail" dfield

It's anticipated that this will be used for JS-populated file metadata in JSON
format in the future (and not just "archive contents" in .ini format).

Also, fix the double-free issue that was occurring when moving files with
extended file descriptions (sbbs_t::movefile()). This was actually the primary
problem I was fixing here, but noticed the metadata issue: metadata would not
have been moved along with the other file info between bases.

Modified Files:


src/sbbs3/filedat.c
diff

Include archive_error_string() in archive_write_header failure msg

Hopefully this helps get to the bottom of Ragnarok's reported problem creating
ZIP QWK files with libarchive.

Modified Files:


src/sbbs3/filedat.c
diff

Remove tbz (tarred bzip) files from supported archive list

Not supported by default on Windows and perhaps not on all *nix systems.
You can still support creation of tbz files if you like, but you'll need to
setup an external "Compressible File Type" in SCFG to do it.

Modified Files:


src/sbbs3/pack_qwk.cpp
diff

Better error handling in QWK packet creation

If libarchive creates a 0-byte file or no file at all, then fall-through to the
"temp file command-line" (if there is at least one Compressible File Type
configured).

If libarchive fails, there will still be a (hopefully helpful) error message
logged, but at least the QWK packet may still get created.

Modified Files:


web/lib/msgslib.ssjs
diff

Fix recently-introduced issue reading the 'mail' base.

Reported by AL@TRMB.

Modified Files:


src/sbbs3/main.cpp
diff

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.

Modified Files:


web/root/msgs/msg.ssjs
diff

Fail with graceful error message if msgbase is not defined.

Modified Files:


exec/delfiles.js
diff

Don't try to delete offline files when removing from filebase.

Reported by DaiTengu via eye are sea.

Modified Files:


web/lib/msgslib.ssjs
diff

Don't open a message base for an invalid sub-board.

Resolves this errors appearing on Vertrauen where the sub=<code> value appears
to be from some *other* Synchronet BBS:
Unrecognized msgbase code: dove-firearms, Request:
/msgs/msgs.ssjs?msg_sub=dove-firearms

Modified Files:


exec/filelist.js
diff

Remove the unused "-arc" option parsing.

Modified Files:


exec/filelist.js
diff

Add -desc option (include summary/descriptions, same as '-vv')

The -hdr option no longer implies -ext (extended descriptions).

Truncate white-space off end of extended descriptions.

Modified Files:


text/menu/sysxfer.asc
diff

Add DIR command - not sure if/where this was documented before

Update the description of the GET and PUT commands too.

Modified Files:


exec/str_cmds.js
diff

Use K_TRIM mode flag with getstr() through-out.

Modified Files:


exec/tempxfer.js
diff

Allow download of any extracted file

Modified Files:


src/sbbs3/ftpsrvr.c
diff

Detect TLS startup errors correct/fully

We can't (apparently) only rely on the return value of start_tls(), we have to
check the value of the crypt session too.
This fix the possibility of this happening:
Jun  7 18:07:26 sbbs synchronet: ftp  0058 TLS ERROR 'No permiss.to perform
this operation' (-21) opening keyset
Jun  7 18:07:26 sbbs synchronet: ftp  0058 <192.168.1.25> initialized TLS
successfully

Instead, we'll detect the failure and disable FTPS support, logging "failed to
initialize TLS successfully".

Modified Files:


src/sbbs3/scandirs.cpp
diff

Increase the progress() reporting frequency for filename/desc searches

This resolve long delays (while searching for files) without any indication of
progress.

Keep new-scan progress interval at 10 dirs however (they're fast).

Modified Files:


src/sbbs3/con_out.cpp
diff

Condition newline before progress display

Sometimes the progress meter (e.g. during file text searches) appears at a
column offset > 0 and doesn't look good. Fix that.

Modified Files:


src/sbbs3/file.cpp
diff

More use of K_TRIM when prompting for filename/spec

Modified Files:


src/sbbs3/data.cpp
diff

Modernize finduser() a bit

Skip any prepended white-space in the passed name to find.
Use better variable names (and length) for the character strings.
Use matchusername() instead of strcmp(); matchusername() follows the liberal
user-id/alias/name matching algorithm.
Use strcasestr() instead of strstr(), so no copy/modification (upper-casing) of
passed name is now required.
When 'Q'uitting the search, set the SS_ABORT flag.

Modified Files:


src/sbbs3/smbutil.c
diff

Resolved sign/unsigned compare warning

Modified Files:


src/sbbs3/text_defaults.c
diff

EmptyDir and BadFilename string updates

Modified Files:


src/sbbs3/execfunc.cpp
diff

src/sbbs3/upload.cpp
diff

Use K_TRIM getstr() mode when prompting for filenames

Don't need no stinking pre/appended white-space in our filenames!

Modified Files:


text/file.can
diff

Add Windows "reserved names"

For info, see:
https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file

Modified Files:


src/sbbs3/file.cpp
diff

src/sbbs3/filedat.c
diff

src/sbbs3/filedat.h
diff

src/sbbs3/ftpsrvr.c
diff

src/sbbs3/sbbsdefs.h
diff

src/sbbs3/scfg/scfgxfr1.c
diff

Give sysop more control over characters allowed in uploaded filenames

5 options:
- Safest Subset
- Most ASCII, Excluding Spaces (the default)
- Most ASCII, Including Spaces
- Most CP437, Excluding Spaces
- Most CP437, Including Spaces

Modified Files:


text/file.can
diff

Add img001.exe a popular FTP-spread trojan

Modified Files:


src/sbbs3/sbbsdefs.h
diff

src/sbbs3/upload.cpp
diff

Include '?' and '*' in ILLEGAL_FILENAME_CHARS definition

Modified Files:


src/xpdev/dirwrap.c
diff

I guess GLOB_PERIOD is Linux or GNU extension. <shrug>

Fix FreeBSD build error:
dirwrap.c:760:13: error: use of undeclared identifier 'GLOB_PERIOD'
Not sure how glob() on FreeBSD treats "dot files" or how to control that.

Modified Files:


ctrl/text.dat
diff

BadFilename is now a format string, EmptyDir uses cond-blankline.

Modified Files:


exec/binkit.js
diff

Don't log an error if after a failure to remove a file, the file doesn't exist

Modified Files:


exec/postfile.js
diff

Set the file cost value to the file size.

Modified Files:


src/xpdev/dirwrap.c
diff

Fix delfiles() to delete files starting with '.' (that match the pattern)

glob() by default, doesn't include 'dot files', so include the GLOB_PERIOD
flag so that when deleting all files in a directory (e.g. node1/temp/*), any
files starting with a dot will be deleted too.

Modified Files:


src/sbbs3/bat_xfer.cpp
diff

Check batch-uploaded filenames for illegal characters/sequences/names

The BadFilename text.dat is now a format string (prints the bad filename).
Use the nice progress() display status while searching for duplicate filenames.

Modified Files:


src/sbbs3/email.cpp
diff

src/sbbs3/upload.cpp
diff

src/sbbs3/writemsg.cpp
diff

BadFilename text.dat string is now a format string

(prints the bad filename)
- also, checkfname() is going to do the file.can (trashcan) checking, so no
need here

Modified Files:


src/sbbs3/logfile.cpp
diff

src/sbbs3/sbbs.h
diff

A little constification.

Modified Files:


src/sbbs3/file.cpp
diff

src/sbbs3/filedat.c
diff

src/sbbs3/filedat.h
diff

src/sbbs3/ftpsrvr.c
diff

More uniform safe/illegal/allowed filename (for upload) determination

sbbs_t::checkfname() now checks the file.can too.
new filedat.c functions:
- safest_filename() - not currently used
- illegal_filename() - returns true for a highly-suspicious (e.g. hack attempt)
filename
- allowed_filename() - returns true if the filename is good for upload (assumed
to be already checked to be legal as well).

Importantly, filenames beginning or ending in a '.' are now unallowed:
- 'dot files' are hidden (by default) on *nix
- files ending in a '.' are problematic on Windows

Modified Files:


exec/load/dns.js
diff

Fix parsing of the return code.

This fixes an issue in TypeClass lookups (such as used by reverse())

Modified Files:


src/sbbs3/smbutil.c
diff

Add new commands to flag/unflag all messages for deletion

The command to delete and remove all messages is changed from 'd' to 'D'.
This operation is not reversable.

New commands are 'd' (flag all msgs for deletion) and 'u' (undelete all msgs).

Modified Files:


src/sbbs3/websrvr.c
diff

Remove incorrect and unnecessary comment.

Modified Files:


src/sbbs3/websrvr.c
diff

Don't allow colons in web-requested path names on Windows

This fixes issue #269 (NTFS Alternate Data Stream vulnerability) and other
potential pathname issues on Windows involving colons.

There are other illegal filename characters on Windows (e.g. <>|"?*), but
filenames with these characters aren't expected to pass the later stat() test,
so should fail with a 404 error.

Modified Files:


src/encode/objects.mk
diff

Add new hex object

Modified Files:


src/sbbs3/GNUmakefile
diff

Web server needs encode lib (for hex_decode)

Added Files:


src/encode/hex.c
diff

src/encode/hex.h
diff

Generic hexadecimal encode/decode string functions (e.g. for URL decode)

These functions can be used for "percent encode/decode" or any other
2-hex-digit encoding/decoding with a single escape character.

Modified Files:


src/sbbs3/websrvr.c
diff

src/sbbs3/websrvr.vcxproj
diff

Filenames with '+' characters in them could not be served unless encoded

The requested path was being URL-decoded with unescape(), but that function
does a '+' to ' ' (space) conversion.
The '+' to space decoding is supposed to occur for the "query" portion of the
URL only, not the "path" portion.
Use the new hex_decode() function instead of unescape() for "path" decoding.

Modified Files:


src/conio/x_events.c
diff

Avoid directly manipulating XImage data for now.

The previous tests to detect RGB8 appear to have been insufficient.
This may have resulted in various artifacts such as transparency with
some drivers on some hardware in X11 mode.

Modified Files:


src/conio/x_events.c
diff

Remove debug output introduced in last commit

Modified Files:


src/syncterm/syncterm.c
diff

Some Atari (and likely C64) fixes

1) setfont() after loading fonts
2) In X11 mode, don't translate ASCII characters from unicode...
   this was causing non-ASCII characters to be translated twice.

Modified Files:


exec/lbshell.js
diff

Only add accessible programs to the xtrn list

Modified Files:


src/syncterm/syncterm.c
diff

Add CIOLIB_MODE_CURSES_ASCII to output_map

Fixes sourceforce issue number 60.

Reported by WitNik (Thanks!)

Modified Files:


exec/load/irclib.js
diff

Fix integer-to-IP conversion

Modified Files:


exec/load/ircd/core.js
diff

Fix undeclared variable in WHO

Modified Files:


src/sbbs3/tmp_xfer.cpp
diff

Increase file list creation progress report frequency

Cosmetic only.

Modified Files:


src/sbbs3/listfile.cpp
diff

Create (export to text) file lists faster

- Don't read/load extended descriptions
- Don't query each file size (use file cost from database)

Modified Files:


web/root/msgs/msgs.ssjs
diff

Elimniate sysop/logged error when just msgs.ssjs is requested

msgs.ssjs line 6: TypeError: msgbase is undefined

Instead, just display an error to the user/browser (assuming there is one).

Modified Files:


exec/ircd.js
diff

Turns out we need to disable infinite loop detection after all

Modified Files:


web/root/msgs/msg.ssjs
diff

Fix errors when reading msgs posted by users that no longer exist

"msg.ssjs line 42: Error: Error -1 reading user number"

Fixed an errant semicolon in the process. (15 year old minor bug)

Modified Files:


src/sbbs3/listfile.cpp
diff

Beautify the file list files (e.g. NEWFILES.TXT) a bit

Use the file size estimates rather than actual byte values (to keep lines < 80
chars).
Use file name file summary/description is blank or doesn't exist.

Modified Files:


src/sbbs3/tmp_xfer.cpp
diff

File list creation scan progress meter display.

Modified Files:


src/sbbs3/sbbsecho.c
diff

Microsoft doesn't have fchmod(), so use chmod() on the closed file

Modified Files:


src/sbbs3/sbbsecho.c
diff

Keep the same area file (areas.bbs) mode flags (permissions)

When executing area manager requests that alter the area file, the temp area
file created (with mkstemp) had mode 0600 (owner read/write only). The sysop
very possibly had different permission flags they want to keep for their area
file, so copy the mode flags from the original area file to the temp/new area
file.

Modified Files:


src/sbbs3/sbbsecho.c
diff

Don't allow downlinks (AreaFix add) of our "Unknown echoes" sub-board

It was recently pointed out to me that a downlink could link to the '*' echo
via Area Manager (AreaFix) request. This was not intentional.

Modified Files:


exec/archive.js
diff

Since viewable file types support wildcards now, just use '*'

This installs as the last viewable file type, so it just becomes a
fall-through for all file extensions/types and attempts to view them
using the Archive class.

Modified Files:


docs/newfilebase.txt
diff

User to User transfer support was restored and other updates

e.g. DIZ's up to 4000 characters are supported.

Modified Files:


src/sbbs3/main.cpp
diff

src/sbbs3/sbbs.h
diff

src/sbbs3/scfg/scfgsys.c
diff

src/sbbs3/scfgdefs.h
diff

src/sbbs3/scfglib1.c
diff

src/sbbs3/scfgsave.c
diff

src/sbbs3/tmp_xfer.cpp
diff

Add optional "temp file transfer" module (defaults to "tempxfer"[.js])

This module (name configurable in SCFG->System->Loadable Modules) replaces the
old hard-coded temp/archive file menu in v3.18 and earlier, going all the way
back to v1a. Good-bye to yet more old and crufty Synchronet C source code. :-/

Change the "Pre Xtrn Prog" and "Post Xtrn Prop" menu option in
SCFG->System->Loadable Modules to just "Pre Xtrn" and "Post Xtrn". The help
text had the wrong names anyway (now fixed).
Save the menu lightbar position in the "Loadable Modules" SCFG menu.

Added Files:


exec/tempxfer.js
diff

A new module to replace the hard-coded temp/archive file menu in v3.18

The last missing file feature in v3.19.

Modified Files:


ctrl/text.dat
diff

More use of conditional blank lines in prompt strings.

Modified Files:


src/sbbs3/text_defaults.c
diff

More use of conditional blank lines in prompt strings.

Modified Files:


src/sbbs3/js_bbs.cpp
diff

Add bbs.export_filelist() method

This is the last thing needed to complete the new temp/archive menu module
implemented in JavaScript for v3.19.

File list generation can easily be done in JS (see filelist.js for example),
but since we have this function already in C++ (i.e. for including filelists in
QWK packets) - might as well make it available for use in JS.

Modified Files:


exec/load/ircd/server.js
diff

Fix LINKS

Modified Files:


exec/load/ircd/core.js
diff

exec/load/ircd/user.js
diff

Only purge queue on connection reset. Clean up user QUIT a bit.

Modified Files:


src/sbbs3/js_bbs.cpp
diff

New bbs.view_file(filename) method.

To list/view file/archive contents.

Modified Files:


exec/ircmsg.js
diff

Wait for the correct PING reponse using a random token.

One potential issue here is if you connect/send to irc.synchro.net,
you'l going to get the PONG response back from a different hostname
(e.g. vert.synchro.net), which won't match the expected PONG response.

Modified Files:


exec/ircmsg.js
diff

Send a PING line after message line(s), wait for response.

Is this what you had in mind, Cyan?

Modified Files:


exec/chat_sec.js
diff

Abort menu loop when disconnected.

Modified Files:


exec/load/text.js
diff

Latest new string number definitions.

Modified Files:


src/sbbs3/text_defaults.c
diff

Updated default BatchFlagPrompt

Modified Files:


exec/load/sbbslist_lib.js
diff

Change the max BBS name length from 40 to 30.

40 was causing a lot of cosmetic issues in sbbslist.js (for 80col terminals).
This is a compromise (not reducing to the old limit of 25 chars at least). :-)

Modified Files:


exec/sbbslist.js
diff

Remove storage from the 6th format, was making lines too long for 80cols.

And nobody cares about your terabytes of free space anyway.

Modified Files:


exec/load/ircd/core.js
diff

Don't set properties until we know the object is valid

Modified Files:


ctrl/modopts.ini
diff

Add comment about adding sections/keys if they don't already exist.

Modified Files:


exec/newslink.js
diff

Twitlist optimization (don't re-read over and over)

Also added debug-level log messages for message loop rejection
reasons, for John Dovey (GATOFUEG).

Modified Files:


src/sbbs3/sbbs.h
diff

src/sbbs3/scfg/scfgxfr1.c
diff

src/sbbs3/viewfile.cpp
diff

Support wildcards in viewable file type extensions (e.g. * and ?)

Now archive.js can be installed as a sort of "fall-through" file viewer (rather
than listing all the supported/possible archive extensions as separately
viewable file-types):
File Extension        *                
Command Line          ?archive list %f 

Modified Files:


exec/load/ircd/core.js
diff

WHOIS now shows if the user is connected via SSL

Modified Files:


ctrl/ipfilter_exempt.cfg
diff

ctrl/text.dat
diff

src/conio/OverhaulNotes.txt
diff

src/conio/bitmap_con.c
diff

src/sbbs3/file.cpp
diff

src/sbbs3/filedat.c
diff

src/sbbs3/js_archive.c
diff

src/sbbs3/js_msgbase.c
diff

src/syncdraw/load.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/bbslist.h
diff

src/syncterm/syncterm.c
diff

src/xpdev/gen_defs.h
diff

Removed Files:



xtrn/sbl/DEPRECATED.TXT
diff

xtrn/sbl/GNUmakefile
diff

xtrn/sbl/Makefile
diff

xtrn/sbl/objects.mk
diff

xtrn/sbl/sbbslist.c
diff

xtrn/sbl/sbl
diff

xtrn/sbl/sbl.bpf
diff

xtrn/sbl/sbl.bpr
diff

xtrn/sbl/sbl.c
diff

xtrn/sbl/sbl.cfg
diff

xtrn/sbl/sbl.doc
diff

xtrn/sbl/sbl.dsp
diff

xtrn/sbl/sbl2smb
diff

xtrn/sbl/sbl2smb.c
diff

xtrn/sbl/sbl2smb.dsp
diff

xtrn/sbl/sbl2txt.c
diff

xtrn/sbl/sbldefs.h
diff

xtrn/sbl/sblpack
diff

xtrn/sbl/sblpack.c
diff

xtrn/sbl/sblpack.dsp
diff

xtrn/sbl/smb2sbl
diff

xtrn/sbl/smb2sbl.c
diff

xtrn/sbl/smb2sbl.dsp
diff

xtrn/sbl/sysop.txt
diff

xtrn/sbl/targets.mk
diff

Merge branch 'master' of gitlab.synchro.net:main/sbbs

Modified Files:


ctrl/ipfilter_exempt.cfg
diff

Add more private IP address ranges to the default filter exemption list

And the IPv4 link local address block (169.254/16) too.

Modified Files:


src/syncdraw/load.c
diff

Treat .msg files the same as .asc when loading.

Removed Files:



xtrn/sbl/DEPRECATED.TXT
diff

xtrn/sbl/GNUmakefile
diff

xtrn/sbl/Makefile
diff

xtrn/sbl/objects.mk
diff

xtrn/sbl/sbbslist.c
diff

xtrn/sbl/sbl
diff

xtrn/sbl/sbl.bpf
diff

xtrn/sbl/sbl.bpr
diff

xtrn/sbl/sbl.c
diff

xtrn/sbl/sbl.cfg
diff

xtrn/sbl/sbl.doc
diff

xtrn/sbl/sbl.dsp
diff

xtrn/sbl/sbl2smb
diff

xtrn/sbl/sbl2smb.c
diff

xtrn/sbl/sbl2smb.dsp
diff

xtrn/sbl/sbl2txt.c
diff

xtrn/sbl/sbldefs.h
diff

xtrn/sbl/sblpack
diff

xtrn/sbl/sblpack.c
diff

xtrn/sbl/sblpack.dsp
diff

xtrn/sbl/smb2sbl
diff

xtrn/sbl/smb2sbl.c
diff

xtrn/sbl/smb2sbl.dsp
diff

xtrn/sbl/sysop.txt
diff

xtrn/sbl/targets.mk
diff

Remove SBL (Synchronet BBS List) - replaced by sbbslist.js years ago

No need to keep this around, someone might try to build or run it!

Modified Files:


src/syncterm/bbslist.c
diff

src/syncterm/bbslist.h
diff

src/syncterm/syncterm.c
diff

Use the screen modes enum, not names array for ini parsing

Any screen mode where the description and the enum value were not
identical (ie: ones with spaces or parenthesis) would not be used
at startup correctly.

Reported by DigitalMan via Discord

Modified Files:


src/sbbs3/js_msgbase.c
diff

Fix mime_version and content_type property names in JSDOCS

Underscores are more JS friendly (and correct).

Modified Files:


src/sbbs3/js_archive.c
diff

Replace la_int64_t with int64_t for rjwboys.

Modified Files:


ctrl/text.dat
diff

Obscure the number of file(s) shown as its non-obvious/helpful

Just include the total files in the listing, not the current display.
Fixes #268.

Modified Files:


src/sbbs3/filedat.c
diff

Use int64_t instead of la_int64_t

Resolve error reported on irc with Ubuntu (don't know what version):
<rjwboys> ok now i get filedat.c:896:3: error: unknown type name
‘la_int64_t’

Modified Files:


src/conio/bitmap_con.c
diff

Default to blocky scaling

For programs that don't specify a scaling type (scfg, syncdraw)
use blocky scaling.

Modified Files:


src/syncterm/syncterm.c
diff

Set the text mode before scaling and size

Setting the scaling and size before changing modes does weird things.
Those things should be fixed, but at least this makes the problem
go away in SyncTERM for now.

Modified Files:


src/conio/OverhaulNotes.txt
diff

13) We need a way to specify the initial mode.

Modified Files:


src/xpdev/gen_defs.h
diff

Character 255 is a non-breaking space in CP437

Now IS_WHITESPACE() will return true for that char and this allows things like
truncsp() to truncate trailing white-space (e.g. from a FILE_ID.DIZ), even if
it contains a terminating 0xFF character for some unknown reason.

Modified Files:


src/sbbs3/file.cpp
diff

Don't abort the file-info edit prompts if no uploader name is entered

Modified Files:


exec/load/ircd/channel.js
diff

exec/load/ircd/server.js
diff

Fix unused/undeclared variables

Modified Files:


src/syncterm/bbslist.c
diff

Set mode by macro, not number

Modified Files:


src/conio/ciolib.h
diff

src/conio/vidmodes.c
diff

src/conio/vidmodes.h
diff

src/syncterm/bbslist.c
diff

src/syncterm/bbslist.h
diff

src/syncterm/syncterm.c
diff

Add "LCD 80x25" to make DigitalMan less angry

He's old, and should keep his blood pressure down.
Do not walk on his lawn.

Modified Files:


exec/ircd.js
diff

exec/load/ircd/channel.js
diff

exec/load/ircd/config.js
diff

exec/load/ircd/core.js
diff

exec/load/ircd/server.js
diff

exec/load/ircd/unregistered.js
diff

exec/load/ircd/user.js
diff

exec/load/irclib.js
diff

text/ircmotd.txt
diff

IRCd 1.9b

Modified Files:


exec/ircd.js
diff

exec/load/ircd/channel.js
diff

exec/load/ircd/config.js
diff

exec/load/ircd/core.js
diff

exec/load/ircd/server.js
diff

exec/load/ircd/unregistered.js
diff

exec/load/ircd/user.js
diff

exec/load/irclib.js
diff

text/ircmotd.txt
diff

Merge branch 'master' into 'master'

IRCd 1.9b

See merge request main/sbbs!126

Modified Files:


src/sbbs3/js_internal.c
diff

Fix console and connect events when using select()

Modified Files:


src/xpdev/sockwrap.c
diff

Make inet_ptoaddr() not crash and work with IPv4 address strings

Modified Files:


src/syncterm/bbslist.c
diff

Fix custom mode font size picker

Modified Files:


exec/msglist.js
diff

Perform deleted-msg filtering for all message bases, not just mail

Issue reported by plt2 via IRC.

Added Files:


src/conio/OverhaulNotes.txt
diff

Add notes for CIOLIBng

This is basically just a scratchpad for things I want to have in
the future ciolib.

Added Files:


exec/load/rip.js
diff

Add a new RIP utility library

Currently, this has RIP.supported which returns true if RIP is
supported by the client, and rip.loadicons(archive) which sends
all the icons (and .RIP files) in the specified archive (either
an Arcive object or a filename) to the remote if they don't
already have them.

If you have a RIP enabled door, a small JS script as a pre-run
command like this:

require("rip.js", "RIP");
rip.loadicons('/sbbs/xtrl/lord/lordicns.zip');

Will automatically send all the icon files in the arcive to the
user if the user doesn't already have them.  For SyncTERM as of
this commit, they will be placed in the cache directory for that
BBS (and only be available for that BBS).

On *nix, the cache dir is ~/.syncterm/cache/<bbsname>
On Win32 it's something like: C:\Users\User\AppData\Local\Microsoft\Windows\INe
tCache\SyncTERM\cache\<bbsname>

No ideal what it is on macOS.

Modified Files:


src/syncterm/ripper.c
diff

Fix auto-transfers

Modified Files:


src/syncterm/ripper.c
diff

Send CRLF instead of the specified CR on a file status.

Modified Files:


src/conio/x_events.c
diff

Fix newly-introduced memory leak...

I was going to start keeping the old rect around again, but that
idea didn't pan out.  This massive memory leak is what I get for
not doing more commits while tuning.

Modified Files:


src/syncterm/syncterm.c
diff

Try to remove the extra "SyncTERM" from the BBS cache path on Win32

Reported by Booch (Thanks!)

Modified Files:


src/conio/scale.c
diff

Fix error: ‘stderr’ undeclared (first use in this function)

Modified Files:


src/sbbs3/file.cpp
diff

Have movefile() leave the original file with the delete attribute

This has the effect of the file being listed as absent/deleted in the listing
that's in memory. As requested by Phil, plt via irc.

Moving a file after this change did result in a crash once, but I couldn't
reproduce it with a debugger attached.

Modified Files:


src/conio/bitmap_con.c
diff

Fix bug in last commit

Pixes set via setpixels() were only set in the "blink off" screen
and not in the "blink on" screen... so they all reverse blinked.

Modified Files:


src/build/Common.gmake
diff

src/conio/bitmap_con.c
diff

src/conio/scale.c
diff

src/conio/x_events.c
diff

Optimizations:

1) Keep a rectangle updated per-screen rather than regenerate each time
2) Strip palette info when putting pixels into rectangles rather than
   during scaling
3) Tighten up the screen locks a bit
4) Don't require a full resend of both screens on an update request
5) Only force a redraw for cursor movement when the cursor is visible
   (And force it whenever the cursor changes)
6) Avoid doubles in interpolation
7) Heavily optimize interpolate_height()
   interpolate_width() likely doesn't need it because it's generally not
   used and also it reads from the next pixel in memory making the
   prefetchers job easier.
8) Fix some memory-leak-on-error issues
9) For ARGB8 XImages, manipulate the data directly rather than through
   XPutPixel()

At this point, the scaling and X11 output time is heavily dominated by
cache misses.  The only really effective way to reduce this hit is to
spread the work across all the L3 caches in the system or move it into
the GPU.

With the latest updates, at the SyncTERM menu, over 90% of the time is
spent in the rendering pipeline, and over 90% of that time is spent
thrashing the caches... the only real easy win left is vectorizing, but
that's highly compiler specific.

To that end, I've switched to -O3 for release builds.  There was a comment
that -finline-functions broke Baja "badly", but that's clearly false since
-f-inline-functions has been part of -O2 for quite a while now, and Baja
doesn't seem any more broken that it ever was.

Modified Files:


src/xpdev/filewrap.h
diff

Create a Win32 wrapper for mkstemp() as MSVC doesn't have it

SBBSecho uses mkstemp() now (instead of tempnam()) - so we need this POSIX
function wrapper for Windows builds to succeed.

Modified Files:


src/sbbs3/sbbsecho.c
diff

Replace tempnam() usage with the suggested mkstemp()

Since we're cleaning up warnings... this will break the Windows build for now.

Modified Files:


src/conio/sdl_con.c
diff

src/conio/xbr.c
diff

src/sbbs3/zmodem.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/ooii.c
diff

src/syncterm/ripper.c
diff

src/syncterm/term.c
diff

src/uifc/filepick.c
diff

Fix all GCC warnings in SyncTERM on FreeBSD

Modified Files:


src/conio/GNUmakefile
diff

Don't disable the GCC suggested parenthesis warning here

Modified Files:


src/syncterm/ripper.c
diff

ifdef out an unused static function for now.

It will be used in the future, so the code shouldn't be deleted,
but not using it causes a warning.

Modified Files:


src/conio/xbr.c
diff

Digitalman and Clang agree that order of operations is too hard to
reply on.  Make it harder to merge this code from upstream in the
future instead.

Modified Files:


src/conio/scale.c
diff

Screw it, if the compiler doesn't optimize it, just use more CPU.

At least the warnings are gone.

Modified Files:


src/conio/GNUmakefile
diff

Disable gcc warning about suggested parenthesis

Deuce doesn't want to add "useless parenthesis" to xbr.c.
Hopefully Clang will just ignore this -Wno-* option?
warning: suggest parentheses around ‘&&’ within ‘||’

Modified Files:


src/conio/scale.c
diff

Declare blend() as inline to eliminate gcc warning

warning: always_inline function might not be inlinable [-Wattributes]

Modified Files:


src/conio/cterm.c
diff

src/conio/cterm.h
diff

src/conio/sdl_con.c
diff

src/sbbs3/umonitor/spyon.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/ripper.c
diff

src/syncterm/syncterm.c
diff

src/syncterm/term.c
diff

Add an argument to cterm_end() to not free fonts

While we're doing that, actually use it in SyncTERM so the offline
scrollback retains fonts.

Also, fix an issue where the current mode would be updated with
the custom parameters when you changed them.

Finally, remove some unused variables.

Modified Files:


src/conio/x_events.c
diff

Eliminate unused variable that I guess clang doesn't warn about but gcc does.

Modified Files:


src/syncterm/ripper.c
diff

Use fexistcase() for RIP assets

It seems that at least LORD has upper-case in the ZIPs and lower-case
in the queries.

Modified Files:


src/conio/cterm.c
diff

src/conio/cterm.h
diff

src/syncterm/ripper.c
diff

src/syncterm/syncterm.c
diff

Add some love for SkyPix

The biggest issue with SkyPix is the fact that it's *not* a character
cell mode.  The cursor is pretty much impossible to position correctly,
and all the text "stuff" basically needs to be bypassed.

Modified Files:


src/syncterm/bbslist.c
diff

Recalculate list height before showing after mode change

Modified Files:


src/uifc/uifc32.c
diff

Draw the shadow if the API bgcolor is blue

WIN_INACT was not getting a border because it changes the value
of bclr which is explicitly compared with BLUE to detect if a
shadow should be drawn.

Instead, draw a shadow any time it would normally be drawn.

Modified Files:


src/conio/x_events.c
diff

No need to resize window just because it moved.

Fixes X11 window dragging blinkiness.

Modified Files:


src/conio/sdlfuncs.c
diff

src/conio/sdlfuncs.h
diff

Fix some return types for SDL functions

Modified Files:


src/syncterm/ripper.c
diff

src/syncterm/term.c
diff

src/syncterm/term.h
diff

Implement upload/download support and use cache dir

Modified Files:


src/syncterm/CHANGES
diff

src/syncterm/CMakeLists.txt
diff

src/syncterm/Info.plist
diff

src/syncterm/ripper.h
diff

src/syncterm/syncterm.c
diff

src/syncterm/syncterm.spec
diff

Known bugs are fixed... it's now Beta.

Modified Files:


src/conio/scale.c
diff

On scale fail, but both buffers back in the pool

If do_scale ever failed, it would result in graphics buffers not
returning to the pool, resulting in them never being reused.

Modified Files:


src/conio/sdl_con.c
diff

src/syncterm/menu.c
diff

src/syncterm/term.c
diff

Go back to ALT-Up/Down for speed adjustment

Now that ALT-Up/Down no longer have an effect on window size, they
can go back to their previous bindings of changing the emulated
output speed.

At the same time, remove AltGr support from keysym parsing since it
should be properly handled by the text input stuff now.

Modified Files:


src/syncterm/syncterm.c
diff

Use aspect_fix() not aspect_correct() for initial window size

Caused the initial window to be the wrong size.

Modified Files:


src/conio/vidmodes.c
diff

Switch from "the CTerm palette" to the colodore one

See https://www.pepto.de/projects/colorvic/ for details.

Modified Files:


src/conio/scale.c
diff

src/conio/scale.h
diff

src/conio/sdl_con.c
diff

src/conio/vidmodes.c
diff

src/conio/vidmodes.h
diff

src/conio/x_events.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/ripper.c
diff

src/syncterm/syncterm.c
diff

src/syncterm/syncterm.h
diff

Use display aspect ratio, not pixel for tracking.

People are used to thinking about the aspect ratio of displays and
used to not thinking at all about that of pixels.  Most of the modes
are simplt 4:3 modes and the text area is the entire window.  The
only oddball here is the Commodore 64 and 128 40-column modes.
Because they have a border around them that's wider on the sides than
the top/bottom, the display aspect ratio is actually narrower than
a normal NTSC screen (6:5).  It seems the PAL version actually has
square pixels, but nobody has asked for a PAL Commodore mode, and I
think that has a different colour palette too so I'm not doing it.

Just to frustrate DigitalMan a bit, the default custom aspect ratio
is now 4:3 (but can be configured).  At present, modifying the custom
mode while *in* the custom modes "works", which no sane person would
want when adjusting the aspect ratio.

Modified Files:


src/syncterm/term.c
diff

Fix ETA calculation.

Use remaining bytes divided by CPS rather than total estimated time
minus elapsed time.

We squirrley on resumed ZModem downloads.

Modified Files:


src/conio/x_events.c
diff

Add a function to translate event coordinates to screen

Fixes issue 56

Modified Files:


xtrn/sbl/sbl2smb.c
diff

xtrn/sbl/smb2sbl.c
diff

Do the Zombie Shuffle

This is fine.

Modified Files:


src/syncterm/syncterm.c
diff

Sync ciolib_to_screen() with screen_to_ciolib()

Modified Files:


src/conio/vidmodes.c
diff

Fix the 132x43 mode...

Was using the 14 row font with 350 screen rows and expecting 43 lines.

Modified Files:


src/syncterm/bbslist.c
diff

Fix off-by-one error in program settings.

Modified Files:


src/syncterm/bbslist.c
diff

Passing a NULL bar prevents *cur from being off the first screen.

Instead, just set bar to the end and let ulist() sort it out.  It
sucks, but it sucks less.

Modified Files:


src/conio/vidmodes.c
diff

Calculate and plug in pixel aspect ratios.

We're leaving the "Awesome Modes" as 1:1, but the rest all assume
a 4:3 display that is filled by the described text.

Modified Files:


src/conio/sdlfuncs.c
diff

Use xp_dlsym() correctly.

Modified Files:


src/conio/sdl_con.c
diff

src/conio/sdlfuncs.c
diff

Set High DPI Awareness before calling SDL_Init()

Modified Files:


src/conio/sdl_con.c
diff

Use the API to insist SyncTERM is "High DPI aware"

Microsoft wants us to use a manifest for this, but I really don't
feel like digging into however one uses manifest files when not
using the Microsoft build files.

Details here: https://docs.microsoft.com/en-us/windows/win32/hidpi/setting-the-
default-dpi-awareness-for-a-process

Modified Files:


src/conio/sdl_con.c
diff

Specify HighDPI mode when creating a window.

This may solve issue where using Windows "DPI scaling" forces the
window to be scaled.

Modified Files:


ctrl/text.dat
diff

Add EditExtDescriptionQ and removed DeleteExtDescriptionQ

Modified Files:


src/sbbs3/file.cpp
diff

src/sbbs3/listfile.cpp
diff

src/sbbs3/text.h
diff

src/sbbs3/text_defaults.c
diff

src/sbbs3/writemsg.cpp
diff

Add support for editing extended file descriptions

New text.dat string:
- EditExtDescriptionQ

Removed text.dat string:
DeleteExtDescriptionQ

Modified Files:


src/sbbs3/main.cpp
diff

Fix the printf format for the "ERROR ... connecting to passthru socket"

Was passing the address string twice, so the port appeared as an invalid
number.

Modified Files:


src/sbbs3/logfile.cpp
diff

Log the current Git branch and hash along with every error

This eliminates any ambiguity about what exact version was in use when each
error is logged.

Modified Files:


src/sbbs3/listfile.cpp
diff

Don't attempt to remove files that have already been removed

This is the likely cause of Phil's reported errors when removing files:
access=-110 info=smb_removefile name not found

The only way I was able to reproduce these errors was to attempt to remove the
same file multiple times in the same file listing. In other worse, use 'R',
remove a file, then 'P'revious back to the first file and try to remove it
again. Or to remove it a file from the batch flag prompt and then attempt to
remove it again. <shrug>

Modified Files:


src/sbbs3/main.cpp
diff

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.

Modified Files:


src/sbbs3/filedat.c
diff

Check return value of fseek() and fstat()

CID 330960, 330967, 330988

Modified Files:


src/sbbs3/filedat.c
diff

Fix typo that caused updatefile() to return false on success

This may have contributed to plt's file editing woes.

Modified Files:


src/sbbs3/file.cpp
diff

Fix potential NULL deref in editfiledesc()

If the file doesn't already have a description, f->desc may be NULL.
Remove the redundant user name in the "removed file" log message.

Modified Files:


src/sbbs3/listfile.cpp
diff

Simplify the file remove/delete logic here.

Modified Files:


src/sbbs3/listfile.cpp
diff

Use correct string for file path when removing file for non dir-ops

Fix issue reported by Jay Crutti.

Modified Files:


src/sbbs3/email.cpp
diff

src/sbbs3/netmail.cpp
diff

src/sbbs3/scfgsave.c
diff

src/sbbs3/writemsg.cpp
diff

Use FOPEN_BUF_SIZE more globally

Modified Files:


src/sbbs3/scfglib1.c
diff

Remove a bunch of old #ifdef/0'd out code

Modified Files:


src/sbbs3/file.cpp
diff

Fix use of uninitialized variable in editfilename()

CID 331582

Modified Files:


3rdp/win32.release/cryptlib/bin/cl32.dll
diff

3rdp/win32.release/cryptlib/bin/cl32.lib
diff

Cryptlib 3.4.5 with the latest patches from Deuce

Modified Files:


src/conio/conio.vcxproj
diff

Fix the new try_x_init() link error

Modified Files:


src/sbbs3/text_defaults.c
diff

Latest text.dat changes.

Modified Files:


src/syncterm/syncterm.c
diff

Blocky scaling is better in my opionion.

Make it the default.

Modified Files:


src/conio/ciolib.c
diff

try_x_init() doesn't need to be guarded by win32 either.

Modified Files:


src/syncterm/ripper.c
diff

Don't guard #pragma

Modified Files:


src/conio/scale.c
diff

src/conio/sdl_con.c
diff

Fix SDL scaling weirdness issues.

Modified Files:


src/conio/scale.c
diff

src/syncterm/ripper.c
diff

Fix for MSVC build using #ifndef _MSC_VER

#pragma clang causes an error
__attribute__ not supported

Modified Files:


src/conio/conio_sdl.vcxproj
diff

Disable X and add new dependent source files

Modified Files:


src/syncterm/SyncTERM.vcxproj
diff

Add new dependent source file: telnets.c

Modified Files:


3rdp/win32.release/cryptlib/include/cryptlib.h
diff

Latest patched cryptlib.h for use with SyncTERM/MSVC build

Modified Files:


src/conio/x_events.c
diff

Fix issue where X11 window would be blank on an Expose event.

Modified Files:


src/conio/bitmap_con.c
diff

src/conio/ciolib.h
diff

src/conio/scale.c
diff

src/conio/vidmodes.c
diff

src/conio/vidmodes.h
diff

src/syncterm/bbslist.c
diff

src/syncterm/bbslist.h
diff

src/syncterm/syncterm.c
diff

Add support for VGA 80x25 mode.

This uses the 720x400 with 9x16 character cell that always made
BBSs look so terrible.

Modified Files:


src/conio/ciolib.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/syncterm.c
diff

Prefer X11 mode over SDL mode

I do, so you should too!

Modified Files:


src/conio/x_events.c
diff

Support ALT-Left/ALR-Right for scaling adjustment in X11 mode.

Modified Files:


src/conio/x_events.c
diff

Clear the entire window when the xim size is potentially changed.

This fixes issues where "leftovers" from the previous mode would
remain in a new screen mode.

Modified Files:


src/conio/sdl_con.c
diff

Use lround(), not roundl().

Also, include math.h

Modified Files:


src/conio/curs_cio.c
diff

src/conio/scale.c
diff

src/conio/x_events.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/ripper.c
diff

src/xpdev/strwrap.h
diff

Fix(?) more Windows warnings.

Modified Files:


src/syncterm/bbslist.c
diff

Don't pass -1 as cur or bar.

Sheesh.

Modified Files:


src/conio/GNUmakefile
diff

Bah, gmake.

Modified Files:


src/conio/GNUmakefile
diff

src/conio/objects.mk
diff

scale.o and xbr.o are (basically) required by bitmap now.

Modified Files:


src/conio/sdl_con.c
diff

sdl_con.c can have stdbool.h too.

Modified Files:


src/conio/scale.c
diff

Include the necessary

Modified Files:


src/conio/scale.c
diff

src/conio/sdl_con.c
diff

src/syncterm/syncterm.c
diff

Have SDL use internal scaling when the window size matches.

Simplest way is to use the alt-left/right snapping.

Modified Files:


src/conio/ciolib.h
diff

src/conio/scale.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/syncterm.c
diff

src/syncterm/syncterm.h
diff

Add a new "Blocky Scaling" option to syncterm

Which required a new ciolib option of course.
Only usable in X11 mode at present, though it may land for SDL
this weekend.

Modified Files:


src/conio/scale.c
diff

Make a generic odd pointy scaler.

Since we have all odd numbers thanks to the pointy scaler, and we
have 2x thanks to xBR, we should now be able to build any integer
scaling from a combination of those two scalers.

This, of course, assumes you have enough CPU and RAM to actually
do the scaling you want... powers of two will likely be the worst
case for scaling.

Also, do pointy scaling before xBR.

Modified Files:


src/conio/scale.c
diff

src/conio/scale.h
diff

src/conio/x_events.c
diff

Now that we have scaling sorted out, the X11 driver doesn't need it

Also, we don't need to use pointers for the scaling.

Modified Files:


src/sbbs3/chat.cpp
diff

src/sbbs3/con_out.cpp
diff

src/sbbs3/inkey.cpp
diff

Fix-ups for the "continuous random tone" sysop chat pager

I haven't played with this chat pager in *years*! The lowest tones were
inaudible (raise the range from 0-800Hz to 400-1200Hz).
Increase the msgabort() tone from 1 to 10ms, sounds like those computerized
voice recognition phone systems now.

Remove the redundant "<user> paged you to chat" notifications (and log message)
when turning *off* the chat pager.

Modified Files:


src/sbbs3/con_out.cpp
diff

src/sbbs3/file.cpp
diff

src/sbbs3/sbbs.h
diff

Simplify the check-and-clear of console abort status

by adding a 'clear' argument to msgabort() that defaults to false. Pass true to
check-and-clear.
checkline() is a NO-OP, so start eliminating "calls" to it.

Modified Files:


src/sbbs3/file.cpp
diff

src/sbbs3/sbbs.h
diff

Clear the abort flag with the various editfile* functions return false

... upon console abort (Ctrl-C).

Modified Files:


ctrl/text.dat
diff

Make (D)esc command short-cut to just edit file descriptions

Modified Files:


src/sbbs3/file.cpp
diff

src/sbbs3/listfile.cpp
diff

src/sbbs3/sbbs.h
diff

src/sbbs3/text_defaults.c
diff

src/sbbs3/viewfile.cpp
diff

Make (D)esc command short-cut to just edit file descriptions

Now, just 'E', then 'D' from the batch flag prompt has you editing a file
description - rather than going through all the prompts for uploader name/date,
etc.

Modified Files:


ctrl/text.dat
diff

Add FileInfoEditPrompt string

Modified Files:


src/sbbs3/file.cpp
diff

src/sbbs3/listfile.cpp
diff

src/sbbs3/sbbs.h
diff

src/sbbs3/text.h
diff

src/sbbs3/text_defaults.c
diff

src/sbbs3/viewfile.cpp
diff

Allow of edit of file info from extended file info prompt

Added text.dat string FileInfoEditPrompt.

Does not allow rename of file from this prompt - perhaps it should?

Resolves issue #263.

Modified Files:


src/sbbs3/addfiles.c
diff

src/sbbs3/filelist.c
diff

Fix misuse of loadfiles()

Modified Files:


src/conio/scale.c
diff

src/conio/vidmodes.c
diff

Add an untested width interpolater

SyncTERM doesn't support any "fat-pixel" modes.

Modified Files:


src/sbbs3/delfiles.c
diff

Fix ERROR ... removing file from database

Wrong arguments passed to loadfiles() caused only index to be read.

As reported by plt via irc:
ERROR (smb_putmsghdr illegal header length increase: 70 (1 blocks, 0 hfields, 0
dfields) vs 0 (0 blocks)) removing file from database

Also, replace old CVS rev with Git branch and hash.

And for everyone: use delfiles.js instead (e.g. with jsexec).

Modified Files:


src/smblib/smbfile.c
diff

smb_removefile() failed if filename length was > 64 chars

And the "name not found" error text was wrong.

Modified Files:


src/conio/bitmap_con.c
diff

src/conio/scale.c
diff

src/conio/scale.h
diff

src/conio/x_events.c
diff

src/conio/xbr.c
diff

Many X11 scaling improvements...

1) Initialize the r2y array for xBR so it actually works.
2) Add a vertical (only) interpolation scaler for aspect ratio enforcement
3) Add a simple muliplier scaler, so that can be removed from x_event.c
4) Use a new graphics buffer free list, which allows tracking last
   drawn screen instead of last bitmap rectangle, removing various hacks
5) Share the Y'CbCr <-> R'dG'dB'd tables between xBR and scale.c

Modified Files:


src/conio/vidmodes.c
diff

src/conio/x_events.c
diff

Align into columns, and add aspect ratios for more modes.

I really need to do a sweep through and update all the aspect
ratios now that the data actually matters.

Modified Files:


src/conio/sdl_con.c
diff

More fallout from non-square pixels...

This fixes all the window adjustment "stuff", but the new aspect
ratio enforcement means that ALT-UP and ALT-DOWN are useless for
resizing the window unless it's not already snapped... use ALT-LEFT
and ALT-RIGHT to decrease/increase the window size.

Modified Files:


src/conio/sdl_con.c
diff

Keep correct aspect ratio when resizing window.

Also, the 8x16 font at 80x25 is *not* a square pixel aspect ratio.
As a result, it now stays "proper", which is likely not what you
want.

This was discovered when I was trying to make DigitalMan happy
because he wanted the apsect ratio retained on scaling.

Sorry for the impending imminent scaling hassle DigitalMan!

Modified Files:


src/sbbs3/js_filebase.c
diff

Don't populate metadata property unless there is metadata 

Or detail level is MAX.

Modified Files:


src/sbbs3/js_filebase.c
diff

The optional JSON-metadata string property is now "metadata"

The old "content" array property goes away.

Modified Files:


src/sbbs3/filedat.c
diff

src/sbbs3/filedat.h
diff

Eliminate list_archive_contents()

Do this in JS and use JSON for format instead of .ini.

Modified Files:


src/sbbs3/js_filebase.c
diff

Allow the metadata property to be set with add() and update()

Sorry for the impending imminent merge hassle Deuce!

Added Files:


src/conio/hqx.c
diff

src/conio/hqx.h
diff

src/conio/scale.c
diff

src/conio/scale.h
diff

src/conio/xbr.c
diff

src/conio/xbr.h
diff

Modified Files:


src/conio/GNUmakefile
diff

src/conio/objects.mk
diff

src/conio/x_events.c
diff

Add smooth scaling to X11 output

Uses "pointyscale" for x3 and x5
Uses xBR from FFmpeg for x2 and x4
HQx is also included, but unused as it's too slow at x4.

Modified Files:


src/sbbs3/main.cpp
diff

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?

Modified Files:


src/sbbs3/filedat.c
diff

src/sbbs3/js_filebase.c
diff

src/sbbs3/upgrade_to_v319.c
diff

Remove the archive contents in SMB file "tail" thing

This was really slowing down the upgrade_to_v319 and there's no current
consumer of the data.

Consider adding back in JSON format later or just leave it to JS things to use
for JSON-formatted metadata.

Modified Files:


src/smblib/smbfile.c
diff

Fix crash in smb_addfile_withlist() when passed a non-NULL empty list

Don't allocate and memset a buffer of -1:
#1  0x0000559222bc167f in strListCombine (list=0x559224220720,
buf=0x559224226550 "", maxlen=18446744073709551615,
    delimit=0x559222bcce0a "\r\n") at str_list.c:447
447             memset(buf, 0, maxlen);
(gdb) print maxlen
$1 = 18446744073709551615
(gdb) up
#2  0x0000559222ba3841 in smb_addfile_withlist (smb=0x7ffce63a7e90,
file=0x7ffce63a5bc0, storage=1, extdesc=0x0,
    list=0x559224220720, path=0x7ffce63a6e80 "/xfer/loons/manhater.mp3") at
smbfile.c:365
365                     strListCombine(list, content, size - 1, "\r\n");
(gdb) print list
$2 = (str_list_t) 0x559224220720
(gdb) print list[0]
$3 = 0x0

Found/reported by plt via irc. Thanks!

Modified Files:


src/sbbs3/filedat.c
diff

Speed up list_archive_contents()

Don't use iniSet* since we know we're not updating existing ini entries. Use
strListAppendFormat() instead.

Modified Files:


src/syncterm/ripper.c
diff

Fix angle calculations on Win32

Do multiple before divide, and use an lround() to get the same results
on Win32 with mingw32 as Clang on FreeBSD.

Modified Files:


src/syncterm/conn.c
diff

Put the kbhit() check back into the connect loop

Was removed along with select() in e2094fb95

Modified Files:


src/sbbs3/smbutil.c
diff

Fix issue with the import message commands introduced in previous commit

The fread() usage here did not need "addressing" (Commit 1b56dc96).

This caused messages posted via smbutil to be short (e.g. 0 or 1 bytes
in length). Reported by Accession via Idle Relay Chat.

Also, don't call strlen() on msgtxt which could be NULL, causing segfault
in final printf() callin in postmsg().

Modified Files:


src/syncterm/ripper.c
diff

Add extended comment on do_fill() and why it's terrible.

Modified Files:


src/syncterm/ripper.c
diff

Draw the ellipse pixel when the angle is equal to start/end as well

Being stritly less/greater causes a one pixel gap at the start/end
of the ellipse, meaning flood fills can escape.

Modified Files:


src/conio/bitmap_con.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/ripper.c
diff

Numerous changes:

Clamp coordinate translation max
Fill using native pattern, not scaled pattern
Add a bad, broken, and slow even/odd fill
Fix logging when RIP is enabled
Use the same ellipse algorithm for partial as full ellipses
Detect diagonal movement in ellipse drawing and don't make the edge fat
Only recurse once in the flood fill
Implement the v2 poly bezier command
Enable filled pie slices
Add RIP_POINT command (basically a scaled pixel using line width)

Modified Files:


exec/tickit.js
diff

Don't allow duplicate extended and normal descriptions

If the extended description and the normal (short) description are the same,
delete the extended (long) description.

Modified Files:


src/sbbs3/echocfg.c
diff

src/sbbs3/rechocfg.c
diff

src/sbbs3/sbbsecho.c
diff

src/sbbs3/sbbsecho.h
diff

Add option to control whether area-linked nodes must be configured

This should address issue # 261.
Before April 19, 2016, area-linked nodes were required to be listed as nodes in
sbbsecho.ini:
http://cvs.synchro.net/cgi-bin/viewcvs.cgi/src/sbbs3/sbbsecho.c?revision=3.6&vi
ew=markup
I'm not sure why the change was made to only require that there be a wildmatch
for each area-linked node. So just make it a configuration option. <shrug>

Also removed the trailing "..." for every top level menu item in echocfg.

Modified Files:


src/sbbs3/listfile.cpp
diff

Make it a conditional *blank* line before the batch flag prompt

It's very tricky to defeat the auto-pause to have your own custom prompt at
rows-1. So going back to having  a blank line before the batch prompt.

Modified Files:


src/sbbs3/listfile.cpp
diff

Change CRLF before batch flag prompt to a conditional new-line.

If a sysop wants a *blank* line before the batch flag prompt, they can put that
(or a conditional blank line, even better) in their text.dat (BatchFlagPrompt).
For now, I'm thinking no blank line before the prompt is better use of the
screen.

Modified Files:


src/syncterm/ripper.c
diff

More improvements...

Consistently use map_rip_color()
Optimize flood fill to not clobber the stack
Fix small full ellipses to match RIPTerm 1.54

Modified Files:


src/syncterm/ripper.c
diff

Fix palette calculations, and add support for RIPv2 'b' command.

Modified Files:


src/sbbs3/filedat.c
diff

Don't sort the keys in the content sections of a file.

Modified Files:


src/sbbs3/text_defaults.c
diff

A couple additional [P]revious commands when list/viewing files

Modified Files:


ctrl/text.dat
diff

A couple additional [P]revious commands when list/viewing files

Modified Files:


src/sbbs3/sbbs_ini.c
diff

sbbs_write_ini() called with a NULL global pointer could have crashed

as get_ini_globals() will de-ref the 'interfaces' element. CID 331352

Modified Files:


src/xpdev/ini_file.c
diff

Fix  a couple of resource leaks in iniSortSections()

One in an error path (CID 331353), the other not.

Modified Files:


src/conio/x_events.c
diff

Optimize pixel calculations.

Modified Files:


src/syncterm/ripper.c
diff

Make some changes

Mostly tightening things up and fixed some text things.

Modified Files:


src/xpdev/ini_file.c
diff

Remove unused variable added in previous commit.

Modified Files:


src/sbbs3/filedat.c
diff

src/sbbs3/filedat.h
diff

src/sbbs3/js_filebase.c
diff

src/sbbs3/upgrade_to_v319.c
diff

Sort stored file (archive) contents, for later comparison purposes

Some archives contain exactly the same files as others, but in a different
order. Believe it or not.

Modified Files:


src/xpdev/str_list.c
diff

NULL pointer checks in strListSort*() and strListDup()

Modified Files:


src/xpdev/ini_file.c
diff

src/xpdev/ini_file.h
diff

Add iniSortSections() which can sort a read ini list and optionally keys

This also exposed problems with the find_section() utility function: when the
"found" section was actually empty and the next immediate line in the ini file
was the beginning of a new section, the "keys" indexed would actually be the
next section. Now fixed.

Modified Files:


src/conio/ciolib.c
diff

src/conio/ciolib.h
diff

src/syncterm/ripper.c
diff

Vectorize beziers

Modified Files:


src/sbbs3/listfile.cpp
diff

Previous file support from the [R]emove and [D]ownload commands.

More support for moving backwards through file searches/listing types.

Modified Files:


src/sbbs3/filedat.c
diff

src/sbbs3/js_filebase.c
diff

Fix issues with file header fields getting repeated (cloned)

e.g. when using the JS FileBase.update() method

Modified Files:


src/smblib/smblib.c
diff

When adding a msg/file header, remove the DELETE attribute

When moving or renewing files, they're first removed and then added and they
were being added with the DELETE attribute set. The filenames are displayed in
red when staged for deletion/removal, so that's how I noticed. Oops.

Modified Files:


text/menu/wwiv/main.asc
diff

text/menu/wwiv/transfer.asc
diff

Insert conditional-newlines (Ctrl-A/) to force line-wraps on >80 col terminals

Apparently these menu files were created exactly 80 columns wide. <shrug>

This fixes issue 257 reported by Keyop.

Modified Files:


src/conio/bitmap_con.c
diff

src/conio/cterm.c
diff

src/conio/cterm.h
diff

src/syncterm/ripper.c
diff

Optimize color_value() and fix RIP reset

color_value() is in the hot drawing path, so should be as fast
as possible... remove all the bit shifting and have the palette
just be colour values.

For RIP, the reset wasn't resetting the global coordinates, so a
scaled image would cause everything to be scaled down later.

Modified Files:


src/syncterm/ripper.c
diff

Add save/resotore mouse fields commands.

Used extensively on Black Flag BBS.

Modified Files:


src/sbbs3/js_filebase.c
diff

Add a 'readd_always' option to the update() method

This will cause archive file contents to be re-captured/added

Modified Files:


src/sbbs3/js_filebase.c
diff

JSDOC updates

Finally document the file-meta-object properties now that things are
solidifying.

Modified Files:


src/sbbs3/addfiles.c
diff

src/sbbs3/filedat.c
diff

src/sbbs3/filedat.h
diff

src/sbbs3/js_filebase.c
diff

src/sbbs3/upgrade_to_v319.c
diff

src/smblib/smbdefs.h
diff

src/smblib/smbfile.c
diff

src/smblib/smblib.c
diff

src/smblib/smblib.h
diff

Store contents (list) of archive files in filebase (in the "msg tail")

This will allow fast/easy display of archive contents without actually reading
the archive files.

Introduces some new functions:
- list_archive_contents()
- smb_adddfile_withlist()

A new SMB convenience variable ("tail", aliased as "content" for a file).
A new file detail level ("file_detail_content", exposed in JS as
FileBase.DETAIL.CONTENTS) which adds a "content" array property to file
metadata objects for JS FileBase.get().

Files already added to the new filebases won't have this archive content
automatically - I'm looking into that now (likely a new or updated JS script to
run).

Modified Files:


src/syncterm/ripper.c
diff

Some optimizations and clearing realloc()ed data.

Modified Files:


src/syncterm/ripper.c
diff

Fix merge issue.

Modified Files:


src/conio/bitmap_con.c
diff

src/syncterm/bbslist.c
diff

src/syncterm/bbslist.h
diff

src/syncterm/ripper.c
diff

src/syncterm/rlogin.c
diff

src/syncterm/term.c
diff

More RIPv3 stuff, make it an option.

The option basically controls forced EGA mode and reported terminal.

Modified Files:


src/syncterm/ripper.c
diff

Implement disabled text for RIP.

We still need to pass ANSI through to the parser though because
Synchronet goes apeshit if it doesn't get a response to a location
request... it tries twice and eats any data it receives that isn't
a valid position report, which obviously breaks menus.

It's also very weird that on Booch's BBS at least it's sent during
the main menu display rather than just on answering.  Not sure why
this is, but it means we can't implement the RIP requirement of
"ignoring all non-RIPscrip bytes" with a zero window without digging
deep into Synchronet guts.

Modified Files:


src/conio/bitmap_con.c
diff

src/syncterm/ripper.c
diff

src/xpdev/xpbeep.c
diff

More improvements for Black Flag...

Pretend we are RipTel v3.
Implement $PCB$
Implement click actions
Fix M areas.

Modified Files:


src/syncterm/ripper.c
diff

Remove debug, optimize

Generate conversion tables rather than do float math twice for each
virtual pixel.

Modified Files:


src/syncterm/ripper.c
diff

Start of minimal RIPv2 support.

Just the easy stuff (scaling, extended colours, border toggle, filled
rectangle)

Modified Files:


exec/load/dns.js
diff

Fix DNS response flag parsing.

Modified Files:


src/sbbs3/listfile.cpp
diff

Search of word in descriptions should match filenames too.

Especially now that now all files even have descriptions (e.g. when super long
filenames suffice).

Modified Files:


src/sbbs3/sbbsecho.c
diff

src/sbbs3/scfg/scfgmsg.c
diff

src/sbbs3/scfg/scfgsub.c
diff

src/sbbs3/scfg/scfgxfr2.c
diff

src/sbbs3/scfgdefs.h
diff

src/sbbs3/scfglib1.c
diff

src/sbbs3/scfgsave.c
diff

src/sbbs3/str_util.c
diff

Add a separate "area tag" string configured per sub-board

According to  Lupine Furmen (FURFOL):
It says in the help that the field is used for both the Newsgroup name AND the
Fido Echo-tag.

These really should be 2 different fields.

Now, the echo/area tag is usually configured in the area file (areas.bbs) and
so does not need to match *any* string configured in SCFG. But since an area
file *can* be generated from SCFG (Export Areas option) and its possible to use
SBBSecho withOUT an area file at all, then it's possible someone would like to
have their FidoNet-style area tags configured different from any other strings
related to a sub-board (e.g. newsgroup name) - why? I'm not clear, but there it
is: another string that  a sysop can set (or not).

Since I'd recently added an area tag field per directory for use by hatchit.js,
this does create some logical symmetry. And there were enough free bytes
per-sub in the msgs.cnf, so why not.

Modified Files:


src/sbbs3/file.cpp
diff

Fix error when moving files

e.g. file.cpp line 239 (removefile) removing "filename.zip" access=-203
info=smb_freemsgdat reading allocation record at offset 1108

Also, if there is an error removing the original file from the filebase, don't
proceed to move the actual file (and log a successful move).

Modified Files:


src/sbbs3/listfile.cpp
diff

When toggling extended file descriptions with the 'T' command, save user

Save the user's "misc" flags to the userbase when they use the 'T' command from
the batch flagging prompt. Without this change, the option would automatically
revert if/when the user record was re-read (which can happen at any time).

Modified Files:


webv4/pages/002-files.xjs
diff

Fix the "undefined" display when a file has no description.

Modified Files:


webv4/root/api/files.ssjs
diff

Fix downloaded filenames in v3.19

Files were being downloaded as undefined.undefined because I didn't emulate
the old 'base' and 'ext' properties - just use a single 'name' property now.

Modified Files:


webv4/pages/002-files.xjs
diff

Use html_encode() for extended descriptions with Ctrl-A codes too.

Synchronet v3.19 now converts ANSI-encoded DIZs to Ctrl-A format upon import.

Modified Files:


src/sbbs3/js_global.c
diff

Add support for conditional new-line Ctrl-A code (/) in html_encode()

Modified Files:


webv4/lib/files.js
diff

Fix count_files() for v3.19 compatibility.

Modified Files:


ctrl/attr.cfg
diff

Change default file description color from darkgreen to lightgray.

Modified Files:


src/sbbs3/addfiles.c
diff

Close stream on error.

Fixes CID 33498

Modified Files:


src/sbbs3/writemsg.cpp
diff

Fix double close (fnopen doesn't leave the descriptor open)

Fixes CID 33724.