rpki-client: disallow AIA in self-signed certs

Per RFC 6487, 4.8.7, self-signed certificates must not have an Authority
Info Access extension. In normal operation this is ensured by ta_parse()
and cert_parse(), respectively. In filemode, only partial checks are
performed, so this is not guaranteed.

Issue flagged by and ok job

sync the boot options list, and some macro fixes;
ok miod

sync

To avoid misunderstanding, keep variables for tcp keepalive in
milliseconds, which is the same unit of tcp_now().  However, keep the
unit of sysctl variables in seconds and convert their unit in
tcp_sysctl().  Additionally revert TCPTV_SRTTDFLT back to 3 seconds,
which was mistakenly changed to 1.5 seconds by tcp_timer.h 1.19.

ok claudio

clockintr: add missing newline to ddb(4) printout

Let's also print the state (running, pending, or established) of each
clockintr.

Don't lookup the signature path's expiry for trust anchor certs

In filemode, also print details for .cer certificates

Point people to boot(8) rather than duplicating information.

Switch alpha to new boot blocks.

Add manual page.

"ask name" option letter is 'n' instead of 'a' on pmax and alpha for historical
(and firmware) reasons.

Check that the CMS signing-time is not after the MFT's nextUpdate

Issuing manifests whose validity window is entirely in the past, as
observed from the purported CMS signing-time would be a form of backdating.

OK tb@

In filemode, display the moment the signature path will expire

Previously this was only shown for ROA+ASPA. Now also show for
GBR, Geofeed, Certs, RSC, and TAK.

OK tb@

Check that the CMS signing-time isn't after the X.509 notAfter

The CMS signing-time is the purported 'now' from the perspective of the
issuer. It doesn't make sense for an issuer to sign objects that have a
validity window that falls entirely in the past (from the perspective of
the signer). Although CMS signing-time is not a trusted timestamp, it
should never be after X.509 notAfter.

OK tb@

typo fix; ok claudio

vmd(8): backout ns8250 changes.

This backs out commit h1IJwTVsYWfnRKWy which intended to fix interrupt
issues on slower host hardware. Unfortunately, this causes a race
condition on much faster host hardware. It still requires investigation.

With an "I told you so" about touching ns8250 from mlarkin@

In filemode for ROAs/ASPAs display when the Signature path will expire

Also rename 'certification path' to 'signature path' for alignment

OK tb@

Extend bgplgd to handle leaked and invalid options to show rib.
OK tb@

Extend bgpctl show rib with two new options:
 - invalid: show only routes that are not valid
 - leaked: show only routes where a route leak was detected
OK tb@

Add F_CTL_LEAKED and F_CTL_INELIGIBLE flags for bgpctl to show leaked
and ineligible paths.
While there rename F_PREF_OTC_LOOP to F_PREF_OTC_LEAK since this indicates
that a route leak was detected.
OK tb@

Try to explain that the neighor role is required to enable both
ASPA validation and RFC9234 OTC leak detection and role capability.
The role on iBGP peers is forced to 'none' since it makes no sense there.
After input from benno@

Merge calling print_certification_path() into adjacent else branch

OK tb@

Fix potential underflow when printing STP data.
Reported by Peter J. Philipp
ok claudio

Refactor whats printed when in filemode

OK tb@

Fix usbpcap handler to properly detect overflows using TCHECK macros.
OK mpi@ mbuhl@

Add "machine poweroff" command on luna88k bootloader.

ok miod@

Revert accidental change from explicit to implicit tagging in r1.86.

ok job

Rename some data fields to closer match their provenance

OK tb@

Stop confusing out and asn_bio in BIO_new_NDEF()

BIO_new_NDEF() sets up an ASN.1 BIO to the output chain and then adds even
more BIOs. Since BIO_push(bio, new_tail) returns bio on success, after the
if ((out = BIO_push(asn_bio, out)) != NULL) the 'out' BIO and the 'asn_bio'
are the same. The code then goes on and uses one or the other. This is very
confusing. Simply stop using out once it's appended to asn_bio.

ok jsing

regen

Add Kingston NV2 NVMe drive.

sync

sh, landisk: set HZ=64 again

The addition of HZ to sys/kernel.h in v1.26 overrides the default
definition of HZ in sh/clock.c, changing landisk from HZ=64 to HZ=100.

Explicitly set HZ=64 in the GENERIC and RAMDISK config(8) files to can
change it from 100 back to 64.

Not sure if this is the best thing, but it does fix the problem.

Problem confirmed by, and fix tested by, miod@.

ok miod@

correct macro request (.nm -> .Nm);

rkcomphy(4)

pk7_cb() and cms_cb()

Add and fix FALLTHROUGH statement. I was confused for way too long since
I hadn't noticed that this case fell through to the next. Also add and
move some empty lines in the cms_cb() to make this resemble KNF more.

Add rkcomphy(3), a driver for the "naneng" combo PHY found on the RK356x
(and RK3588).  This is a PIPE PHY with support for PCIe, SATA, USB3, SGMII
and QSGMII.  For now only PCIe, SATA and USB3 support are implemented.
SATA support has not been tested.

Also add the refernce clocks needed by the PHYs to rkclock(4).

ok mlarkin@

zap a tab

Alphabetically sort various bits and pieces - mechanical change

Might make later refactors easier to follow

OK tb@

Add crl.c in various places to gain access to the new helper

Refactor expiration calculation

Unify common code paths which find the exact expiry moment into a new
helper function. Additionally, the new helper offers more accuracy by
checking more applicable CRLs whether their 'nextupdate' is 'sooner'.

tb@ noted: The helper adds a multiplier of log(#crls), but that's
certainly acceptable as it is still very cheap.

OK tb@

Avoid an 1 byte out-of-bounds read in ASN1_PRINTABLE_type()

In case the input is not NUL terminated, the reversed check for length
and terminating NUL results in a one-byte overread. The documentation
says that the input should be a string, but in ASN.1 land you never
know...

Reported by Guido Vranken a while back

ok beck

Initialize the local signtime variable in the individual *_parse()
functions

ok job

Invalidate the signtime output parameter in cms_parse_validate_internal()
and in cms_get_signtime().

ok job

Enable mvtemp(4)

Update supported hardware list and fix spelling of Marvell while I'm here

Match on the Armada 380 temperature sensor, which works the same as the
CP110 sensor.

ok kettenis@

Put upper bound on number of entries in SSH2_MSG_EXT_INFO request.
This is already constrained by the maximum SSH packet size but this
makes it explicit.  Prompted by Coverity CID 291868, ok djm@ markus@

calloc can return NULL but xcalloc cannot.  From Coverity CID 291881,
ok djm@

Remove a few extra spaces

New alpha secondary boot blocks using the MI code base. Not linked to the
build yet.

Remove long dead "tool".

Allow strip and objcopy to be overrridden, for easier cross-buildability. NFC

Switch an early return into goto err

Tiny cleanup for readability

Turn a malloc() into calloc() and check two function calls directly
forever instead of a combined check afterward.

Use "if (ptr == NULL)" instead of "if (!ptr)"

Requested by jsing

Fix double free after BIO_new_NDEF()

Once the asn_bio is prepended to the out chain, and before the asn1_cb()
has done its thing, asn_bio needs to be popped off again on error. Failing
to do this can cause write after frees or double frees when the out BIO
is used after the function returned.

Based on a very complicated diff by Matt Caswell and Viktor Dukhovni.
This was part of the fixes in OpenSSL 1.1.1t.

ok jsing

Fix an off-by-one in dsa_check_key()

The private key is a random number in [1, q-1], so 1 must be allowed.
Since q is at least an 160-bit prime and 2^159 + 1 is not prime (159
is not a power of 2), the probability that this is hit is < 2^-159,
but a tiny little bit wrong is still wrong.

Found while investigating a report by bluhm
ok jsing

Call CRYPTO_cleanup_all_ex_data() from OPENSSL_cleanup().

Issue reported by Graham Percival (@gperciva)

ok tb@

Add OPENSSL_cleanup() calls to some regress.

This gets us some minimal test coverage.

Avoid -0 in BN_div_word().

Currently, the use of BN_div_word() can result in -0 - avoid this by
setting negative again, at the end of the computation.

Should fix oss-fuzz 56667.

ok tb@

Correct sign handling in BN_add_word().

A sign handling bug was introduced to BN_add_word() in bn_word.c r1.18.
When handling addition to a negative bignum, the BN_sub_word() call can
result in the sign being flipped, which we need to account for. Use the
same code in BN_sub_word() - while not technically needed here it keeps
the code consistent.

Issue discovered by tb@

ok tb@

Remove a pesky space.

Add regress coverage for BN_{add,sub,mul,div,mod}_word().

This also provides some indirect coverage for BN_hex2bn(), BN_bn2hex() and
BN_get_word().

Two of these tests are currently failing and will be fixed shortly.

Mark test table as static const.

Add missing $OpenBSD$ tag.

Bump version to 7.8

Factor out the cache reset logic into rtr_reset_cache() and use it
consitently in all places where the cache should be reset.

Additionally adjust the FSM to handle a connection close event in the
protocol negotiation phase as a protocol mismatch. Some caches fail
to send an error PDU and this allows them to work. A drawback is that
if the connection is closed because of a different reason the system
will fall back to a lower then necessary version. The RFC and draft
mandates this behaviour.

OK tb@

btrace(8) retrieves the types of the arguments of each tracepoint
from the kernel.  Add them to a cache when needed, and print them
with btrace -l.  If the argument is of type int, print the value
as %d signed integer.  This gives pretty output for refcnt changes.
All other types still use %ul.
The probe and argument number can be used as array index.  Remove
the qsort and search for probe number to simplify the code.  Use
array index for both probes and argument types.
OK mpi@

Add new dt(4) ioctl DTIOCGARGS to get the types of probe arguments.
They are already tracked as strings in the kernel.  Export them to
userland using one ioctl(2) for all arguments of each probe.
OK mpi@

Remove unused make variables FSTYPE (=install360) and FSDISKTYPE
(=mini34).  Preparatory to removing some obsolete/unused disktab
entries.

Fix possible leak of spibuf and flowbuf in error case.

ok patrick@ bluhm@

sync

Crankl libcrypto/libssl/libtls minors after symbol addition

Update Symbols.list

Expose various X509_STORE_*check_issued()

Expose X509_CRL_get0_sigalg() and X509_get0_uids

Expose UI_null()

Expose the EVP_CIPHER_meth_* API (setter only) in evp.h

mechanical change, rename struct members to match the original X509 names

OK tb@

Show the X.509 notBefore in filemode

OK tb@

ASN.1 BIO: properly wire up prefix_free and suffix_free

If something goes wrong before the ASN.1 BIO state machine has passed
both flushing states, asn1_bio_free() forgets to free the ndef_aux
and the ex_arg since the prefix_free() and suffix_free callbacks are
not called.

This can lead to leaks, notably in streaming bios.

Part of https://github.com/openssl/openssl/pull/15999
I have a regress covering this but it is not yet ready to land.

ok beck jsing

Add the necessary stack offsets for macppc so dt(4).
Tested on a G5 and G4 macppc.
OK miod@

Update the stack offset for the profile probe to account for the clockintr
API functions. Fixes flamegraphs on archs I could test.
OK bluhm@ miod@

Implement setting the parent clock for RK356x.  This includes code for
setting external parent clocks, which is needed to switch to the clock
provided by the Ethernet PHY.

ok patrick@

Fix error on installer abort/exit

^C during questiong yields
/install: /tmp/cppid: cannot open $(<) input

The trap handler must first test existence, then read the PID file.
Quote like other PID file constructs in install.sub do.

Use newlines to make the trap function readable.

OK miod

Take controller out of reset; makes it work on rk356x.

ok patrick@, mlarkin@

Return the correct type for ASN.1 BOOLEANs

ASN.1 BOOLEANs and ASN.1 NULL are handled specially in the ASN.1 sausage
factory and they are special in that they don't have a->value.ptr set.
Both need to be special cased here since they fail the a->type.ptr != NULL
check.

Apart from fixing an obvious bug in ASN1_TYPE_get(), this fixes another
crash in openssl(1) asn1parse. There is more to do in the vicinity, but
that is more complex and will have to wait for OpenBSD 7.3-current.

with/ok jsing

openssl(1) asn1parse: avoid crash with ASN.1 BOOLEANS

When pointing openssl asn1parse -strparse at DER octets 01 01, it crashes:

$ printf '<\x01\x01>' | openssl asn1parse -inform der -strparse 1

Refuse to parse BOOLEAN types instead, which avoids a crash in hensonian
/* hmm... this is a little evil, but it works */ code.

Found while poking at CMS timestamps to understand one of job's diffs.

with/ok jsing

Compile the output filter rules into per peer filter rules.

especially on route-servers the output filters are in the hot path so
reducing the number of rules to check has a big impact. I have seen a
25% to 30% speedup in my big IXP testbench.
The output ruleset is applied and copied for each peer during config reload
and when a peer is initially added.
OK tb@

Explicitly ignore return from fcntl(... FD_CLOEXEC) here too.
Coverity CID 291853.

Plug mem leak on error path.  Coverity CID 405026, ok djm@.

Expliticly ignore return code from fcntl(.. FD_CLOEXEC) since there's
not much we can do anyway.  From Coverity CID 291857, ok djm@

Like sshd_config, some ssh_config options are not first-match-wins.
sshd_config.5 was fixed in r1.348, this is the same for this file

Remove no-op (int) > INT_MAX checks since they can never be true.
From Coverity CID 405031, ok djm@

modify parentheses in conditionals to make it clearer what is being
assigned and what is being checked

ok djm dtucker

remove obsolete libLLVM.so.[0-6].0 during upgrade

ok kn@ deraadt@

Check that a PCIe port isn't disabled in the device tree.

ok patrick@

Unwrap a few lines and zap a tab

Add missing error checking in PKCS7

Check the return value of BIO_set_md(). Prompted by OpenSSL's fix for
CVE-2023-0401 (the crash in that bug is an OpenSSL 3-only problem due
to provider design).

ok beck jsing