snake: Avoid creating definitions prefixed with _ inside a program

Remove extra semicolon.

Do not pass usleep() values >= 1 million. From Henning Petersen in PR 51286

WARNS=5, and make WARNS=5 the default for src/games.

branches: 1.27.6;
Fix a few of the games to use getprogname() in their usage() functions.

Suggested by Joerg.

XXX There are probably many more that have their progname hard-coded, but
XXX I'm not up for a wholesale roto-till at the moment.

sprinkle static

if initscr() fails, exit with a message rather than crash in the
next curses call

Remove the \n and tabs from the __COPYRIGHT() strings.

branches: 1.23.6;
convert __attribute__s to applicable cdefs.h macros

branches: 1.22.4;
Replve duplicate #includes
From Slava Semushin <slava.semushin@gmail.com>, via private mail

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

Control -d by #ifdef DEBUG; adjust usage message accordingly. Bug
reported by Malcolm Parsons <malcolm@ivywell.screaming.net> in Debian
bug 84852.

Remove uses of __P.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

Always lseek back to the beginning of the file in post(), which may be
called twice in a game if a bonus occurs. Patch from Malcolm Parsons
<malcolm@ivywell.screaming.net>.

Use setgid(), not setregid().

fix handling of '-t' flag, and pause for a little bit after drawing happysnake

branches: 1.14.2;
Convert snake to use curses, and generally clean it up somewhat.

Includes bugfixes from or inspired by OpenBSD.

Security improvements for games (largely from or inspired by OpenBSD).

Games which run setgid from dm, but don't need to, should drop their
privileges at startup.

Games which have a scorefile should open it at startup, then drop all
privileges leaving just the open writable file descriptor. If the
game can invoke subprocesses, this should be made close-on-exec.

Games with scorefiles should make sure they do not get a file
descriptor < 3. (Otherwise, they could get confused and corrupt the
scorefile when using stdin, stdout or stderr.)

Some old setuid revokes from the days of setuid games change into gid
revokes.

Use the symbolic names `SEEK_SET' and `O_RDONLY' where appropriate in
the games.

Add `__noreturn__' and `__unused__' attributes where appropriate to
the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Most noreturn attributes were previously added in bin/6144, with some
others that were missed then in bin/8082. Previous `unused'
attributes were covered in bin/6557, bin/8058 and other PRs (all these
PRs have already been handled and closed).

Add use of `const' where appropriate to the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Some such changes were previously covered in PRs bin/6041, bin/6146,
bin/6148, bin/6150, bin/6151, bin/6580, bin/6660, bin/7993, bin/7994,
bin/8039, bin/8057 and bin/8093.

WARNSify, KNFify

Use POSIX tty semantics.

Various changes to make games compile w/o warnings on the alpha:
Include appropriate includes, delete bogus function declarations,
change sizes of variables and casts.

fix up botched include

clean up import, NetBSD RCS Ids, etc.

evil lseek casts...

Add RCS identifiers.

Cleanup for GCC 2.

branches: 1.1.1;
Initial revision

Remove extra semicolon.

Do not pass usleep() values >= 1 million. From Henning Petersen in PR 51286

WARNS=5, and make WARNS=5 the default for src/games.

branches: 1.27.6;
Fix a few of the games to use getprogname() in their usage() functions.

Suggested by Joerg.

XXX There are probably many more that have their progname hard-coded, but
XXX I'm not up for a wholesale roto-till at the moment.

sprinkle static

if initscr() fails, exit with a message rather than crash in the
next curses call

Remove the \n and tabs from the __COPYRIGHT() strings.

branches: 1.23.6;
convert __attribute__s to applicable cdefs.h macros

branches: 1.22.4;
Replve duplicate #includes
From Slava Semushin <slava.semushin@gmail.com>, via private mail

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

Control -d by #ifdef DEBUG; adjust usage message accordingly. Bug
reported by Malcolm Parsons <malcolm@ivywell.screaming.net> in Debian
bug 84852.

Remove uses of __P.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

Always lseek back to the beginning of the file in post(), which may be
called twice in a game if a bonus occurs. Patch from Malcolm Parsons
<malcolm@ivywell.screaming.net>.

Use setgid(), not setregid().

fix handling of '-t' flag, and pause for a little bit after drawing happysnake

branches: 1.14.2;
Convert snake to use curses, and generally clean it up somewhat.

Includes bugfixes from or inspired by OpenBSD.

Security improvements for games (largely from or inspired by OpenBSD).

Games which run setgid from dm, but don't need to, should drop their
privileges at startup.

Games which have a scorefile should open it at startup, then drop all
privileges leaving just the open writable file descriptor. If the
game can invoke subprocesses, this should be made close-on-exec.

Games with scorefiles should make sure they do not get a file
descriptor < 3. (Otherwise, they could get confused and corrupt the
scorefile when using stdin, stdout or stderr.)

Some old setuid revokes from the days of setuid games change into gid
revokes.

Use the symbolic names `SEEK_SET' and `O_RDONLY' where appropriate in
the games.

Add `__noreturn__' and `__unused__' attributes where appropriate to
the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Most noreturn attributes were previously added in bin/6144, with some
others that were missed then in bin/8082. Previous `unused'
attributes were covered in bin/6557, bin/8058 and other PRs (all these
PRs have already been handled and closed).

Add use of `const' where appropriate to the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Some such changes were previously covered in PRs bin/6041, bin/6146,
bin/6148, bin/6150, bin/6151, bin/6580, bin/6660, bin/7993, bin/7994,
bin/8039, bin/8057 and bin/8093.

WARNSify, KNFify

Use POSIX tty semantics.

Various changes to make games compile w/o warnings on the alpha:
Include appropriate includes, delete bogus function declarations,
change sizes of variables and casts.

fix up botched include

clean up import, NetBSD RCS Ids, etc.

evil lseek casts...

Add RCS identifiers.

Cleanup for GCC 2.

branches: 1.1.1;
Initial revision

Do not pass usleep() values >= 1 million. From Henning Petersen in PR 51286

WARNS=5, and make WARNS=5 the default for src/games.

branches: 1.27.6;
Fix a few of the games to use getprogname() in their usage() functions.

Suggested by Joerg.

XXX There are probably many more that have their progname hard-coded, but
XXX I'm not up for a wholesale roto-till at the moment.

sprinkle static

if initscr() fails, exit with a message rather than crash in the
next curses call

Remove the \n and tabs from the __COPYRIGHT() strings.

branches: 1.23.6;
convert __attribute__s to applicable cdefs.h macros

branches: 1.22.4;
Replve duplicate #includes
From Slava Semushin <slava.semushin@gmail.com>, via private mail

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

Control -d by #ifdef DEBUG; adjust usage message accordingly. Bug
reported by Malcolm Parsons <malcolm@ivywell.screaming.net> in Debian
bug 84852.

Remove uses of __P.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

Always lseek back to the beginning of the file in post(), which may be
called twice in a game if a bonus occurs. Patch from Malcolm Parsons
<malcolm@ivywell.screaming.net>.

Use setgid(), not setregid().

fix handling of '-t' flag, and pause for a little bit after drawing happysnake

branches: 1.14.2;
Convert snake to use curses, and generally clean it up somewhat.

Includes bugfixes from or inspired by OpenBSD.

Security improvements for games (largely from or inspired by OpenBSD).

Games which run setgid from dm, but don't need to, should drop their
privileges at startup.

Games which have a scorefile should open it at startup, then drop all
privileges leaving just the open writable file descriptor. If the
game can invoke subprocesses, this should be made close-on-exec.

Games with scorefiles should make sure they do not get a file
descriptor < 3. (Otherwise, they could get confused and corrupt the
scorefile when using stdin, stdout or stderr.)

Some old setuid revokes from the days of setuid games change into gid
revokes.

Use the symbolic names `SEEK_SET' and `O_RDONLY' where appropriate in
the games.

Add `__noreturn__' and `__unused__' attributes where appropriate to
the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Most noreturn attributes were previously added in bin/6144, with some
others that were missed then in bin/8082. Previous `unused'
attributes were covered in bin/6557, bin/8058 and other PRs (all these
PRs have already been handled and closed).

Add use of `const' where appropriate to the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Some such changes were previously covered in PRs bin/6041, bin/6146,
bin/6148, bin/6150, bin/6151, bin/6580, bin/6660, bin/7993, bin/7994,
bin/8039, bin/8057 and bin/8093.

WARNSify, KNFify

Use POSIX tty semantics.

Various changes to make games compile w/o warnings on the alpha:
Include appropriate includes, delete bogus function declarations,
change sizes of variables and casts.

fix up botched include

clean up import, NetBSD RCS Ids, etc.

evil lseek casts...

Add RCS identifiers.

Cleanup for GCC 2.

branches: 1.1.1;
Initial revision