Rewrite X509_PUBKEY_set0_param() to use asn1_abs_set_unused_bits()

This streamlines the logic and uses ASN1_STRING_set0() and
asn1_abs_set_unused_bits() instead of inlining them.

ok jsing

Use asn1_abs_set_unused_bits() in asn1_str2type()

ok jsing

fdisk(8) no longer uses the partition table it finds in
/usr/mdec/mbr. Make the partition table all zeros and
see if any hidden uses fall out.

ok deraadt@

Set the correct alignment on __jmpxor (.align's meaning is MD!)
Put .L_setjmp_magic in the correct ELF section for duplicate suppression
Rename the "botch" label to delete it from the symbol table

ok kettenis@ miod@

Delete superfluous/duplicated #includes

Add END() tags to give these functions sizes in the ELF symbol table

ok kettenis@ miod@

Address the clang 13 "changed binding to STB_WEAK" warning on arm (32bit):
 * add _?ENTRY_NB to arm/asm.h
 * make sure ld.so's arm asm bits see the same includes as libc
 * switch libc's arm bits to the generic DEFS.h
 * switch arm ASM bits from ENTRY to ENTRY_NB as necessary

ok kettenis@ miod@

fix some dobeep_msgs calls

dobeep_msgs isn't printf-like: it just prints the two arguments
separated by a space.  When it was introduced, some calls from ewprintf
were incorrectly translated and the "%s" remained.

ok florian@

automatically delete trailing whitespaces on RET in c-mode and
auto-indent-mode (only after computing the auto indent.)

tested also by Mikhail (thanks!).  ok tb@

While not strictly required clear all pfds in the poll loop and not just
once at the start.
OK tb@

Introduce MAX_HTTP_REQUESTS and MAX_RSYNC_REQUESTS.
These just replace MAX_CONNECTIONS and MAX_RSYNC_PROCESSES to be more unified.
OK tb@

Recent changes to truncate(2) swapped the ordering of some validations
causing EACCESS as opposed of ESDIR to be returned while trying to
truncate a directory as a user lacking write permissions to the same
directory. As this behavior is reasonable, change the truncate directory
from /etc/ to /tmp which makes the test pass both as root and non-root.

Neither macppc nor the retired loongson have any remaining useful
information in /usr/mdec/mbr. Stop telling fdisk(8) that macppc
and loongson HAS_MBR, and don't bother including the file in the
base set.

macppc build/install tests and ok gkoehler@
loongson is gone deraadt@

Respect RLIMIT_FSIZE when extending a file via truncat(2)/ftruncate(2).
This refactors the commin parts of sys_truncate() and sys_ftruncate()
into dotruncate().  If the new size of the file is larger than the
RLIMIT_FSIZE limit _and_ the file is being extended, not truncated,
return EFBIG.  Adapted from a diff by Piotr Durlej.
With help from and OK by deraadt@ guenther@.

Improve #ifdefs for STACK_OF() helpers.

Discussed with claudio

whitespaces found when I went checking for something else

There is no need to be quiet by default anymore and also stdout is
perfectly fine. So switch the stats output at the end of the run to
simply use printtf(3) and no longer depend on -v flag.
OK tb@

add support for the ehci controller on marvell 3720 boards.

the marvell controllers have two quirks compared to standard ehci
controllers, but they're small enough that they can be dealt with
here rather than creating a specific glue driver (like imxehci has).

the first quirk is that the ehci registers are offset in the window
advertised by the device tree. this is handled by having ehci_fdt
always present a bus space subregion to ehci, and defaults the
offset to 0. marvell controllers provide the right offset into their
register window so the subregion code can find the right place for
ehci to operate on.

the other quirk is that marvell controllers need to be forced to
host mode with the extra EHCI_USBMODE register.

this makes ehci work on espressobins and gl-mv1000 boards.

tested by me and dtucker@
ok kettenis@ patrick@

In pf the kernel paniced if IP options in packet within ICMP payload
were truncated.  Drop such packets instead.
Reported-by: syzbot+91abd3aa2fdfe900f9ce@syzkaller.appspotmail.com
OK sashan@ claudio@

Fix white space.

Convert KVA allocation to km_alloc(9).

ok visa@

Lock kernel in nfsrv_rcv() because NFS subsystem is not MP-safe yet.

Tested in snaps for a week.

OK bluhm@

Convert KVA allocation to km_alloc(9).

ok mpi@

Document rc_exec.

Repair a leak of a FILE.  The FILE contains a fd via fdopen(), the
handling of the fd is correct, so this temporary fdopen now must use
a dup() so we can fclose().
ok otto

Remove some unhelpful comments and spell NULL correctly.

Factor out ASN1_ITYPE_EXTERN handling.

Factor out the ef->asn1_ex_d2i() callback handling - this allows us to pull
out all of the related variables into a self-contained function.

ok tb@

${rcexec} -> rc_exec

Use 'at' for ASN1_TEMPLATE variable names rather than 'tt'.

Also use array indexes for it->templates, rather than trying to be extra
clever in for loops (suggested by tb@ during a review).

No functional change.

ok tb@

Remove _ex_ from all of the internal function names.

It no longer makes sense to have "extended" versions of functions
internally.

No functional change.

ok tb@

Replace the $rcexec variable by an rc_exec function.
It is much cleaner to the eyes and makes more sense from a functionnal point of
view.
This will allow to extend rc_exec with other functionnalities (like upcoming
rc_startdir).
Bonus point: daemon_logger will now work with manually crafted rc_start
functions.

This will require a mechanical change from ${rcexec} to rc_exec in rc.d scripts.
ports will be fixed right after this commit but we will keep compatibility to
give a chance to people to fix their custom scripts.

positive tests from a few
ok robert@

No need for true(1).

repair a few more backwards compares that create headaches
ok millert

system(3) should ignore SIGINT and SIGQUIT until the shell exits.
This got broken when system.c was converted from signal(3) to sigaction(2).
Also add SIGINT and SIGQUIT to the set of blocked signals and unblock
them in the parent after the signal handlers are installed.
Based on a diff from Leon Fischer.  OK deraadt@

vmm: remove spinout paranoia

Move all the spinout counters for ipis (enable/disable vmm, remote
vmcs clear) behind MP_LOCKDEBUG.

All three areas (vmm_start, vmm_stop, vmx_remote_vmclear) migrate
to infinite busy waits.

tested by and ok mlarkin@

vmm: load the vmcs before reading vcpu registers

The vmm(4) ioctl for reading a vcpu's registers didn't flush and
load the vmcs before issuing vmread instructions. This adds a flag
to vcpu_readregs_vmx to signal if the vmcs needs to be synchronized
before attempting the reads. (This is the same approach used for
vcpu_writeregs_vmx function.)

This fixes `vmctl send` on Intel hosts using vmd(8).

While here, I noticed the vcpu_writeregs_vmx function doesn't
properly set the vmcs state variable to VMCS_CLEARED after running
a vmclear instruction. This can cause errors on vm re-entry.

ok mlarkin@

use correct logging or error calls

ftruncate sets errno, so use fatal() and not fatalx().

Checking for under-reads of read(2) isn't going to set errno so
use errx().

ok abieber@

as promised, extend the precaching using quirks stems extension.

(as it happens, some py-sip thingy got the treatment so I could test
that the code does something) !

Add expected failures for octeon and riscv64.

Re-classify the roff(7) \r (reverse line feed) escape sequence
from "ignore" to "unsupported" because when an input file uses it,
mandoc(1) is likely to significantly misformat the output,
usually showing parts of the output in a different order
than the author intended.

Test the handling of some additional one-character escape sequences
that take no argument and are ignored: \% \& \^ \a \d \t \u \{ \| \}
No change to parsing or formatting needed.

fold some of the useful "obsolete" information into the main text

Add RFC 7318

New ldapsearch does not support -h and -p for host and port, but
accepts -H ldapuri.  Use startup and cleanup framework.  Pidfile
does not work anymore, use pkill ldapd.  Delete deactivated and
failing cpan test.

Drop *out == NULL check in ASN1_STRING_to_UTF8()

Unfortunately, several things in the ecosystem depend on the existing
API behavior of being able to pass in an uninitialized pointer on the
stack: haproxy, grpc, mongo-tools and others show up on the first two
pages of Debian codesearch.

ok jsing

Don't pass uninitialized pointer to ASN1_STRING_to_UTF8()

Exposed by recent rewrite of ASN1_STRING_to_UTF8(). Found via grep
after fixing CID 352831.

ok jsing

Don't pass uninitialized pointer to ASN1_STRING_to_UTF8()

Exposed by recent rewrite of ASN1_STRING_to_UTF8().

CID 352831

ok jsing

Rename hid_none to hid_all as it represents all possible hid kinds by
now.

uhid_maxrepid() is currently not able to observe all report IDs as it
passes hid_none as the wanted hid kind to hid_start_parse(). This
becomes a problem for descriptors with multiple report IDs per
collection. Therefore change the semantics of the hid_none kind to let
it represents all possible kinds.

Gets rid of "uhidev_intr: bad repid" console spam as reported by sthen@
on bugs@.

Have been in snaps for a week.

Reorder functions within file.

Order functions by use, moving public API to the bottom and utility
functions to the top. This makes the code more logical/readable, plus we
can remove all except one of the static function prototypes.

No functional change.

Rewrite the asn1_template_*() functions with CBS.

Rewrite the asn1_template_*() functions with CBS, readable variable names
and free then alloc.

This was the last caller of asn1_check_eoc() and asn1_check_tag(), hence
remove them and rename the _cbs suffixed versions in their place.

ok tb@

following the fixed parsing direction of roff_expand() in roff.c rev. 1.260,
some diagnostics now appear in a more reasonable order, too

Adjust a column number in an error message
after the roff_expand() reorganization in roff.c rev. 1.260.

The new parsing direction has two effects:
1. Correct output when a line contains more than one expanding
escape sequence that has a side effect.
2. Column numbers in diagnostic messages now report the changed
column numbers after any expansions left of them have taken place;
in the past, column numbers refered to the original input line.

Arguably, item 2 was a bit better in its old state, but slightly
less helpful diagnostics are a small price to pay for correct
output.  Besides, when the expansion of user-defined strings or
macros is involved, in many cases, mandoc(1) is already unable to
report meaningful line and column numbers, so item 2 is not a
noteworthy regression.  The effort and code complication for fixing
that would probably be excessive, in particular since well-written
manual pages are not supposed to use such features in the first place.

fix a wrong column number that got fixed as a side effect
of the roff_expand() reorganization in roff.c rev. 1.260

remove a bogus warning that went away as a side effect
of the roff_expand() reorganization in roff.c rev. 1.260

Make roff_expand() parse left-to-right rather than right-to-left.
Some escape sequences have side effects on global state, implying
that the order of evaluation matters.  For example, this fixes the
long-standing bug that "\n+x\n+x\n+x" after ".nr x 0 1" used to
print "321"; now it correctly prints "123".

Right-to-left parsing was convenient because it implicitly handled
nested escape sequences.  With correct left-to-right parsing, nesting
now requires an explicit implementation, here solved as follows:
1. Handle nested expanding escape sequences iteratively.
When finding one, expand it, then retry parsing the enclosing escape
sequence from the beginning, which will ultimately succeed as soon
as it no longer contains any nested expanding escape sequences.
2. Handle nested non-expanding escape sequences recursively.
When finding one, the escape sequence parser calls itself to find
the end of the inner sequence, then continues parsing the outer
sequence after that point.

This requires the mandoc_escape() function to operate in two different
modes.  The roff(7) parser uses it in a mode where it generates
diagnostics and may return an expansion request instead of a parse
result.  All other callers, in particular the formatters, use it
in a simpler mode that never generates diagnostics and always returns
a definite parsing result, but that requires all expanding escape
sequences to already have been expanded earlier.  The bulk of the
code is the same for both modes.
Since this required a major rewrite of the function anyway, move
it into its own new file roff_escape.c and out of the file mandoc.c,
which was misnamed in the first place and lacks a clear focus.

As a side benefit, this also fixes a number of assertion failures
that tb@ found with afl(1), for example "\n\\\\*0", "\v\-\\*0",
and "\w\-\\\\\$0*0".

As another side benefit, it also resolves some code duplication
between mandoc_escape() and roff_expand() and centralizes all
handling of escape sequences (except for expansion) in roff_escape.c,
hopefully easing maintenance and feature improvements in the future.

While here, also move end-of-input handling out of the complicated
function roff_expand() and into the simpler function roff_parse_comment(),
making the logic easier to understand.

Since this is a major reorganization of a central component of
mandoc(1), stability of the program might slightly suffer for a few
weeks, but i believe that's not a problem at this point of the
release cycle.  The new code already satisfies the regression suite,
but more tweaking and regression testing to further improve the
handling of various escape sequences will likely follow in the near
future.

improve a comment explaining a particularly nasty hack; no code change

Remove unused macro (log_debuginfo)

Now MULTIPROCESSOR kernel boots with CPU modules installed in
arbitrary slots.

Plamen Mihaylov's LUNA-88K2 originally has 2 CPU modules in slot #0
and #2, but MULTIPROCESSOR kernel panics because they are not placed
in contiguous slots.  This diff fixes it.

Original diff from miod@, tested by Plamen Mihaylov and me.

document what this file stands for

Document that imsg_add(3) frees its msg argument on failure.

ok op@ claudio@

Adjust or remove some stale comments

Add missing space between No macro and "authority".

Deal with STACK_OF divergence between LibreSSL and OpenSSL

Rewrite mft.c and roa.c with ASN.1 templates

The ASN.1 templates are a rather direct translation of the ASN.1 in the
relevant RFCs and they allow deserializing the Manifest and ROA eContent
in a single step instead of numerous opaque d2i_ASN1_SEQUENCE_ANY() calls.
Once the eContent is deserialized, we can walk the structs, validate it
as before and populate the internal data structures.

Positive feedback job
ok claudio

Not so recent evolution of the PowerPC and POWER architectures have defined
exception addresses past EXC_LAST, making its definition wrong.

Replace it with EXC_END, which points to the end of hardware exception
addresses, and adjust logic accordingly.

ok kettenis@

Revert "drm/amd/pm: keep the BACO feature enabled for suspend"

From Alex Deucher
0fad10b263a33492107d62caa460d5e4c717ce29 in linux 5.15.y/5.15.41
a56f445f807b0276fc0660c330bf93a9ea78e8ea in mainline linux

Force maximum payload size to 128 bytes for AR816X/AR817x as it triggers
dma write error rendering the network unusable.

Bug reported and patch tested by Scott C. MacCallum.

document what PLIST_REPOSITORY does a bit better

Remove #ifdef DDB specific includes, added in 1.968 but related code bits
removed in 1.970.

ok bluhm@

Refactor asn1_d2i_ex_primitive()

Split the object content handling off into asn1_d2i_ex_primitive_content(),
move the handling ov V_ASN1_ANY into asn1_d2i_ex_any() and move the MSTRING
handling into asn1_d2i_ex_mstring(). This way we parse the header once
(rather than twice for ANY and MSTRING), then process the content, while
also avoiding complex special cases in a single code path.

ok tb@

document /etc/login.conf.d/
help robert ajacoutot

ok ajacoutot

ditch most of the obsolete stuff
there are still a few odds and ends which should make their way to the
rest of the manpage eventually (or to the faq), useful information that's
not immediately available elsewhere

Move towards OpenIKED 7.1

Refactor ASN.1 template functions before rewriting.

Change asn1_template_ex_d2i() so that we short circuit in the no explicit
tagging case.

Split out the SET OF/SEQUENCE OF handling from asn1_template_noexp_d2i()
into a asn1_template_stack_of_d2i() function and simplify the remaining
code.

ok tb@

Use a higher match priority such that mvclock(4) wins from syscon(4).

ok patrick@, dtucker@

Expose asn1_abs_set_unused_bits() in asn1_locl.h

Should have been part of a previous commit.

ok jsing

Rewrite make_addressRange() using CBS

Factor the trimming of the end and the counting of unused bits into
helper functions and reuse an ASN.1 bit string API to set the unused
bits and the ASN1_STRING_FLAG_BITS_SET. With a couple of explanatory
comments it becomes much clearer what the code is actually doing and
why.

ok jsing

Simplify make_addressPrefix()

In order to set the BIT STRING containing an address prefix, use existing
helper functions from the ASN.1 code instead of redoing everything by
hand. Make the function single exit and rename a few variables to make
it clearer what is being done.

ok jsing

Simplify unbound rc.d script.

input/ok sthen@ halex@

Add a simple test that exercises ASN1_STRING_to_UTF8() a little.

Clean up and fix ASN1_STRING_to_UTF8()

Instead of using a temporary variable on the stack, we can use the usual
Henson mechanism for allocating the struct. Make the function single exit
and throw an error instead of crashing or leaking if out is NULL or *out
is non-NULL.

tweaks/ok jsing

Remove a stray space

Avoid use of uninitialized in ASN1_STRING_to_UTF8()

A long standing failure to initialize a struct on the stack fully was
exposed by a recent refactoring. Fortunately, the uninitialized 'flag'
member is only used to decide whether or not to call freezero(NULL, 0),
so it is completely harmless. This is a first trivial fix, a better
version will be landed separately with regress.

Reported by Steffen Jaeckel, GH #760

ok beck

Rewrite asn1_item_ex_d2i_sequence() using CBS and readable variable names.

Now that combine no longer exists, we can also free and reallocate.

ok tb@

Remove incorrect line about 'dhcp' in hostname.if triggering the use of
dhclient.

OK tb, kettenis

The pf IPv4 option and IPv6 extension header filter has stricter
checks for IGMP and ICMP6 MLD packets.  Use ttl, hlim, link-local,
and multicast features in test where necessary.

document failed experiment

GC old stats code, I already removed the collection part

pfi_kif_alloc() may be called with M_NOWAIT.  Add NULL check to
handle malloc(9) failure.
from markus@; OK sashan@

regen

Unlock umask(2). sys_umask() only modifies `fd_cmask', which
modification is already protected by `fd_lock' rwlock(9).

ok bluhm@

use a safer idiom for reading pid from file before killing
pointed out by kn, tested in snaps for a week

regress test for in-place transfers and clobbering larger files with
smaller ones; would have caught last regression in scp(1)

fix in-place copies; r1.163 incorrectly skipped truncation in all cases,
not just at the start of a transfer. This could cause overwrites of larger
files to leave junk at the end. Spotted by tb@

Use strncmp() and IFNAMSIZ for if_xname in veb(4) consistently.
OK dlg@

More KNF and whitespace fixes.

Remove misplaced continue

ok job

whitespace spotted during read-thru

Specify which version of RPKI RSC draft is supported.