Fix the length check for reinjected ICMP packets: sizeof(struct icmp) is
28 but an ICMP packet can be as small as 8 bytes (e.g. an ICMP echo
request packet with no payload), so check against ICMP_MINLEN instead.

Prior to this fix, divert(4) would erroneously discard valid ICMP
packets that are shorter than 20 bytes.

ICMPv6 is not affected, so this change applies to ICMP over IPv4 only.

ok florian@ henning@

Format time_t with %lld after casting to long long

Rename p_hdrlen to min_hdrlen to better reflect its purpose.

No object file change
ok florian@ henning@

Delete duplicate conditional

diff from frantisek holop (minusf (at) obiit.org)

Replace F_DUPFD followed by setting FD_CLOEXEC with just F_DUPFD_CLOEXEC

ok matthew@ millert@

Add Xr to skey and stty

Delete extern declarations for variables that don't exist (anymore?)

Only need <stdint.h> and not all of <inttypes.h> here

msts(4) appears to support TIOCSTSTAMP

From Maurice Janssen (maurice (at) z74.net)

Add F_DUPFD_CLOEXEC handling

Add newer errnos

Switch from calloc() to reallocarray() where the zeroing isn't needed

Use time_t for storing times

Parse times with strtoll() instead of strtol()

Regenerate to pick up time_t changes

bfd-in2.h is generated from this; merge back a change that was made there

Store timestamps in time_t

bfd-in2.h is generated from this; merge back a change that was made there

After a cleanup by deraadt I noticed the four-space-indent had started
infesting the scripts. As we generally use a single tab for line
continuation indent in the scripts, let's reclaim 30 precious bytes!

ok rpe@ krw@

Fix readelf --debug-dump=frames-interp output: once a register is given
a column, it needs to included in all rows, even after the register is
restored.

ok jsg@

less chatty; ok ajacoutot

mmap(2) requires MAP_PRIVATE ^ MAP_SHARED for flags;
found by kristaps@ on Mac OS X

Extend comment.

Initial support for handling configuration files from packages
(aka @sample) -- `sysmerge -p`
Worked done at g2k14.
Committing early so we have plenty of time for testing.

with help from espie@
thanks to landry@ for feedback on initial testing

sysctl.conf is an example.
Tweak error output.
Mention that `-S' has no effect when sets aren't used.

Add support for non-root hubs now that uhub(4) can deal with them.  For
the moment only Super and High Speed devices are properly recognized.

Some TT love is required for Full and Low speed devices.

Correctly recognize Super Speed devices, this is part of the work to
be able to use USB 3.0 devices behind an external hub.

This is a bit tricky because the SS status use a different power bit
that maps to the Low speed one.  So no longer accept devices without
power bit and fallback to the parent hub's speed in case the status
does not report any particular speed.

Note that xhci(4) root hubs still set the traditionnal UPS_PORT_POWER
bit with the correct device speed.

Handle super speed hub descriptors.

Do not store the whole USB hub descriptor in the "struct usbd_hub"
to help integrating super speed hubs that use a different descriptor.

some minor tweaks;

sort "prefork", and remove a useless macro;

basic cleanup;

Allow to inclue the types section anywhere in the configuration file.

Found by chris@
OK doug@

Remove support for the continuous reporting "any" mouse mode which never
really worked properly and is rarely used.

Backport from 2.17: mark the _GLOBAL_OFFSET_TABLE_, _DYNAMIC, and
_PROCEDURE_LINKAGE_TABLE_ symbols as (at least) hidden.

Backport from binutils-2.17 the correct i386/amd64 register->int assignments
for CFI.  This changes the unwind information generated on amd64.

We don't use the __openbsd_randomdata_{start,end} symbols, but rather
get that info via the PT_OPENBSD_RANDOM segment info, so kill the symbols
While here, delete the commented out lines for __DYNAMIC as the question
they ask is answered 'no'

ok matthew@ back in May

Match format width of symbol visibility to the column header
This corrects the alignment of the columns of hidden symbols

Only need <stdint.h> and not <inttypes.h> here

update currency exchange rates;

some basic fixes;

zap trailing whitespace;

When opening directories, re-match the location after the index file
has been appended.  This allows to use a fastcgi target as the default
index, for example index.php.

OK florian@

No events were added for DNS UDP so it stopped working after the first
request.  Additionally, the DNS code tried to use an invalid timeout.

Fix from mm@freebsd.org
Reported by Johan Schuijt

Absurdly, the return value of sqlite3_column_text()
is "const unsigned char *", which causes warnings with GCC on Linux.
Explicitly cast to "const char *" to avoid this.
Issue noticed by kristaps@.

mansearch.h uses uint64_t, so it needs stdint.h; found on Linux

Drop some useless verbiage.
sysctl.conf is an example now.

drm/radeon: fix typo in radeon_connector_is_dp12_capable()

From Alex Deucher
94dfc49785ea1acc1dd2c086ffd8d61ea3a5ee8f in ubuntu 3.8
af5d36539dfe043f1cf0f8b7334d6bb12cd14e75 in mainline linux

Tweak verbiage.

drm/radeon/dp: fix lane/clock setup for dp 1.2 capable devices

From Alex Deucher
c9a1adc31f78a30f33c591b61171f02d13a5b1a7 in ubuntu 3.8
3b6d9fd23e015b5397c438fd3cd74147d2c805b6 in mainline linux

drm/radeon/atom: fix dithering on certain panels

From Alex Deucher
9102ef0d290f01247918f5a519d8fa4a96eaf370 in ubuntu 3.8
642528355c694f5ed68f6bff9ff520326a249f99 in mainline linux

drm/radeon: only apply hdmi bpc pll flags when encoder mode is hdmi

From Alex Deucher
85cdd5e933c0f9fe3262067e707eed565db46378 in ubuntu 3.8
7d5ab3009a8ca777174f6f469277b3922d56fd4b in mainline linux

some corrections and improvements with respect to prologue macros;
found while working on mandoc(1) messages

Remove redundant verbiage in the style of "a file is a file,
a line is a line, a column is a column"; suggested by jmc@

Bring the handling of defective prologues even closer to groff,
in particular relaxing the distinction between prologue and body
and further improving messages.
* The last .Dd wins and the last .Os wins, even in the body.
* The last .Dt before the first body macro wins.
* Missing title in .Dt defaults to UNTITLED.  Warn about it.
* Missing section in .Dt does not default to 1.  But warn about it.
* Do not warn multiple times about the same mdoc(7) prologue macro.
* Warn about missing .Os.
* Incomplete .TH defaults to empty strings.  Warn about it.

Simplify by allowing only one post-handler.
Saves 36 static arrays and 10 lines of code
at the expense of only five new trivial static functions.
No functional change.

Simplify by allowing only one pre-handler.
Saves 12 static arrays and 19 lines of code.
No functional change.

Simplify man(7) validation:
Drop pre-handlers, they were almost unused.
Drop the needless complexity of allowing more than one post-handler.

This saves one internal interface function, one static function, one
private struct definition, sixteen static arrays, and 45 lines of code.
No functional change.

regen

Fix floating point handling: When converting double to size_t,
properly round to the nearest M (=0.001m), which is the smallest
available unit.

This avoids weirdness like (size_t)(0.6 * 10.0) == 5
by instead calculating (size_t)(0.6 * 10.0 + 0.0005) == 6,
and so it fixes the indentation of the readline(3) manual.

add bochs and microsoft emulated vga

Clarity with respect to floating point handling:
Write double constants as double rather than integer literals.
Remove useless explicit (double) cast done at one place and nowhere else.
No functional change.

demacrify: get rid of man_nmsg(), man_pmsg(), mdoc_nmsg(), mdoc_pmsg()

mention requests and macros in more messages

Dynamically allocate the stack of roff(7) .ie condition values
and thus get rid of the last useless fatal error.

Allow to serve emtpy (0 bytes) files.

Found by jasper@
OK florian@

Split MANDOCERR_IGNARGV into one message for .An and one for .Bl
and report the macro name and argument.

In .Bl -column, if some of the column width declarations are given
right after the -column argument and some at the very end of the
argument list, after some other arguments like -compact, concatenate
the column lists.
This gets rid of one of the last useless FATAL errors
and actually shortens the code by a few lines.

This fixes an issue introduced more than five years ago, at first
causing an assert() since bsd.lv mdoc_action.c rev. 1.14 (June 17, 2009),
then later a FATAL error since mdoc_validate rev. 1.130 (Nov. 30, 2010),
and marked as "TODO" ever since.

Remove the useless FATAL error "argument count wrong, violates syntax".
The last remaining instance was .It in .Bl -column with more than one
excessive .Ta.  However, simply downgrading from FATAL to ERROR, it just
works fine, almost the same way as in groff, without any other changes.

Improve handling of next-line scope broken by end of file.
Detect the condition earlier, report in the error message
which block is broken, and delete the broken block.
Consequently, empty section headers can no longer happen.

Simplify: replace one global flag by one local variable
and remove three unused global flags.  No functional change.

Get rid of the useless FATAL error "child violates parent syntax".
When finding items outside lists, simply skip them and throw an ERROR.
Handle subsections before the first section instead of bailing out.

Remove two useless FATAL errors.
When a file contains neither text nor macros, treat it as an empty document.
When the mdoc(7) document prologue is incomplete, use some default values.

use nitems() instead of various local constructs

ok stsp@

better name and wording for the last two non-generic errors

Various improvements related to .Ex and .Rv:
* let .Nm fall back to the empty string, not to UNKNOWN
* never let .Rv copy an argument from .Nm
* avoid spurious \fR after empty .Nm in -Tman
* correct handling of .Ex and .Rv in -Tman
* correct the wording of the output for .Rv without arguments
* use non-breaking spaces in .Ex and .Rv output where required
* split MANDOCERR_NONAME into a warning for .Ex and an error for .Nm

Partial implementation of .Bd -centered.

In groff, .Bd -centered operates in fill mode, which is relatively
hard to implement, while this implementation operates in non-fill
mode so far.  As long as you pay attention that your lines do not
overflow, it works.  To make sure that rendering is the same for
mandoc and groff, it is recommended to insert .br between lines
for now.  This implementation will need improvement later.

* consistently use semantic mdoc(7) macros
* add HISTORY and AUTHORS
* some mdoc(7) style fixes
* some wording fixes and avoid stray future tense
feedback and ok rpe@ jmc@, ok jasper@

Make sure asynchronous commands do not race with synchronous ones.

Since asynchronous commands can be submitted from interrupt context
it was possible to race with a process waiting for the completion of
a previously submitted command.  So stop relying on the per-softc
TRB pointer for asynchronous commands and simply get the address of
the command TRB from the event TRB.

Improve the logic to determine the maximum endpoint service interface
time payload.  Super speed companion descriptor are still not used but
at least we can properly initialize super speed interrupt pipes.

Implement polling.

Super Speed hub descriptor definition and routine, required for upcoming
external USB 3.0 hub support.

Even if the endpoint it reseted before the stack gets informed that a
transfer stalled, report that a stall happen because umass(4) relies
on this behavior...

Fix debug printfs.

Add support for using - as shorthand for stdin/stdout in tradcpp.

When looking into switching the /usr/bin/cpp wrapper to
tradcpp I came across "| ${CPP} ${CPPFLAGS} -" in usr.bin/which.

gcc documents this behaviour for cpp here:
https://gcc.gnu.org/onlinedocs/cpp/Invocation.html

Versions of the John F. Reiser derived cpp in 32V and CSRG SCCS
accept this usage as well, as does the cpp in PCC.

Fix CVE-2014-3507, avoid allocating and then leaking a fresh fragment
structure when a zero-length fragment is received.

Based on
https://git.openssl.org/gitweb/?p=openssl.git;a=commitdiff;h=d0a4b7d1a2948fce38515b8d862f43e7ba0ebf74

diff by miod@, ok guenther@ bcook@ deraadt@

Fix CVE-2014-3508, pretty printing and OID validation:
 - make sure the output buffer is always NUL terminated if buf_len
   was initially greater than zero.
 - reject OIDs that are too long, too short, or not in proper base-127

Based on
https://git.openssl.org/gitweb/?p=openssl.git;a=commit;h=0042fb5fd1c9d257d713b15a1f45da05cf5c1c87

ok bcook@

Correct test reversed during merge of fix for CVE-2014-3509

pointed out by Watson Ladd (watson (at) matasano.com)
ok deraadt@

Fix CVE-2014-3506, DTLS handshake message size checks.  From
https://git.openssl.org/gitweb/?p=openssl.git;a=commit;h=1250f12613b61758675848f6600ebd914ccd7636
with comment/whitespace style tweaks

ok bcook@ miod@

Oops, revert changes commited by mistake. The previous commit was supposed
to only apply to s23_srvr.c.

When you expect a function to return a particular value, don't put a comment
saying that you expect it to return that value and compare it against zero
because it is supposedly faster, for this leads to bugs (especially given the
high rate of sloppy cut'n'paste within ssl3 and dtls1 routines in this
library).

Instead, compare for the exact value it ought to return upon success.

ok deraadt@

Bump example pubkey filenames to /etc/signify/openbsd-56-base.pub for
5.6.

ok deraadt@

Fix and simplify the description of httpd(8)'s signal handling.  httpd
does not re-executed itself on SIGHUP, it simply reload the
configuration and sends it to its child processes.

ok deraadt@

Don't try to ouput FCGI_STDERR into error.log if there is no data.
Problem noticed by naddy@, OK reyk@

Opportunistically try to parse "Status: $code" in the very first
response from the fcgi daemon and use that code as HTTP response
code. If it doesn't work out fall back to code 200.
This might fix naddy@'s issue with redirects in cvsweb.
To be revisited after unlock.
Discussed with & grudgingly OK reyk@

shorten signal text a bit

Fix CVE-2014-3511; TLS downgrade, verbatim diff
https://git.openssl.org/gitweb/?p=openssl.git;a=commit;h=280b1f1ad12131defcd986676a8fc9717aaa601b
ok guenther miod

merge CVE-2014-3510; Fix DTLS anonymous EC(DH) denial of service
https://git.openssl.org/gitweb/?p=openssl.git;a=commit;h=17160033765480453be0a41335fa6b833691c049
ok bcook

merge fix for CVE-2014-3509 -- basically a missing s->hit check; ok guenther

Mention how httpd responds to SIGHUP and SIGUSR1.

Description from reyk@