Attribute ``A language that doesn't affect the way you think about
programming is not worth knowing.'' to Alan Perlis, per Diomidis Spinellis'
blog at http://www.spinellis.gr/blog/20060424/

Fix typo or mispelling.

Increase username length to match modern systems, and create #defines for a
couple of lengths.

Actually accept "?" argument as documented in the manpage.
Don't attempt to ignore an untrappable signal.
Fix 6 possible buffer overflows.

Don't follow symlinks on systems that have O_NOFOLLOW.
Don't bother looping with lockf() since first iteration would return.
Prevent two buffer overflows.

prevent "stack" overflow

bracket else block

WARNS=4

Better check data read from tetris.scores before use as array indices etc.
This is CVE-2006-1539, files against Gentoo Linux, the patch is from
Gentoo.
A standard NetBSD installation is not as much risk because tetris is
sgid "games", and users shouldn't be in that group.

read config more strictly, from Maximillian Dornseif

don't try to switch a char and have a case of EOF - use an int.

Remove dup Hitler fortune. One of the two fortunes I moved yesterday, was
already in the offensive set. From a NetBSD fan who prefers to be anonymous.

PR/33495: Daniel Weiss: Offensive quote in netbsd fortune package
- Move 2 offensive to women Adolf Hitler quotes to fortune2-o
- s/Adolph/Adolf/

Add a virtual destructor to avoid "future ABI issues". I love c++.

add virtual destructors so that g++ shuts up.

Don't use iostream just for the usage message. Use stdio instead.

XXX: GCC uninitialized

Coverity CID 3382: Fix memory leak.

Coverity CID 3383: Fix memory leak.

Coverity CID 3508: Fix file leak.

Coverity CID 3269: Fix memory leak.

sprinkle some -fno-strict-aliasing and -Wno-pointer-sign with GCC4.

lrint() -> larn_lrint().

end the argument list to exec*(3) with a NULL instead of a bare '0', as
the latter isn't a pointer context in these varargs functions.

UC, PC and BC are provided my libtermcap, don't duplicate them.

avoid a pointer sign difference.

change (mostly) int to socklen_t.  GCC 4 doesn't like that int and
socklen_t are different signness.

Credit Alan Perlis for ``give him a lollipop'' programming language quote.
Hat tip: Jerry Leichter.

Coverity CID 3032: Prevent double free: Next to the call of page_more()
there is a comment saying /* does fclose */, but the next line calls
fclose(fp);. Go figure.

It's "its."

It's "its."

fix buffer overflow (CVE-2006-1744), from Debian

Grammar nit.  Never mind that it's still not true.

Coverity CID 2788: If no room gets returned, don't try to place a monster.

Coverity CID 2791: Fix file pointer leak.

Coverity CID 2452: possible negative array index; CID 1518 and CID 1517: possible overrun of static array

Coverity CID 1288: possible negative array index

Coverity CID 993: dereference of NULL pointer

Coverity CID 1293: not checking for an error return

Coverity CID 1287: not checking for error return

Coverity CID 1327: check for error return

Bah!  Compare, don't assign!

Coverity CID 1328: check for error return

Coverity CID 1326: check for error return

Fix Coverity issue 891 -- FORWARD_NULL.
Approved by christos@.

Fix Coverity issues 2366 and 2365 -- REVERSE_INULL.
Approved by Christos@.

Fix Coverity issue 2584 -- USE_AFTER_FREE and issue 889 -- FORWARD_NULL.
Approved by christos@.

Add missing parens.

Simplify (Masao Uebayashi)

Coverity CID 692: Another && that should be ||. Boy...

Coverity CID 558: && should be obviously ||

Coverity CID 2735: Remove dead code.

Coverity CID 2737: Handle linked-lists properly. Use calloc instead of malloc
so that we don't end up storing garbage accidentally and the next pointer is
initialized. If there is an inconsistency in the file abort instead
of dereferencing NULL.

remove duplicate #include <stdlib.h>

Now that we use exit(2), we need to include <stdlib.h> for its definition.

include <stdlib.h> to get a prototype for exit

Coverity CID 1197: Don't pass -1 to close.

Coverity CID 1467: Elide static buffer overflow.

Coverity CID 868: Fix possible NULL deref (after INFTIM passes :-)

Coverity CID 980: Prevent an impossible NULL point deref.

Coverity CID 2073: Fix memory leak.

Coverity CID 999: Prevent NULL deref.

Coverity CID 695: Remove impossible condition. This changes the scoring as
it was probably intended originally.

Coverity CID 998: Possible NULL pointer deref. Actually this was a bug
where a house to be sold would be skipped. Change the sell loop to be like
the buy side loop.

fix typo

Coverity CID 1004: Fix NULL deref.

Coverity CID 1005: Fix NULL deref and incorrect variable use.

Coverity CID 1443: Prevent static overrun.

Coverity CID 827: Prevent NULL pointer deref.

Coverity CID 1699: Fix memory leaks.

Coverity CID 2544: Fix memory leak.

Coverity CID 1786: Close file.

Coverity CID 570: Remove impossible check.

Coverity CID 1195: Check for possible negative index.

Coverity CID 777: Fix NULL pointer deref and de-obfuscate code.

Coverity CID 1230: Remove spurious close().

Coverity CID 1294: Avoid negative variable used as index.

Coverity CID 1295: Don't allow cchose to return a negative number. It is
used as an index.

fclose() open FILE * even though we are returning to fail anyway.
silences coverity, CID 1590 / run 5

don't access out of bounds element of static alloc array.
found by Coverity, CID 1519 / Run 5

XXX possible that intention was to start with i = 0.

If none of the boarding parties are set do not fall off the end of the
list. Addresses Coverity CID 1507. Also put pointers before shorts in
a struct rather than mixing them up

add more "ifdef notdef" around unused sections - addresses Coverity CID 564

Add a missing bp->b_score not null check - addresses Coverity CID 1003

if getpwuid() returns null, print out the uid rather than dereferencing
Addresses Coverity CID 930

Fix overrun in players[] array. Addresses Coverty CID 1457

If getpwuid() returns null, its not usually a good idea to dereference
that to try to assign a default value. Just copy the default value into
the final destination.
Addresses Coverty CID 925

Call fclose() appropriately if there is a problem with the save file.
Coverty CID 2063

Use MKPRIVATELIB instead of NOLINKLIB/NOMAN/NOPIC/NOLINT.

Change DPADD from LIBTERM (which doesn't exist) and LIBCOMPAT (which it
doesn't use) to LIBTERMCAP (which it does use).

Fix typo in the instructions.

Remove duplicate and uncredited "Plan for the Improvement of English
Spelling", which is also in the file "fortunes".

Correct The Hitchhiker's Guide to the Galaxy entry about dolphins as per
page 119 of the book.

moduli utils moved from games to usr.bin.

Remove qsiefe/qsafe from games. Will be put back in usr.bin.

Bump date for previous.

Drop trailing whitespace. Remove superfluous .br.

better handle ^d in getinp(), so we dont loop endlessly.
i have no idea why i wanted to play this game.

oops, make this install to /usr/games and not /...
noted by hannken@, thanks.

Descend to moduli too.

Add qsieve and qsafe, two programs used for generating the moduli file.

These were written by William Allen Simpson and submitted in PR 21983,
and are added with minor adjustments and nits from christos@ and myself.

Approved by christos@ and groo@.