The _exit codes in syslogd privsep.c were the wrong way around.
Fatal errors should exit with 1, and regular shutdown should result
in exit with 0.
OK deraadt@

Replace manually written function names with __func__.
OK sthen@

Fix trailing white spaces.
OK sthen@

Make the perl modules consistent for the multiple regression tests.
This includes coding style, better error messages and variable
naming.

Fix read access to uninitialized memory found by kristaps@ with valgrind,
patch from kristaps@.
Theoretically, this could lead to a buffer overrun and segfault,
but only for very long output lines (about 1000 charecters) of
exactly the right length or if by ill chance, the complete unused,
uninitialized tail of the line output buffer (at least 1000 characters
total length) would contain all blank characters.  Anyway, the
uninitialized data read wasn't used for anything.

Add a description of what the relayd and sosplice regression tests
do.  Also explain the optional environment variables.

Make out of band data tests more reliable.  As multiple tcp urgent
bytes cannot be handled correctly, allow another variation.

Make the forking tests more reliable.  During copy the read(2)
system call might fail with EAGAIN when the other forked process
has already read the data.

Make the multiple UDP packet test more reliable.  The receive buffer
limit also counts the mbufs in it, so packets could be dropped.
Increase both send and receive socket buffer size.

Add test cases for multiple small HTTP requests.  Many PUT requests
with ony byte content length triggered relayd to hang.  This has
been fixed already.

Variables to count seen packets should be unsigned.

ok krw@

sync

replace more ROTATE macros with plain-old C code.

Let the compiler optimize these. Even older versions of gcc generate
equal or better quality code than the inline asm.

ok miod@

remove return value from HOST_c2l/l2c macros

These macros and asm inlines simulate a function returning a value, but
nothing ever uses this return value. Remove the pseudo-returns and
(void) casts discarding the unused values.

This, maybe unsurprisingly, speeds things up a bit. It also removes the
GCC 4.9 warnings about unused values.

ok miod@ deraadt@

Explicitely need <sys/pool.h> now.

Sigh, ignoring instruction fetch bus errors for the kernel code should not
depend upon the address being at the beginning of a cache line, for we may
arrive in the middle of a line thanks to a branch. Noticed the hard way...

When the first child of the node being validated gets deleted during
validation, man_node_unlink() switches to MAN_NEXT_CHILD.  After
that, we have to switch back to MAN_NEXT_SIBLING after completing
validation, or subsequent parsing would add content into an already
closed node, clobbering potentially existing children, causing
information loss and a memory leak.  Bug found by kristaps@ with
valgrind in groff(7) on Mac OS X.

Note that the switch back must be conditional, for if the node being
validated itself gets deleted, we must *not* go to MAN_NEXT_SIBLING,
which would not only yield wrong results in general but also crash
in malformed manuals having an empty paragraph before the first .SH,
for example OpenBSD c++filt(1).

While here, add the missing <sys/types.h> as required before mandoc.h.

pool debug back on

a small tweak to improve malloc in multithreaded programs. we don't need
to hold the malloc lock across mmap syscalls in all cases. dropping it
allows another thread to access the existing chunk cache if necessary.
could be improved to be a bit more aggressive, but i've been testing this
simple diff for some time now with good results.

this uses pools, but relied on mbuf.h to provide them.

found by benoit@

kristaps@ found this with valgrind, merge his patch from bsd.lv:
Fix a corner case where \H<nil> (where <nil> is the \0 character) would
cause mandoc_escape() to read past the end of an allocated string.
Found when a script scanning of all Mac OSX manuals accidentally also
scanned binary (gzip'd) files, discussed with schwarze@ on tech@mdocml.

Sync proc.c with httpd.  httpd needs SIGUSR1 but snmpd will ignore it
now instead of terminating the process.

ok blambert@

Sync proc.c with httpd.  httpd needs SIGUSR1 but relayd will ignore it
now instead of terminating the process.

ok florian@

Sync proc.c with httpd.  httpd needs SIGUSR1 but iked will ignore it
now instead of terminating the process.

ok mikeb@

dont rely on mbuf.h to provide pool.h.

ok miod@, who has offerred to help with any MD fallout
ok guenther@

Add mention of execvpe() setting errno on failure.

from Justus Baumgartner (meddev50 (at) gmail.com)

its a bit silly to include mbuf.h to get to malloc.h. just include
malloc.h instead.

introduce mbuf list and queue apis. both manage fifo lists of mbufs
and a count of the mbufs.

struct mbuf_list and the ml_foo() apis can be used to build lists of
mbufs where you dont need locking (eg, on the stack).

struct mbuf_queue and mq_foo() wrap mbuf_lists with a mutex, and
limits the number of mbufs that can be queued. they can be useful
for moving mbufs between contexts/subsystems.

with help from jmc@ for the manpage bits
mpi@ is keen

Add fancy printing of ktrace()'s ops argument
mquery() has the exact same argument layout as mmap(), so share the case
Fix a couple brace placement glitches

external page headers use an RB tree to find the page header
containing an item when its returned to the pool. this means you
need to do an inexact comparison between an items address and the
page address, cos a pool page can contain many items.

previously this used RB_FIND with a compare function that would do math
on every node comparison to see if one node (the key) was within the other
node (the tree element).

this cuts it over to using RB_NFIND to find the closest tree node
instead of the exact tree node. the node compares turns into simple
< and > operations, which inline very nicely with the RB_NFIND. the
constraint (an item must be within a page) is then checked only
once after the NFIND call.

feedback from matthew@ and tedu@

Use %#o instead of %#x for mode_t

Display symbolicly the mode argument of mkdir, mkfifo, mknod, and umask

typo, sorry

While all current callers pass valid data to ascii_hspan() only,
it's safer to assume incoming enum data might be invalid
and catch it instead of happily returning an unitialized int.
No functional change right now.

typo; Steven Honeyman <stevenhoneyman at gmail dot com>

Sync regress/usr.sbin/relayd with recent http.h changes

don;t mark up {};

Segfault fixes from kristaps@.
Note that .It and .Nm blocks without bodies cannot be generated by
valid mdoc(7) syntax but are a adequate representations of invalid
mdoc(7) constructs like Bl -hang It Bo Sh and Sh SYNOPSIS Nm Bo Sh.

Execute /etc/netstart using sh(1) instead of sourcing it.
Committing early to make sure we have time to fix any side-effect.

ok deraadt@

When running interactively, display the output of the "check" action
(ok or failed) like we do with all other actions.

ok jung@ rpe@

Fix regression tests for ld.so

It now runs correctly with 'make depend regress'

On kernels compiled with R10000 support, ignore (by simply returning)
`bus error upon instruction fetch' exceptions where the faulting address is
in the kernel, and at the very beginning of an I$ cache line.
(I've experienced these on an R16000 Fuel since several months already)

i broke the userland shim used for the extent regress test when i
made it so struct pool was only visible to _KERNEL. tedu broke it
too when he added the size argument to the kernel free
functions.

this fixes both issues. the main change is to provide a local version of
struct pool with just the bit (pr_size) needed for extent to run.

if extents take advantage of more malloc/pool features (eg, {M,PR}_ZERO
then this will need to be updated again.

found by and based on a diff from Theo Buehler
ok mpi@

sync with hackathons.html

update the built-ins list:
- "times" is both promoted to posix and special (ooh!)
- "pwd" is promoted to posix, but not special (aah!)

while here, jiggle the text somewhat to clarify that "non-POSIX"
actually meant when posix mode is off, not mandated by posix

joint work with guenther

repair operation of kern.arandom, which will only allow a buffer of
512 bytes.  As a result, it stopped working...
ok miod

Cover execvpe() in the description of what the envp arg means.

from Justus Baumgartner (meddev50 (at) gmail.com)
ok jmc@

If a stray .It follows .El, we are no longer in the list,
even though the list is still the last processed macro.
This fixes a regression introduced in mdoc_macro.c rev. 1.95:
Ulrich Spoerlein <uqs at FreeBSD> reports that various of their
kernel manuals trigger assertions.

replace sprintf/strdup with asprintf in engine test

only build the getrandom path if SYS_getrandom is defined.

like the sysctl path

Fixed typo in "distinct".

ok jmc@

use C rather than C++ array initialization syntax

this causes errors with stricter C compilers

getrandom(2) support for getentropy_linux

This enables support for the new getrandom(2) syscall in Linux 3.17.

If the call exists and fails, return a failure in getentropy(2) emulation as
well. This adds a EINTR check in case the urandom pool is not initialized.

Tested on Fedora Rawhide with 3.17rc0 and Ubuntu 14.04

ok deraadt@

reduce cutoff for "hours only" start times to reduce window of ambiguity.
ok deraadt

I found a number of interactive events which can cause signals, and go
down paths not previously marked as signal handled unsafe.  Try to clean
up a few of them especially regarding errno, mark others as unsafe, and
repair a few by avoiding stdio.  Glanced at by misc people in Slovenia,
but considered too risky before release..

sync

use void * instead of char *. ok jsing

Update to tzdata2014f from ftp.iana.org.

Public key operations are no longer supported

Remove support for public key operations

Remove support for public key operations

Convert 3DES regress test from /dev/crypto to using kernel source code
directly with a simplified CBC implementation.

Convert AES-CTR regress test from /dev/crypto to using kernel source
code directly.  This test is converted the same way jsing@ has recently
converted an XTS test by pulling in xform.c code.

Convert AES regress test from /dev/crypto to using kernel source code
directly.  This test case uses ECB vectors, therefore no chaining is
required.

lofn(4) and nofn(4) are gone

With deprecation of crypto(4) interface lofn(4) and nofn(4) become
obsolete.  No objections from the usual suspects.

unlink lofn(4) and nofn(4)

lofn(4) and nofn(4) are going away

Align install.md files with installer style.

- { foo ; bar ; } -> { foo; bar; }
- if foo ; then -> if foo; then

OK halex@

shorten share/doc/{html,mg} to share/doc, since nothing currently
gets installed in html (i think!);

admission of hawaii happened in 1959, not 1984;

from the freebsd bugs database, bug 192651:
Reported: 2014-08-14 13:17 UTC by Richard

XPG requires insque() and remque() to work with linear lists and not just
circular lists.  Amazingly, they managed to extend the requirements to no
longer match the behavior of the VAX instructions they were modeled after,
so the trivial VAX ASM versions have to go.  Nice job breaking it, X/Open!

Based on a diff from enh (at) google.com
ok miod@

Use O_CLOEXEC wherever we open a file and then call fcntl(F_SETFD, FD_CLOEXEC)
on it, simplifying error checking, reducing system calls, and improving
thread-safety for libraries.

ok miod@

Create a function which loads sgd in the mfi_iop_ops struct so that skinny
adapters can use "IEEE sgl".

tested dlg yasuoka
ok dlg jsg

Some compilers apparently worry that abort() might return
and then throw a "may be used uninitialized" warning, so
sprinkle some /* NOTREACHED */.  No functional change.
Noticed by Thomas Klausner <wiz at NetBSD dot org>.

Clarify that .Em and .Sy are physical, not semantic markup,
explain appropriate usage, and provide some examples.
ok jmc@

regen

Add USB hubs found on Intel Z97 chipset boards (one attached to
each EHCI root hub).  OK deraadt@ jsg@

double word fix-o;

Backport from binutils-2.17:
2005-08-17  Alan Modra  <amodra@bigpond.net.au>
* elfxx-mips.c (_bfd_mips_elf_hide_symbol): Cope with being called
without any got section.

ok miod@

remove two duplicate command entries;
reported by Denis Fondras <openbsd at ledeuns dot net>

fixed overrid(d)en typo

millert@ and jmc@ agree that "overriden" is wrong

Unused variable, and more fixes in the attach failure path.

Bring back 1.173 (reverting 1.174) - mips64 systems are still unhappy when
the hint returned is over VM_MAXUSER_ADDRESS, apparently; better be safe for
now while this is investigated further.

No need for <netinet/in_systm.h> nor <netinet/tcpip.h>.

No use for <netinet/in_systm.h> nor <netinet/tcpip.h>.

Kill bandwidth monitoring meters.

Add bounce matching for [] and {} like mg already does with () and like
Emacs does.
ok florian@

Compare number of allocated clusters with a low watermark, not
a magic number 4 since sometimes we can't fit a single packet
(jumbo frame) into 4 clusters.

OK dlg

Sync with httpd, including the following change from doug@:
"Sync with RFC 7230-7235 phrases and IANA registered status codes.
ok reyk@"

Implement rxrinfo ioctl for cluster usage statistics

Sync with RFC 7230-7235 phrases and IANA registered status codes.

ok reyk@

No need for raw_cb.h

Kill MRT_{ADD,DEL}_BW_UPCALL interfaces and the bandwidth monitoring
code that comes with them.

ok mikeb@, henning@

Show only active pools by default, pressing 'A' shows all of them.

ok deraadt@

Do not set RTF_MASK, it is not used anymore.

ok yasuoka@

fix logging strings (correct function name via __func__ + a typo)

ok florian@ henning@

Remove obsolete struct stat parameters.

ok reyk@

add some definitions from libdrm 2.4.56

fix date that got changed by Mdocdate
and no longer matches the desired output