tidy up the flags section a little: supply a little more indent, and reduce
the amount of modal verbs going on;

When running flood ping with count packets (-c) set an alarm after
sending all packets otherwise ping will wait forever to see all
answers - which might not arrive on lossy links.

Problem pointed out by, input & OK claudio

make the logic for deleting first slightly more obvious
in particular, there's nothing to try if we don't have older packages

Remove support for running ssh(1) setuid and fatal if attempted.
Do not link uidwap.c into ssh any more.  Neuters UsePrivilegedPort,
which will be marked as deprecated shortly. ok markus@ djm@

Replace VATTR_NULL() with memset(3) in fusefs_getattr(). VATTR_NULL()
sets all members of struct vattr to VNOVAL (-1) instead of 0, which is
what is appropriate here. The VATTR_NULL() macro is intended for
initialising struct vattr when setting attributes.

ok mpi@

Don't log_warn on normal conditions, it should be log_debug.
Pointed out by semarie@

stop assuming prefix lengths, remove inet6_makenetandmask()

This removes any logic that implies IPv6 destination host addresses to be
/64 subnets so they are taken as is.
RFC 3587 deprecated this in 2003 and our manual page actually states:

  The route is assumed to be to a network if any of the following apply to
  destination:

  *   [...]
  *   it is an IPv6 address with a “/XX” suffix (where XX is the number of
      bits in the network portion of the address and is less than 128)
  *   [...]

  If destination is a valid IP address or host name, it is presumed to be a
  route to a host.

Stripping relevant code from `inet6_makenetandmask()' left the function as
dummy wrapper around `prefixlen()', so zap it completely.

Discussed with and positive feedback from many, OK benno henning

TX packets, which can be either Ethernet or control packets, must be
sent in order.  Otherwise it is possible that the key is set before
we send out the EAPOL packet, or that packets are sent out before
the key is set.  Thus modify the SDIO backend to put both types into
the same internal TX queue, which will be sent asynchronously.

Discussed with bluhm@

When a key isn't in the first table, we need to try the same key again
not the any key. Also rename some labels. Fixes GitHub issue 1406
reeported by Mark Kelly.

some more style fixes

Add missing $OpenBSD$ markers.

remove unused, empty file

Replace getprogname() to argv[0] in bnaddsub

ok tb@

allow shell globs to match program and hostname selector tags via
fnmatch(3); ok sthen@ bluhm@

vmd(8): fix vmctl -b option for i386 kernels.

ok pd@

Don't send FBT_DESTROY if the FUSE connection is still PENDING. Also
don't attempt to determine the size of the root inode. This is because
we don't need to know the size of directories and FBT_GETATTR will also
cause a deadlock if fuse_unmount(3) is called before the file system
has a chance to process fbufs.

Add corresponding regression test.

ok mpi@

libfuse now supports -f command line option; uncomment test.

unvname_new() is passed something that is always 64-bits so the
parameter should be declared uint64_t and not int.

From & for semarie@ who isn't committing at the moment and ok beck@
who is on the road.

Switch to MSG_PROTOCOL_BGP4MP_ET formats for update and state mrt messages.
The _ET format adds an additional microsecond time field which makes those
message dumps more informative. The various table dumps are not modified
since there the time especially between entries plays a secondary role.
OK benno@

we have splraise() now

previously the code would do a series of comparisons to IPL_FOO and
IPL_BAR, and use splfoo() and splbar() as needed, but for like a
dozen different IPl levels (i may be exaggerating). splraise() rolls
all of that up.

ok deraadt@ mpi@

Slot 0 in the hostbased key array was previously RSA1, but that is
now gone and the slot is unused so remove it.  Remove two now-unused
macros, and add an array bounds check to the two remaining ones
(array is statically sized, so mostly a safety check on future changes).
ok markus@

Document behavior change of EC_POINTs_mul() again.

Recommit Billy Brumley's ECC constant time patch with a fix for sparc64
from Nicola Tuveri (who spotted the omission of ecp_nist.c from the PR).

discussed with jsing
tested by jsg

use ANSI C function declaration; no binary change

use 'switch' rathan than an if-else construct to match on the netflow version to in pflow_get_mbuf(),
to match the rest of the file

ok benno@ florian@

Insert the appropriate uvm_vnp_uncache(9) and uvm_vnp_setsize(9)
kernel calls to ensure that the UVM cache for memory mapped files is
up to date.

ok mpi@

Add support for the BCM43455 SDIO chip to bwfm(4).

In certain modes SDIO-connected bwfm(4) chips might expect us to read
another packet (even though nextlen is set to none) to issue more RX
interrupts.

Be more explicit in FETCH_TIMEOUT description.
It only happens on the command line.
If you want to put it in a config file, it's a DEFAULT or localhost
property called fetch_timeout instead.

Implement FBT_FSYNC, which is called on fsync(2) and fdatasync(2).
Currently ignores the a_waitfor argument and always invokes the file
system's fsync implementation synchronously.

ok mpi@

the AP sends their crypto parameters as a mask, so let us still select it
in case we want WPA2 and they have WPA1 still enabled

don't save or use auto-join when the card is not in station (aka client) mode

OK stsp@

fix thinko

okay mestre@

Re-set the pointer to the bwfm event after the strict alignment
check.

Remove support for loading HostBasedAuthentication keys directly in
ssh(1) and always use ssh-keysign.  This removes one of the few remaining
reasons why ssh(1) might be setuid.  ok markus@

network.conf

byebye network.conf.template

Use struct filterstate in rde_update_dispatch() and pass it down to
functions like rde_update_update and path_update. This will allow to
move more pointers from rde_aspath to the prefix.
Looks good benno@

make this build again after changes made in rev 1.74
ok mpi@ phessler@

Add an "Any" key to run a command if a key is pressed that is not bound
in the current key table. GitHub issue 1404.

reduce duplicate code, fix typo/free correct buffer

In filteropts_to_rule():

* Merge `once' handling from `anchorrule' and `pfrule'
* Remove/shorten duplicate code block
* Fix typo I introduced with r1.678 that frees the wrong buffer (twice)

OK sashan

rtadvd is only interested in IPv6 so limit route socket to AF_INET6.
OK florian@

Fix for uninitialized variables.

Found by and ok jsg@, ok markus@

typos, from Michael W. Bombardieri.

keep options.identity_file_userprovided array in sync when we load
keys, fixing some spurious error messages; ok markus

memleak in unittest; found by valgrind

memleaks; found by valgrind

vioscsi driver has been stable enough so we can move a good portion of these
log_debug messages to be DPRINTF (compile time).

Prompted by reyk@.

Ok reyk@.

re-commit the removal of the EC_POINTs_mul() regression tests with num > 1

Document xcrypt in the amd64 version of cpu.4.

Bits taken from the i386 version.

OK jmc@

recommit label indentation part of the backout; clearly unrelated to the
breakage.

Revert unrelated change to vioscsi.c

(I had this in my tree to silence vioscsi.c log_debug, but Carlos
already has a better diff for that)

Track resources and enforce cpu/memory/interface limits for non-root users.

The limits are currently hard-coded and undocumented (4 CPUs/VMs, 2G
memory, 8 interfaces) but will be configurable in an upcoming diff.
These limits are tracked in total usage; for example, a user will be
able to run up to 4 VMs with 512M of memory or a single VM with 2G.

OK ccardenas@ mlarkin@

Restore correct behaviour to pledge for access and stat, which was broken
by the unveil diff.   Noticed at, and diff from semarie@

tweak previous;

adjust the example hostname.if to show "wpakey" on the same line as "nwid",
and provide an IPv6 autoconf example.

while here, split interface settings from IP assignment.

OK jmc@

adjust hostname.if examples to show "wpakey" on the same line as "nwid"

OK jmc@

Implement RFC 8106: IPv6 Router Advertisement Options for DNS
Configuration.

mark up managed / other configuration

It's a packet, not package.

nuke newd leftover

remove unused variable

Fix new-window -k, GitHub issue 1403.

$OpenBSD$

Also revert regression tests so that EC_POINTs_mul() with longer vectors
gets exercised again.

back out ecc constant time changes

after the constant time commits various regress tests started failing
on sparc64 ssh t9, libcrypto ec ecdh ecdsa and trying to ssh out
resulted in 'invalid elliptic curve value'

ok tb@

don't run cvs_client_sendfile() if its file_type is CVS_DIR, but do not
depend on it being CVS_FILE explicitly.

unbreaks remote operations on removed files.

Document that route monitor also takes -T rtable as an argument.

Make route monitor display changes in all routing domains by forcing
the route filter to set RTABLE_ANY. Previously only the routing
table/rdomain of the route process was displayed (that being the
kernel default).
ok kn@ claudio@ and henning@

get rid of two more implicit ktable_get with rdomain 0.
should not change anything when run in rdomain 0.
ok henning@ phessler@ claudio@

Don't pass an uninitialised size value to free(9).  Pointer argument is
NULL in this path so free will return early without accessing it.

ok jca@ tb@

Remove mention of old man directory

OK espie

sync

.El should have been zapped too;

zap network configuration section, it's a level of tinkering that's
not really supported.

add another ref to PORTS_PRIVSEP here, as it is really a good idea.

Remove cscope leftover and a stray comma.

From "kshe" on tech@.

openssl app timers: TM_START -> TM_RESET, TM_STOP -> TM_GET

Much more apt than the current operation names.

Names suggested by jca@ ages ago.

ok jca, jsing

tweak previous; ok reyk

Correct some typos and also ran it through mandoc -Tlint

OK millert@ deraadt@ and jmc@ who also gave me a cluebat stick

Add & update author credits for Mary Ann Horton.

Add credits for leave(1) and w(1); update credit for script(1).

Supported by the Spinellis repo and CSRG SCCS logs.

ok brynet millert jca jmc schwarze

network -> interface; ok reyk@

explicitly link libm for sqrt()/fmax() calls added in bgpctl.c rev 1.204

unbreak gcc4 build

Check the disk/kernel/cdrom file permissions after openening the fd.

This prevents time of TOCTOU attacks for instances.

OK mlarkin@

Make the default failure for unveil while disabled return success
so that people don't get screwed when playing with it on their
machines

sync

Crank minor for unveil
ok deraadt@

Unveiling unveil(2).
This brings unveil into the tree, disabled by default - Currently
this will return EPERM on all attempts to use it until we are
fully certain it is ready for people to start using, but this
now allows for others to do more tweaking and experimentation.

Still needs to send the unveil's across forks and execs before
fully enabling.

Many thanks to robert@ and deraadt@ for extensive testing.
ok deraadt@

no longer interpret 0.192.168.4 in hosts(5) as 192.168.4/24
OK phessler@ claudio@ benno@ kn@
"steh' nicht rum, committe das" henning@

Monitor the route socket and regenerate the list of interfaces and
messages to send when an interface shows up or disappears or
and address shows up or disappears.

add missing IMSG_STARTUP; unclear why this ever worked

Use "rtable" not "tableid" as argument name for rtable

Make rtable(4) usage documentation consistent with other programs.
This is to have `man -k ar~rtable' show the full list without having to
look for other variations of the same argument type.

OK bluhm

Unused variables.

ok henning@

Unused variable.

ok martijn@ henning@

tests for next steps in unveil

Some USB network interfaces like rum(4) report ENXIO from their
ioctl function after the device has been pulled out.  Also accept
this error code in bpf_detachd() to prevent a kernel panic.  tcpdump(8)
may run while the interface is detached.
from Moritz Buhl; OK stsp@

Unused variables.

ok halex@

Fix some comments referencing sockets which are not used by the ber api. The
api uses read and write buffers (byte streams) that are utilized by calling
applications which may or may not use sockets.

ok claudio@

buffer byte streams that applications then use for

When we nack a domain service because we don't support the requested major
version, provide a major version that we do support.

Eliminate the weird condition in the BN_swap_ct() API that at most one bit
be set in condition. This makes the constant time bit-twiddling a bit
trickier, but it's not too bad. Thanks to halex for an extensive rubber
ducking session over a non-spicy spicy tabouleh falafel..

ok jsing, kn