Use register names without the % prefix in the global register variable
declarations since clang doesn't register the %-prefixed ones.

Now that struct winlink has a session pointer, can remove some arguments.

There is no real need for window_printable_flags to allocate, make it
return a buffer from the stack.

If a #() command doesn't exit, use its most recent line of output (it
must be a full line). Don't let it redraw the status line more than once
a second.

Requested by someone about 10 years ago...

use strtonum instead of strtoll to parse the argument to -j

ok deraadt@ benno@

sync

Add sizes to free()

ok deraadt@ visa@

Add size to free()

ok deraadt@ visa@

load_cfg returns < 0 on error, not != 0. Problem reported by Kaushal Modi.

SA group has been renamed to bundle.  Adapt test.

Rename all SA groups to bundles consistently.  The first kernel
commit in 2000 that introduced the features already called them SA
bundles.  The word group is taken by Diffie-Hellman, reusing it
causes confusion.
OK hshoexer@

icmp6_rip6_input() was mostly duplicated code from rip6_input().
Merge these functions together and remove icmp6_rip6_input().
OK mpi@

Add support for dynamic "NAT" interfaces (-L/local interface).

When a local interface is configured, vmd configures a /31 address on
the tap(4) interface of the host and provides another IP in the same
subnet via DHCP (BOOTP) to the VM.  vmd runs an internal BOOTP server
that replies with IP, gateway, and DNS addresses to the VM.  The
built-in server only ever responds to the VM on the inside and cannot
leak its DHCP responses to the outside.

Thanks to Uwe Werler, Josh Grosse, and some others for testing!

OK deraadt@

Use the rt_rmx defines that hide the struct rt_kmetrics indirection.
No binary change.
OK mpi@

Update to terminfo.src 2017-04-01.

Add a suspend helper function, and do not allow detaching or suspending
while already doing so.

Style nits and a missing cast.

Make the test pass when an obj directory exists.

better example; from hiltjo posthuma
ok sthen

spelling fixes from michael w. bombardieri

When the data we have buffered to write to a terminal grows beyond a
reasonable amount (currently width * height * 8 bytes), discard all
output to the terminal and start trying to redraw periodically
instead. Continue with this until the amount of data we are trying to
write falls to a low level again.

This helps to prevent tmux sitting on a huge buffer of data when there
are processes with fast output running inside tmux but the outside
terminal is slow.

A new client_discarded format holds the amount of data that has been
discarded due to this mechanism.

The three variables (when to start this, when to stop, and how often to
redraw) are basically "works for me" at the moment, this is going in to
see how it goes and if it causes problems for anyone else.

Switch base tools from /dev/bpf0 to /dev/bpf. Now that /dev/bpf has been
around for two releases, it should be safe to do so.

ok bluhm deraadt sthen tb yasuoka

Bump numbers for /usr/obj allocation with *big* compilers.

ok otto

enlarge obj to satisfy new world order that has *big* compilers and
remove condition for static linking; ok tb@

More thoroughly reject direct access to unintended files, such that
URIs like http://man.openbsd.org/OpenBSD-current/mandoc.db and
http://man.openbsd.org/OpenBSD-current/man1/ do not cause display
of garbage.

Simplify patching of motd(5), also making it agree better with the
documentation if the first line of the file is blank.
Quirk reported by Anthony Coulter <bsd at anthonycoulter dot name>.
OK rpe@

Revert use of DECSLRM on iTerm2, it doesn't help as much as we throught,
and there are some question marks about it's support.

On terminals without DECSLRM, when a pane that is less than the full
with of the terminal scrolls, tmux needs to redraw the entire pane. This
results in a large amount of output data which can cause slow terminals
to struggle, particularly when many lines are scrolled together quickly.

This can be reduced by only redrawing when tmux doesn't hold any
buffered data for the terminal. If a redraw is required and data is
buffered, the redraw is deferred until all that data is consumed (it is
checked after every event loop, a timer is used to ensure this happens
at some point). While a redraw is pending, no additional data will be
written to the terminal.

The redraw still happens, now it is just pushed back if it is possible
it would just add more data on top of a terminal that is already
behind. This both gives the terminal a chance to catch up, and allows
tmux to process more scrolling (that would require additional redraws)
in the meantime.

Helps with a problem reported by Greg Hurrell.

Detect iTerm2 and use DECSLRM for it as well.

spelling fix;

Now that global -i is gone, pass -i through to the apropos(1)
expression parser, such that "apropos -i 'Nm~dump\>'"
finds kdump(1) and WCOREDUMP(2) and you don't need
to type the counter-intuitive "apropos -- -i 'Nm~dump\>'".

don't forget to fill in canary bytes for posix_memalign(3); reported by
and ok jeremy@

Add a format for number of bytes writtent to client, useful for debugging.

Add sparc64 support.

Do not check for BCE for a background colour that isn't needed, use
colour 8 instead.

Delete the undocumented and unimplemented man(1) -i (interactive
apropos) option.  It will not be implemented.  Featurism isn't the
plan for the future; simplicity is.

installation of the compiler creates include/g++ if needed; mtree does not
need to do this.

recent perl have signal names we don't have, protect against that.
also, allow child_error to take a parameter so that this can be used to decode
an arbitrary wait() result.

The apropos(1) manual still documents the unary -i operator for
regular expression search terms, but it appears that somewhere
along the way, the implementation got lost, so restore it.
Bug found while investigating other reports from Gonzalo Tornaria.

Style nits; no binary change.

ship clang with i386 and amd64.  It does not become the main compiler YET.
ok kettenis

Tweak parameters to decode_*, add a check or two, and thus gain most of the
sanity improvements reyk@ recently put into dhcrelay to ensure no more than
the captured packet is processed.

Fix previous: i forgot explicit NUL termination;
noticed by Gonzalo <Tornaria at cmat dot edu dot uy>, thanks!

After 11 years of pondering about it I think that brookdavis@freebsd.org
had it correct. Don't BPF_WORDALIGN() the value for the number of
bytes read() into the buffer. This could theoretically cause the
processing of 1 - 3 more bytes than were read.

ensure the buffer cache backs off all the way with the correct type
of memory, handling the fact that both queues are actually in dma
space when not flipping buffers high

Include client name in key logging.

Support packet aggregation for umb(4) on tx.

tested by bluhm@, ststp@ and Bryan Vyhmeister.
ok bluhm@ ststp@

Cleaning out /usr/obj is necessary the first time due to permission issues.
Make this more clearly visible and also state explicitly that this is no
longer necessary after completing a full release build.

ok deraadt

wipe some more macros that do not belong in C++ forwarding headers.
(breaks lincity-ng)
okay kettenis@

same hack logic as maketars. noticed by landry@/semarie@

add a clang comp list for arm64

use freezero() instead of explicit_bzero+free

removed unused function; from Edgar Pettijohn

Warn if pipex(4) is disabled but the configuration uses it.

Delete #include lines which are unnecessary.

Including sys/types.h and net/if.h are needed to use pipex(4).

sort

use freezero()

use freezero() instead of explicit_bzero + free

put clang binaries into clang set

use freezero() instead of 4-line conditional explicit_bzero + free

free(NULL) is ok so use it; from Michael W. Bombardieri

Fix pipex(4) not to cause panics on strict alignment architectures.

ok claudio

Rearrange ure_iff to resemble other drivers a bit more.
from Brad

/dev/rmt* is no longer relevant; from miod

some freezero() calls

use freezero()

memset() of password field should be explicit_bzero().
[Not using freezero() in yacc files yet]

freezero() can be used here

skip mtree creation of two include dirs.  gcc3 may still use them, but
if so gcc3 should create them itself.

The raw ip input functions are called from several places.  Use the
address family passed down with pr_input to check that the correct
one is used.
OK florian@

Use the address family passed down with pr_input to simplify
tcp_input().
OK florian@

Use the address family passed down with pr_input to simplify
udp_input().
OK florian@

matthieu@ observes % pax (without any arguments) hits pledge violation,
because it tries MTIOCTOP against stdin, the tty.  It is very inconvenient
to use isatty to distinguish this difference in userland, so return ENOTTY
for tty devices.
ok natano

Restore -kO Nd, -kO Nm, -kO sec, and -kO arch to working order.
They got broken in the SQLite removal.
As opposed to the rest of -kO, they are no longer very useful,
but they are certainly not supposed to fail assertions.
Issue reported by Gonzalo Tornaria <tornaria at cmat dot edu dot uy>.

Take a more radical approach to disabling colours: pretend we don't
HAVE_TERMINFO in our local config.h.  Some of the lower levels of LLVM
don't seem to respect our default setting.

As a result, delete the -ltermlib dependency.

Note that this breaks -fcolor-diagnostics=auto.  However, fixing the whole
codebase is out of scope for now ...

Prompted by an instance found in the AsmParser by deraadt@

ok kettenis@ patrick@

create a gross mechanism that can handle gcc-only, clang-only,
and gcc+clang architectures
with kettenis

Use better types for checksum() parameters and local var from
dhclient. Eliminate extra prototypes for checksum() and wrapsum().

consictently use .Dv NULL and a few other tweaks; ok schwarze@

don't manipulate hdr.len, it's used internally by libutil now; ok florian@

whitespace fixes

don't manipulate hdr.len, it's used internally by libutil now; ok florian@

Change build infrastructure to allow building both gcc and clang.  This
doesn't actually flip the switch yet, so aarch64 continues to be the only
architecture for which we build clang.

ok jsg@, deraadt@

Recognize nl_langinfo(CODESET) return values "646" and "" as aliases
for "US-ASCII", useful for different versions of NetBSD and Solaris.
Found by dtucker@ and by Tom G. Christensen <tgc at jupiterrise dot com>.
OK dtucker@ deraadt@

Test syslogd(8) "last message repeated" feature and -r option.

Add syslogd(8) option -r to suppress the summary line for pipe and
remote loghost as they are most commonly used for automated log
processing.  With -rr the "last message repeated" feature can be
disabled completely.
OK sthen@ deraadt@ jmc@

test display of long URIs

implement display of long .Lk URIs to match -Tascii

Skip objc tests with clang.

Fix handling of trailing punctuation in .Lk.
This macro is unusual in so far as trailing punction needs to remain
inside the scope because it must be inside, not after the display
of long URIs in terminal output mode.
Improves formatting of fw_update(1), help(1), less(1), sendbug(1),
acx(4), inet6(4), ipsec(4), oce(4), isakmpd.conf(5), afterboot(8),
release(8), traceroute(8).

Change COMPILER_VERSION tests which limited additional warnings to gcc4
to instead skip them on gcc3 as clang can handle -Wpointer-sign and
-Wold-style-definition.

We need to update the ia6_updatetime when changing the vltime/pltime
vio ioctl(2). Otherwise ifconfig shows the wrong times since
vltime/pltime are calculated from this value on export.

The handling of router advertisements does the right thing.
OK bluhm

Don't bother moving the cursor for empty lines.

Remove a couple of redraw flags that no longer have any effect.

Memory leak, from David CARLIER.

Use EL1 to clear lines when redrawing the leftmost pane, rather than
spaces.

from brad: document rgephy

Disable colored diagnostics in LLVM's lld.

Requested by deraadt@

According to termcap(3), char PC, *BC, *UP need to be extern. Fixes an
ld(1) error found by mestre a while ago.

fix suggested and ok naddy

Always evaluate expression in BUG_ON() macro to avoid unused variable
warnings.  Makes non diagnostic kernel compile.
OK kettenis@