Enable EPAN if it is available.

ok patrick@

openssl ca: avoid double free for spkac files without default section

ok jsing

strmode takes a mode_t, not an int; prompted by Collin Funk.
ok kettenis@ deraadt@ tb@

lower parts of the manual page clearly say why using malloc() to
allocate sigaltstack regions is bad... so the example code should
not use malloc()...
ok semarie

Remove the less-than-useful change log.

ok tb@

Rewrite BN_bn2mpi() using CBB.

The content is effectively a u32 length prefixed field, so use
CBB_add_u32_length_prefixed(). Use BN_bn2binpad() rather than manually
padding if we need to extend and use sensible variable names so that the
code becomes more readable.

Note that since CBB can fail we now need to be able to indicate failure.
This means that BN_bn2mpi() can now return -1 when it would not have
previously (correct callers will check that BN_bn2mpi() returns a positive
length).

ok tb@

Sync bytestring from libssl.

Provide CBB_add_u32_length_prefixed().

This is needed for an upcoming change in libcrypto.

ok tb@

delete ancient netbsd pcibios tweaking trash

correct elfcore_procinfo naming (in comments)

remove space between function names and argument list

My earlier commit [1.1169 of pf.c (2023/01/05)] makes pf(4) to report wrong
rule and anchor number when packet matches rule found and anchor depth 2
and more.  The issue has been noticed and reported by Giannis Kapetanakis
(billias _at_ edu.physics.uoc.gr), who also co-developed and tested
the final fix presented in this commit.

To fix the issue pf(4) must also remember the anchor where matching rule
belongs while rules are traversed to find a match for given packet.
The information on anchor is now kept in anchor stack frame.w

OK sthen@

recognise Qualcomm Oryon; ok kettenis@

Update ruby-module.5 for default Ruby version switch to 3.3

Instead of allocating an arbitrary amount of extra space, let the EFI
devicetree fixup protocol less us how much space it needs.

Pointed out by Heinrich Schuchardt
ok tobhe@, mlarkin@

Instead of allocating an arbitrary amount of extra space, let the EFI
devicetree fixup protocol less us how much space it needs.

Pointed out by Heinrich Schuchardt
ok tobhe@, mlarkin@

Instead of allocating an arbitrary amount of extra space, let the EFI
devicetree fixup protocol less us how much space it needs.

Pointed out by Heinrich Schuchardt
ok tobhe@, mlarkin@

Add missing ref & reorder

OK tb@ claudio@

Read IPv6 forwarding value only once while processing a packet.

IPv4 uses IP_FORWARDING to pass down a consistent value of
net.inet.ip.forwarding down the stack.  This is needed for unlocking
sysctl.  Do the same for IPv6.

Read ip6_forwarding once in ip6_input_if() and pass down IPV6_FORWARDING
as flags to ip6_ours(), ip6_hbhchcheck(), ip6_forward().  Replace
the srcrt value with IPV6_REDIRECT flag for consistency with IPv4.

To have common syntax with IPv4, use ip6_forwarding == 0 checks
instead of !ip6_forwarding.  This will also make it easier to
implement net.inet6.ip6.forwarding=2 for IPsec only forwarding
later.

In nd6_ns_input() and nd6_na_input() read ip6_forwarding once and
store it in i_am_router.  The variable name has been chosen to avoid
confusion with is_router, which indicates router flag of the packet.
Reading of ip6_forwarding is done independently from ip6_input_if(),
consistency does not really matter.  One is for ND router behavior
the other for forwarding.  Again use the ip6_forwarding != 0 check,
so when ip6_forwarding IPsec only value 2 gets implemented, it will
behave like a router.

OK deraadt@ sashan@ florian@ claudio@

Do not send ICMP redirect if IP forwarding is IPsec only.

If sysctl net.inet.ip.forwarding is set to 2, only packets processed
by IPsec are forwarded.  I this case behave more like a router than
a host and do not accept ICMP redirect packets.

OK deraadt@ sashan@ florian@ claudio@

vmd(8): protect global vm and vcpu state with mutex.

The vm process uses multiple pthreads to emulate vcpu and also drive
an event loop doing various io emulation. At points, one or the
other needs to read/write global vm state bits and toggle if a vcpu
is halted or "done."

This adds in a another mutex to protected the global state and
untangles areas where the mutex for protecting a condition variable
was being used around modifying some global state.

ok mlarkin@

Enable uvm percpu caches on luna88k.

It survives 3.5 days "make build" and makes about 1.5% faster on 3 CPU
machine:-)

ok miod@ phessler@ dlg@

Work around dbclient cipher and mac query bug.

Unlike earlier versions, recent Dropbear (at least v2024.85) requires
a host arg when querying supported ciphers and macs via "-c/-m
help".  Earlier versions accept but do not require it, so always
provide it.  If these queries fail, skip the test with a warning.

Remove dropbear key types not supported by current OpenSSH.
Allows subsequent test runs to work if OpenSSH is rebuilt w/out OpenSSL.

stricter check for overfull tables in penalty record path

put back reaping of preauth child process when writes from the monitor
fail. Not sure how this got lost in the avalanche of patches.

The GICv3 redistributor spacing on the X1E80100 (Snapdragon X Elite)
does not follow the regular scheme and needs to be read out of the
redistributor-stride property.

ok kettenis@

Add PNP IDs to handle the X1E80100 (Snapdragon X Elite) USB controllers.

ok kettenis@

Add _HID for X1E80100 (Snapdragon X Elite) SoCs.

ok kettenis@

The X1E80100 (Snapdragon X Elite) joins the group of chips that require
the SMMU quirk.

ok kettenis@

remove externs for global hibernate_state
removed in subr_hibernate.c rev 1.35

Kill the SIGPIPE signal handler which is installed around write opertations.
Instead just SIG_IGN SIGPIPE in main.c for all of acme-client.
More work to be done here but at least this distraction is gone.
OK florian@ deraadt@ op@

Provide defaults for ciphers and macs if querying for them fails since
on some versions of Dropbear (at least v2024.85) "-m help" doesn't seem
to work.  Enable all supported pubkey algorithms in the server.

Use ed25519 keys for kex tests since that's supported by OpenSSH even when
built without OpenSSL.  Only test diffie-hellman kex if OpenSSH is compiled
with support for it.

Rework dropbear key setup to always generate ed25519 keys, other types
only if OpenSSH has support for the corresponding key type.

x_all.c: remove a bunch of unnecessary parentheses

Handle interfaces going down and up again.

Missing feature reported by, testing & OK semarie

Better words for downloaded snap is older message.
ok deraadt@ florian@ tb@

#if notyet -> #ifdef notyet for consistency and -Wundef

Stop trying to deprecate "test -L" in favour of "test -h" (or vice versa).
It's hopeless because POSIX requires both since Issue 6 (2001).
Both always worked on OpenBSD, no matter which base system shell was used.

According to research done by jsg@, it seems likely that actually,
"test -L" has precedence over "test -h" by about one year:
v8 (Feb 1985) had -L, SunOS 3.0 (Feb 1986) had -h; but SVR4 (1989)
already had both, so we are talking about 35 years of petrification.
More details: https://marc.info/?l=openbsd-bugs&m=171867441927989

Resolving a question raised by Tim dot theCHASEs dot com on bugs@.
OK deraadt@ millert@ jsg@ jmc@  and also works for Tim Chase.

Use BUILDINFO to make sure we are not going backwards in time.

Input & OK deraadt

remove prototypes with no matching function

v3_generic_extension() use ASN1_STRING_set0()

This aligns it with do_ext_i2d()

v3_generic_extension() rename the X509_EXTENSION

now that ext is free, we can use it like everywhere else

Rename 'ext' to 'name' in v3_generic_extension()

In this code 'ext' is usually used for an X509_EXTENSION object.

remove wdc_ata_addref()/wdc_ata_delref() #if 0'd since rev 1.1

Make local BIT_STRING_BITNAME variables const

There's no reason for them not to be const. This is a piece of a larger
diff that I carry in several of my trees to move more things to rodata
or relro. The full diff requires a change to a public header and it's
very annoying to have to 'make includes' and recompile the entire lib
all the time when hopping from tree to tree.

Re-enable ssh-dss tests if ssh is compiled with DSA support

Stop using DSA in dropbear interop tests.

x509_conf: rename ext_struc into ext_struct

requested by jsing on review

x509_conf: rename all ext_nid to nid

There are no nid variables in this file, so no need to disambiguate.

do_ext_i2d(): move empty line to the proper place

do_ext_i2d(): malloc -> calloc

requested by jsing on review

do_ext_i2d(): populate ext_oct with ASN1_STRING_set0()

ok jsing

do_ext_i2d(): avoid leaks and add some missing error checking

If ASN1_OCTET_STRING_new() failed, ext_der would be leaked, fix this.
If i2d(foo, NULL) succeeded, the same is not guaranteed for the second
with appropriately sized buffer since i2d() may make further allocations
internally. So use the proper error check. Also transfer the ownership of
ext_der to the octet string to avoid a now possible double free.

ok jsing

Indent labels in x509_conf.c

do_ext_i2d(): make various NULL checks explicit

ok jsing

do_ext_i2d(): unwrap a line

iked: do not attempt to read multiple SANs

No extension in a valid certificate appears more than once per RFC 5280
section 4.2. So don't go walking the extension stack and try to inspect
multiple subject alternative names because crappy OpenSSL API encourages
you to do so. Instead call the API in the only correct way possible and
report multiple SANs in log_info(). This is unlikely to be hit since the
extension caching in LibreSSL has rejected repeated OIDs in a cert for a
long time.

ok tobhe

Quiet compiler warnings when built with WARNINGS=Yes
Most are from functions that take no args but used the old
K&R style foo() instead of foo(void).  From espie@

grow the iso again

Rename mf_fgets() and cu_fgets() to mf_getline() and cu_getline().
These functions now use getline(), not fgets().  From espie@

Invalidate the right stack[] member

There was a shift of the index in the for loop, and invalidating the wrong
member could result in a double free in auth_tree_free() on process exit.

ok claudio job

Put back NULL checks in crl_get() and build_chain()

Turns out filemode still relies on these. Rather than adding to the
spaghetti in filemode.c, begrudgingly put back the NULL checks with
an additional XXX.

ok claudio job

Document the new argument .St -p1003.1-2024 introduced in st.c rev. 1.15
and polish the description of .St -p1003.1-2008 a bit.
OK job@ jmc@

remove one more mention of DSA

Fix typo: s/fetesteexcept/fetestexcept/

In vmx(4) TSO must pullup headers into first mbuf.

Forwarding IPv6 packets from vmx with LRO to vmx with TSO did not
work.  vmx(4) has the requirement that all headers are in the first
mbuf.  ip6_forward() is quite dumb.  It calls m_copym() to create
a mbuf that might be used for sending ICMP6 later.  After passing
the forwarded packet down to ether_encap(), m_prepend() is used to
restore the ethernet header.  As the mbuf cluster has been copied,
it is read only now.  That means m_prepend() does not provide the
empty space at the beginning of the cluster, but allocates a new
mbuf that contains only the ethernet header.  vmx(4) cannot transmit
such a TSO packet and drops it.

Solution is to call m_pullup() in vmxnet3_start().  If we ended up
in such a miserable condition, use the first mbuf in the chain and
move all headers into it.

OK jan@

KNF

KNF

When loading a device tree using the "mach dtb" command, give firmware
a chance to make modifications (such as applying memory reservations)
by using the EFI devicetree fixup protocol.

ok jca@

The fix comes from Giannis Kapetanakis (bilias _from_ edu.physics.uoc.gr).
When relayd(8) handles 'host disable/enable' command issued by relayctl(8),
it disables redirect it finds in tables for particular host.  However there can
be multiple redirect instances which use the same host in relayd(8) tables.
This change makes relayd(8) to walk through all tables and disable all redirects
which match the host.

OK giovanni@, OK sashan@

disable the DSA signature algorithm by default; ok markus@

(yes, I know this expands to "the Digitial Signature Algorithm
signature algorithm)

promote connection-closed messages from verbose to info log level;
they could be the only record of the connection terminating if the
client doesn't send a SSH2_MSG_DISCONNECT message. ok dtucker@

Change adds a 'log' option to relayd.conf(5) rule. The relayd(8) then uses
the option to set corresponding `log` action in pf(4) rules it generates
to handle network traffic.

The patch comes from Giannis Kapetanakis (bilias _from_ edu.physics.uoc.gr).

OK sashan@

drm/amdkfd: handle duplicate BOs in reserve_bo_and_cond_vms

From Lang Yu
3194771798ef256af75577a0a8a2e8ce45726c78 in linux-6.6.y/6.6.34
2a705f3e49d20b59cd9e5cc3061b2d92ebe1e5f0 in mainline linux

Revert "drm/amdkfd: fix gfx_target_version for certain 11.0.3 devices"

From Alex Deucher
bb430ea4ba31dcc439d26e02bcbc4473cad4c6d9 in linux-6.6.y/6.6.34
dd2b75fd9a79bf418e088656822af06fc253dbe3 in mainline linux

drm/amd: Fix shutdown (again) on some SMU v13.0.4/11 platforms

From Mario Limonciello
7bc52dce073222c1b6dd23a1e76a68495fc14794 in linux-6.6.y/6.6.34
267cace556e8a53d703119f7435ab556209e5b6a in mainline linux

drm/amdgpu/atomfirmware: add intergrated info v2.3 table

From Li Ma
4eff07025c844dfeaab8adbb6fbb617775a42423 in linux-6.6.y/6.6.34
e64e8f7c178e5228e0b2dbb504b9dc75953a319f in mainline linux

drm/fbdev-generic: Do not set physical framebuffer address

From Thomas Zimmermann
bd2ad553f18c43acd57d1ea46c59a1ecd860cea6 in linux-6.6.y/6.6.34
87cb4a612a89690b123e68f6602d9f6581b03597 in mainline linux

drm/amdgpu: add error handle to avoid out-of-bounds

From Bob Zhou
5b0a3dc3e87821acb80e841b464d335aff242691 in linux-6.6.y/6.6.34
8b2faf1a4f3b6c748c0da36cda865a226534d520 in mainline linux

drm/i915/hwmon: Get rid of devm

From Ashutosh Dixit
cfa73607eb21a4ce1d6294a2c5733628897b48a2 in linux-6.6.y/6.6.34
5bc9de065b8bb9b8dd8799ecb4592d0403b54281 in mainline linux

define CONFIG_PM_SLEEP to 1 for IS_ENABLED()

Replace x with x509_exts in X509V3_add1_i2d() and X509V3_get_d2i()

requested by jsing on review

Rewrite X509V3_get_d2i()

This API is wrapped by nine *_get{,1}_ext_d2i() functions and they all
have the same defect: if an idx variable is passed in, multiple extensions
are handled incorrectly.

Clean up the mess that was the current implementation by replacing the
reimplementation of X509v3_get_ext_by_NID() with extra twists by actual
calls to the real thing. This way the madness is implemented explicitly
and can be explained in comments. The code still gets shorter.

In brief: always call this API with a known nid, pass crit, and a NULL idx.
If NULL is returned, crit != -1 is an error (malformed cert or allocation
failure).

ok jsing

Add regress coverage for X509V3_get_d2i()

Add note about timeout(1)'s standards compliance

OK jmc@

Add new argument for IEEE 1003.1-2024 aka POSIX.1

OK jmc@ schwarze@

Disable MSI on Zhaoxin ZX-100/ZX-200/ZX-E StorX to unhang SSD

The Unchartevice 6640MA's BIOS forces one of three SATA speeds:
- Gen1/2: bsd.rd reaches installer, but SSD does not attach
- Gen3:   bsd.rd attaches SSD but hangs

MSI works for iwm(4) and and xhci(4), only ahci(4) bugs out, so add a quirk
for this controller as done for a few other devices already:

 ahci0 at pci0 dev 15 function 0 "Zhaoxin StorX AHCI" rev 0x01: apic 9 int 21, AHCI 1.3.1
-ahci0: device not communicating on port 0
+ahci0: port 0: 6.0Gb/s
 scsibus0 at ahci0: 32 targets
+sd0 at scsibus0 targ 0 lun 0: <ATA, 256GB SSD, V1.3> naa.5000000000000000
+sd0: 244198MB, 512 bytes/sector, 500118192 sectors, thin

OK kettenis

zap a stray space

regen

Add Zhaoxin vendor and AHCI product found in Unchartevice 6640MA notebook

https://www.devicekb.com/hardware/pci-vendors/ven_1d17 shows
"ZX-100/ZX-200/ZX-E StorX AHCI Controller" and the notebook has a bunch of
other devices, but only this one needs fixing so far.

Feedback jsg
OK deraadt

Make GENERIC boot on ZHAOXIN KaiXian KX-6640MA

The Unchartevice 6640MA notebook comes with such a CentaurHauls CPU,
installs via RAMDISK_CD (with AHCI fix), but GENERIC would hang after
cpu0: 4MB 64b/line 16-way L2 cache

Pretty sure Intel TPM sensor code should run on Intel CPUs, anyway.

Idea from brynet
OK deraadt brynet

same treatment for this test

penalty test is still a bit racy

To my current knowledge, UFSHCI has a single target design.  Reflect this
in the code, and remove some XXXs by that.  No functional changes.

We should block non-wakeup interrupts until we're in the resume path and
have disabled interrupts at the CPU level again.  So instead of
cpu_suspended use a new intr_suspended variable that is set and cleared in
intr_enable_wakeup() and intr_disable_wakeup().

ok mglocker@, mlarkin@

Install BUILDINFO if available.

We already keep /var/db/installed.SHA256 for sysupgrade(8) as a cheap
check to not install the same snapshot over and over again. With
/mnt/var/db/installed.BUILDINFO we can ensure that we are not
installing an older snapshot over a newer snapshot.

Intentionally not yet committing the sysupgrade(8) bits so that we
exercise the sysupgrade(8) / install.sub interaction as it will happen
during the 7.5 -> 7.6 transition. I.e. a newer install.sub getting
installed by an older sysupgrade(8).

deraadt@ likes it.

openssl-ruby: prepare test for default ruby switch

drm/i915/gt: Fix CCS id's calculation for CCS mode setting

From Andi Shyti
0b01a41e2ce40fda06dd46d9baf8523b95080ac8 in linux-6.6.y/6.6.33
ee01b6a386eaf9984b58a2476e8f531149679da9 in mainline linux