Unbreak route6d.

Instead of breaking sendmsg(2) by adding unneeded space to its cmsg
item, add space to the cmsg used by recvmsg(2), where it will be used
to get the incoming packet hop limit.

Reported by several over the last years, and more recently by 'bsdsx',
who tested it against NetBSD route6d.  Also works against Quagga ripng.

ok deraadt@ sthen@

save some file descriptors. instead of a pipe, use kevent to watch parent

naddy would like the child to exit when the parent dies.
hook up a pipe between them and watch for eof in the child.

Missing local.

ok schwarze@

No longer talk about -b flag, it's gone.

Drop usage of TMPDIR.
While here, stop refering to /tmp/sysmerge.XXXXXXXXXX, that's a script
internal we don't need to know about.

sync

test mixing of tabs with Ta

Make sched_barrier() use its own task queue to avoid deadlocks.

Prevent a deadlock from occuring when intr_barrier() is called from
a non-primary CPU in the watchdog task, also enqueued on ``systq''.

ok kettenis@

life is simpler if all requests go in the fifo, and then just remove them
in the error case instead of duplicating code.

better handle anchored FQDNs (e.g. 'cvs.openbsd.org.') in hostname
canonicalisation - treat them as already canonical and remove the
trailing '.' before matching ssh_config; ok markus@

deraadt tells me i'm supposed to check if connect() actually worked.

two phase handling for tcp so that slow connects don't stall the process

Cast isspace() argument to unsigned char.

ok jca@

Move -t and -w functionality to -a. Both flags are in the way for a
merge with ping(8). Let's see if we can shove every weird and special v6
functionality into -a.
suggested by and OK sthen@

Modernize allocation by:

 * removing unneeded casts of void* return values
 * replacing varied and creative error messages with the allocation
   function's name
 * replacing errx() with err() so that the errno string is reported

ok beck@, jung@, millert@

Cast iscntrl()'s arg to unsigned char.

ok nicm@

0 -> NULL when comparing with a char*.

ok dtucker@, djm@.

Repair the pty check for kernels without pty support.

Check file sizes only for regular files. The current code breaks savecore
due to its kvm handling.

ok deraadt

Pledge the syslogd privsep process with "stdio rpath wpath cpath
inet dns getpw sendfd proc exec".
OK deraadt@

The hosts.lpd examples file does not contain a single example.
The file format is so simple that no example is needed.
All relevant documentation is already available
from the proper place, which is the lpd(8) manual.
Consequently, delete the empty file.
OK millert@ dcoppa@ beck@ deraadt@

Allow PTMGET with "tty rpath wpath" but restrict only to /dev/ptm by
checking cdevsw. ok deraadt

save request length in cache. naddy noticed we weren't getting any hits.

Remove pointless externs - the structs are declared in the same files a
few lines above.

Expand DECLARE_ASN1_ALLOC_FUNCTIONS and DECLARE_ASN1_FUNCTIONS_const
macros. The only change in the generated assembly is due to line numbering.

Remove pointless uses of DECLARE_ASN1_ENCODE_FUNCTIONS_const.

DECLARE_ASN1_FUNCTIONS_const already includes this macro so using both
means we end up with duplicate function prototypes and externs.

wrap a long line

Fix use of pointer value after BIO_free, and remove senseless NULL checks.
ok bcook@

Always allow a r/w opening of /dev/null though the namei check.  This
pattern is common, especially because of daemon(3) usage.  Will probably
help some daemons move their pledge() calls further upwards.
ok doug,

ugly white space

delete pledge_bind_check() function and remove pledge_bind_check() call from sys_bind().

bind(2) still require PLEDGE_INET or PLEDGE_UNIX in order to be called, due to
SYS_bind entry in pledge_syscalls array. The diff restores also the ability for
PLEDGE_UNIX to call bind(2) (pledge_bind_check function missed that).

problem spotted by doug@
OK deraadt@

Place TIOCSTI reminder block better

For "tty" pledges, treat TIOCGPGRP and TIOCGWINSZ like TIOCGETA -
returning ENOTTY instead of killing the process.

Merge nlist out of boundary access fix with other nlist implementations.
While at it, merge style and typo fixes back into nlist(3), too.

ok deraadt, jsing, millert

Put tls_config_verify_client_optional() in the right place.

Fix tpyo.

Implement real "flock" request and add it to userland programs that
use pledge and file locking.  OK deraadt@

actually include the prerequisite dependency for BIO instead of doing nastyness

If a DOWN route entry is passed to a L2 output function, be dumb and
simply use it.

In most of the cases doing a route lookup at this point is a noop as
it will return you the same DOWN entry you already have.

The exception is the case where the route has been removed from tree
since your kernel looked for it.  So what?  It's just a blue packet.

Note that this "exception" can only happen if your sending path does
not run under the KERNEL_LOCK.

ok mikeb@

Put some iwm(4) debug code into #ifdef IWM_DEBUG.
ok mpi@

Clean up iwm(4) scanning logic a bit: Reset sc_scanband in callers of
iwm_mvm_scan_request() and always call ieee80211_end_scan() when done.
ok mpi@

Oops, committed old version of previous diff with a typo in it: NLL -> NULL

In iwm(4), correctly size and map the mbuf used for large firmware commands.
Fixes occasional firmware errors while bringing the interface up or scanning.
ok phessler@

Don't free after calling paste_set but do after evbuffer_add, from Theo
Buehler.

Validate parsed ELF values to prevent out of boundary accesses.
While at it, return proper return value when encountering a stripped
binary. Instead of -1 (illegal file), it should be the amount of symbols
that were tried to be resolved.

ok millert

Disable !-command to escape to a shell. You are supposed to play, press
^Z, or open up another terminal if there is something else to do.

ok deraadt

Add native support for ed-style diffs. No need to pledge "proc exec" anymore.

ok deraadt

doug and I think the kernel has enough features to support
pledge "stdio rpath wpath cpath getpw proc exec tty" now.
It will be hard to drop many of those features unless cu becomes
privsep for the "upload" commands.

FIOSETOWN/FIOGETOWN were added to "ioctl", but study finds no programs
currently needing them.  delete 'em for now.
ok doug

Add TIOCCBRK and TIOCSDTR to the whitelist for pledge ioctl.

cu(1) uses these.

ok deraadt@

Pledge support for the parent/resolver in identd(8).

This limits the resolver to just "stdio getpw" or "stdio getpw rpath"
depending on whether ~/.noident files are checked.

The child/listener cannot use pledge yet because it calls a sysctl that
hasn't been whitelisted.

"commit" deraadt@

Pledge for ftp(1) in non-interactive mode.

We will iterate and remove some of the pledges in the future.  This is
conservative for now.

Tested by sthen@ and myself.
ok deraadt@

Remove -B from EXAMPLES; reminded by jmc@

Add allocarray(), an overflow-safe allocation function.

We avoided reallocation support because it demands more fancy footwork
to deal with the prepended link struct.

This has been on my mind for a while, and a 2010 security review of mksh
by the Android security team's Chris Palmer suggested it.

ok nicm@. Also discussed with millert@ and tedu@.

sync

fine tune the logging some more

simplify logging functions. once a daemon, always a daemon

safety check that we're dealing with the filter we expect

most things should be static

exit(1) is better for the impossible condition

fix some signed/unsigned integer type mismatches in format
strings; reported by Nicholas Lemonias

Do not abuse .Nm for emphasis;
patch from Michael Reed <m dot reed at mykolab dot com>.
Also drop .Tn while here.

Delete two preprocessor constants that are no longer used.
Patch from Michael Reed <m dot reed at mykolab dot com>.

argument to sshkey_from_private() and sshkey_demote() can't be NULL

After spawning, the parent can pledge "stdio rpath wpath cpath"
from rob pierce

Remove three distracting aliases for NULL.

ok nicm@

Simplify the part of args() that is handling .Bl -column phrases:
Delete manual "Ta" handling because macro handling should
not be done in an argument parser but should be left to the
macro parsers, which exist anyway and work well.
No functional change, minus 40 lines of code.

Confusing and redundant code found while investigating
an old bug report from tim@.

When blk_full() handles an .It line in .Bl -column and indirectly
calls phrase_ta() to handle a .Ta child macro, advance the body
pointer accordingly, such that a subsequent tab character rewinds
the right body block and doesn't fail an assertion.  That happened
when there was nothing between the .Ta and the tab character.
Bug reported by tim@ some time ago.

it is perhaps better style to not call close() on -1, even if harmless

make sure req is zeroed in tcp case

better memory handling of the request/cache chain

do not insert entry into cache until it's fully formed

doh, not all requests are the same size. check len first.

assert is the wrong tool

the inet sockets don't work well with inet6 addrs. pick family from addr.
detected by naddy

everybody can build rebound

no mail for _rebound. deraadt

introduce logerr, since most logging is followed by exit

don't allow NSD to pick up libevent from /usr/local if the libevent2
package is installed.

now with _rebound user, we can try a little harder at privdrop

_rebound user and group (52)

make the HUP interlocking in the parent work better.

trivial KNF

In syslogd replace the dprintf() macro with a logdebug() function
as dprintf(3) is in libc now and does something different.
OK guenther@

Pledge login_token with "stdio rpath wpath cpath fattr getpw tty".
OK deraadt@

just a space in usage, from deraadt

Remove an unused included header (sys/stat.h).

add a hint about the config file, until it changes

child can be pledged down a bit to just sockets and io

When using a pf rule with both nat-to and rdr-to, it could happen
that the nated source port was reused as destination port.  Do not
initialize nport at the beginning of the function, but where it is
needed.
OK sashan@

import rebound, a lightweight dns proxy, for further polishing

Introduce an unsigned char variable for the ctype function calls.

ok millert@

Don't Xr flock, since that is not the locking method used.
ok millert

Remove disklabel -B (NUMBOOT) support. All the platforms which used to need
it are now using MI installboot for that purpose.

ok krw@ deraadt@

Avoid a race between fopen(3) and fchmod(2).  Use umask(2) and
unlink(2) and fopen(3) to prevent an attacker to open an old file
with wrong permissions before the secret is written into it.  This
also guarantees that a new file with correct permissions is created.
Without fchmod(2) "fattr" can be removed from pledge.
with and OK deraadt@

No need to create links for xxboot now that MI installboot is the preferred
way to install boot blocks.

Use MI installboot instead of disklabel -B to install boot blocks.
ok krw@ deraadt@

Add an extra argument to bootstrap() to allow for a limited overlap between an
existing partition and the boot blocks span, and update all callers to require
an overlap limit of zero sectors (thus not changing their behaviour).

Then, add proper support for vax: copy the 2nd-stage boot block to /boot and
install the 1st-stage boot block at the beginning of the disk, retaining the
disklabel; allow for an overlap of up to 16 sectors, which is perfectly fine
as long as your `a' partition is FFS.

Note that regular installs will not even have such an overlap, because the
default OpenBSD span on a disk on vax starts at sector 16, but installation
media use sperific layout which require this.

ok krw@

add missing comma and missing range restriction, found by smilint