We need to loop over the files once to work out the width of the labels
before testing them.

Trying to drop privileges means we can't handle ARG_MAX arguments, so
remove it for now.

Add a couple of missing spaces (style nits).

Use symbolic constants for st_mode flags, no binary change.

If ~/.magic exists but can't be used, fail rather than silently falling
back to /etc/magic.

Fix a couple of errors in the magic source files.

New implementation of the file(1) utility. This is a simplified,
modernised version with a nearly complete magic(5) parser but omits some
of the complex builtin tests (notably ELF) and has a reduced set of
options.

ok deraadt

Revert back to using GCC builtins.  This code triggers an off by one in
device_unref() as found by deraadt@.

another thinkpad event to ignore; from mp39590@gmail

vio: If enqueue fails, drop packet.

From brad@

Enable the NX bit and use it in the PAE pmap code.  PAE is still disabled
while we're chasing at least one remaining bug.

ok mlarkin@, deraadt@

improve wording.

ok jmc@

read_defaults has the thing for strtonum to chew on in cp, not
optarg.

fixes a segfault introduced by the atoi to strtonum change and
reported by ajacatout@

combine -Dd onto one line and update usage();

add ssh-agent -D to leave ssh-agent in foreground without enabling
debug mode; bz#2381 ok dtucker@

Use "softintr_pic0" instead of "softintr_fakepic" when faking a
struct device so there is enough space in the buffer for a NUL and
the unit is included in the string.

discussed with deraadt and millert

2*len -> use xreallocarray()
ok djm

rename xrealloc() to xreallocarray() since it follows that form.
ok djm

Replace atoi() usage with strtonum().  OK deraadt@

We cannot log errors with relay_close() before allocating
se_log evbuffer.
(Same problem as the one just fixed in httpd(8))
OK benno

We cannot log errors with server_close() before allocating
clt_log evbuffer.
server_close() calls server_log() which uses ctl_log.
Crash reported by Daniel Jakots <vigdis AT chown DOT me>, thanks!
OK benno

Unify mdoc_deroff() and man_deroff() into a common function deroff().
No functional change except that for mdoc(7), it now skips leading
escape sequences just like it already did for man(7).
Escape sequences rarely occur in mdoc(7) code and if they do,
skipping them is an improvement in this context.
Minus 30 lines of code.

Get rid of two empty wrapper functions.  No functional change.

Fix 13 years old typo that should be responsible for the unhappiness
of UVM on PowerPC architectures by breaking pmap_is_referenced() and
friends.

ok kettenis@

tedu remnants of the previous attempt to implement page zeroing in
the idle thread.

ok deraadt@

replace the use of struct ifqueue in pipex with mbuf_queues.

this has a slight semantic change. previously pipex would only
process up to 128 packets on the input and output queues at a time
and would reschedule the softint if there were any left. now it
mq_delists the current set of pending packets and only processes
them. if anything is added to the queues later they'll cause the
softint to run again.

this in turn lets us deprecate sysctl_ifq since nothing uses it
anymore. because niqueues are mostly wrappers around mbuf_queues,
we can provide sysctl_mq and just #define sysctl_niq to it.

pipex bits are ok yasuoka@

use reallocarray instead of calloc; avoid the zero before infill
ok nicm

Do not need to buf[0] = 0 before strlcpy(buf, ...

delete a NOTDEF of unused code

KNF and other style adjustments to make this more scrutible.
reviewed by millert

Add a simple regression test for sshd's configuration parser.  Right now,
all it does is run the output of sshd -T back through itself and ensure
the output is valid and invariant.

Two small fixes for sshd -T: ListenAddress'es are added to a list head so
reverse the order when printing them to ensure the behaviour remains the same,
and print StreamLocalBindMask as octal with leading zero.  ok deraadt@

Check for and reject missing arguments for VersionAddendum and ForceCommand.
bz#2281, patch from plautrba at redhat com, ok djm@

avoid an uneeded deref
ok blambert@ guenther@

error in strtonum() conversion; found by Nathanael Rensen

dont need \n on panic strings

Remove forgotten empty files, from miod, thanks

Expand SLIST macro of unix pcb members before printing the pointer.
Fixes a gcc warning.  No binary change.
Found by and OK deraadt@

Use .Ar for placeholders and .Cm for arguments to be provided verbatim.
No text change.
Inconsistent markup noticed by Alex dot Greif at greifdesign dot net.
OK jmc@ ajacoutot@.

Make session_has return a flag, returning the first winlink found is a
recipe for errors.

Change the windows array into an RB tree and fix some places where we
were only looking at the first winlink for a window in a session.

window_index is only used in one place (window_destroy) so inline it there.

Update termtypes.master to terminfo.src from ncurses-5.9-20150418.

Init a variable to zero so values aren't or'd into uninitialised memory.
From jeundery at gmail dot com

Add the necessary glue to keep carp(4) working while other pseudo-drivers
are converted to if_input(), this time without breaking parent-less setup.

ok sthen@, dlg@

Add missing header for NBRIDGE chunk, found the hardway and diff from
Rafael Zalamena.

Implement the MI atomic API for PowerPC to avoid using gcc builtins that
include extra sync operations.

ok kettenis@

Reduce differences between pae and no-pae modes.

discussed with deraadt

sync

Init rt to NULL in arpresolve().  Not a problem in practice
as la will be NULL when rt is uninitialised which should cause
the function to return, but makes the code easier to follow.

ok mpi@ miod@

Init error to 0 for sdmmc_mem_single_{read,write}_block
otherwise the value would be uninitialised in the unlikely
case of being called with length 0.

having macros provide semicolons is dangerous.

use correct key for nested certificate test

unknown certificate extensions are non-fatal, so don't fatal
when they are encountered; bz#2387 reported by Bob Van Zant;
ok dtucker@

Look up indexes as number before name, makes more sense if windows are
named starting with numbers. From Thomas Adam.

Always format real layout even when zoomed.

Do not die on USR1 if any of the socket parent directories are
missing. Reported by Robin Powell.

Simplify error messages when socket connect fails, suggested by "Karthik K".

The free callback could end up being fired before the done callback
(happens on Cygwin), so use a reference count instead of a single
flag. SF bug 188 reported by "iceboy".

Fix moving windows to nonexistent indexes when renumber-windows is
off. From Thomas Adam, reported by Daniel Levai and Theo Buehler.

Remove an extra lcr3 that snuck into pmap_switch during yesterday's
cleanup, responsible for various reaper panics pointed out on bugs@ this
morning.

ok deraadt@

Improve divert-to specification parsing w.r.t. rule address family.
ok henning

According to RFC 4861 and the actuall implementation rltime may not
exceed 9000 seconds.
Reported by Alexander Lobodzinski on bugs@, thanks!
OK benno@

Avoid a use after free when the target node is deleted during validation.
Bug reported by jsg@.

Bind mouse dragging so that it is passed through to applications if they
want it rather than entering copy mode.

Don't eat the mouse event that triggers a drag end because we may want
to pass it on to application inside the pane.

Put mouse_any_flag back, don't know where it went to (still in man page).

Pass mouse events through to commands for if-shell.

cmd_mouse_pane can return NULL, check for that.

Section 1, not section one: section numbers are always numerical.
For example, "man -s one" wouldn't work either.
Patch from Alex Greif <alex dot greif at greifdesign dot net>.

disable *8 again for now. incoherent archs arent having much fun with it.

append the setdir ("5.7/amd64", "snapshots/vax") as get-parameter "path"
when fetching the autoinstall response file.
a webserver that serves static files doesn't give a damn. if I map that
to something that dynamically creates the response file, i can use that to
construct the sets path, or play other arch and/or version dependent
games.
ok krw uwe

Back in January, Kaspars at Bankovskis dot net reported that the wc(1)
manual claims that -m would count characters, while our version of the
command actually doesn't, lacking POSIX multibyte support.  Since
nobody stood up to fix the code, sync the docs with reality for now.
OK jmc@

When diff(1) finds differences, it returns an exit status of 1.
In that particular case, refrain from printing "diff: exit code 1"
because that exit status doesn't indicate an error condition.

Issue noticed by and patch OK'ed by ajacoutot@.
"I agree with the goal, and I suspect the diff actually achieves it" guenther@.

Revert previous.  Packets going through carp_input() but not destinated
to the carp interface would end up enqueued without being decapsulated.

Found the hard way by shten@

Add back a backslash removed in rev 1.42 so KEX_SERVER_ENCRYPT will
include aes again.

ok deraadt@

Remove a duplicate variable and #define that snuck in, in a previous
commit.

ok deraadt@

fix a memory leak if tls_read() fails.  ok henning@

fix a memory leak.  ok blambert@

No need to do "size = (size_t)sb.st_size" both before and after
a fstat() call.
ok mikeb@ markus@

init both fds passed to pipe as -1 instead of initing one twice
ok deraadt@ guenther@ miod@ millert@

The ELF psABI for PPC specifies that the stack shall always be 16-byte aligned.

ok mpi@ deraadt@

rework sgi mutexes to use the owner pointer as the lock.

this is like src/sys/arch/alpha/alpha/mutex.c r1.14.

this changes sgi mutexes so they record which cpu owns the lock
rather than just if the lock is held or not. the diagnostics compare
the owner to the current cpus curcpu() address so they can actually
tell if the current cpu holds the lock instead of whether any cpu
holds the lock.

instead of using custom asm to implement a cas this uses atomic_cas_ptr.

while im here i also shuffled the code. on MULTIPROCESSOR systems
instead of duplicating code between mtx_enter and mtx_enter_try,
mtx_enter simply loops on mtx_enter_try until it succeeds.

this also provides an alternative implementation of mutexes on
!MULTIPROCESSOR systems that avoids interlocking opcodes. mutexes
wont contend on UP boxes, theyre basically wrappers around spls.
we can just do the splraise, stash the owner as a guard value for
DIAGNOSTIC and return. similarly, mtx_enter_try on UP will never
fail, so we can just call mtx_enter and return 1.

tested by and ok miod@

Eliminate the last uses of *fork's second syscall return register; the pid
is zero in the child

ok deraadt@ miod@

Reduce differences between i386 pmap modes.

ok kettenis@

Remove most static, gives us much better ddb tracebacks from secondary
processors.

Log if we refuse to continue the exchange when another one that corresponds
to the same policy is already active.  OK markus, hshoexer

Remove unused-but-set variables, from Thomas Adam.

Support for multiple key tables to commands to be bound to sequences of
keys. The default key bindings become the "prefix" table and -n the
"root" table. Keys may be bound in new tables with bind -T and
switch-client -T used to specify the table in which the next key should
be looked up. Based on a diff from Keith Amling.

Fix asking for list of http servers via '?'. The 'more' now used
on the install media does not (currently) support '-e' or other
posix nifties, but does exit after displaying the last line.

Should fix scanning for wireless networks too.

Reported by Adam Wolk on misc@.

ok deraadt@

Style nit - unnecessary brackets.

totally forgot about the !uid notation in newuser when adding the early
cache for pkg_create. Reminded by sthen@

Fix previous:  Don't let man(1) warn twice about non-existant names.
Again noticed by deraadt@.

Avoid out-of-bounds read access before the beginning of the
mdoc_macros[] array.  This sometimes prevented proper warnings
about text nodes preceding the first section header.

Make jump-to-backward/jump-to-forward repeatable with
jump-reverse/jump-again, from Jacob Niehus.

Remove a bad typo introduced in rev 1.185 and found by jsg@.

Because of this semicolon, a local route was *always* created.  This
should have no impact except for IPv6 addresses on loopback interfaces
for which the IPv6 code was trying to create a route to "::1" twice.

This should now be fixed so we can fix the bug that was hidding a bug
who was hidding a bug... lalala...

ok henning@

Do not treat loopback interfaces as p2p interfaces and create only
one route to "::1".

Due to a clever BSD trick, the `ifa_dstaddr` field of addresses on
IFF_LOOPBACK ifps is set to the same value that `ifa_addr`.  That's
why filtering for broadcast addresses is so complicated, because
guess what, `ifa_broadaddr` is the same as `ifa_dstaddr`!

Sadly our IPv6 code was "only" checking for `ifa_dstaddr` without
looking if the ifa was attached to a IFF_POINTOTPOINT interface.
So it always tried to create two routes to "::1" and, with the
recent RTF_LOCAL work, succeed.

You should now have only one local route to "::1".

 ::1             ::1         UHl       14  0 32768     1 lo0
-::1             ::1         UH         0  0 32768     4 lo0

ok henning@

Always call rt_ifa_dellocal(9) when removing an IPv6 address.

The routing layer already check for the correct ifa when asked
to delete a local route, so do not try to be clever here.

This change also prevent having a NULL ifp pointer in your routing
table when you delete loopback interfaces having the same address.

ok henning@

Use a more sensible buffer size for flags string.

port src/sbin/ping/ping.c r1.119 and r1.121 to ping6:

> add a mac to the timestamp payload and calculate it with siphash.
>
> this lets us have some confidence that the timestamp hasnt been
> damaged or tampered with in transit.
>
> discussed with jp aumasson
> ok florian@