sync icmp_mtudisc_clone() with icmp6_mtudisc_clone(); prompted by bluhm@

Move AF-specific mask logic from callers into set_ipmask()

Instead of doing the same dance with every caller, check for user provided
mask or address familiy specific maximum inside the function itself.

Feedback and OK claudio

synchronize resolver code with smtpd

Implement most prefixlen operations as OP_RANGE (prefixlen A - B).
Simplify the RDE logic this way and make it possible to load such ranges
into a much faster lookup trie for prefix-sets.
When printing the config bgpd tries to use the nices way to express the rule:
e.g.    match from any prefix 18.0.0.0/8 prefixlen 8 - 32
becomes match from any prefix 18.0.0.0/8 or-longer
Apart from that there is no user visible change because of this.
OK sthen@

Use the newer/more sensible names for EVP_MD_CTX_* functions.

 EVP_MD_CTX_create -> EVP_MD_CTX_new
 EVP_MD_CTX_destroy -> EVP_MD_CTX_free

This should make the intent more obvious and reduce head scratching during
code reviews.

Raised by tb@

Correctly clear the current cipher state, when changing cipher state.

When a renegotiation results in a change of cipher suite, the renegotation
would fail if it switched from AEAD to non-AEAD or vice versa. This is due
to the fact that the previous EVP_AEAD or EVP_CIPHER state remained,
resulting in incorrect logic that caused MAC failures.

Rename ssl_clear_cipher_ctx() to ssl_clear_cipher_state() and split it
into separate read/write components, then call these functions from the
appropriate places when a ChangeCipherSpec message is being processed.
Also, remove the separate ssl_clear_hash_ctx() calls and fold these into
the ssl_clear_cipher_{read,write}_state() functions.

Issue reported by Bernard Spil, who also tested this diff.

ok tb@

Back out 1.4, it broke things

Serialize synchronous commands with a rwlock.

Prevent triggering an assert if two drivers try to submit a command
at the same time.

Issue reported by Tom Murphy.

rename flag

ok gilles@

Adjust after change of struct filter_as.
OK phessler@

Change the way as_compare() and aspath_match() handle 'neighbor-as'. Instead
of doing the condition before calling aspath_match() just pass the neighbor-as
down to as_compare() which then has all needed data for the lookup. While
doing this also remove one of the as fields in struct filter_as since the
min/max fields can be reused for unary operations.
OK denis@ phessler@

factor error code in icmp6_mtudisc_clone(); ok kn@ florian@

Add "op" column for CP_SPIN.  Diff from fukaumi at soum.co.jp

ok solene

rename token to match naming convention

ok eric@

Use error label in host_if()

This brings it in line with host() and host_dns().

OK sashan miko

Change verification of communities a bit. Move the flag checking first since
it is currently terminal and then do the length check. If that one triggers
do a treat-as-withdraw but at the same time drop the bad attribute since it
is better to not have invalid attributes in the Adj-RIB-In since most code
does not expect that.
OK benno@

some text improvements from matt schwartz; ok gilles

do not use the literal string "label" as our example label; ok gilles

rework the text about "label" somewhat for clarity; ok eric

Add defines for dealing with PCID support in cr3

ok mlarkin@

use timing-safe compares for checking results in signature verification
(there are no known attacks, this is just inexpensive prudence)

feedback and ok tb@ jsing@

Forgot to rename pf_frent_holes() prototype in previous commit.

Avoid traversing the list of fragment entris to check whether the
pf(4) reassembly is complete.  Instead count the holes that are
created when inserting a fragment.  If there are no holes left, the
fragments are continuous.
idea from claudio@; OK claudio@ sashan@

remove useless session flag

ok gilles@

move readmes detection to find_extractible, so that only changed/new readmes
are recorded.

Be more specific and list those readmes in alphabetic order at end.

document PKGSTEM changes

upon mda failure, smtpd would assume tempfail and retry. this is at odds
with the other MTA which assume a permfail unless the exit status is one
of a specific set. make smtpd honour the same exit statuses as postfix.

note that all errors that occur before the user mda is executed (fork, pipe
and related) are still considered tempfail, only errors coming from the mda
itself are handled as permfail.

this commit is a temporary solution as i believe the SIGCHLD handler is way
more complex than it should be and we'll simplify it after 6.4 is out.

ok eric@

Correct usbd_abort_pipe() signature.

From Artturi Alm.

Introduce inet4applymask() which does the same as inet6applymask() and
can be used instead of doing direct fiddling around with struct in_addr.
Use it in a few cases where it makes the code more similar between INET
and INET6 case.
OK denis@

Instead of bzero() use an assinment to zero struct in_addr.

remove unused flags

kevent: Don't poll for nonzero timeouts < 1us.

Instead of truncating nanosecond timeouts to zero here and polling, we
should round up to a delay of at least a tick, just like all the other
timespec syscalls.

Fixed in NetBSD kern_event.c v1.62 and FreeBSD r247804.

ok millert@ visa@

Remove bits about MODMONO_NANT in the lang/mono section; devel/nant does
not exist anymore.
ok jca@ daniel@

regen

add another toshiba nvme

simplify the smtp_command() function by splitting cmd handlers into
two sets of functions smtp_check_<verb> and smtp_proceed_<verb>. we
use the check functions to validate that verb is acceptable at this
point of a session and proceed to actually move the session forward

ok eric@

Stop using composite EVP_CIPHER AEADs.

The composite AEADs are "stitched" mode ciphers, that are only supported on
some architectures/CPUs and are designed to be faster than a separate
EVP_CIPHER and EVP_MD implementation. The three AEADs are used for less
than ideal cipher suites (if you have hardware support that these use
there are better cipher suite options), plus continuing to support AEADs
via EVP_CIPHER is creating additional code complexity.

ok inoguchi@ tb@

Stop handling AES-GCM via ssl_cipher_get_evp().

All of the AES-GCM ciphersuites use the EVP_AEAD interface, so there is no
need to support them via EVP_CIPHER.

ok inoguchi@ tb@

bump max line size

ok eric@

Clean up SSL_DES and SSL_IDEA remnants.

All ciphersuites that used these encryption algorithms were removed some
time ago.

ok bcook@ inoguchi@ tb@

document "wrapper" in the local section; from matt schwartz
ok gilles

while here, fix some spelling at eol whitespace introduced in previous;

use new smarthost syntax in log message

Update and document syntax for smarthost string in smtpd.conf:

- the +auth specifier is removed: it is implied by the presence of an
  auth label in the rest of the string
- secure:// is removed: use smtp+tls:// or smtps://
- tls:// is replaced by smtp+tls://
- smtp:// becomes SMTP with opportunistic STARTTLS
- smtp+tls:// becomes SMTP with mandatory STARTTLS

Adjust your config file accordingly.

ok gilles@

Allow a large line number to go to the end with goto-line, from Mark
Kelly in GitHub issue 1460.

Fix selection test, from Takeshi Banse.

Remove round_buffersize function from drivers using the default buffersize
provided by the upper layer audio driver.

ok ratchov@

mailaddr matching was working for the exact and domain match, but not when
only a user-part was supplied.

ok millert@ and eric@

Remove a few unnecessary casts

Print SKIPPED if package wycheproof-testvectors is missing.  This
is the magic string that is recognized by my test framework.
OK tb@

Remove ECDH from TODO list. Done!

Unify FAIL printfs.

After libcrypto/ecdh/ech_key.c -r1.8 fixed the failing test cases, remove
two noisy INFO and reorder things a bit.

Elliptic curve arithmetic only makes sense between points that belong to
the same curve. Some Wycheproof tests violate this assumption, making
ECDH_compute_key() compute and return garbage. Check that pub_key lies
on the curve of the private key so that the calculations make sense.
Most paths that get here have this checked (in particular those from
OpenSSH and libssl), but one might get here after using d2i_* or manual
computation.

discussed with & ok jsing;
"good catch!" markus

Run Wycheproof ECDH tests against libcrypto. Some tests currently fail,
will be fixed with the next commit to libcrypto.

Use a Boolean rather than repeated string comparison.

Document how SIGHUP and log rotation works with syslogd(8).  Do not
mention the PID file.  While there, use .Nm consistently.
wording from schwarze@; deraadt@ cheloha@ millert@ agree

update tradcpp to 0.5.2
ok miko@ bcallah@ deraadt@

Update nmea(4) wrt talker ids.

Prodded by semarie@, reworded by jmc@

properly deal with MAILER-DAEMON sender in LMTP

reported and fix tested by Mark Kane

Match on interfaces with Digitizers/Touchscreen collections, as long
as they have an X usage.

Should get basic touchscreen functionality on some common
HID-over-i2c devices like laptops.

For touchscreen devices, use the first HUG_X/Y usage pages found
when looking for logical min/max of screen.

ok kettenis

clarify the forward-only delivery action

Prevent ospf6d from starting when another process is listening on the
control socket.

ok florian@

Accidentally relayd(8) closed file descriptor 0 in the pfe child
process.  If env->sc_snmp is initialized with 0, snmp_init() closes
it.  Set it to -1 to prevent the close(2).
OK reyk@ benno@ millert@

Fix errno for post-lock unveil calls
from Jan Klemkow

Remove 'overlap_allowance' parameter from bootstrap().

It was added to support Vax. Vax is dead Jim.

ok deraadt@

Tweak comment.

sync fatalx() message between smtp_client_{state,response}; ok stsp@ gilles@

Make 'ifconfig nwid' override 'ifconfig join'.

There was no way to override a decision made by join's network
selection algorithm (apart from disabling the feature by clearing
the join list). Automatic selection is based on heuristics which
cannot always guess correctly so we need to provide an override.

One specific problem was that if 'nwid foo' and 'nwid bar' were
both visible in the scan and only 'nwid foo' was a member of the
join list, then there was no way at all to connect to 'nwid bar'.
The wireless stack would keep selecting 'nwid foo' instead.

'ifconfig iwm0 nwid bar' command will now disable automatic
network selection and force the use of ESSID 'bar'.
Any of these commands will re-enable automatic network selection:
   ifconfig iwm0 -nwid
   ifconfig iwm0 nwid ''
   ifconfig iwm0 join some-network-id

ok phessler@ deraadt@

update currency exchange rates;

Recognize more talker IDs when parsing NMEA RMC messages

The NMEA 0183 standard says that the first two chars correspond to the
'source' of the message, right now we were only looking for 'GP' prefix
for 'GPS', but this can also be 'GL' for Glonass, 'BD' for BeiDou, 'GA'
for Galileo, or 'GN' for a generic GNSS source.

Match the RMC messages from all those variants, with this i'm able to
use my navilock nl-8002u (which uses GNRMC) as a timedelta sensor for
ntpd, and i have my GPS position in the nmea(4) sensors.

ok deraadt@

Remove RSA-PSS from todo-list

Run Wycheproof RSASSA-PSS testvectors against libcrypto.

jmc points out that the Nd should match the description in pci(4)

Unsetting Initialized during syslogd die() is a relic from the time
when we had real signal handlers.  But now we use libevent, so
remove the old logic.
OK deraadt@ millert@

the main process must chdir to /, since it cannot have daemon() do the
job at startup.  After much anguish I accept dlg's solution of chdir
for the problem ("starting ntpd on a filesystem I want to unmount"),
but we cannot change the main-process daemon() call.  Why?  Because
the ntpd privsep design predates more modern designs where the config
file is parsed once, and configuration marshalled to the fork+exec
children.  Instead each ntpd process re-parses the config, and if
we chdir before fork+exec startup, it will move the basedir causing
-f "relativepath" to fail.

discussed with florian

Remove unused argument to tls1_change_cipher_state_cipher().

Instead of enumerating the files to clean by hand, set PROGS=${TESTS}.

Suggested by jsing

remove shadow variables for 'error' in usbioctl(); ok mpi@ ratchov@

clarify that patterns are extended regular expressions; ok millert@ phessler@

Dont print "default" for "0.0.0.0" if it is a host route.
diff from asou at soum.co.jp

ok claudio kn

Cadd a TDB `tdb' instead of `sa'.

No functionnal change.

ok deraadt@, bluhm@, visa@

Pass the correct size to free(9) in the error path of db_ctf_decompress().
OK jasper@

switch to improved incoming message parser:
- simpler interface not using callbacks
- no hard-coded line length
- avoid unnecessary string copy

ok gilles@

ich variable is only used in DPRINTF() and uaudio_get_cluster_nchan()
has no side effect, so setting value for ich can be moved under
UAUDIO_DEBUG.

ok ratchov

fix fd leak in an error path
ok mpi@

Move kcov device definitions into <sys/conf.h>.

OK anton@ deraadt@ kettenis@ mpi@

sync

Add more uhid devices, 8 total

ok deraadt

mention bnxt(4)

enable bnxt(4)

Use correct environment variable to access remote machine.

Make sure to clean up the .d files with 'make clean'

Nuke ssl_pending/ssl_shutdown function pointers.

ssl3_pending() is used for all protocols and dtls1_shutdown() just calls
ssl3_shutdown(), so just call the appropriate function directly instead.

ok beck@ inoguchi@ tb@

Move .dynstr before _edata to fix image size calculation.  The PE
header contains an image size field that is calculated using the
difference between the start of the header and edata.  Since we
copy out .dynstr into the EFI binary, make sure that .dynstr is
before edata so that it's included in the image size.  This makes
it consistent with efiboot on armv7.

ok kettenis@

Practice safe free()'ing by setting alloc to NULL to
prevent double free.

Spotted by jsg@, simpler fix by myself and miko@.

ok jsg@

Never let FS_RAID partition be named, a.k.a. given a mount point.

Nuke pointless and inconsistant error message before one get_mp()
invocation.

ok otto@

fix double free in mib_carpgroupget() error path
ok claudio@ miko@

fix memory leak in an error path
ok patrick@

fix use-after-free

spotted by jsg@
input from jsg@ miko@