Revert utf-8 fix for X509_NAME_get_index_by_NID to avoid libtls
regress for the moment.  this will come back after we rethink
the failure versus not there case.

ok tb@ jsing@

Introduce json_do_string() a function that JSON escapes a string.

Implement json_do_printf() using json_do_string() and vasprintf().
json_do_string() only escapes the basic control chars (\b, \f, \n, \r and \t)
other control chars are considered an error. Also the forward slash is
not escaped since the JSON data is not embedded into HTML or XML.
With feedback from tb@ & millert@
OK tb@

On read failure just abort the rrdp request.

Before a read error would not be removed from the poll fds and trigger
constantly. RRDP_STATE_PARSE_ERROR should only be used for errors from
xml parser since then the remaining data from the socket still needs to
be consumed.
OK tb@

Bring back length check tb ok'ed and I managed to remove while
changing tests.

ok tb@

Fix a few KNF/whitespace issues

Sometimes the touchpad doesn't attach.  Try to solve this issue be
reloading the firmware.

ok tobhe@

Absolutely astounding that custom code was written for the softraid case
to require "no", instead of "n" or "no" which works at every other damn prompt
in the installer.  This seems to be an artifact of relentlessly pushing people
towards softraid by default, and I think that is a bit nasty and pushy.

Change X509_NAME_get_index_by[NID|OBJ] to be safer.

Currently these functions return raw ASN1_STRING bytes as
a C string and ignore the encoding in a "hold my beer I am
a toolkit not a functioning API surely it's just for testing
and you'd never send nasty bytes" kind of way.

Sadly some callers seem to use them to fetch things liks
subject name components for comparisons, and often just
use the result as a C string.

Instead, encode the resulting bytes as UTF-8 so it is
something like "text",

Add a failure case if the length provided is inadequate
or if the resulting text would contain an nul byte.

based on boringssl.

nits by dlg@
ok tb@

Mark the BIO_F_* function codes as intentionally undocumented
and for now, skip the the BIO_R_* reason codes.
It looks like all public symbols in the BIO library
are now documented or marked as intentionally undocumented.

free the correct pointer.  Missed in rev 1.16.
ok dv@

Rename P into generator

ok jsing

Remove bogus newline in printf.

ok jmatthew@

Simplify EC_GROUP_new_by_curve_name()

Pull the setting of the name a.k.a. nid into ec_group_new_from_data().
This way, we can return early on finding the nid in the curve_list[].
This also avoids a silly bug where a bogus ERR_R_UNKNOWN_BUG is pushed
onto the error stack when ec_group_new_from_data() failed.

While there rework the exit path of ec_group_new_from_data() a bit.
Instead of an ok variable we can use an additional pointer to keep
track of the return value and free the EC_GROUP unconditionally.

ok jsing

Style tweaks for SMIME_write_PKCS7()

Initialize the mdalgs stack at the top and test and assign for ctype_nid.
Use an empty line to separate variable declarations from the actual code
and zap an extra empty line.

ok jsing

Switch K&R function definition to ANSI to make clang 15 happier

Unwrap a line

Simplify slightly and use i2d_PKCS7_bio_stream()

This is a wrapper of i2d_ASN1_bio_stream() that doesn't require us to
pass in PKCS7_it.

add Brussels South Charleroi airport

sync with userland

Sync with upstream

Update some links in the README, remove a duplicate word in a zlib.h doc
comment. The only code change is guarded by #if defined(_WIN32).

Call nd6_ns_output() without kernel lock from nd6_resolve().
OK kn@

stray whitespace

Bump to 8.0

Add a missing pair of braces.

Use uppercase for the CURVE_LIST_LENGTH macro

Consistently use lowercase hex digits for curve parameters

Now that we have C99 initializers, garbage collect some comments

Rework the curve list to use actual structs instead of a custom
serialized format.

ok jsing

Drop the now unnecessary and unused field_type from the curve data

ok jsing

Convert EC_CURVE_DATA to C99 initializers

Also clean up the definition of EC_CURVE_DATA a bit.

ok jsing

Simplify ec_group_new_from_data() further

We have a BN_CTX available, so we may as well use it. This simplifies
the cleanup path at the cost of a bit more code in the setup. Also use
an extra BIGNUM for the cofactor. Reusing x for this is just silly. If
you were really going to avoid extra allocations, this entire function
could easily have been written with three BIGNUMs.

ok jsing

Make warnings more precise

Import regenerated moduli.

The built-in 10G Ethernet on Apple arm64 hardware does not have a MAC
address programmed into the hardware.  Get it from the device tree instead.

ok dlg@, jmatthew@

Drop some dead code

No member of the curve_list[] table has a method set. Thus, curve.meth
is always NULL and we never take the EC_GROUP_new(meth) code path.

ok jsing

Remove pointless/wrong .meth = 0 entries from curves_list[]

Mechanically convert curve_list[] to C99 initializers

ok jsing

Clean up handling of nist_curves[]

There's no point in introducing a typedef only for two sizeof() calls.
We might as well use an anonymous struct for this list. Make it const
while there, drop some braces and compare strcmp() return value to 0.

ok jsing

Remove ASN1_item_ndef_i2d(3) documentation

This was the last public API explicitly named ndef/NDEF for indefinite
length encoding, so remove that explanation as well.

sync

First pass of removing low-level ASN.1 streaming docs

regen

Intel Braswell SDIO

spelling

add acpi(4) listing; ok miod

drm/fb-helper: set x/yres_virtual in drm_fb_helper_check_var

From Daniel Vetter
d27acf15c8fac00a251e2a24da09fcc1bb3337dd in linux-6.1.y/6.1.27
1935f0deb6116dd785ea64d8035eab0ff441255b in mainline linux

avoid use after free
ok jmatthew@

avoid use after free
ok florian@

regen

add Ryzen 7040 "Phoenix" APU device id

spotted in notebookcheck review of
Asus ROG Zephyrus G14 (2023) GA402XY, Ryzen 9 7940HS (Radeon 780M)

adjust ftruncate() logic to handle servers that reorder requests.

sftp/scp will ftruncate the destination file after a transfer completes,
to deal with the case where a longer destination file already existed.
We tracked the highest contiguous block transferred to deal with this
case, but our naive tracking doesn't deal with servers that reorder
requests - a misfeature strictly permitted by the protocol but seldom
implemented.

Adjust the logic to ftruncate() at the highest absolute block received
when the transfer is successful. feedback deraadt@ ok markus@

prompted by https://github.com/openssh/openssh-portable/commit/9b733#commitcomment-110679778

FSSIZE was not updated when 'fakeramdisk' was enlarged from 5760
blocks to 6080 blocks with etc.alpha/disktab r1.24.

Noticed by deraadt@

Fix typo in MRDISKTYPE.

'rdroot' (size 10,240 blocks) is not the same as 'rdboot' (size
2,048 blocks).

Noticed by deraadt@

x509_asn1: make this test pass again after reinstating DER preservation

check_complete.pl: update for recent changes in bn

mandoc -Tlint tells me I forgot to zap a comma

document that - with recent changes - the -A option now also excludes
the ASPA data from the JSON output.
ok claudio@

sync

Remove most documentation pertaining to proxy certificates.

Update EXFLAG_PROXY and X509_V_FLAG_ALLOW_PROXY_CERTS documentation since
we need to keep them for the time being.

Remove proxy cert api remmnants

Remove documentation of BN_generate_prime(), BN_is_prime{,_fasttest}()

Remove documentation of BN_zero_ex() and update BN_one() and BN_zero()
which are no longer macros (and the latter is no longer deprecated and
no longer attempts to allocate memory).

Garbage collect BN_zero_ex()

Remove the EFI RTC implementation on amd64.  Since all amd64 systems we
know have a MC146818A compatible RTC this code isn't actually used.  But
there are systems that have a buggy EFI implementation that blows up when
we call the GetTime runtime service to check whether the RTC functionality
is implemented.

ok mlarkin@, dlg@

msdosfs: Never allocate clusters outside the volume

- Assert that usemap_alloc() and usemap_free() cluster number argument
  is valid.
- In chainlength(), return 0 if cluster start is after the max cluster.
- In chainlength(), cut the calculated cluster chain length at the max
  cluster.

Adapted from FreeBSD commit 097a1d5fbb7990980f8f806c6878537c964adf32

ok miod@

Remove __dead again. Apparently this causes issues for some upstreams.

Thanks to orbea for the report

Revert disablement of the encoding cache

Without the cache, we verify CRL signatures on bytes that have been
pulled through d2i_ -> i2d_, this can cause reordering, which in turn
invalidates the signature. for example if in the original CRL revocation
entries were sorted by date instead of ascending serial number order.

There are probably multiple things we can do here, but they will need
careful consideration and planning.

OK jsing@

Send x509_subject_cmp() to the attic

This helper has been inside #if 0 for nearly 25 years. Let it go. If we
should ever need it, I'm quite confident that we will be able to come up
with its one line body on our own.

sync

The policy tree is no more

Mop up documentation mentioning it or any of its numerous accessors that
almost nothing ever used.

Zap extra blank line

Make the descriptions of BIO_get_retry_BIO(3) and BIO_get_retry_reason(3)
more precise.  Among other improvements, describe the three BIO_RR_*
constants serving as reason codes.

Slightly improve the documentation of the "oper" parameter by
explicitly listing the valid arguments, i.e. the BIO_CB_* constants.

Document the eight BIO_CONN_S_* constants that are passed to BIO_info_cb(3)
as the "state" argument.  Document them here because connect BIOs are
the only built-in BIO type using these constants.

Mark the five BIO_GHBN_* constants as intentionally undocumented.
They are intended to be used by BIO_gethostbyname(), which is deprecated
in OpenSSL and already marked as intentionally undocumented in LibreSSL.
Besides, these constants are completely unused by anything.

Remove artifical limit of 2 hours on a PIO lifetime, as recommended by
draft-ietf-6man-slaac-renum-05 and implemented by Linux in 2020.

OK florian@

whitespace

Sort alphabetically

Remove unnecessary target

policy test: simplify Makefile

Fix that atactl sd0 readattr didn't work for some disks.  Change it to
check the cksums of the attribute values instead of comparing the
revisions.  diff from NetBSD through naito.yuichiro at gmail.com. test
by kolipe.c at exoticsilicon.com.

ok kevlo miod deraadt

bump version to 8.4

New manual page written by Ted Bullock,
dropping the empty RETURN VALUES section
and adding the missing "#include <stdilib.h>" below EXAMPLES.

Provide function prototypes for macros that take arguments,
rename the "ev" argument to "event" to make some text read better,
and get rid of colons at the ends of list tags.

OK jmc@ and Ted Bullock.

Mention a few standard BIO_ctrl(3) command constants
that provide type-specific functionality here.
While here, fix some wrong return types in the SYNOPSIS.

Mention a few standard BIO_ctrl(3) command constants
that provide type-specific functionality here,
and add the missing return type to one function prototype.

Add "counter-timer" to openboot_special[] in order to not mention it as
unconfigured during boot, now that timer(4) is gone.

Mention the type-specific BIO_ctrl(3) command constants
in the manual pages of the respective BIO types.

Mention the type-specific BIO_ctrl(3) command constants
in the manual pages of the respective BIO type.
While here, fix some wrong return types in the SYNOPSIS.

Remove net lock from DIOCGETQUEUE

Same logic and argument as for the parent *S ioctl unlocked in r1.400,
might as well have committed them together:

  Both ticket and number of queues stem from the pf_queues_active list which
  is effectively static to pf_ioctl.c and fully protected by the pf lock.

OK sashan

remove some 19 year old #if 0 code

ok deraadt

whitespace

as noticed by sdk@, a package with an exact numbers of 64K chunks would
produce a spurious error (so 1 chance in 2^26)

It's like read/write: we need to recognize 0 as EOF and not try to checksum
a non-existing block.

while there, also make sure that we got all the signed blocks at EOF
before exit(0)

Note that none of those two bugs affect the actual security of signed
packages: the basic assertion that only signed data gets written
through the pipe is still 100% valid !

but it's a good idea to not emit spurious messages for valid files, and also
to recognize truncated files !

okay tb@ (thanks a lot)

Print VHE feature in dmesg.

ok mlarkin@, patrick@

Run open rsync and ports rsync programs against each other using
the --rsync-path option.  So we can see whether the tests pass in
all interoperability combinations.
Suggested by claudio@

Mark OpenSSLDie() as __dead

This tells gcc that OPENSSL_assert() will not return and thus avoids a
silly warning that triggers scary gentoo QA warnings.

From claudio

vmd(8): fix specifying boot image in vm.conf

Previous change to allow overriding changed the way we parsed and
stored the boot image path. The lifetime of the path was...much too
short. Heap allocate the kernel path.

Found by Mischa Peters.

ok mlarkin@

adjust after man_validate.c rev. 1.128 improved the error messages

Make LLVM 15 happier by changing from K&R to ANSI prototypes

Do not rewrite MAN_LP and MAN_P to MAN_PP because doing that causes
confusing warning messages complaining about macros that don't even
appear in the input file.
As a welcome side effect, this also shortens the code...

Fixing a minibug
reported by Alejandro Colomar <alx dot manpages at gmail dot com>.