Simplify apple firmware copying

Less duplicate code, easier to add new firmware.

OK afresh1

Remove unnecessary HIDDEN_DECLS.

Removes some unwanted spaces.

Whack sha1dgst.c with the style(9) stick again.

Add regress coverage for some BIO_f_asn1() bugs fixed before release

Minor whitespace tidying

Tidy up includes.

Inline sha_local.h in sha1dgst.c.

Nothing other than sha1dst.c uses this header - pull it in to sha1dgst.c
directly (sha_local.h will be removed at a later date).

Fix a typo in dump_exp_results()

Add more extensive regress coverage for BN_mod_exp2_mont()

Rename dump_results() into dump_exp_results()

This prepares for consistency with an upcoming diff. While here fix an
argument order bug.

Move the N_MOD_EXP_TESTS up to where it belongs

Make several calls to BN_nnmod() unconditional

This removes a potential branch in a sensitive function and makes the
code a lot simpler. It is a really bad idea optimize here for what
davidben aptly calls "calculator" purposes.

ok jsing

Correctly reduce negative inpot to BN_mod_exp2_mont()

Negative bases could result in a negative modulus being returned. This is
not strictly speaking incorrect but slightly surprising. This is all a
consequence of the shortcut of defining BN_mod() as a macro using BN_div().

Fixes ossfuzz #55997

ok jsing

Add regress coverage for an issue with BN_mod_exp2_mont() reported
by Guido Vranken in ossfuzz #55997. This test currently fails and
will be fixed momentarily.

This also checks sensible behavior with respect to zero exponents for
this functions.

bn_primes: there is no need to link this statically. It can pull the
primes table in directly.

amd64: identify IBT capability in cpu(4) dmesg lines

requested by and ok deraadt@

Another missing #include <openssl/err.h>

Add missing #include <openssl/err.h>

Add license to sha256.c/sha512.c.

Use multiple statements instead of comma separated expressions.

No change to generated assembly.

The slightly less deprecated spelling of ERR_remove_state(0) is
ERR_remove_thread_state(NULL).

Add blank lines for readability.

Add some blank lines for readability, along with some more style(9) tweaks.

Whack sha with a style(9) stick.

No change in generated assembly.

Pass lbuf/lbuflen to PRT_print_[mbr|gpt]_menu() to allow changes
to the original input. Just zap it for now.

No functional change.

bn_mod_exp_ret() fix stupid logic error

Add EFI_SYSTEM_PARTITION_GUID #define and use it to check for
protection.

No functional change.

Add MICROSOFT_BASIC_DATA_GUID #define to make clear what menu
items result in the same GUID. Easier to spot and more resistant
to simple typos in a 36 character hex string.

No functional change.

Add support for the RK3568 32k RTC clock.  This clock uses a fractional
divider so rework the RK3399 support for fractional dividers to be more
general and reuse it.

ok dlg@

fix memory leaks in error paths
ok miod@

fix a memory leak
feedback kettenis@ ok miod@

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