Perform forward-confirmed reverse DNS verification on incoming connections.
Only flag the session for now.

input from and ok gilles@ sunil@

In the validators, translate obsolete macro aliases (Lp, Ot, LP, P)
to the standard forms (Pp, Ft, PP) up front, such that later code
does not need to look for the obsolete versions.
This reduces the risk of incomplete handling.

adapt to the change in mansearch.c rev. 1.62;
reminded by bluhm@, thanks!

The field dom_rtkeylen has been removed from struct domain.  Convert
to designated initializers and adapt the test.  Fix white space.

Convert more MH_ALIGN() to m_align(). Also switch from m_gethdr/M_GETHDR
calls to m_get/M_GET calls because M_MOVE_PKTHDR() is initialising
the pkthdr and so it is not needed when allocation the header.
OK bluhm@

Send SSL_AD_DECODE alerts in the case of a bad hello request or an
invalid change cipher spec. Found due to dead assignment warnings
by the Clang static analyzer.

ok inoguchi (previous version), jsing

sync

Render .br as <br/>, not as an empty <div>.
The element <br/> was already employed for many other purposes,
so there is nothing wrong with using it.
Also, it is safer because <br/> is permitted in phrasing content,
whereas <div> is only allowed in flow content.

This is the first part of the HTML syntax audit which i wanted
to do for a long time.  Reminded by a loosely related bug report
from Mark Harris <mark dot hsj at gmail dot com>.

Examples of where this caused HTML nesting syntax errors:
* in man(7) code between .nf and .fi
* in mdoc(7) code between .Bd -unfilled and .Ed
* in mdoc(7) code between .Ql Xo and .Xc
* in mdoc(7) code between .Rs and .Re

Do not process the top 32 bits of the 64 bit interrupt mask register
on the O2. The CRIME interrupt controller only has 32 interrupt sources.

From miod@

Remove the unused interrupt type (edge/level) from the mace interrupt
handler registration. The code is inherited from isa(4), but on the O2,
although some interrupt sources on CRIME are edge-triggered, all
the MACE interrupts are level-triggered.

From miod@

In PRU_DISCONNECT don't fall through into PRU_ABORT since the latter frees
the inpcb apart from the disconnect. Just call soisdisconnected() and
clear the inp->inp_faddr since the socket is still valid after a disconnect.
Problem found by syzkaller via Greg Steuck
OK visa@
Fixes:
Reported-by: syzbot+2cd350dfe5c96f6469f2@syzkaller.appspotmail.com
Reported-by: syzbot+139ac2d7d3d60162334b@syzkaller.appspotmail.com
Reported-by: syzbot+02168317bd0156c13b69@syzkaller.appspotmail.com
Reported-by: syzbot+de8d2459ecf4cdc576a1@syzkaller.appspotmail.com

DT_MIPS_RLD_MAP is an offset, so relocate it when present.
This fixes handling of lld-linked executables on mips64.

problem reported by visa@ and worked out with him

Merge post-2.17 but pre-GPLv3 diffs that add support for DT_GNU_HASH
to ld and improve the readelf support:
https://sourceware.org/ml/binutils/2006-07/msg00129.html
https://sourceware.org/ml/binutils/2006-07/msg00181.html

Disable the ld support on mips64, as its ABI requires a symbol
ordering that conflicts with the requirements of DT_GNU_HASH.

Tested on macppc, alpha, and sparc64 to verify operation with all
three ELF word size combos

testing help miod@
ok kettenis@

Avoid uint32_t in _lock.h so users of rwlock.h mutex.h etc don't have to
include sys/types.h first.

ok guenther@ deraadt@

Include "com.h" to fix using a puc(4) based serial port as serial console
which was broken by the previous commit.

ok jsg@, mpi@

Remove artefacts from 'C', 'c' and 'F' characters, to match the other
sizes.

OK patrick@

Add Spleen 8x16, 12x24, 16x32, and 32x64 variants to wsfont.

Together with the already imported Spleen 5x8, it will allow using the
same font family across a wide range of screen resolutions, from small
OLED displays to 4K monitors.

The fonts are 2-Clause BSD licensed and are my original creation.

OK jcs@, brynet@, bcallah@, kettenis@

mention REGRESS_SKIP_TARGETS; ok bluhm@

err -> errx

diff from Lauri Tirkkonen <lotheac@iki.fi>

Use strtonum(3) for -t seconds, improve errors on invalid number input

Base 10 suffices, negative numbers should be invalid (not converted) and
zero not treated specially.

This also unifies error messages and removes unnecessary EINVAL from them
since strtonum()'s errstr is explicit enough already.

Feedback and OK martjin, tb

update currency exchange rates;

regen

Add another Xeon E3-1200 v6/7 host device.
Add nVidia Quadro M1200

from Peter Ezetta, thanks.

Remove erroneous quote added in previous

Spotted by jmc, thanks!

Fix a typo and remove broken example.

From Edgar Pettijohn.

log current power status going into and out of suspend, since it may be
interesting to review.
it seems the battery status is sometimes stale, but that's another bug.
ok kn phessler

on link-connect report events, add an fc-rdns field, currently empty

prepare for smtp-out reporting and while at it, make a few changes to the
report format

/usr/local/ -> ${LOCALBASE}

ok espie@

Update after INDEX change

ok espie@

Document m_align().
With and OK bluhm@, jmc@

MH_ALIGN -> m_align
OK bluhm@

MH_ALIGN -> m_align. In revarprequest() set the ph_rtableid so that
the function is doing the same initialisation as arprequest().
OK bluhm@

MH_ALIGN -> m_align and make the spacing look the same in all three places.
OK bluhm@

Switch MH_ALIGN to m_align which is the same.
OK bluhm@

Trivial MH_ALIGN/M_ALIGN to m_align conversions.
OK bluhm@

in getopts, when a option is followed by a colon the parameter is mandatory

ok guenther@ deraadt@

Clear PANE_EXITED flag when starting new child process in case the pane
has been respawned.

tweak previous;

sync

New midicat utility to send to or receive from MIDI ports.

ok mpi@

Remove midiplay.

ok mpi@

Restore function prototype for ASN1_dup, fixing usage on I32LP64 platforms.
This prototype was removed inadvertantly in r1.50.

OK jsing@

don't attempt to connect to empty SSH_AUTH_SOCK; bz#293

Fix test when using malloc_conf=J
ok otto@

Do not draw horizontal lines through vertical spans
which are requested in the data section rather than in the layout.
Mini-feature found in misc/pfm(1).

Mention dependency on portslist package for outdated-perl-ports(1).
requested by espie@

This test does not terminate if malloc conf is preconfigured with
J.  Clear that option to allow running full regress with paranoid
malloc flags.  This is the same fix as for malloc_ulimit1.

remove double .Pp. oops - sorry for not spotting!

Now that it is better understood how borders work,
rewrite tbl_hrule() in a simpler way.
Fix several bugs in the process.

No more special flags, just use the existing TBL_OPT_* from mandoc.h.
Reduce the number of tracked rows from three to two, which is more logical:
one above the line and one below is sufficient to figure out crossings.
No more magic quirks, all conditions are readily comprehensible now.
Add comments.

Improve introduction so there is more clarity about the patterns of
unveil calls should follow.  ok various, including jmc

Honor SIGINT/QUIT when running in foreground

Allows stopping `syslogd -F' with ^C.

OK bluhm

Fix various overflows in perl

Addresses:
* CVE-2018-18311 (Perl RT #133204)
* CVE-2018-18312 (Perl RT #133423)
* CVE-2018-18313 (Perl RT #133192)
* CVE-2018-18314 (Perl RT #131649)

sync

missed adding tls_default_ca_cert_file here. found by sthen

In peer_down() we have to flush the Adj-RIB-Out. As soon as the peer state
is PEER_DOWN no updates will be processed anymore and quick flush can be made.
Fixes a memory leak and a use-after-free of the peer id in a bgpctl code path.
OK job@

update for libtls default cert changes.
bonus: this exposed a few missing const qualifiers.

expose the default cert file as a function, not a define. it's really
an internal detail of the library, so the string should live inside it,
not in the application code.
ok jsing

introduce FILTER_COMMIT which will allow taking a decision at DATA commit
time, unusable yet but necessary for the upcoming serie of diffs.

ok eric@

community_ext_delete_non_trans() allocates new memory in all cases where
nlen > 0. So in those cases it is important to free() that memory after
use in all cases.
OK phessler@

simplify

ok gilles@

Refactor a tangle in ssl3_send_client_verify() into one function for each
type, sigalgs/rsa/ec/gost.  Move a few special dances for GOST where they
belong now.  This prompted a fix for a long-standing bug with GOST client
certificate authentication where tls1_transcript_data() fails since the
transcript was already freed before.  Add a bit of missing error checking
and leave some further cleanup for later.

idea, guidance & ok jsing

We're manipulating malloc flags ourselves, start with restting them all.
ok bluhm@

Better handle automatic column width assignments in the presence of
horizontal spans, by implementing a moderately difficult iterative
algoritm.  The benefit is that spans containing long text no longer
cause an excessive width of their starting column.

The result is likely not optimal, in particular in the presence
of many spans overlapping in complicated ways nor when spans
interact with equalizing or maximizing colums.  But i doubt the
practical usefulness of making this more complicated.

Issue originally reported in synaptics(4), which now looks better,
by tedu@ three years ago, and reminded by Pali Rohar this summer.

allow configuration of ecn processing

this is a step toward better rfc6040 support

ok claudio@

handle tunnel ecn configuration and reporting.

tested with normal and special ifconfig builds.

ok claudio@

restrict setting ecn to root

ok claudio@

allow configuration of ecn propagation on tunnel interfaces.

ok claudio@

sync

Allow reverse sort order

Prefixing the field with a dash will sort processes in reverse order.
`o -pid' will therefore list PID 1 on top.

"looks good" tedu, improvements and OK cheloha

fuse_parse_cmd_line(3) is called fuse_parse_cmdline(3).

Based on a diff from Edgar Pettijohn III.

Correct lock initialisation for libcrypto.

The current crypto_lock_init() function is not called early enough, meaning
that locks are already in use before it gets called. Worse, locks could be
in use when they are then initialised. Furthermore, since functions like
CRYPTO_lock() are public API, these could be called directly bypassing
initialisation.

Avoid these issues by using static initialisers.

ok bcook@

Bugfix: never set termp->enc to the ambiguous value TERMENC_LOCALE,
but instead set it to TERMENC_UTF8 or TERMENC_ASCII.
Makes tbl(7) box drawing work under -T locale (that is, by default
when LC_CTYPE is defined appropriately).

additional check needed after the previous (box drawing) patch

This test does not terminate if malloc conf is preconfigured with
J.  Clear that option to allow running full regress with paranoid
malloc flags.  Also fix whitespace.
OK otto@

Handle UTF-8 in word-separators option, GitHub issue 1551.

Adjust bgpctl to handle the community changes done in bgpd.
OK job@, phessler@

Start reworking community handling. Merge standard communities and large
communities into one filter_community struct and allow it that more then
one community can be used in filter rules (currently up to 3).
Also rework the code handling bgpctl show rib commands. The special IMSG
types for the various filters are gone and the code is in general simpler.
OK job@, phessler@

Further cleanup of icmp_do_error.
- Use m_align() since it handles all cases
- Use same rounding logic in the size check as in m_align() so all data will
  filt always.
- consolidate pkthdr initalisation into one place
- use m_prepend() instead of direct pointer manipulation (including the panic
  in case an underflow happens).
OK bluhm@

Test for EMSGSIZE return when sending more fds in a message then the receiver
can handle and also test the case where the reciever is using read() instead
of recvmsg() to test the unp_discard() codepath in the kernel. The latter test
is just instrumenting the code but has no way to check if the discarding was
successful.
OK guenther@ bluhm@

Discovered prefixes should not override explicitly configured
prefixes. That way one can have multiple prefixes configured on an
interface and set "autonomous address-configuration no" in rad.conf to
tell clients to not form addresses from some of these prefixes.

Pointed out by Ross L Richardson (openbsd AT rlr.id.au), thanks!
OK phessler

don't truncate user or host name in "user@host's password: " prompts.
requested by Marcel Logen; ok dtucker@

In -T utf8 output mode, render tbl(7) borders with the Unicode
box drawing characters, U+2500 to U+257F.

Originally suggested by bentley@ four years ago,
reminded this summer by Pali Rohar.

Binary and decimal arithmetics are boring,
so let's use some ternary arithmetics for a change.

That said, some other aspects are too complicated for my liking,
so this could use some polishing in the future.

Implement support for DT_GNU_HASH, taking all the interesting bits
from Matt Dillon's implementation in DragonFlyBSD commit 7629c631.
One difference is that as long as DT_HASH is still present, ld.so
will use that to get the total number of symbols rather than walking
the GNU hash chains.  Note that the GPLv2 binutils we have doesn't
support DT_GNU_HASH, so this only helps archs were lld is used.

ok kettenis@ mpi@

Refactor "find the right pool" code into a function. ok djm@ tb@

EVFILT_TIMER: Remove extra tick from tvtohz(9) on timeout reload.

tvtohz(9) adds an extra tick to account for the present tick, but this
tick needs to be removed when the timeout is reloaded thereafter.  We
already do this for periodic setitimer(2) timeouts.

Prompted by Paul Herman's writeup on clock aliasing for DragonflyBSD:
https://frenchfries.net/paul/dfly/nanosleep.html

Also fixed in FreeBSD r238424.

Style tweaks from visa.

ok visa@, guenther@

Free sizes for the subdevs array.

ok anton@, visa@

Attach the driver only if the stack managed to set the configuration.

Tested by stsp@

Sync with ldap(1)

Sync with ldap(1)

Sync aldap and ber with ldap(1).

OK claudio@

LDAP returns values as OCTET STRINGs, which can be LDAPStrings, but isn't
always the case. This caused some malformed output when querying databases
returning data containing NUL-bytes. Fix this by also returning the
actual size.

With this diff we should produce basically identical output to openldap's
ldapsearch.

Lots of back and forths with claudio@

OK claudio@

handle -s and -i in a uniform fashion, do not try to change the node type
but use the Targ_* functions correctly.

This fixes an oddity in end node handling noticed by kn@

review and okay kn@

Implement tbl(7) lines in -T html output,
as far as they are on the edges of table cells
rather than going through the middle of cells:

* the box, doublebox, and allbox options;
* the | and || layout modifiers;
* and the _ and = data lines;
- but not yet _ and = in individual layout and data cells.

Missing feature reported by Pali dot Rohar at gmail dot com.

When a conditional block is closed by putting "\}" on a text line
by itself (which is somewhat unusual but not invalid; most authors
use the empty macro line ".\}" instead), agree more closely with
groff and do not produce a double space in the output.

Quirk reported by millert@.

While here, tweak the rest of the function body of roff_cond_text()
to more closely match roff_cond_sub().  The subtly different handling
could make people (including myself) wonder whether there is any
point in being different.  Testing shows there is not.

Place mandoc.css into the public domain.

The reason for doing this rather than using the ISC license
is that i guess that in some contexts, a requirement to preserve
a Copyright and license header might be inconvenient, and i really
don't care at all how people use it.

What matters is that they do use it, or something similar - attempts
to use mandoc without any CSS are a constant source of grief and
bogus bug reports because HTML without CSS doesn't look very good:
the more structural and semantic and the less presentational and
old-fashioned the HTML, the more so.

Thanks to Mark Harris <mark dot hsj at gmail dot com> for pointing out
that the permissions on this particular file were unclear.

Move the {qcow2,raw} create functions from vmctl into vmd/vio{qcow2,raw}.c

This way they are in the appropriate place and code can be shared with vmd.

Ok ori@ mlarkin@ ccardenas@

Keep a list of known vms, and reuse the VM IDs.

This means that when using '-L', the IP addresses of the VMs are stable.

ok reyk@

Simplify writing of tbl(7) cells by using the new feature of passing
a NULL pointer for the value of a style attribute, in which case
the attribute is omitted from the HTML element.
Minus 12 lines of ugly and repetitive code, no functional change.

Support more than one style attribute one the same HTML element.
In fact, this is already required when a table uses non-default
horizontal and vertical alignment in the same cell.