sourceaddr:

- show that -ifp and "address" do not mix
- show how to reset value (from claudio)
- tweak text

ok claudio

Rename bn_mod_exp_zero to the more appropriate bn_mod_exp

Retire the bn_mod_exp test.

Its is fully covered by bn_mod_exp_zero now.

Reimplement a variant of the bn_mod_exp tests from scratch

This exercises the same corner cases as bn_mod_exp and a few more.
With input from jsing

fixes for mandoc -Tlint
ok tb@

remove -beta tag

Document M_LOG flag; from Peter J. Philipp.

Move annoying yet harmless diagnostic message into #ifdef DEBUG.

Adjust sourceaddr argument parser.

Only print the sourceaddrs if no argument was passed. If arguemnts
are set make sure that either and address is present or that an ifp
was given. This make the command behave a more like other route commands.
OK deraadt@

Fix rtr_parse_aspa(), the spas array is actually not copied over into
the rtr_aspa struct so access them directly from the buf using offset
as the address of the first element.
OK tb@

after updating amdgpu-firmware from 20221214 to 20230310 a new
warning appeared in dmesg on renoir:

[drm] psp gfx command LOAD_TA(0x1) failed and response status is (0x7)
[drm] psp gfx command INVOKE_CMD(0x3) failed and response status is (0x4)
psp_securedisplay_parse_resp_status *ERROR* Secure display: Generic Failure.
psp_securedisplay_initialize *ERROR* SECUREDISPLAY: query securedisplay TA failed. ret 0x0

This is likely related to not implementing all the HDCP paths.
Return early in psp_securedisplay_initialize() to avoid this.

I suspect the newer renoir firmware adds a TA_FW_TYPE_PSP_SECUREDISPLAY
component the older one didn't have.

reported by jmc@

Add code to bring up the PCIe controller on the RK356x.

ok dlg@

When syslogd is sending messages via UDP to a remote loghost, it
stops if there is a permanent error.  Add EACCES generated by pf
to the list of transient errors.  This restores pre-6.5 behavior
and continues logging after pf.conf has been fixed.
OK millert@ deraadt@ mvs@

Update for 7.3.

Consistent phrasing: function -> function pointer

Add X509_STORE_{set,get}_check_issued and X509_STORE_CTX_get_check_issued to manpage

with and OK tb@

sync

Install EVP_CIPHER_meth_new.3

Add EVP_CIPHER_meth_* documentation from OpenSSL 1.1

This is essentially the original text with a few tweaks and fixes by me,
removing parts inapplicable to LibreSSL. There are dangling references to
EVP_CIPHER_CTX_copy(3) and EVP_CIPHER_CTX_get_cipher_data(3). This all
isn't great, but it's better than nothing. Probably good enough for these
rarely used functions.

Update manpage for X509_CRL_get0_tbs_sigalg()

OK tb@

Add missing dependecy for rkcomphy(4); pointed out by dlg@

Bump LibreSSL version to 3.7.2

revert previous, stsp made WEP work on bwfm(4)

make WEP encryption work on bwfm(4)

based on an initial diff by jsg@
brokenness pointed out by kn@
ok jsg@

tweak sourceaddr:

- remove "-inet|-inet6" as it is not generally meaningful
- remove the qualifier "if set": sourceaddr now works nicely in cases
  where no "address is given

Fix a number of out of bound reads in DNS response parsing.
Originally from djm@.  OK deraadt@ florian@ bluhm@

state that binary updates are not available for all platforms/architectures.
triggered by a bug report by 'kodcode', thanks.
wording adjusted by jmc@, ok phessler@

openssh-9.3

slightly earlier, and tweak some dates

Do not leak screen in popups, GitHub issue 3492.

Fix the length check when computing a fake challenge for users not
in the S/Key database.  If the system hostname is longer than 126
characters this could result in NUL bytes being written past the
end of a stack buffer.  There is no impact on systems with a hostname
126 characters or less.  Found by Qualys.  OK deraadt@

Add missing nameref resolving for the _DEP() package elements that indicate
dependencies between ACPI devices.

ok dlg@, deraadt@

Show MFTs as part of the signature path

no objection tb@

For now pass AF_UNSPEC to printsource() to ensure that both inet and inet6
addrs are fetched from the kernel. This needs more cleanup.

Make route sourceaddr always print the used addresses for inet and inet6.
Print 'default' if no sourceaddr is set and the default algorithm is used.
With and OK kn@ deraadt@

drm/amdgpu/vcn: Disable indirect SRAM on Vangogh broken BIOSes

From Guilherme G. Piccoli in amd-staging-drm-next
https://gitlab.freedesktop.org/drm/amd/-/issues/2385

This fixes amdgpu failing to init on Steam Deck after the drm 6.1 update:

[drm] failed to load ucode VCN0_RAM(0x3A) [drm] psp gfx command LOAD_IP_FW(0x6)
failed and response status is (0xFFFF0000)
[drm] *ERROR* ring vcn_dec_0 test failed (-60)
[drm] *ERROR* hw_init of IP block <vcn_v3_0> failed -60
drm:pid0:amdgpu_device_init *ERROR* amdgpu_device_ip_init failed
drm:pid0:amdgpu_attachhook *ERROR* Fatal error during GPU init

reported and tested by bentley@ on:
bios0: vendor Valve version "F7A0113" date 11/04/2022
bios0: Valve Jupiter

handle dmi_get_system_info(DMI_BIOS_VERSION)

store bios version, needed for amdgpu Steam Deck quirk

Fix command prompt not to always append argument but only if there has
actually been expansion. GitHub issue 3493.

Return the signature length after successful signing operation

This is required behavior of the EVP_DigestSign() API, but seemingly
almost nothing uses this. Well, turns out ldns does.

Reported by Stephane. Helpful comments by sthen.

ok jsing

Add comments that explain why things are done in this strange order.

There's some method to this madness.

ok jsing

Push calloc() of ndef_aux down as far as possible and
pull the setting of the ex_arg up, so we can do error
checking.

ok jsing

Error check BIO_asn1_set_{prefix,suffix}() calls

ok jsing

Streaming BIOs assume they can write to NULL BIOs

At least SMIME_text() relies on this. Pushing an error on the stack trips
PKCS7 regress in py-cryptography, so indicate nothing was written instead
of throwing an error.

Reported by Alex Gaynor a while back

ok jsing

Ensure negative input to BN_mod_exp_mont_consttime() is correctly reduced.

A negative input to BN_mod_exp_mont_consttime() is not correctly reduced,
remaining negative (when it should be in the range [0, m)). Fix this by
unconditionally calling BN_nnmod() on the input.

Fixes ossfuzz #55997.

ok tb@

Include tests with negative values in BN_mod_exp* regress.

This currently fails.

bn_mod_exp_zero: rename result into got

Prep for rpki-client 8.3

WEP in bfwm(4) never worked, sync with reality

"ifconfig bwfm0 [-]nwkey ..." does not work and thus the installer no longer
offers WEP, so unmention support while leaving the recommendation against it.

Free KRL itself in addition to its contents. From Coverity CID 291841,
ok djm@

Check pointer for NULL before attempting to deref.  None of the existing
callers seem to do that, but it's worth checking.  From Coverity CID
291834, ok djm@

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@