vmd(8): guard against bad virtio drivers

Add protections against guests with bad virtio-{blk,net,scsi}
drivers, specifically avoiding invalid descriptor chains and
invalid vionet packet sizes. This helps prevent possible lockup
of the host vm process due to a spinning device event loop thread.

Also fix an unneeded cast in the vioblk handling in case of invalid
buffer lengths.

OK mlarkin@

Only hash known CH extensions

RFC 4.1.2 specifies the ways in which the extensions in the first and
the second ClientHello may differ.  It basically says that extensions
not known to a server must not change.  This in turn makes it impossible
to introduce new extensions that do change.  It makes little sense to
enforce that extensions we don't know and care about aren't modified,
so make the hashing more lenient and restrict it to the extensions we
do care about.  Arguably, enforcing no change in an unknown extension
is incompatible with the requirement that it be ignored.

ok bcook jsing

kqueue: Make timer re-addition reset existing timer

When an existing EVFILT_TIMER filter is re-added, cancel the existing
timer and any pending event, and restart the timer using the new timeout
period. This makes the new timeout period take effect immediately and
matches the behaviour of FreeBSD. Previously, the new setting was
applied only after the existing timer expired.

The timer rescheduling is done by using an f_modify callback. The
reading of timer events is moved from f_event to f_process. f_event of
timer_filtops becomes redundant. Unlike most other event sources, timers
activate knotes directly without using a klist and knote(9).

OK mpi@

Mark umb(4) as network device instead of a generic one.  This also makes
it show up with the proper category in hotplug scripts.

From Tilo Stritzky
ok groth@ sthen@

Keep under #ifdef TMPFS functions to grow/shrink uaobj.

ok patrick@

Indent and simplify the grammar.

Eliminate S/R conflicts and simplify filter grammar.

Remove trailing whitespace in comment blocks.

reenable POOL_DEBUG

Simplify token declaration.

Simplify now that TID and PID are now only being parsed as builtin.

Remove support for in-kernel filters.

This might be added back in a future if copying events to userland becomes
a performance issue.  However note that it is not always possible to filter
in-kernel if, for example. a variable has to be evaluated when a rule fires.

Add a test that roundtrips a bunch of points on all builtin curves
via point2oct and oct2point and that checks the corner case in hybrid
encoding that was fixed in ec2_oct.c r1.13.

Clean up TLSv1.2 certificate request handshake data.

Currently cert_req is used by clients and cert_request is used by servers.
Replace this by a single cert_request used by either client or server.
Remove the certificate types as they are currently unused. This also fixes
a bug whereby if the number of certificate types exceeds SSL3_CT_NUMBER
the number of bytes read in is insufficient, which will break decoding.

ok inoguchi@ tb@

Fix packet size checks and remove bad casts.

Because dhcpsz was an uninitialized ssize_t, it was possible that a
garbage "packet" would be queued on the receiving end of the virtio
network device.

Change the type to size_t and add proper checks based on it being
greater than zero. Remove the cast of ssize_t to uint64_t that also
caused garbage sizes when dhcpsz was unintialized and set at runtime
to something < 0.

Turn panic for unknown event type into a printf and return.

ok patrick@

Use the plenteous BUFSIZE found in mg in lieu of 128.

remove -p from usage;

Extend filters to support any conditionnal test including global variables.

Stop using in-kernel filtering for the moment except for not tracing the
tracer.

Keep track of the number of filtered events.

Always initialized min value for an histogram.

Test local variables, boolean and print() of variables other than maps.

typo

Support for local (scratch) variables: "$var_name".

Every rule gets its own list of (local) variables.

Extend print() to support any kind of variable.

Support first shell argument as $1 in order to use it in filters.

Remove '-p' option now that scripts can filter by pid/tid.

Check that path MTU discovery across routing domains inserts a
dynamic IPv4 route.  Add IPv6 test that runs tcpbench over pair
interfaces in different routing domains.  For that setup, PMTU
currently does not work.  IPv6 TCP packets get fragmented.

Convert remaining uvm_km_zalloc(9) to km_alloc(9).

Tested by bluhm@, jj@, kettenis@ and Scott Bennett.

ok kettenis@

Improve ntpd offset handling.  Call the index of the offset loops
"shift" consistently.  Merge the two offset loops in client_update()
into one.  Use a simple assignment for the best value instead of
memcpy().  Use the same mechanism to loop over the offset array
everywhere to avoid an invalid best value.
tested by weerd@; OK claudio@

HTTP errors are logged by the http module. No need to repeat them.
Instead add a logx when a fallback from delta sync to a snapshot happens.

When setting RRDP_STATE_PARSE_ERROR a warning is issued so there is no need
to warn a 2nd time here, it adds no additional information.

unplug unused certificate verification code, now that this is done by libtls.

ok tb@ millert@

Fix const in previous. Pointed out by asou

Fix __builtin_bitreverse32 on 32-bit PowerPC

This is a backport from LLVM 11.  Before this fix, code using
__builtin_bitreverse32 was crashing SIGILL because clang-10 emitted a
64-bit rldicl/clrldi instruction.  The SIGILL only happened on 32-bit
cpus, not on the G5.  The code for LLVM 11 uses __builtin_bitreverse,
so clang-10 needs this fix to build clang-11.

https://github.com/llvm/llvm-project/commit/a5d161c119d5a
https://reviews.llvm.org/D77946

ok kettenis@

Move TAILQ initialization to files where they are used.

These priv-sep daemons all follow a similar design and use TAILQs
for tracking control process connections. In most cases, the TAILQs
are initialized separate from where they are used. Since the scope
of use is generally confined to a specific control process file,
this commit also removes any extern definitions and exposing the
TAILQ structures to other compilation units.

ok bluhm@, tb@

Match on "amlogic,meson-sm1-mmc" compatible.

Fix indent of EC_METHODs as requested by jsing.

While there zap trailing whitespace from a KNF approximation gone wrong.

Adjust ectest.c for set_compressed_coordinates

Compare pointer against NULL and fix a KNF issue.

ok jsing

Prepare to provide EC_POINT_set_compressed_coordinates

ok jsing

Adjust ectest.c for get_Jprojective coordinate change

Compare function pointers against NULL, not 0.

ok jsing

Provide EC_POINT_{g,s}et_Jprojective_coordinates for internal use

ok jsing

Simplify code after adding EC_POINT_{s,g}et_affine_coordinates()

ok jsing

Adjust ecdhtest.c for affine_coordinates change

Adjust ectest.c for affine_coordinates change

Compare function pointers against NULL, not 0.

ok jsing

Prepare to provide EC_POINT_{g,s}et_affine_coordinates

Similar to part of OpenSSL commit 8e3cced75fb5fee5da59ebef9605d403a999391b

ok jsing

Simplify after EC_POINT_get_curve() addition

ok jsing

Adjust ectest.c for EC_GROUP_{g,s}et_curve change

Add prototypes for EC_GROUP_get_curve_{GF2m,GFp}().

These will be removed once EC_GROUP_get_curve() is public.

Compare function pointers against NULL, not 0.

ok jsing

Prepare to provide EC_GROUP_{get,set}_curve(3)

There are numerous functions in ec/ that exist with _GF2m and _GFp
variants for no good reason.  The code of both variants is the same.
The EC_METHODs contain a pointer to the appropriate version.  This
commit hides the _GF2m and _GFp variants from internal use and
provides versions that work for both curve types. These will be made
public in an upcoming library bump.

Similar to part of OpenSSL commit 8e3cced75fb5fee5da59ebef9605d403a999391b

ok jsing

Indicate if an mg function is unsuitable for a startup file (requires
user interaction).

Add keep-alive support to the HTTP module.
Requests are split away from connections. When a request is received try
to reuse an IDLE connection. If none is around start a new one (unless
there are too many connections inflight).

Idle connections are kept for 10sec and closed after that time. For
rpki-client this is plenty of time since RRDP exchanges will be a burst
of requests. So the connection used to fetch the notification XML file
will be reused to fetch all delta XML files.

This reduces the CPU load since far less TLS handshakes need to happen.

OK job@ deraadt@

Allow more than one block of code to exist on a single line. Also,
move the code which expands variables to be ran when variables are
discovered instead of in multiarg() just before execution. This means
a variable who's value is included in anothers', won't change the
others value if its own is changed.

I have also included code, which is commented out at the moment, which
implements a function map specific to interpreter. Not sure if I can
ultimately avoid using it though.

Switch some warnings to logx() to reduce log noise on runs without -v.
OK job@

Update comment to be less confusing (I hope)

Properly shutdown on connection loss instead of crashing.
Found by bluhm@ while playing with setting noclose to 1 for daemon(3):
Also included in this commit

OK bluhm@

Don't overwrite environment CFLAGS.

OK bluhm@

Add 'dired-jump' from Philip K. <philip@warpmail.net> sent to tech@
over a year ago. Comments and testing from gkoehler@. Thanks to both.
Also, some modifications from me.

prefix_insert() and prefix_remove() emulate a tail queue by keeping the
tail pointer (pointer to last element) around and depending on the state
of the list insert at head or insert after tailp. Now gcc has a hard time
to realize that the tail pointer is not used uninitalized. So rewrite the
code to be more explicit about tailp handling (also rename the pointer
to be more explicit). All in all this should be more readable and silences
the gcc warning as well.

Use LIST instead of SLIST for requests. The way SLIST_REMOVE was used did
a double traverse of the list which now is replaced with no traversal at all.
Also stop double wrapping requests just for the list.
OK millert@

Ignore SIGPIPE by default and restore default behaviour before executing
the CGI.
OK bluhm@ florian@

Add quirk to enable all Thinkpad X1 Extreme 1 speakers & atmos dolby

From Ivo Sbalzarini <ivo.sbalzarini@gmail.com>, thanks!

Allow INTEL 400SERIES cAVS to attach the azalia(4) driver

From Ivo Sbalzarini <ivo.sbalzarini at gmail.com>

more KNF

Remove new_sym_enc and new_aead.

These can be replaced with accessors that allow this information to be
retrieved from the new record layer.

ok inoguchi@ tb@

Avoid division by zero in hybrid point encoding

In hybrid and compressed point encodings, the form octet contains a bit
of information allowing to calculate y from x.  For a point on a binary
curve, this bit is zero if x is zero, otherwise it must match the
rightmost bit of of the field element y / x. The existing code only
considers the second possibility. It could thus fail with a division by
zero error as found by Guido Vranken's cryptofuzz.

This commit adds a few explanatory comments to oct2point and fixes some
KNF issues. The only actual code change is in the last hunk which adds a
BN_is_zero(x) check to avoid the division by zero.

ok jsing

code review results in KNF, and moving local variables into lowest scope
ok claudio

Recognize BCM57762 A0 and A1.  My diff for this only added the A0 revision,
but the A1 revision has been seen in the wild and Brad Smith provided
a diff that adds both.

ok jsg@

Move new_mac_secret_size into the TLSv1.2 handshake struct.

Drop the 'new_' prefix in the process.

ok inoguchi@ tb@

Move reuse_message, message_type, message_size and cert_verify into the
TLSv1.2 handshake struct.

ok inoguchi@ tb@

Set alpn_selected_len to zero when freeing alpn_selected.

This is not strictly necessary since we proceed to zero the entire struct,
however it keeps the code consistent and easily auditable.

ok tb@

The powerpc64 ELFv2 ABI explicitly states that exception enable bits
and rounding control bits are not restored by longjmp(3).  So expect
the some failures on that platform.

ok bluhm@

Multicast decryption fixes for iwx(4).

Pick the correct key for multicast frames in iwx_ccmp_decap().
Comparing the PN of a multicast frame against the last-seen PN of
the pairwise key is obviously wrong. We need to check the multicast
frame's PN against the last-seen PN of the group key.

Update crypto-offloading checks in iwx_rx_frame() to match recent
WPA1/TKIP groupcipher fixes made in athn(4).
The code inherited from iwm(4) only looked at the pairwise key, and
unlike iwx(4) and athn(4), iwm(4) only offloads pairwise crypto.

Found while investigating a question from zxystd at OpenIntelWireless.

RS ^-anchoring needs to know if it's reading the first record of a file.
Without this fix, when reading the first record of an input file named
on the command line, the regular expression engine will be
misconfigured, precluding a successful match.  From Miguel Pineiro Jr

Fix TLS error message race.  Due to less handshakes in TLS 1.3 an
EPIPE error may be reported instead of a TLS alert.

Same internal consistency check as libc malloc: size in hash table
should correspond to size of chunk meta data

Simpler error handling for suspend()/hibernate()

Save errno when we get an error so we can pass it to the apm(8) client.
ok kn@

post 6.9 development continues...

sync

even more shrink

sync

more shrink

Adjust Raspberry Pi installation instructions, direct users towards the
U-Boot based method primarily, though keep a quick mention of the UEFI
firmware (U-Boot method not working on Pi400 currently whereas UEFI
sort-of works). Add some notes about framebuffer consoles, they do
work on RPi these days but there are still some rough edges.

Embiggen arm64 ramdisk kernel and miniroot/install img files. Provide
U-Boot binaries that work on Raspberry Pi 3 and 4 (and possibly others)
and firmware for Raspberry Pi 4. This allows the same installation
method as used on Raspberry Pi 3 without separate UEFI firmware
(although UEFI can still be used). Help from kettenis@ jsg@ deraadt@

sync

some name shortening

rewrite the text describing the "address" parameter into something
easier to read;

When loading a config poll iscsid for up to 10 seconds and check if the
sessions all managed to connect to the targets.
With this slow session establishment should not cause mount errors
during startup.
Initial version from Dr Ashton Fagg ashton (at) fagg id au
OK dlg@ deraadt@

Implement a control message to get the state of iscsid. This is used
by iscsictl to poll for completion after reload commands.
With this slow session establishment should not cause mount errors
during startup.
Initial version from Dr Ashton Fagg ashton (at) fagg id au
OK dlg@ deraadt@

Turn on the direct ACK on every other segment.

This is a backout of rev 1.366 which turned this feature off.
Although sending less ACKs makes TCP faster if the CPU is busy with
processing packets, there are corner cases where TCP gets slower.

Especially OpenBSD 6.8 and older has a maxbust limitiation that
scales badly if the other side sends too few ACKs.  Also regress
test relayd run-args-http-slow-consumer.pl uses strange socket
buffer sizes that triggers slow performance with the new algorithm.

For OpenBSD 6.9 release switch back to 6.8 delayed ACK behavior.

discussed with deraadt@ benno@ claudio@ jan@

Add a workaround to avoid wrong code generated by m88k gcc.

NATIVE_TO_UNI is defined as follows in utf8.h:
--
#define NATIVE_TO_UNI(ch)       ((UV) ((ch) | 0))
--
and UV is 'unsigned long' on m88k.

Details are at:
https://github.com/Perl/perl5/issues/18655

help and ok afresh1@

Unhook ieee80211_mira.c from the build. All consumers have switched to RA.

Keeping files in CVS HEAD for now until we are certain we're not going back.

ok deraadt@

Add back IMSG_RECONF_ROA_ITEM in the parent imsg handler. It is needed
for origin-sets. Found by and reminder procter@
OK deraadt@

openssh-8.6

Since iwi(4) doesn't call into net80211_newstate() the interface link state
must be updated by the driver in order to get packets to flow.

In case of WPA the link state was updated as a side-effect of a successful
WPA handshake. This commit fixes the WEP and plaintext cases.

Similar fix as recently committed to ipw(4).

Additionally, check for errors from iwi_auth_and_assoc() and keep scanning
if this function fails.

Problem confirmed and fix tested by matthieu@
ok deraadt@

Switch athn(4) 802.11n Tx rate adaptation from MiRA to RA.

Tests:
AR5418: Uwe Werler
AR9280: kn, jmatthew, Lauri Tirkkonen, Scott Bennett, Mikolaj Kucharski
AR9285: kevlo, trondd, myself

ok deraadt@

Make athn(4) set the channel when hostap or monitor modes enter RUN state.

Fixes a problem where the hardware would end up using a different channel
than the one selected by net80211.

Found while investigating issues reported by trondd testing my athn RA patch.

ok deraadt@

Make the description of the ELF header reflect reality, removing the
traces of the long gone FreeBSD "ELF brand" code.

Based on a diff from George Brown.
ok jmc@