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@

When starting an AP or IBSS network, reconfigure the operation mode to
11ac/n/a/g/b as applicable. Fixes an issue where hostap would end up
running in the rather meaningless MODE_AUTO unless a mode was explicitly
configured with ifconfig.

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

ok deraadt@

Implement version 2 of virtio(4) at fdt, as used by Parallels on the
Apple M1.  With this vio(4) shows up and we can properly install and
use OpenBSD as VM.

"not afraid of the virtio diff" deraadt@
"okie dokie" jcs@

mention DTLS1_2_VERSION

Mention DTLS1_2_VERSION here, too

Document SSL_OP_NO_DTLSv1{,_2}

Document DTLSv1_2_{,client_,server_}method(3)

do not pass file/func to monitor; noted by Ilja van Sprundel; ok djm@

Merge documentation for SSL_is_dtls() from OpenSSL

Reshuffle and reindent code. No functional change.

Rework the http code to require poll() only when really needed.
Especially tls_read() and tls_write() do not map 1:1 to read() and write()
calls and so assuming that after a tls_read() one needs to poll for more
data is wrong. Instead call tls_read() until it returns a TLS_WANT_*
return.

While here also ignore SIGPIPE. It is almost impossible to properly guard
from SIGPIPE by looking at POLLHUP. Instead just let write() handle it and
return an error.

Putting this in now so this can be tested widely.

Switch back to the legacy verifier for the release.

This is disappointing as a lot of work was put into the new verifier
during this cycle. However, there are still too many known bugs and
incompatibilities. It is better to be faced with known broken behavior
than with new broken behavior and to switch now rather than via errata.
This way we have another cycle to iron out the kinks and to fix some of
the remaining bugs.

ok jsing

Fix bgpctl show mrt for UPDATE messages. The call to output->attr() was
incorrect. Adjust output->attr() to take a reqflag argument instead of
a struct parse_result pointer since that is the only bit needed.
Found by and OK procter@, OK deraadt@

On powerpc64 regress/usr.sbin/bgpd/config failed.  It parses a
config file, writes bgpd's config to stdout and compares it with
an expected output.  On big endian machines the order of the set
of communities is different.  The parser used memcmp(3) to sort a
struct of integers.  This depends of the endianess.  The correct
way is to compare the integer fields in native byte order.  With
this change, the resulting order is the same on i386 and powerpc64.
OK claudio@

Make rpki-client -V output nicer.
OK job@, kn@, deraadt@