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@.

Fix some minor buglets in wump:
1) Don't put two pits in the same room.
2) Don't put bats and pits in the same room.  (you will never hit the bat)
3) Don't start the player in a room with a pit or bad, if possible.  Some
caves are so crowded the loop may go on forever, so we give up after 100
tries to put them in a safe location (as long as it's not with the wumpus).
4) Make the manpage reflect reality WRT the default number of rooms.

Bug #3 pointed out by salo.

Apply patch from PR bin/26501 to fix hang in wump if you play too many
games in a row.  Also modify change made in rev 1.18 to work correctly.
take_action() returns 1 if the player dies, causing the game to allways
exit after any death, now the game correctly asks if the player would
like to play again.

Simplify error checking.

Replaced one instance of err() with errx(), as the value of errno might not
be correct at that time.

Typos, duplicate words, consistent formatting.

Remove trailing spaces, double spaces between words, fix some typos.

Correct typos.

Removed the unnecessary #include <ctype.h>.

Fix two lines which were broken by previous commit. Noticed by Thomas
Klausner.

misc/31566: Jaap Boender: update African and Asian capitals to the
current situation.

Added an empty line to conform to KNF.

Drop trailing whitespace.

Use standard AUTHORS section header. From YOMURA Masanori in private mail
Sort sections if necessary. Use more/better markup.

Sulfur|Sulphur -> Sul[f|ph]ur

Sulfur -> Sulfur | Sulphur

er, forgot the C{a}esium change in the last

1) Update atomic weights.
   The weights here are taken from
     http://www.iupac.org/publications/pac/2003/7508/7508x1107.html
   and have been rounded to four significant figures in all cases.
   In the case of elements that have no isotopes stable enough for
   reasonable measurement, numbers have been updated from Zumdahl,
   "Chemistry", Fifth edition and are presented (as before) in parentheses.
2) Cesium is now C{a}esium
3) A number of new elements have been added at the end of the periodic
   table.

Aluminum -> Alumin{i}um

Aluminium is really the official name, but doing it this way allows
entry of either answer.

Sulphur -> Sulfur

Although the traditional spelling in commonwealth countries is
"Sulphur", the official IUPAC name of the element is "Sulfur", and
even the Royal Society of Chemistry now spells it "Sulfur".

Added a missing '-' in score list output.

Pass lint(1).

Enable WARNS=3.

Fixed an output bug where a new-line was added due the usage of puts(3).

Pass lint(1).

Enable WARNS=3.

add a new style cast to memcpy.

Enable WARNS=3 and pass lint(1).

ok christos@

Pass WARNS=3

Comment from Al.

Aligned the usage message from the program with the one mentioned in the
man page.

- caesar(6) is not a SETGIDGAME, so we don't need to revoke any privileges.
- replaced direct POSIX read/write with stdio, which made the code simpler.
- added even more error checking.
- restructured the code to make each function fit on one screen
  (well, except one).
- now the code reflects the intended purpose of the program.
- return 0 instead of EXIT_FAILURE on success.

Quote macro.

Tidy up curses state when we quit, instead of just doing exit().

Sort sections. Use mdoc macros. Reformat.

Removing trailing spaces.

KNF conversion