And the variable too.

SELECT_FD_TYPE is obsolete

Back out a chunk of rev 1.34 that wasn't supposed to go in with the
capitalization corrections.

printf wasn't supposed to go in.

Move #include <syslog.h> to the one file that needs it

Add support for tracing libraries in static PIE binaries.  rcrt does not
contain "LD_TRACE_LOADED_OBJECTS" support, so this gets done by calling
RTLD_TRACE directly.
ok guenther

#include <sys/syslimits.h> to allow operation without requiring the noise
of <sys/param.h>

DEFAULT_PRIORITY and DEFAULT_QLIMIT no longer used

Routing sockets are here to stay, so delete the wrapper that would use the
defunct ioctl(SIOCSARP)

ok millert@

Explicitly mention that PATH_MAX includes the NUL in the ENAMETOOLONG
description.

Prefer linux_sockaddr over osockaddr, so the latter can go away.
sys_socket() supports SOCK_{CLOEXEC,NONBLOCK} now, so build on that.
doaccept() supports the non-inheriting-of-O_NONBLOCK, so build on that.
Merge compat_sys_{accept,send,recv}() into their only callers.
Lie a little and use struct sockaddr in the linux socket syscall args when
 it's a return argument to avoid a bunch a casts.
Delete many other unnecessary sockaddr casts

verified with a curl Linux binary

back it out properly

never tested with a make release

Talk about thread-local errno and do not mention "extern int errno".
Also correct the string version of error 0.  OK guenther@

The kernel doesn't actually care what a sockaddr's sa_len is on input,
so don't waste code setting it

improvment and ok florian@

No need to include pfvar.h, another leftover from relayd.  It was also
used for portrange operators which weren't used in httpd.

OK florian@

Use initial assignment of action to check for errors.

This simplifies things and make action = -1 no longer a dead store.

Also, spell FALLTHROUGH consistently.

reported by fritjof@alokat.org

finally enable the KEX tests I wrote some years ago...

adapt to new error message (SSH_ERR_MAC_INVALID)

switch ssh-keyscan from setjmp to multiple ssh transport layer instances
ok djm@

add experimental api for packet layer; ok djm@

Missing initialization; tjenahej@speedmail.se

Add arc4random/getentropy shims for NetBSD.

The latest NetBSD (6.1.5) arc4random does not appear to reseed the CRNG state
after a fork, so provide an override until the fork-safe version in CVS appears
in a release.

These are the same as the FreeBSD shims.

ok deraadt@

store compat flags in struct ssh; ok djm@

Oops, missed the new #include when manually applying the diff from
Helg (xx404 (at) msn.com)

adapt kex to sshbuf and struct ssh; ok djm@

`wellspring' Apple keyboards found on Macbook Air need slightly different
processing for Fn-key chords. From William Orr <will@worrbase.com>

Fix bracing in ASSERT_VP_ISLOCKED(vp) macro to not always panic()

From Helg (xx404 (at) msn.com)

move dispatch to struct ssh; ok djm@

Log the remote user in the access.log.
Pointed out by, tweak & OK reyk@

s/clt_fcgi_remote_user/clt_remote_user/
OK reyk@

Move doaccept() declaration from its .c file to sys/socketvar.h for use
by compat/linux

update packet.c & isolate, introduce struct ssh
a) switch packet.c to buffer api and isolate per-connection info into struct ssh
b) (de)serialization of the state is moved from monitor to packet.c
c) the old packet.c API is implemented in opacket.[ch]
d) compress.c/h is removed and integrated into packet.c
with and ok djm@

Delete ABI compat for osockaddr: 20 years and many ABI bumps since sa_len added

ok krw@ bluhm@ deraadt@

Prefer strdup() to malloc() + strcpy() and ftello() to ftell()
ok deraadt@ guenther@ millert@

Decouple auth parameters from struct server_config into struct auth.

OK florian@

Does not need netinet/ip_ipsp.h

Also print the fspace member of map entries when PRINT_VM_MAP_ENTRY is
requested.

ok deraadt@

Remove dead store causing clang to warn; dhill

use SYMLOOP_MAX rather than MAXSYMLINKS

mikeb points out that 'struct ipsec_policy' can also be hidden by _KERNEL

remove #ifndef #define blocks trying to handle lack of BUFSIZ
ok millert

mlk jr day is the third monday, and not just for new york.
from richard

SYMLOOP_MAX == MAXSYMLINKS, so use SYMLOOP_MAX

Kill loadavg.h and just use the fscale value directly.
There's no need for crazy casts of p_pctcpu in ORDERKEY_PCTCPU
since kinfo_proc exports it as uint32_t.  OK deraadt@

sparc also needs <machine/param.h> for CPU_SUN*.

fix format strings in (disabled) debugging

this test was broken in at least two ways, such that it
wasn't checking that a KRL was not excluding valid keys

First step of hiding many kernel-only parts of <netinet/ip_ipsp.h>
under _KERNEL, and adjust the one consumer (netstat) so that it requests
the exposure.  Will take a few more rounds to get this right.
ok mikeb

cleanup of the SMALL hiding from Mathias Svensson

Replace HOST_NAME_MAX+1 with NI_MAXHOST when the hostname is used
with getaddrinfo() or getnameinfo(), but keep HOST_NAME_MAX+1 when
the hostname comes from gethostname().
No binary change;  OK deraadt@

Make use of an msr available on recent Intel processors to obtain the
maximum supported temperature, Tj(Max).  As the temperature values are
relative to this value this should make the sensor values more accurate.

From Simon Mages.

Explicitly mention that PATH_MAX includes the NUL in the ENAMETOOLONG
description.

Make some $OpenBSD$ lines prettier/standardier by eliminating
superflous '*' after '/*' and adding blank after terminating '$'.
Also eases parsing of the lines by simple awk scripts.

Aesthetic approval from tedu@.

Remove unnecessary <netinet/ip_ipsp.h> includes

Back out changes unrelated to the ENAMETOOLONG diff.

<netinet6/in6_ifattach.h> is not needed here.

remove a couple of unused headers.

reported by Jonas 'Sortie' Termansen

Make a tmux-%u directory under TMUX_TMPDIR, like TMPDIR.

Only care about other permissions, allow group to be set.

Remove unnecessary duplicate S_ISDIR check, from Dmitri Paduchikh.

add a specialized version of version compares that should not update
everything all the time, at the bequest of aja.

(basically, we updated everything when runtime dependencies  changed, even
though we actually only care for wantlib, and hard dependencies, but those
trigger bumps all over the tree)

Experimental for now, turn on with -DSHORTENED. Likely to become the new
default if nothing bad comes out of it.

If SIGUSR1 and SIGUSR2 are unavailable, don't invent something crazy
honouring systems before 4.3BSD.

replace local MIN() with MINIMUM()

pass the size of the softc to free.

ok deraadt@

be a bit more careful in these tests to ensure that
known_hosts is clean

white space fixes. no binary change.

gcc does not understand that the loop conditionally sets bsod before
using it, so it warns...  Copy the warning silencing code from the
same function in ../ld.so

like in ps(1), fetch the FSCALE value using sysctl rather than using
the header version
ok guenther

move to <limits.h> universe
ok yasuoka

CMASK went out of scope with the recent <sys/param.h> removal.  But oh
golly gee, ftpd can handle that itself due to the glorious old practice
of #ifndef #define.  Remove that junk.
ok guenther

unneccessary cmask variable; ok guenther

If UID_MAX is not defined, this should fail to compile.
ok guenther

normalize formatting

cleanup sys/param.h comment

djm, your /usr/include tree is old

The kernel doesn't actually care what a sockaddr's sa_len is on input,
so don't waste code setting it

regression test for known_host file editing using
ssh-keygen (-H / -R / -F) after hostkeys_foreach() change;
feedback and ok markus@

some feedback from markus@: comment hostkeys_foreach()
context and avoid a member in it.

make ssh-keygen use hostkeys_foreach(). Removes some
horrendous code; ok markus@

convert load_hostkeys() (hostkey ordering and known_host
matching) to use the new hostkey_foreach() iterator;
ok markus

Back out previous.  HOST_NAME_MAX is only defined for newer POSIX
revisions so it is not safe to define MAXHOSTNAMELEN in terms of it.

introduce hostkeys_foreach() to allow iteration over a
known_hosts file or controlled subset thereof. This will
allow us to pull out some ugly and duplicated code, and
will be used to implement hostkey rotation later.

feedback and ok markus

Forward declare 'struct file' so that libkvm and others that indirectly pull
in sys/uio.h with _KERNEL defined don't warn.

ok deraadt@

Do not trust the content of event TRBs coming from the hardware and
maintain a list of possibly submitted commands.

Stop including <sys/param.h> from <netdb.h>.  Portable software must either
include <sys/param.h> (to operate in legacy mode) or if it wishes to operate
in the POSIX world use <limits.h>
ok guenther millert doug naddy

Eliminate a gcc warnings by not #defining _KERNEL when pulling in
<sys/socket.h>: it doesn't expose anything kdump cares about.
Also, COMPAT_43 is dead

warning noted by deraadt@

more and better key tests

test signatures and verification
test certificate generation
flesh out nested cert test

removes most of the XXX todo markers

make the signature fuzzing test much more rigorous:
ensure that the fuzzed input cases do not match the
original (using new fuzz_matches_original() function)
and check that the verification fails in each case

add a fuzz_matches_original() function to the fuzzer to
detect fuzz cases that are identical to the original data.
Hacky implementation, but very useful when you need the
fuzz to be different, e.g. when verifying signature

better dumps from the fuzzer (shown on errors) - include the
original data as well as the fuzzed copy.

didn't mean to commit these changes

enable hostkey-agent.sh test

Implement sending syslog messages over TLS.
OK reyk@

tweak previous
with help from jmc@

This size check was obviously commented out because of the incorrect
size computed for reports with reportID.

Tested by Benjamin Baier.

Do not calculated the length of a report with an extra byte for the
reportID because the kernel skips it.

Problem reported and fix tested by Benjamin Baier.

When restoring spsr values when handling traps use spsr_fsxc instead
of spsr_all so all the bits are restored.  Using the msr instruction
with spsr_all is treated the same as spsr_fc and does not include the
status and extension fields (bits 23:8).

This fixes the problem of some i.MX6 machines powering up with the
big endian bit set in the extension field causing them to crash
on returning from the first interrupt.

From NetBSD.

Switch some uses of msr that only deal with interrupts/mode to use
just the control field ('c' bits 7:0) instead of 'all' which includes
the flags field ('f' bits 31:24) which isn't modified.

Do not even try to dereference a NULL pointer.

Found the hard way by Peter N. M. Hansteen.

ok claudio@, phessler@