allow out-of-class defaulting of comparison operators

ok deraadt@, mortimer@

this is backport of the following upstream commit:

commit 5fbe21a7748f91adbd1b16c95bbfe180642320a3
Author: Nathan Sidwell <nathan@acm.org>
Date:   Wed Jun 16 10:43:43 2021 -0700

    [clang] p2085 out-of-class comparison operator defaulting

    This implements p2085, allowing out-of-class defaulting of comparison
    operators, primarily so they need not be inline, IIUC intent. this was
    mostly straigh forward, but required reimplementing
    Sema::CheckExplicitlyDefaultedComparison, as now there's a case where
    we have no a priori clue as to what class a defaulted comparison may
    be for. We have to inspect the parameter types to find out. Eg:

    class X { ... };
    bool operator==(X, X) = default;

    Thus reimplemented the parameter type checking, and added 'is this a
    friend' functionality for the above case.

unifdef
ok stsp@

bn_prime.pl: fix shebang and a couple more whitespace tweaks

Add comments documenting possible sources for MBR partition types
and GPT partition GUIDs.

No functional change.

Fix erroneous comment for the PT_OPENBSD_MUTABLE definition.

Move LINEBUFSZ to be with its friends in misc.h.

Nuke unused HELPBUFSZ.

No functional change.

Rename ascii_id() find_mbr_type(), make it a bit more like its
sibling find_gpt_type(), tweaking PRT_print_part() as needed.

No functional change.

Last arg is also a pointer, so pass NULL instead of 0; ok deraadt@

pine64: Silence dd(1) properly

We must not discard all errors, just disable informational messages.

OK tb

simplify final MAKEDEV call

No need to loop here, the script takes multiple args.

OK tb afresh1

sync nd6_resolve() EINVAL handling with arpresolve()

Less diff between them;  merging three returns into one also reduces
upcoming unlock diffs.  OK bluhm

sync nd6_resolve() uptime handling with arpresolve()

makes the two familiar functions look more alike; OK bluhm

Make *_name fields char * instead of char[]. Specify desired
width of output in printf() rather than "%-*s" and sizeof(name) +
1.  Clearer, may even save some bytes as many names are shorter
than the max.

Rename PRT_print_mbrtypes() to PRT_print_mbrmenu() and
PRT_print_gpttypes() to PRT_print_gptmenu() to remove implied
completeness of list presented in menu.

No functional change.

Change malloc chunk sizes to be fine grained.

The basic idea is simple: one of the reasons the recent sshd bug
is potentially exploitable is that a (erroneously) freed malloc
chunk gets re-used in a different role. malloc has power of two
chunk sizes and so one page of chunks holds many different types
of allocations. Userland malloc has no knowledge of types, we only
know about sizes. So I changed that to use finer-grained chunk
sizes.

This has some performance impact as we need to allocate chunk pages
in more cases. Gain it back by allocation chunk_info pages in a
bundle, and use less buckets is !malloc option S. The chunk sizes
used are 16, 32, 48, 64, 80, 96, 112, 128, 160, 192, 224, 256, 320,
384, 448, 512, 640, 768, 896, 1024, 1280, 1536, 1792, 2048 (and a
few more for sparc64 with its 8k sized pages and loongson with its
16k pages).

If malloc option S (or rather cache size 0) is used we use strict
multiple of 16 sized chunks, to get as many buckets as possible.
ssh(d) enabled malloc option S, in general security sensitive
programs should.

See the find_bucket() and bin_of() functions. Thanks to Tony Finch
for pointing me to code to compute nice bucket sizes.

ok tb@

things will be ready a bit earlier

Rename *_sname fields/functions to *_name to remove implied
shortness of the names.

No functional change.

Use strict and warnings

Make an attempt at reducing the eyebleed in bn_prime.pl

Use a style more resembling KNF and drop lots of parentheses. Simplify
a few things. No change in generated output on success.

Use Eric Young's usual license in the proper place rather than a weird
commented-out license stub in a HERE document.

Add RCSID

Add checks to ensure the uint16_t array isn't overflowed when this
script is run. This is more of an issue with uint16_t now than it
was with prime_t aka BN_ULONG before r1.6.

Zap an empty line

Drop unnecessary casts from and to void *

Unindent asn1_bio_get_ex()

The "snps,reset-*" properties are deprecatedand are being replaced with
"reset-*" proprties on the PHY device tree nodes.  Add support for this.

ok dlg@

Guard GF2m-specifics with OPENSSL_NO_EC2M

Enable FRP256v2 tests

This was previously disabled because tb apparently can't grep. Exercise
this curve as well as part of the new test cases in ECDH wycheproof.

Remove dead/unreachable code

A copy-paste error would have resulted in a modified msg in case ctLen == 0
or msgLen == 0. So obviously this is unreachable code.

Pull in <openssl/rsa.h> directly

This is needed for many reasons. It is currently pulled in via x509.h
but only when OPENSSL_NO_DEPRECATED is undefined. Again this should be
fixed in the public header as well.

BN_free() is defined in <openssl/bn.h>

This is currently pulled in via dsa.h and ecdsa.h, but only when
OPENSSL_NO_DEPRECATED is not defined. We should fix this in the
public header, too - let's wait a bit with that.

we are now hacking on 7.3-current

correct day of week

This year Lebanon springs forward April 20/21 not March 25/26.
This change is not yet part of an iana-tz or global-tz release.

Update to 2023agtz from https://github.com/JodaOrg/global-tz
 *  Egypt now uses DST again, from April through October.
 *  This year Morocco springs forward April 23, not April 30.
 *  Palestine delays the start of DST this year.
 *  Much of Greenland still uses DST from 2024 on.
 *  America/Yellowknife now links to America/Edmonton.

correct rk3308 clk_32k_sel mask
ok jmatthew@

update sppp(4) docs to mention remote address range hack

As already documented in pppoe(4). I missed updating sppp(4) years
ago when support for the wildcard address range was added.

avoid use after free
ok miod@ kettenis@

fix off-by-one in pf_state_expires() bounds test
such a value would have triggered a KASSERT()
ok sashan@ deraadt@

plug another leak in addctag; found + ok tb@

fix GMAC_CHAN_RX_CONTROL write
ok patrick@ kettenis@

avoid memleak / crash in addctag

the l pointer is advanced, so if the line is malformed `goto cleanup'
will free(NULL) or a pointer inside l.

semplification and ok tb@

plug memleak in error path

based on a diff by lux (lx [at] shellcodes [dot] org), thanks!
Diff via, tweak and ok tb@

delete trailing whitespaces; no functional change

Bump version

Reverse bus_space_unmap(9) order

When `unmap_1` is reached we should bus_space_unmap() sc_hwrm_* and not sc_db_*.

ok miod@

update the description for bwfm(4);

Document support for the BCM4378 / BCM4387 chipsets.

from Brad

Improve length checks for ATTR_MP_REACH_NLRI.
Based on a report by cjt (melissa_cjt at 163.com)
OK tb@

remove uneeded includes

sync

Aggressively randomize the location of the stack on all 64-bit architectures
except alpha.  This will put the stack at a random location in the upper
1/4th of the userland virtual address space providing up to 26 additional
bits of randomness in the address.  Skip alpha for now since it currently
puts the stack at a (for a 64-bit architecture) very low address.  Skip
32-bit architectures for now as well since those have a much smaller
virtual address space and we need more time to figure out what a safe
amount of extra randomizations is.  These architectures will continue to
use a mildly randomized stack address through the existing stackgap random
mechanism.  We will revisit this after 7.3 is released.

This should make it harder for an attacker to find the stack.

ok deraadt@, miod@

mda_expand_format: simplify token name extraction for %{name}
It is simpler to compute the length based on the start and end
pointers and pass that value to memcpy(), adding an explicit NUL
terminator after copying.  OK op@

rkpciephy(4)

Enable rkpciephy(40.

Add rkpciephy(4), ad friver for the PCIe 3.0 PHY dound on the RK356x.
The driver in principle supports bifurcation but this is (largely) untested.

ok dlg@

improve dmesg output to help with debugging.

for SoCs that can provide multiple instances of rkclock, let them
provide a name that will be printed during attach so you can tell
which one is doing what.

when rkclock_set_frequency isn't handling a clock, have it print
which rkclock instance isn't handling a clock. while here, print
the clock index the same way the #define refers to them.

ok kettenis@

Use a task to switch clocks on RK3568 as the clock API needs process context.

ok jmatthew@

regen

add another Navi 33 device id

0x7483 rev 0xcf is Radeon RX 7600M according to
Radeon Software for Linux version 22.40.3 (5.4.3) libdrm-amdgpu-common

Add a few more RK3568 clocks.

ok dlg@

Use a task to switch clocks on RK3568 as the clock API needs process context.

ok jmatthew@

disable POOL_DEBUG for release
ok deraadt@

Fix a potential NULL dereference in the unpriv child expanding %{mda}.
It is not legal to use %{mda} in anything but an mda wrapper.
mda_expand_token() will now return an error when %{mda} is used and
mda_command is NULL.  OK op@

KNF: some missing spaces after comma

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