Make cttest work better with the portable test harness

Fix cttest to use public header

Rename the largely misnamed bn_print.c to bn_convert.c

This file primarily contains the various BN_bn2*() and BN_*2bn() functions
(along with BN_print() and BN_options()). More function shuffling will
follow.

Discussed with tb@

Provide and use bn_copy_words() in BN_copy().

This is simpler than the current code, while still being well optimised by
compilers, across a range of architectures. In many cases we even get a
performance gain for the BN sizes that we primarily care about.

Joint work with tb@

Add support for truncated SHA512 variants.

This adds support for SHA512/224 and SHA512/256, as specified in FIPS
FIPS 180-4. These are truncated versions of the SHA512 hash.

ok tb@

Provide soon to be used crypto_store_htobe32().

ok tb@

Use memset() and only initialise non-zero struct members.

ok tb@

Fix double free in error path in openssl(1) x509

A conversion from X509_REQ_get_pubkey() to X509_REQ_get0_pubkey() missed
one free of pkey in an unlikely error path. After the conversion pkey is
no longer owned by us, so we mustn't free it.

ok jsing

add support for "rockchip,cryptov2-rng"

the steps to operate rockchip,cryptov2-rng are basically the same
as the existing rockchip,cryptov1-rng support, but the registers
and bits have moved around. add some abstraction for the register
differences and have the state machine call the different backends.

this is present on rk356x chips as the "True Random Number Generator
(TRNG)".

tested on a bunch of different rk3568 boards.
ok kettenis@

A tab snuck in

add the "local experiments" ethertypes

Avoid an overflow in the ELF SYSV ABI hash function.
The hash function is supposed to return a value less than or equal
to 0x0fffffff.  Due to a bug in the sample code supplied with the
ELF SYSV ABI documentation, the hash function can overflow on 64-bit
systems.  Apply the same fix used by GNU libc, MUSL libc and FreeBSD.
Prompted by https://maskray.me/blog/2023-04-12-elf-hash-function
OK tb@ miod@

Sprinkle UL suffix to constant literals which don't fit in int. NFCI

Catch up with box drawing characters which have been standardized in unicode
after the original wscons code was written and chose placeholder values.

From NetBSD (wsemul_vt100_chars.c r1.8 and r1.14) via Crystal Kolipe, thanks!

Use ANSI-style functions for Base64 wrappers

Silences a few -Wdeprecated-non-prototype warnings emitted by clang 15.

ok bluhm miod

Check whether products listed on a manifest were issued by the same authority as the manifest itself

OK tb@

Simplify how IMSG_CTL_SHOW_RIB_COMMUNITIES is constructed. This can just
call imsg_compose() and be done with it.
OK tb@

Grammar fixes in comments.

pmap_copy() has never, ever, been implemented in any of the platforms OpenBSD
ever ran on, and it's unlikely to ever be implemented, so remove it.
ok jsg@

The NBs have been duly noted and ignored. Drop them.

Zap trailing whitespace

Remove intentionally undocumented pci_{io,mem}_find and convert their last
few users to pci_mapreg_info().

ok jsg@

Move USRSTACK to the end of userland address space.

Remove files that definitely contain no code anymore
(experts disagree whether they ever did)

Drop now useless files from the Makefile

Fold ECDSA sign and verify mess into ecs_ossl.c

discussed with jsing

Move RSA_generate_key() from rsa_depr.c to rsa_gen.c

Discussed with jsing

Move DSA_generate_parameters() from dsa_depr.c to dsa_gen.c

Discussed with jsing

Move DH_generate_parameters() from dh_depr.c to dh_gen.c

discussed with jsing

Sort commands alphabetically both in the tables and manpage.
OK tb@

Fix compilation on sparc64.

zap leftover lines from previous

ec_lib.c: fix a few NULL misspellings

Fix various early return issues spotted by coverity

A large mechanical diff led to sloppy review and gave coverity an
opportunity to be right for once. First time in a good many weeks.

same diff/ok jsing

ectest: missing error checking

CID 452228

remove duplicate includes
ok deraadt@ miod@ krw@

bgpctl network bulk requires now the specification of 'add' or 'delete'.
In the add case the extra attributes can be specified afterwards.
This makes the parser behave cleaner since 'add' and 'delete' are removed
from the attribute set table.
OK tb@

Pull MP-safe arprequest() out of kernel lock

Defer sending after unlock, reuse `refresh' from similar construct.

OK bluhm

regen

Add BCM4388.

ok patrick@, deraadt@

correct the entry for chilepeso; from jan stary

remove duplicate signal.h include

Simplify code by folding uuid_attr() into PRT_protected_uuid().

No functional change.

Clarify logic in PRT_uuid_to_menudflt() by using find_gpt_menuitem().

No functional change.

regen

Add PCI IDs for the 2nd generation Aquantio 10G NICs.

ok mlarkin@

remove duplicate lines

Add test for multiprotocol announce statements

remove duplicate lines

fix double words
ok dtucker@

Fix 24bit maximum/mask in manpage. Noticed by wangqr on github.

Remove now unused sha_local.h.

Provide and use crypto_ro{l,r}_u{32,64}().

Various code in libcrypto needs bitwise rotation - rather than defining
different versions across the code base, provide a common set that can
be reused. Any sensible compiler optimises these to a single instruction
where the architecture supports it, which means we can ditch the inline
assembly.

On the chance that we need to provide a platform specific versions, this
follows the approach used in BN where a MD crypto_arch.h header could be
added in the future, which would then provide more specific versions of
these functions.

ok tb@

Provide and use crypto_store_htobe64().

It is common to need to store data in a specific endianness - rather than
handrolling and deduplicating code to do this, provide a
crypto_store_htobe64() function that converts from host endian to big
endian, before storing the data to a location with unknown alignment.

ok tb@

Align printing of geofeed records in filemode

remove unused buffer selection code
ok visa@

Abstract find_[gpt|mbr]_menuitem() to simplify code and
prepare for future uses.

No functional change.

Handle BN_CTX at the EC API boundary.

The EC API allows callers to optionally pass in a BN_CTX, which means that
any code needing a BN_CTX has to check if one was provided, allocate one if
not, then free it again. Rather than doing this dance throughout the EC
code, handle the BN_CTX existance at the EC API boundary. This means that
lower level implementation code can simply assume that the BN_CTX is
available.

ok tb@

Clean up unused BIGNUM.

Simplify code by passing pointers to the appropriate type to
find_gpt_desc() and find_mbr_desc().

No functional change.

Document the RETURN VALUES of BIO_method_type(3) and BIO_method_name(3)
for the various BIO types.

Shuffle mbr and gpt function prototypes and declarations together.

No functional change.

Recommit jsing's r1.27 - portable is ready

Use htobe64() instead of testing BYTE_ORDER and then handrolling htobe64().

Thanks to tobhe for providing most of the fix via openiked-portable

While all the BIO_TYPE_* constants are part of the API, most of their
values are only part of the ABI and not of the API, so delete them
from the SYNOPSIS: application programmers must not rely on the
specific values.

Instead of listing the specific values, properly describe the meaning
of all these constants.

However, the values of BIO_TYPE_NONE and BIO_TYPE_START are hard-coded
into the API and application programmers need to be aware of their
values, so those remain in the SYNOPSIS.

be more forceful in pointing people to bulk(8) and proot(1).

Back out r1.27 using htobe64() - apparently some OS don't have it.

ok jsing

give people a chance to find bulk(8) from proot(1)

Consolidate sha1 into a single file.

Consolidate sha1 into a single file.

Simplify handling of big vs little endian.

Rather than sprinkling BYTE_ORDER checks throughout the implementation,
always define PULL64 - on big endian platforms it just becomes a no-op.

ok tb@

Use htobe64() instead of testing BYTE_ORDER and then handrolling htobe64().

ok tb@

Omit sha512_block_data_order() prototype when assembly is not being used.

In the case that the pure C implementation of SHA512 is being used, the
prototype is unnecessary as the function is declared static and exists
in dependency order. Simply omit the prototype rather than using #ifndef
to toggle the static prefix.

ok tb@

Remove less than useful implementation notes.

ok tb@

bn_mod_sqrt: remove no longer necessary complications due to the
non-deterministic nature of the old implementation.

Add a new implementation of BN_mod_sqrt()

This is a reimplementation from scratch of the Tonelli-Shanks algorithm
based on Henri Cohen "A Course in Computational Algebraic Number Theory",
Springer GTM 138, section 1.5.1. It is API compatible with the previous
implementation, so no documentation change is required.

Contrary to the old implementation, this does not have any infinite loops
and has various additional sanity checks to prevent misbehavior in case
the input modulus is not a prime. It contains extensive comments and the
individual parts of the algorithm are split into digestible chunks instead
of having one huge function.

One difference of note is that it BN_mod_sqrt() now always returns the
smaller of the two possible answers. In other words, while its core is
non-deterministic, its answer is not.

ok jsing

Correct benchmark result computation on 32 bit platforms.

Nuke unused function prototype.
Only acknowledge interrupts when one was active.

ok kettenis@

fix double words in comments
feedback and ok jmc@ miod, ok millert@

add missing write of SAB_DAFO
ok miod@ who tested on Ultra 5

Some more cleanup in bn_to_string

Drop BN_NO_DEPRECATED dance from bn_test

Make GPT display of menu hex octet the same as the MBR display.
i.e.  two digits.

Provide benchmarks for BN_copy()

Fix indentation of structs and unions in x509v3.h

No change according to diff -w

Enable caps lock LED on modern Apple laptop keyboards.

ok kettenis@ patrick@

Make bn_to_string() static

This function is no longer used directly by regress, so it can now be local
to this file.

bn_to_string no longer needs to be linked statically

Rework the bn_to_string() to use public API

We can use the undocumented functions {i2s,s2i}_ASN1_INTEGER(3) to
exercise bn_to_string(). This way we use public API and remove the
need of linking statically.

Various wording tweaks for clarity and precision, and a few for conciseness.
OK jmc@ and Ted Bullock

More deduplication: talk about environment variable in one place, not two.
Clarify that the variables only affect the event_base structure currently
being created.  They do not disable "library support" as a whole.
Sort the variables alphabetically.

OK jmc@ and Ted Bullock

fix 'in in'

fix setting parity bits
ok mglocker@

spelling

Treat symlinks better in $ORIGIN determination in ld.so

Now symlinking an executable away from the rest of its installation
tree no longer prevents it from finding the libraries. This matches
the behavior of other OS linkers. Prompted by a behavior change in
lang/ghc test suite.

Swapped the order of dirname/realpath in _dl_origin_path.

Added some regress tests that pass and then bin3 that fails without
this change and reflects the behavior needd for lang/ghc.

Suggestion by semarie@, OK deraadt@

So clang 13 miscompiles dev/usb/umass_scsi.c with the combination if -Oz
and -mbranch-protection=bti.  So turn off the BTI protection in ramdisk
kernels for now.

ok deraadt@, miod@, phessler@

Move a few functions out of OPENSSL_NO_DEPRECATED

Geoff Thorpe added OPENSSL_NO_DEPRECATED nearly two decades ago. The hope
was that at some point some functions can be dropped. Most of the functions
marked deprecated are actually unused nowadays but unfortunately some of
them are still used in the ecosystem. Move them out of OPENSSL_NO_DEPRECATED
so we can define it without breaking the consumers in the next bump.

ERR_remove_state() is still used by a dozen or so ports. This isn't a big
deal since it is just a stupid wrapper for the not quite as deprecated
ERR_remove_thread_state(). It's not worth patching these ports.

Annoyingly, {DH,DSA}_generate_parameters() and RSA_generate_key() are still
used. They "make use" of the old-style BN_GENCB callback, which is therefore
more difficult to remove - in case you don't know know: that's the thing
responsible for printing pretty '.', '+' and '*' when you generate keys.

Most annoyingly, DH_generate_parameters() was added to rust-openssl in 2020
for "advanced DH support". This is very unfortunate since cargo bundles a
rust-openssl and updates it only every few years or so. As a consequence
we're going to be stuck with this nonsense for a good while.

ok beck jsing

Remove some doubled empty lines

Remove X9.31 support from openssl(1)

The X9.31 standard has long been retired and deprecated and libcrypto will
drop support for it soon. This prepares userland.

ok jsing

Drop X9.31 support from libtls

The TLS signer isn't exposed in public API (we should finally fix it...)
and it supports X9.31, a standard that has been retired and deprecated for
a very long time. libcrypto will stop supporting it soon, this step is
needed to prepare userland.

ok jsing

don't attach temperature sensors for which there is no TMP# node in
the DSDT, and verify the result of the fetch when updating it later.

ok kn