- loginAttemptListCount()
- loginAttemptListClear()
- loginAttempts()

Now returns 0 on list lock faliure: loginFailure() and loginBanned().

events when not in "appy-time".

So now if you need to clear the failed-login/temp-ban list, just
   'touch /sbbs/ctrl/clear'
Other changes to the semfiles:
The terminal server is now identified as "term" rather than "telnet" in the
(optional) server/protocol specific semfile naming.
Removed support for the old telnet.rec semfile (long deprecated).

email/netmail messages. Does *not* support the "From:" kludge.
I know this function needs need to be completely replaced and it's silly to
have QWK kludge parsing logic in multiple places, but ugh, I didn't want to
rewrite this just yet.

sub-menus.

internal code suffixes or grp/lib internal code prefixes.
More strcpy->SAFECOPY and sprintf -> SAFEPRINTF replacements.
More use of uifcYesNoOpts.

if any sub-boards or directories have *blank* names or internal code suffixes
(a sign of a corrupted configuration), exclude them from the saved subs/dirs
automatically. I'm not sure how this can happen (use of cnflib.js?) - but
running "scfg -f" (force save) should fix this situation. For Android8675.

DOOR.ID file created by Synchronet.

plain-text portion (only) for MIME-encoded messages (emails). If/when there
are ever QWK readers that support MIME-decoding, I'll make this behavior
configurable. :-)
When passing mail between QWKnet nodes, leave the original MIME-encoding in
tact (if there is any).

If the modify callback function returned a pointer *within* the list item's
allocated buffer, the strcpy() would copy from potentialy free'd memory as
realloc may change the location of the heap buffer when resizing.
Fixed by allocating a copy of the returned pointer before freeing the original
list item buffer and then just assign the allocated pointer (no copying
needed). This likely will result in more heap fragmentation for modified list
items are now newly-allocated buffers rather than reallocated existing buffers
but the other option would have been to allocate a temporary copy of the string
before reallocating and then copying and that would've been a lot more overhead
than with this approach.

This likely fixes any crashes seen in recent revs of v3.17c (e.g. when
imported QWK or REP packets and text/*.can and the twitlist.cfg are parsed).
The sighting on Vertrauen was only in the Windows build and appeared when
importing QWK/REP packets.

If the attached file begins with a '^', ignore/skip that character in the file
path. This should resolve the error reported by Ken  of Section One BBS:
ERROR line 850, attachment file not found: ^C:/SBBS/XTRN/BRE777/...

SBBSecho v2.x included this work-around and it was not included in the v3
re-factoring/re-write. Of course, BRE is doing something wrong and is totally
at fault here, but no one expects BRE to be fixed... ever. :-)

i've made a few adjustments to xtrn.cpp for external.bat...

- remove unneeded redirection of dosemu output.
   still uses same log file name and stops zero
   byte file dosemu.log from being created in node
   directories.
- fix faulty "keystroke" dosemu -I command. it
   didn't actually do the "\r" but sent it as
   characters to the command line.
- added code for SBBS specific lredirs to be
   done more cleanly like dosemu's default
   autoexec.bat. only deletes redir if it exists.

His editor also appears to have removed trailling white-space from this file,
which is fine.

commit.

suitable for passing to findstr_in_list().
SBBSecho peformance improvement: don't open/read the twitlist.cfg file for
*each* imported message: just read it once during initialization (using the new
findstr_list() function of course).
Reversed course on the findstr()/trashcan() matching logic: significant leading
white-space was not backwards compatible (and was the cause of recent lost
messages in DOVE-Net) - so I decided to go a different route and support
C-style character escape sequences (e.g. \r, \n, \t, \x##, etc.) in findstr
comparison strings, so the new way to represent a leading space character
in a filter file (e.g. twitlist.cfg, name.can) would be: "\ ". So to match (e.g.
filter/disallow) all strings with a leading space: "\ *". "\x20 *" would also
work (0x20 is ASCII for "space").
Now, again, leading white-space in filter files (e.g. text/*.can, twitlist.cfg)
is ignored. <sigh>

a caller-supplied function (callback) which can modify each string or leave
the string in tact by returning NULL. Shrinking strings is trivial while
expanding strings (making them bigger) is a little more involved, probably
using the cbdata to pass a temporary buffer around to store each expanded
string or returning a new globally-valid pointer.
Removed the malloc.h #include as we haven't used alloca() here for a while.

files was being skipped, causing false-positive filtering of imported
QWK messages on Vertrauen. Sorry about that!

message numbers should be printed differently for each context, so added a
'reading' boolean argument (defaults to true) to indicate it was called from
readmsgs() and to use the index offset (+1) as the printed msg number in that
case.

The (O)perator (D)elete message (range) command will now skip permanent msgs.
Fixed bug with (L)ist msgs command while in (S)earch for Your Messages mode
(the msg numbers listed did not match the msg numbers used by the Read prompt).

full path (sans file suffix/extension), it would use that path for the menu
file to be displayed.

twitlist.cfg) - this allows adding a filter of say " *" which will filter
out names (e.g. mail "Fromt" names) beginning with a space character.

warning: ‘sprintf’ may write a terminating nul past the end of the destination

warning: ‘*’ in boolean context, suggest ‘&&’ instead
note: ‘sprintf’ output between 5 and 132 bytes into a destination of size 128

warning: ‘%s’ directive writing up to 4096 bytes into a region of size 3969

ciolib_attrfont(), caught by GCC v7.3.0:
	warning: ‘*’ in boolean context, suggest ‘&&’ instead

sub-board of the "Reading" prompt. This allows a sysop or sub-board
operator to quickly delete a range of messages from a message base. e.g. for
easier clean-up of msg-net hiccups or intentional spamming of subs.

(e.g. as the result of a search for text or messages to you).

HEX_DIGITS() and DEC_DIGITS()

sbbs_t::cmdstr() and cmdstr().

Use case-insensitive check of executable file extension for ".bat".
The obvious "fix" would have been to use strcasestr() instead of strstr(),
but that would leave the unlikely corner case of x.bat.exe producing a
false-positive, for example. Use the proper functions for getting a file
extension and performing a full case-insensitive string compare.

"overly long" cmdlines.
Also fixed a bug (for Windows) where external() would return 0 (success) even
when CreateProcess() fails - must re-restore the "last_error" value before
returning.

(excessively long fspec/%s value).

smb_netaddr_type() was changed to *never* return NET_NONE. Locally-posted
messages would result in a (M)ail reply trying to go to a invalid Internet mail
addres (the user's name/alias).