no need to declare "extern optarg" and "extern optind"
when <unistd.h> is included;
patch from Jan Stary <hans at stare dot cz>;
OK millert

Nuke whitespace foolish enough to expose itself during the great
"warning:" rectification.

remove unused variables

ok millert@

Instead of creating a socket with socket() or accept() and then
setting the O_NONBLOCK flag on it with fcntl(F_SETFL) afterwards,
just pass SOCK_NONBLOCK to socket() or accept4() and get it right
to begin with.

ok millert@ krw@ beck@ deraadt@ jca@

ANSIfy hunt(6)

Note: casted 2 args to struct sockaddr * on list.c to shut off compiler
warnings, and also changed an int len to socklen_t since recvfrom(2) receives
the last argument as the latter.

I'm running out of credits, but this was once again with great help and OK from
tb@

Headers cleanup and sorting

OK tb@

Some basic code maintenance in games/

- in main() replace exit with return
- drop some /* NOTREACHED */ lint comments along the way.
- make more use of standard CFLAGS, esp. -Wimplicit-function-declaration
- add and sort some headers when needed
- add straightforward pledges to some programs used at compile time

discussed with and ok mestre@

Use __progname instead of hand-rolled parsing of argv[0].
Inspired by similar diffs by tobias@ and millert@.

"I like the concept" pjanzen@
ok tedu@

Fix -D in the synopsis line:
* Whitespace is allowed between -D and the variable name.
* Each -D option only takes one single assignment.
* Drop excessive .Sm macro.

stop using libwrap. ok deraadt

replace srandomdev()+random() with the arc4random*() family
tweaks and ok millert@, ok deraadt@

fix potential bugs spotted by parfait
ok millert@ pjanzen@

Remove useless setsid() and setpgrp() calls and add -b option to
run huntd in the background like a true daemon. OK deraadt@

tidy up synopis and usage(); sort options; from Igor Sobrado

ISO C police: log and expl are reserved for the C library.
expl -> expl_string
log -> logit

New license from the official hunt release at
ftp://ftp.cgl.ucsf.edu/pub/hunt.shar.Z -- it has been approved by all three
authors (Conrad Huang, Ken Arnold, and Greg Couch).

double words and spelling fixes
ok pjanzen@

Replace things like srandom(time(NULL)) with srandomdev(3). random(3)
good enough for games but we should at least use a decent seed.
pjanzen@ OK

No dm -> no need to revoke setegid.

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

Quiet alpha warnings, principally with correct includes

A little more select() reinitialization.

re-initialize timeout before each invocation of select()

User netdb to get well known port.
Remove unused 3rd arg to main().
Revoke any setgid privs (unnecessary?)
Add -D for command-line configuration options.
Properly handle new, spawning connections.
Fix that evil repeated-quit problem in two places.
Fix a bug where a server would quit only if it were questioned for stats.
Allow port sharing.
Clean up havechar().
Don't allow write() to block.

reviewed by pjanzen@

More accurate man page; more informative logging; use random() instead of
old internal routine (seems to minimize instant death on re-entry).

Access control works now; also, some minor tidying and man page fixes.

avoid denial of service attacks on server port; extra conf vars; fix otto

major changes: security, curses, config

ident for OpenBSD
mandoc-ify manual pages
use -locurses

branches: 1.1.1;
Initial revision

Nuke whitespace foolish enough to expose itself during the great
"warning:" rectification.

remove unused variables

ok millert@

Instead of creating a socket with socket() or accept() and then
setting the O_NONBLOCK flag on it with fcntl(F_SETFL) afterwards,
just pass SOCK_NONBLOCK to socket() or accept4() and get it right
to begin with.

ok millert@ krw@ beck@ deraadt@ jca@

ANSIfy hunt(6)

Note: casted 2 args to struct sockaddr * on list.c to shut off compiler
warnings, and also changed an int len to socklen_t since recvfrom(2) receives
the last argument as the latter.

I'm running out of credits, but this was once again with great help and OK from
tb@

Headers cleanup and sorting

OK tb@

Some basic code maintenance in games/

- in main() replace exit with return
- drop some /* NOTREACHED */ lint comments along the way.
- make more use of standard CFLAGS, esp. -Wimplicit-function-declaration
- add and sort some headers when needed
- add straightforward pledges to some programs used at compile time

discussed with and ok mestre@

Use __progname instead of hand-rolled parsing of argv[0].
Inspired by similar diffs by tobias@ and millert@.

"I like the concept" pjanzen@
ok tedu@

Fix -D in the synopsis line:
* Whitespace is allowed between -D and the variable name.
* Each -D option only takes one single assignment.
* Drop excessive .Sm macro.

stop using libwrap. ok deraadt

replace srandomdev()+random() with the arc4random*() family
tweaks and ok millert@, ok deraadt@

fix potential bugs spotted by parfait
ok millert@ pjanzen@

Remove useless setsid() and setpgrp() calls and add -b option to
run huntd in the background like a true daemon. OK deraadt@

tidy up synopis and usage(); sort options; from Igor Sobrado

ISO C police: log and expl are reserved for the C library.
expl -> expl_string
log -> logit

New license from the official hunt release at
ftp://ftp.cgl.ucsf.edu/pub/hunt.shar.Z -- it has been approved by all three
authors (Conrad Huang, Ken Arnold, and Greg Couch).

double words and spelling fixes
ok pjanzen@

Replace things like srandom(time(NULL)) with srandomdev(3). random(3)
good enough for games but we should at least use a decent seed.
pjanzen@ OK

No dm -> no need to revoke setegid.

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

Quiet alpha warnings, principally with correct includes

A little more select() reinitialization.

re-initialize timeout before each invocation of select()

User netdb to get well known port.
Remove unused 3rd arg to main().
Revoke any setgid privs (unnecessary?)
Add -D for command-line configuration options.
Properly handle new, spawning connections.
Fix that evil repeated-quit problem in two places.
Fix a bug where a server would quit only if it were questioned for stats.
Allow port sharing.
Clean up havechar().
Don't allow write() to block.

reviewed by pjanzen@

More accurate man page; more informative logging; use random() instead of
old internal routine (seems to minimize instant death on re-entry).

Access control works now; also, some minor tidying and man page fixes.

avoid denial of service attacks on server port; extra conf vars; fix otto

major changes: security, curses, config

ident for OpenBSD
mandoc-ify manual pages
use -locurses

branches: 1.1.1;
Initial revision