fix placement of opening parentheses, and drop some .Xo while here

Move from <sys/param.h>.  (The binary change is due to a line number
passed to assert, found by doug)
ok millert

Arguments are just ".Ar", not ".Brq Ar" or even ".Ns { Ns Ar ... Ns }".
The .Ar macro already causes distinctive formatting in a standard way,
so there is no need for additional braces.
This also fixes the only mandoc warning in src/sbin.

switch to <limits.h>; ok millert

remove useless escaping; mandoc warned about some of this

Properly escape punctuation when given as an argument to a macro;
this was the only mandoc warning in src/bin.

Less code, more better. No longer need to worry about what mysterious
things will happen when machines have 8 byte longs.

The make code has "bootstrap", to allow it to be brought up on other
systems.  Rarely used & tested -- perhaps once a decade. Perhaps not
even once this decade?  Anyways,
    #define PATH_MAX (MAXPATHLEN+1)
is quite wrong.  Delete the chunk, assuming any system this is ported
to has PATH_MAX.

SSL_CTX_use_certificate_chain() has been added to LibreSSL and there
is no need to keep a local copy in ssl_privsep.c.  This adds a little
burden on OpenSMTPD-portable because it will have to put it in
openbsd-compat for compatibility with legacy OpenSSL.

OK gilles@

Adapt to <limits.h> universe.
ok millert

Replace <sys/param.h> with <limits.h>
millert spotted the accidental <ctype.h> removal that caused binary change.

The SSL/TLS session Id context is limited to 32 bytes.  Instead of
using the name of relayd relay or smtpd pki, use a 32 byte arc4random
buffer that should be unique for the context.  This fixes an issue in
OpenSMTPD when a long pki name could break the configuration.

OK gilles@ benno@

Tweak previous: Do not put punctuation on its own line, put it at the end
of the preceding macro line; no output change with mandoc, fixes output
with groff.  Also, if you want spacing back after .Sm off, do not add
an argument containing a blank character, simply rely on .Sm on.

Add regression tests for syslog over TLS.

Binary code patching on amd64

This commit adds generic infrastructure to do binary code patching on amd64.
The existing code patching for SMAP is converted to the new infrastruture.

More consumers and support for i386 will follow later.

This version of the diff has some simplifications in codepatch_fill_nop()
compared to a version that was:

OK @kettenis @mlarkin @jsg

Clean up macros in isakmpd(8).

- Fix mandoc warnings ("WARNING: skipping empty macro: No")
- Mark up arguments with Ar, not Aq Ic
- Mark up pathnames with Pa

ok jmc@

Replace <sys/param.h> with <limits.h>

This patch is from Theo.  I helped verify that removing <sys/param.h>
doesn't change anything.

This produces the same binaries before and after with clang 3.5 and gcc
4.9.4 on amd64.  There is a slight difference in the way it is generated by
our old gcc 4.2.1 despite the fact that the preprocessed input is almost
identical.

ok deraadt@

when hostname canonicalisation is enabled, try to parse hostnames
as addresses before looking them up for canonicalisation.
fixes bz#2074 and avoids needless DNS lookups in some cases;
ok markus

<sys/param.h> is not needed here either.
ok guenther millert doug

Replace <sys/param.h> with <limits.h> and other less dirty headers where
possible.  Annotate <sys/param.h> lines with their current reasons.  Switch
to PATH_MAX, NGROUPS_MAX, HOST_NAME_MAX+1, LOGIN_NAME_MAX, etc.  Change
MIN() and MAX() to local definitions of MINIMUM() and MAXIMUM() where
sensible to avoid pulling in the pollution.  These are the files confirmed
through binary verification.
ok guenther, millert, doug (helped with the verification protocol)

improve checksum parsing slightly. now handles filenames with spaces.
(though not names with ')'; sorry.)

just to be careful, add a cpp guard that the sscanf sizes are ok

adjust to HOST_NAME_MAX+1 & LOGIN_NAME_MAX

If MAXPATHLEN is undefined, do not set it to 512. Dangerous.

Complete the list of functions in the paragraph that mentions that
errbuf needs to hold at least PCAP_ERRBUF_SIZE chars.

Rename pcap_create()'s ebuf argument to errbuf to match the rest of the
public pcap_* functions that use errbuf.  Mainline libpcap also uses
"errbuf" for pcap_create().

No object file change.

Add more missing argument names.

Remove pointless casts for several malloc/calloc/free calls.  No object
file change.

The BPF paper referenced in the SEE ALSO section was most likely an unpublished
draft.  Replace it with the authors' 1993 Winter USENIX paper, which is a more
authoritative reference on BPF.

ok deraadt@ jmc@ millert@

Fix a use after free, where the already freed p->opt.source was used by
pcap_cleanup_bpf() to disable monitor mode on 802.11 devices.

feedback blambert@
ok deraadt@ mikeb@ millert@

Delete the MANLINT variable and the related SUFFIXES rules because
since yesterday, "mandoc -Tlint -Wfatal" can no longer fail.
Instead, as suggested by deraadt@, provide a manlint target
that is *not* run during make build, but can be run
whenever you want to check syntax of manuals.
"nice stuff" deraadt@

sync

Merge in a commit from upstream..

- Fix bug#637: fix that nsd.db grows limitlessly, an off by one
  on one megabyte free chunks, created during AXFRs of large zones,
  that caused the one megabyte chunk to be leaked.

ok sthen@

Since <netdb.h> soon won't include <sys/param.h>, MAXHOSTNAMELEN may
not be in scope, so hardcode as 256.  Other options have more downside.

move to PATH_MAX, etc; normalize includes for life in the <limits.h> universe
ok guenther millert

Move to using PATH_MAX
ok millert guenther

first batch of cleanup to programs based upon the namespace cleanups
in net/pfvar.h sys/proc.h sys/ucred.h arpa/nameser.h
change to PATH_MAX, reduce use of MIN() and MAX(), HOST_NAME_MAX+1,
LOGIN_NAME_MAX, etc etc
ok millert guenther, some review by doug

first batch of programs adapting to the namespace cleanup
(pfvar.h nameser.h proc.h ucred.h)
ok guenther millert, and some review from doug as well.

Switch to including <sys/types.h> instead of <sys/param.h>
This hides the unavoidably cruddy sys/param.h namespace in some programs.
Little impact, because many programs are cruddy and still get it via
other includes.
ports testing thanks to sthen and naddy
ok guenther millert

switch to PATH_MAX

include <sys/syslimits.h> and switch to use of LOGIN_NAME_MAX, making
this far more standalone.  discussed with guenther
ok millert

include <sys/syslimits.h> to get access to (and then use) NGROUPS_MAX
discussed at length with guenther
ok millert

Stop including <sys/param.h> and other stuff.  Assume this will be included
with sys limits in scope, so use PATH_MAX instead.
ok millert guenther

do not assume <netmls/mpls.h> will pull in <limits.h> via <sys/param.h>
get <limits.h> directly.
ok millert

Do not include 5 header files, including <sys/param.h>
ok millert

add kex unit tests

remove static version string

simplify des headers by stripping out all the unnecessary userland bits
ok deraadt

fully remove disabled mschap support, which does weird DES things.
(already won't build for some time since the removal of md4)
ok naddy

Remove a variable no longer used in re_tick().

From Fabian Raetz <fabian dot raetz at gmail dot com>

use PATH_MAX for path buffers. in case, you know...

If some mythical system lacks PATH_MAX, do not set it to MAXPATHLEN+1
ok millert

remove xmalloc, switch to sshbuf

switch to sshbuf

UVM_INH_* and UVM_ADV_* are gone; use MAP_INHERIT_* and MADV_*

Rework the disk label handling once again. Disk label is read at attach time
because it teaches us the drive geometry, but from then on there is no reason
not to implement disklabel spoof support and a proper DIOCGPDINFO ioctl.

As a result, this makes disklabel -A now work on SMD disks.

The drawback of this, is that, since native labels do not have a pcylinders
value, the code will now always assume pcylinders == ncylinders + acylinders.
(this ought to be the case by default for labels produced by SunOS format(8).)

Be sure to print time_t with the appropriate format specifier. Repairs
dump of eeprom variables on sun4.

Test shmat() with a non-NULL address, both failure and success

Use .An in the AUTHORS section where appropriate;
from Kaspars at Bankovskis dot net

The flags variable in shmat was not actually used.  We need UVM_FLAG_*
flags, not mmap-style flags for UVM_MAPFLAG().  Remove the nonsensical
MAP_ANON|MAP_SHARED value and convert MAP_FIXED to UVM_FLAG_FIXED.
OK guenther@

we don't have an alloca() symbol anywhere in libc, don't prototype it.
instead, #define to __builtin_alloca. if that doesn't work, you're hosed.
from an old discussion about the perils of alloca. (don't use alloca!)

use .In where appropriate; from Kaspars at Bankovskis dot net

typo; from Kaspars at Bankovskis dot net

oops, forgotten back-pointer to state for error messages
problem noticed by jca@

remove .Tn; from Jan Stary <hans at stare dot cz>

handle UMAC128 initialization like UMAC; ok djm@ markus@

Fix subtle breakage for polled commands introduced during iopool conversion.

remove unused variable

ok tedu@

remove unused variable

ok tedu@

all modern systems can do this SUSPEND thing

NR_OF_EOFS is a festering boil, lance & drain.
ok millert

Map the sigcode page with MAP_INHERIT_COPY to make sure it isn't shared ater
a fork.

ok deraadt@

Don't include i82489 headers in cpu.h

Only pmap.c was not including them explicitly. Fix that.

"The direction is good" deraadt@

Define and print HV cpuid flag on i386.

This is set by many hypervisors, including kvm, vmware, hyper-v.

Remove an unnecessary variable and shorten a line.

Shorten some long lines.

Add a test for syslogd that reconnects to a TCP syslog server after
an established connection has been closed.

Use explicit_bzero instead of memset in hash Final and End functions.
OK deraadt@ djm@

tell the truth about DES.

joint work with djm@ and jsing@

ok djm@

When the connection to a TCP syslog server fails or the TCP connection
terminates, try to reconnect after an increasing timeout.
test and OK jasper@

fix regression reported by brad@ for passworded keys without
agent present

Map the sigcode page copy-on-write.  This allows userland to put breakpoints
in the signal trampoline.  Solves some long-standing issues with debugging
signal handlers in GDB.

ok deraadt@

sync ssh-keysign, ssh-keygen and some dependencies to the new
buffer/key API; mostly mechanical, ok markus@

Extend the description of _timeout.

when enqueueing offline mails from within the daemon session, we should not
rely on getlogin() otherwise mail will end up enqueued as coming from user
who started smtpd.

bug spotted by deraadt@, diff ok todd@

remove commented-out test code now that it has moved to a
proper unit test

unit tests for KRL bitmap

back in september I did the large abstraction refactoring to allow these
other systems to fit into the same mold, so add copyright

Fatal errors no longer exist.
If a file can be opened, mandoc will produce some output;
at worst, the output may be almost empty.
Simplifies error handling and frees a message type for future use.

Sync with libc version.

Make strlcpy/strlcat slightly easier to read.

POSIX does not require an explicit inclusion of sys/types.h to get
uid_t or gid_t (and neither do we).  OK deraadt@

downgrade .so failure from FATAL to ERROR

Define a new wscons mouse type for Synaptics clickpad devices that lack
physical buttons.  This will be used if the acpi pnpid for the mouse
matches a list provided by Synaptics found in Linux.

Combined with the xenocara changes this will remove the need for an xorg
config file for the x240/t440/t540 generation of thinkpads.

ok matthieu@ shadchin@ kettenis@

I bet the reference to <sys/ttycom.h> near the ldisc list in tty(4)
led this program and the associated manual pages to including that
file directly, rather than the canonical include path of <sys/ioctl.h>
discussed with millert

do not be specific regarding which ioctl backend .h file; the list
is good enough, and we do not want to mislead.  the file to include
is at the top of the file.

not supposed to include sys/ttycom.h (these may have rpcgen history)

wrong place to mention nmea.  cua is different, that hints at how it
works to many.

delete overly-specific details that cause espie angst.
ok millert tedu

change _yp_domain[] to HOST_NAME_MAX+1, and re-arrange those who include
it to get limits.h early enough