pf(4) ignores 'keep state' and 'nat-to' actions for unsolicited
icmp error responses. Fix tightens rule matching logic so icmp
error responses no longer match 'keep state' rule. In typical
scenarios icmp errors (if solicited) should match existing state.
The change is going to bite firewalls which deal with asymmetric
routes. In those cases the 'keep state' action should be relaxed
to sloppy or new 'no state' rule to explicitly match icmp
errors should be added.

The issue has been reported by Peter J. Philip (pjp _at_ delphinusdns.org).

Discussed with bluhm@ and florian@

OK bluhm@

Accept 65 for VT525 as well.

convert to ansi function decl

regen

add Realtek 8852AE

ok kevlo@

mention ConnectX-6 Dx and ConnectX-6 Lx

match on Mellanox ConnectX-6 Lx
from and tested by Olivier Croquin
ok dlg@

regen

add Mellanox ConnectX-6 Lx
from Olivier Croquin

Fix missing function declarations to let mkhybrid build with clang-16

ok millert@

regen

add a Navi 32 device id

747e rev c8 is Radeon RX 7800 XT
747e rev ff is Radeon RX 7700 XT
found in AMD Software: Adrenalin Edition 23.9.1

regression test for override of subsystem in match blocks

allow override of Sybsystem directives in sshd Match blocks

allocate the subsystems array as necessary and remove the fixed limit of
subsystems. Saves a few kb of memory in the server and makes it more like
the other options.

preserve quoting of Subsystem commands and arguments. This may change
behaviour of exotic configurations, but the most common subsystem
configuration (sftp-server) is unlikely to be affected.

downgrade duplicate Subsystem directives from being a fatal error
to being a debug message to match behaviour with just about all other
directives.

handle cr+lf (instead of just cr) in sshsig signature files

point GH_* to DIST_TUPLE for complicated stuff
remove description of MTREE_FILE that was removed a few years ago

ROACH_URL and ROACH_SITES, to help portroach figure out the new things.

Correct error message: vionet -> vioblk. No functional change.

vmd(8): clean up struct ioinfo.

In prep for fixing some vioblk device issues, simplify the ioinfo
struct by dropping members that aren't needed.

ok mlarkin@

Mark EVP_CIPHER_set_asn1_iv() and EVP_CIPHER_get_asn1_iv() as intentionally
undocumented because they are unused outside libcrypto according
to codesearch.debian.net and should probably not be public: they seem
hardly useful even for implementing custom EVP_CIPHER algorithms.

tb@ came to similar conclusions regarding these two functions.

minimal linting, no output change: .Ed with an argument;
stray .Pp before .It; new sentence, new line; trailing whitespace

Initialize afi & safi to zero

OK tb@

Avoid use-of-uninitialized in i2r_IPAddrBlocks()

Reported by Viktor Szakats in
https://github.com/libressl/portable/issues/910

ok job

reflect new reality

Use casts to force two function pointers assignements and fix build with clang-16

"looks fine to me" robert@, ok kettenis@

bsd.port.mk.5: fix markup of NO_ARCH variable

OK kn@ schwarze@

explicitly cast ULLONG_MAX to double when comparing to a double

ok krw@

Correct the RETURN VALUES of OBJ_add_object(3).
The text was misleading before and after the improvement
in obj_dat.c rev. 1.61.  The way i'm fixing the documentation
here takes that improvement into account.

Also add a CAVEATS section about adding incomplete objects.

Use -std=gnu89 to fix the build with clang-16

This is ancient GNU software that tends to break whenever clang adds
more warnings about deprecated features in new languages versions or
turns them into errors.  Using -std=gnu89 (the defaults for base-gcc)
for those packages means we'll probably avoid most new warnings, errors,
and possible miscompilations when updating clang.

ok robert@

regen

be more specific in describing the magic behind EXTRACT_CASES

add Intel Optane SSD P1600X
from Andreas Bartelt

remove uneeded function decls
ok tb@

Build with -std=gnu89 to fix the build with clang-16

This is ancient GNU software that tends to break whenever clang adds
more warnings about deprecated features in new languages versions or
turns them into errors.  Using -std=gnu89 (the defaults for base-gcc)
for those packages means we'll probably avoid most new warnings, errors,
and possible miscompilations when updating clang.

This fixes the build of libiberty and texinfo with clang-16.

ok robert@

nits

Use shared net lock for ip_send() and ip6_send().

When called with NULL options, ip_output() and ip6_output() are MP
safe.  Convert exclusive to shared net lock in send dispatch.

OK mpi@

use ansi function decls

use ansi function decls

Add missing <sys/time.h> include for timespecsub()

document ERRORS better

Remove -mabi=elfv2 option.  This is the default for OpenBSD and clang 16
generates a (spurious) error about it in certain contexts.  This is fixed
in later versions (see https://reviews.llvm.org/D156351) but it is easier
to just drop the option.

ok miod@, jsg@

remove doubled entry in config.h.in

regen

remove extra blank line to reduce changes to upstream in a locally-patched file

whitespace fixes, problem with util/tube.c pointed out by jsg (probably
introduced by the diff/patch process I use to avoid stomping on the local
changes in updates)

use ansi function decls

remove unused variable

remove unused variable

add void to function decls with no args

remove unused variable

vmm(4)/vmd(8): include pending interrupt in vm_run_parmams.

To remove an ioctl(2) from the vcpu thread hotpath in vmd(8), add
a flag in the vm_run_params structure to indicate if there's another
interrupt pending. This reduces latency in vcpu work related to
i/o as we save a trip into the kernel just to flip the interrupt
pending flag on or off.

Tested by phessler@, mbuhl@, stsp@, and Mischa Peters.

ok mlarkin@

remove unused variable

clockintr: replace u_int with standard types

The clockintr code already uses uint64_t everywhere, so we may as well
be consistent: replace u_int with uint32_t everywhere it is trivial to
do so; leave the sysctl(2) hook and ddb(4) code alone for now.

Suggested by mpi@.

ok mpi@

clockintr: clockintr_establish: change first argument to a cpu_info pointer

All CPUs control a single clockintr_queue.  clockintr_establish()
callers don't need to know about the underlying clockintr_queue.
Accepting a cpu_info pointer as argument simplifies the API.

From mpi@.

ok mpi@

revert disabling warnings for zlib on clang >= 15
no longer needed with zlib 1.3
ok tb@

DISNAME -> DISTNAME

According to the C11 standard, char32_t and char16_t are not part
of the C language but are part of the C library and have to be
declared in <uchar.h> - see paragraph 7.28.2.

In stark contrast, according to the C++11 standard, char32_t and char16_t
are part of the C++ language, namely, keywords - see paragraph 2.12.1.
Consequently, they must not be declared in a header file.

To resolve this vile contradiction, use the predefined macro __cplusplus
to find out which language is in use for the current compilation unit -
see C11 paragraph 6.10.8.3 and C++11 paragraph 16.8.1.

Reminded of the problem by naddy@.
OK naddy@ who tested in make build / make release.
Looks reasonable to millert@.

clockintr: remove clockintr_expiration()

With the introduction of clockintr_advance_random(), this interface is
now unused.  Remove it.

Suggested by mpi@.

Thread: https://marc.info/?l=openbsd-tech&m=169392340028978&w=2

ok mpi@ mlarkin@

clockintr: remove clockintr_nsecuptime()

With the introduction of clockintr_advance_random(), this interface is
now unused.  Remove it.

Suggested by mpi@.

Thread: https://marc.info/?l=openbsd-tech&m=169392340028978&w=2

ok mpi@ mlarkin@

clockintr: add clockintr_advance_random()

Add clockintr_advance_random().  Its sole purpose is to simplify the
implementation of statclock's pseudorandom period.

Ideally, nothing else will use it and we will get rid of it and the
pseudorandom statclock period at some point in the near future.

Suggested by mpi@.

Thread: https://marc.info/?l=openbsd-tech&m=169392340028978&w=2

ok mpi@ mlarkin@

github.io is https
zap the dot so that dumb cut&paste works

Log a dummy "<unknown>" IP address in the unlikely event that genameinfo(3) fails

clang-16 warning reported by robert@, ok tb@ millert@

Fix indentation, found with -Wmisleading-indentation.

Already commited upstream.

Prevent warnings from -Wmissing-prototypes

https://github.com/NLnetLabs/unbound/pull/931

Fix built.

The api changed to handle cookies and extended error codes.

Passing in NULL for config_file disables cookie handling so we do not
need to pass a valid comm_reply, NULL will do.

sync to unbound 1.18.0; heavy lifting by sthen

fix mismatching declaration of argument with mismatched bounds

ok tb@

function definitions without prototypes are deprecated, so shut clang16 up

ok kettenis@

add a BUGS section warning about the creation of incomplete objects

Improve error handling in OBJ_add_object()

There is no need for a helper function to obfuscate lh_ADDED_OBJ_new().
Just call the real thing directly.

Adding an object with a NID of NID_undef basically amounts to disabling
a built-in OID. It does so in an incoherent fashion and the caller can't
easily tell success from failure of the operation. Arguably the result is
a corrupted objects table.

Let's not allow adding such an object in an attempt at keeping things
slightly more coherent.

Issue noted and initial diff by schwarze while writing documentation

ok schwarze

document EVP_CIPHER_name(3)

add the missing .Dv macros to the list of flags

improve the descriptions of almost all flags,
in particular saying which API functions each flag affects

unifdef HAS_INLINES
inline is part of gnu89 and c99
ok kn@ espie@

vmm(4): switch the APMI CPUID mask to an include mask

dv points out that there are other bits there that imply the existence
of other MSRs, so switching this to an include list is a better idea.

reorder

automated MASTER_SITE* -> SITE* renaming
(need some alphabetical reordering)

Partial rewrite:
* Integrate the leftovers of the former NOTES section into the main text,
resulting in a more logical order of information.
* Make many descriptions more precise and tweak many wordings.
For example, the description of OBJ_cmp(3) was totally misleading.

Add a CAVEATS section explaining the scary ownership contracts
of the functions returning ASN1_OBJECT pointers.

Move the discussion of NID_undef to the BUGS section because the
statement "objects which are not in the table have the NID value
NID_undef" was misleading in more than one way.

Considering that an API as fundamental as this one contains such a
gigantic amount of quirks and traps and gaps makes me shudder.

avoid fatal errors at all costs

DIST_TUPLE's handling of HOMEPAGE is optional, of course

update informal toc

stop putting multicast addresses into the Receive Address Registers

Hash them all into the Multicast Table Array.  This matches what
FreeBSD does.

This fixes a problem where the I217-LM would fail to receive packets
for some of the programmed addresses.

Approach agreed by jmatthew@ dlg@
OK claudio@

Make ksmn(4) attach to 19h/4x devices

Tested on:
cpu0: AMD Ryzen 5 PRO 6650U with Radeon Graphics, 2900.00 MHz, 19-44-01, patch 0a404102

ok mlarkin

pof for check-lib-depends

be blunter about IS_INTERACTIVE

drop very old history

strongly hint that people take advantage of sqlports

document moving mechanisms

use terminology for other projects: it's still "fake" to us, but
acknowledge the more common name "staging area"

There's no index in bsd.port.mk these days.

Be more specific in some places.
In  particular, telling people not to use pre-fake unless
they know what they're doing is idiotic.

oops, forgot to rm these before cvs import

merge unbound-1.18.0

import unbound 1.18.0, ok florian

Fix touchpad on newer device trees. The *gpio fields moved up one layer.
The driver will work with both formats for now but we plan to remove
the old one in the future.

ok kettenis@

readmes hasn't existed for a long time, say goodbye to TEMPLATES

nits, explain when DISTFILES is automatically set