Apparently, some versions or configurations of Mystic BBS will include an
Origin Line in FTN NetMail messages. That would've been okay for SBBSecho,
except these origin lines do not contain an FTN address (in parenthesis).

So, firstly: when importing NetMail messages (subnum=INVALID_SUB), don't
attempt to parse any Origin Lines. Secondly, fix up the Origin Line parse
code a bit better detect the case when no FTN address is actually present.

Don't set the object's private data with freed data pointers.

Wow. This is another one of those "forever bugs".

Left it there, but commented out, just in case anyone needs it later.
But it was reported that the setcap line failed anyway:
Sep 24 09:20:15 bbs setcap[21466]: Failed to set capabilities on file
`/sbbs/repo/src/sbbs3/gcc.linux.*.exe.*/sbbs' (No such file or directory)
Sep 24 09:20:15 bbs setcap[21466]: The value of the capability argument is not
permitted for a file. Or the file is not a regular (non-symlink) file

At the request of alterego (ALTERANT), if a sub-board has a "post semaphore"
configured, signal (touch) it whenever a message is imported via SBBSecho into
that sub-board.

I changed the name from "Cyber-Net" to "CyberNet" because this value is used
as the 5D domain name which is limited to 8 characters. Everything else is
as Chad provided it.

Don't re-read the configuration - we already have it in memory, just pass it
by reference to createdefaults(), so we don't need the SBBSCTRL environment
variable (again). As reported by Michael Long via alt.bbs.synchronet.

Also addressed the truncation of the new user chat handle as reported by GCC.

Just a port of someone else's work, with minor alterations.
For nolageek, best geek.

Bug introduced in commit a2f5990b (Sept-11):
By calling putuserrec() before modifying user->misc and then calling
noyes(), we're giving an opportunity for the low-level node sync code to
read the modified "useron" back from the database, thus losing the change
we just made to user->misc. Instead, move the putuserrec() call to the end
of the case statement. Another option would have been to turn off the NETMAIL
flag before the first call to putuserrec().

Bug reported by Nugax (BYTEXCHG)

e.g. Hitting Ctrl-U while online, any nodes in "quiet mode" would not
be shown to the user even though that user was a sysop. This is not
how sbbs_t::whos_online() from getnode.cpp behaves. Now changed to
behave the same as whos_online().

(e.g. logon, new-user, quiet mode) - these are value node status values
with a user on the node.

Also address GCC warnings about unsafe sprintf usage, e.g.
warning: ‘%s’ directive writing up to 79 bytes into a region of size 59

Treat .asc, .msg, and .txt the same.
For .asc, .msg, .txt, split the file on /\r*\n/ instead of
assuming CRLF. We know what phil says about assumtion.

Doesn't seem to work. Maybe the service has to be install and up first
(cause it worked at one time). Oh well, with the menu option added, the
sysop can just reset it the first time manually.

Side note: Is it "Trade Wars", "TradeWars", or "Tradewars"? All 3 are used
interachangeably. I'm going with the former.

The game is much easier to play when the monsters don't move. :-)

A clock_t is bigger than an int on 64-bit Linux systems, so there
was some weirdness with the tick offset that caused monsters to
never move.

Use the same terminal ctrl key values as sbbs.
getstr() might need some love here (especiall left/right arrow).