Don't crash if we're asked to restore from /dev/null. (woops)

Fix a bit more lint.

Fix some more lint. (There's plenty to go around.)

If we try to load a game and it bombs, keep prompting for another one,
because if we try to continue with a half-loaded game we'll probably
end up in trouble.

Don't cast the return value of calloc.

fix some lint

need errno.h

Update for today's changes: 1-player games are not allowed, and cards.pck
no longer appears.

Fix the OpenBSD housing shortage code.

ret_card(), which I moved earlier, was consequently filed under the wrong
source file name. fix.

We do not need special code for pdp11. Also, because random() returns all
good bits (unlike rand()) we can simplify this.

Typo patrol

This file is also no longer used.

Abolish the initdeck program and the weird little binary file it generates.
The card deck data is now compiled in. (And it always should have been,
even back when memory usage counted. It's small.)

This changes the save format slightly. (I am riding the previous
breakage an hour ago and not providing compat. There should be no
further breakage.)

Source files should be named *.c or *.h, not random other things.

Fold monop.ext (which was extern data decls) into monop.h and deck.h.
Fold monop.def (which had definitions for the extern data) into monop.c.
Fold brd.dat/mon.dat/prop.dat (which were array initializers for some of
the data) into monop.c.

Rearrange includes accordingly. No functional change.

Remove last traces of old save/load code. We no longer need a private
copy of malloc.

Sort out the memory leak fix properly.

New save/load code. Related to (but not the same as) the new save/load
code in OpenBSD.

Note: this breaks compatibility with old save files. However, the old
save files were missing critical information, had a completely insane
format, and didn't work anyway.

I will be riding this breakage with another breakage within a day or
two when I fix the insanity in the card deck handling.

Fixes from OpenBSD.
   - If a player goes bankrupt buying a property (which shouldn't be
allowed, but I guess it is), don't cause them to resign to themselves.
   - Use "NULL", not "0", for pointers.
   - Avoid undefined behavior adjusting the current player after resigning.
   - Don't leak the memory used to store player names.

The last of these is disabled for the moment because we have the pointers
in question tagged const. Will see to this when I'm finished merging.

Fix from OpenBSD: don't say "1 houses".

Bug fixes from OpenBSD:
   - don't truncate unreasonably long player names when bidding;
   - don't allow players to bid more cash than they have.

Bug fix: having $0 does not mean you need to mortgage something. Plus some
tidyup.
From OpenBSD.

More fixes from OpenBSD:
  - initialize in the proper order
  - don't allow 1-player games
  - make the check for players named "done" work properly
  - a couple other minor things

Sanity fixes for input handling. Suggested by OpenBSD, but different code.

A first stab at supporting the housing shortage rules. From OpenBSD.
(note: this is not fully correct - I will fix it when I finish merging)

Don't increment booleans. From OpenBSD

Sanity fixes for input buffer handling. From OpenBSD

Improve the behavior of get-out-of-jail-free cards by returning them to
the bottom of the deck when used. There are still a few other problems.
From OpenBSD.

Rename ->last_card to ->top_card to match OpenBSD (and also to make
somewhat more sense...)

Move ret_card() to cards.c, where it more rationally belongs. From OpenBSD.

A few more whitespace/KNF. I keep hoping I've found them all...

Add the sizes to the extern array declarations. This may not be
necessary, but it's safe and reduces unnecessary divergence.
From OpenBSD.

Use random() rather than rand(). Suggested by diffing against OpenBSD.

Remove unused swap() macro.

Don't misuse random strings as printf formats, even where it's harmless.
Found by diffing against OpenBSD.

One more (hopefully the last) whitespace fix.

getinp() no longer magically treats the prompt arg as a printf format, so
don't escape % as %%. From OpenBSD.

Move function heading comments to something vaguely like the right place.

KNF/whitespace nits. No functional changes.

Fix bugs in debug printouts. From OpenBSD.

Don't use sprintf; use snprintf.

Don't use perror(); use err/warn instead.
Suggested by diffing against OpenBSD.

Fix capitalization of street names. from pjanzen@openbsd

More typo fixes

Fix formatting bug apparently introduced when the page was mdoc'd. (Had
.Nm, should have been .Pp.)

Also bump date; for some reason this hadn't been done since the original
import.

Remove transient thingy that wasn't supposed to see the light of day, and
finish dealing with some local variables that shadow a global. (hi dholland)

Fix (another) typo in comment

Fix typo in comment

Split some code from moreobj.c into action.c, as per larn 12.2.

Typo correction: s/than/that/ in one instance.

More assorted cleanup merged from larn 12.2.

Once upon a time, larn 12.0 had functions named getchar() and putchar()
that conflicted with libc. We changed them to lgetchar() and xputchar()
respectively; larn 12.2 changed them to ttgetch() and ttputch(). After
reflecting on this for a while I've decided to adopt the larn 12.2
names; not so much for compatibility but for consistency going forward.
So, massrename them. Also make ttputch() static.

Don't create files mode 666 in /var. (The code in question is not enabled
by default, though.)

Larn does not need its own private versions of functions from <ctype.h>.
Also, remove the function gettokstr(), which is not used. From larn 12.2.

A long time ago, larn had code to prohibit playing it during work hours.
Most of this code was removed ages ago, probably when dm(6) was invented;
long before -r1.1 in the NetBSD tree in any event.

Now remove the last fragment of it.

Also nuke the file of US holidays 1987-1993 that we've been carrying
around for no reason.

Some assorted minor cleanups, from larn 12.2.

Make a bunch of functions static. Merged from larn 12.2.

Use mkstemp to make the temporary files used when you win.

Since games are (now) setgid, not setuid, it is no longer necessary to
manipulate the effective uid, only the effective gid.

Document -d.

Add a -d flag so that primes will print the difference between the current
prime and the previous prime.  [I needed that for some reason I don't recall
and these changes lying about.  Since they might be useful/interesting to
someone, I might as well as commit them.]

Fix typo: sive -> sieve.

Pedanticism about time_t (not really a bug, but now it won't show up when
audited.)

Fix y2038 bug: time_t != int.

Make the default WARNS for games 4. The only game that needs to set WARNS
to anything else now is rogue, so clear WARNS from the other makefiles.

build with WARNS=4

Make hack build with WARNS=4.

Make this build with WARNS=4 by not reusing names from <math.h>.

Make this build with WARNS=4 by not reusing names from ctype.h.

Make this build with WARNS=4.

make mille build with WARNS=4

Call srandom() as well as initializing the private random generator
(which is crappy and should be removed, but that's for another day...)
This way e.g. fortune cookies no longer appear always in the same order.

Larn now builds with WARNS=4.

Apparently the reason the printf arguments were all wrong is that larn
had its own private and not-quite-standard implementation of printf.
Remove it surgically with a chainsaw.

Add gcc printf format checking, and fix the abundant problems this revealed.
(It appears that someone sometime thought that you use %d to print a long.)

clean for WARNS=4

Work around gcc signedness warning: if you check for uid_t < 0, gcc warns
because it's unsigned, but I don't really want to rely on uid_t being
unsigned on every platform this code might propagate to.

Build with WARNS=4. Sort out the game's shadowing of its own variables,
which fixes at least one bug.

Build with WARNS=4. (All fixes are cosmetic, except that it's definitely
bad karma to shadow a local variable with another of the same name in a
nested block...)

Build with WARNS=4.

This builds with WARNS=4; set it in the Makefile so it stays that way.

Build with WARNS=4.

I'm guessing that there should be no space between the hyphen and
the second word in "hand-lettered".

Only pull in <sys/cdefs.h> on NetBSD, to fix build on Solaris 10.
Tweak replacement definition of __dead.

ANSIfy. Remove unnecessary casts. Clean up for -Wsign-compare. Make more
things file-static. Other minor tidyups, and fix a couple minor bugs found
along the way.

Whitespace/KNF nits.

Fun with parts of speech: `stupid' here is an adjective, not a noun.

Comprehensive (or at least extensive) string handling cleanup for rogue.

This patch dates (mostly) back to 2002; the critical parts of it were
handled back then by security-officer. As far as I know, there's
nothing exploitable fixed herein.

A slightly earlier version of this patch was reviewed by Christian Biere
when I filed it as PR 34750.

more __dead fallout: these two files are built for the native host, not
for the target host - and thus do not necessarily have __dead defined in
<sys/cdefs.h>.

convert __attribute__s to applicable cdefs.h macros

include sys/cdefs.h so that __attribute__ can be fixed later

A terrible kludge: don't compile/link malloc.c if arch=sun2. My apologies
to the legions of sun2 monop fans who will probably be unable to save/restore
games.

__attribute -> __attribute__

Teach wtf(6) about pkgsrc's help facility:

% cd /usr/pkgsrc/audio/qsynth
% wtf subst
subst: ===> mk/subst.mk:
# This Makefile fragment implements a general text replacement facility.
...

Increased WARNS to 4, since it passes both gcc333 and gcc411 on i386.

Add new Makefile knob, USE_FORT, which extends USE_SSP by turning on the
FORTIFY_SOURCE feature of libssp, thus checking the size of arguments to
various string and memory copy and set functions (as well as a few system
calls and other miscellany) where known at function entry.  RedHat has
evidently built all "core system packages" with this option for some time.

This option should be used at the top of Makefiles (or Makefile.inc where
this is used for subdirectories) but after any setting of LIB.

This is only useful for userland code, and cannot be used in libc or in
any code which includes the libc internals, because it overrides certain
libc functions with macros.  Some effort has been made to make USE_FORT=yes
work correctly for a full-system build by having the bsd.sys.mk logic
disable the feature where it should not be used (libc, libssp iteself,
the kernel) but no attempt has been made to build the entire system with
USE_FORT and doing so will doubtless expose numerous bugs and misfeatures.

Adjust the system build so that all programs and libraries that are setuid,
directly handle network data (including serial comm data), perform
authentication, or appear likely to have (or have a history of having)
data-driven bugs (e.g. file(1)) are built with USE_FORT=yes by default,
with the exception of libc, which cannot use USE_FORT and thus uses
only USE_SSP by default.  Tested on i386 with no ill results; USE_FORT=no
per-directory or in a system build will disable if desired.

Per PR 36185, OKed by martin@

Xref shuffle(1) from random(6), and vice versa

NetBSD JIHBED

NetBSD: Dead on Target
                -- Gimpy

 - Don't call basename $0 at many places. Use variable $PROGNAME
 - Better check when neither $ACRONYMDB is define nor files in
   /usr/share/misc/ are found
 - Replace cat | fgrep to using fgrep only
 - Replace sort | uniq to using sort -u

By Slava Semushin <slava.semushin@gmail.com> in private email.