Do not let the -m option or MANPATH with leading, trailing, or double
colon override the default manpath, let them add to the default manpath.
Only override the default manpath by the -M option, by MANPATH without
leading, trailing, or double colon, or by "manpath" in man.conf(5).

Problem reported by Jan Stary <hans at stare dot cz>.
Patch OK'ed by millert@.

Style spacing nits.

Include the timestamp TCP option in keep alive packets as well.

According to RFC 7323 "once TSopt has been successfully negotiated,
... [it] MUST be sent in every non-<RST> segment for the duration
of the connection."  Which means that keep alives which are just
ACK packets must include that too.

Pointed out and tested by Lauri Tirkkonen <lotheac at iki ! fi>, thanks!
ok mpi

Pass a thread pointer instead of its file descriptor table to getvnode(9).

Input and ok millert@

array.h can be local to window-choose.c now.

Use a TAILQ not array for find-window.

Simplify environ_push so it doesn't need the ARRAY_* functions.

If status line is at the top, the offset needs to be adjusted when
drawing pane numbers.  Based on a diff from John O'Meara.

remove badly indented printf and it's preceeding test
as suggested by miod@

fix indentation
ok miod@

fix indentation of a line
ok krw@ miod@

add missing braces to code under #ifdef TRM_DEBUG0
ok krw@

remove a surplus tab
from miod@

add missing braces
ok miod@

Remove ARRAY_* from history and expand completion to complete a) layout
names and b) targets beginning with -t or -s.

fix a fd leak in an error path in code under #ifdef DEBUG

regen

#include <sys/event.h> was removed from init_sysent.c but not
syscalls.master where the file is generated from.  Fix this by removing
it from syscalls.master so it doesn't come back.

From Nicolas Bedos.

Pass a thread pointer instead of its file descriptor table to getsock(9).

Diff from Vitaliy Makkoveev.

Manpage tweak and ok millert@

use res_hnok() to valid domain part in valid_domain()

ok eric@

Add a format window_linked which is 1 if a window has been linked
multiple times, also remove the default space in window_flags and use a
conditional to add it in window-status-format (this means additional
flags can be added in the option without extra spaces). From Thomas Adam
with tweaks by me.

Document that priority 1 is reserved for kernel use.

Display thread IDs instead of the name of the process's owner when "-H"
is used.

The rationnal is that when you're looking at threads you're generally
already filtereing by PID and this allow you to see which thread is a
pig.

Written some time ago with mikeb@

ok sthen@, krw@, guenther@

Turn cursor off during redraw, pointed out by George Nachman.

Clarify pseudo-terminal request behaviour and use "pseudo-terminal"
consistently.  bz#1716, ok jmc@ "I like it" deraadt@.

Blacklist DH-GEX for specific PuTTY versions known to send non-RFC4419
DH-GEX messages rather than all versions of PuTTY.  According to Simon
Tatham, 0.65 and newer versions will send RFC4419 DH-GEX messages.  ok djm@

make SWAP_KEY_GET symmetrical with SWAP_KEY_PUT by having it call
a function in uvm_swap_encrypt.c instead of doing stuff inline.

lets me remove an extern from the header too.

ok kettenis@

put mpi's atomics back in, but with the return value of add (and therefore sub, inc, and dec) fixed.

the asm read the value from memory into a register, added to it,
and then tried to write it back. after succeeding it doesnt have
to add again before returning.

this splits sub, inc, and dec off from add. sub can use the subf
opcode, and inc and dec can use the addic opcode. explicitely
identify where the modified memory is so we can avoid using "memory"
as a clobber.

ok mpi@

Add missing braces so this will correctly fall back to the "name"
property of a node if "device_type" is not present.

ok mpi@ miod@

add missing braces
ok guenther@ miod@

Use | when combining flags not +.  Luckily in this case 1 + 2 == 1 | 2.

ok mlarkin@

Fix a memory leak in _dl_load_dep_libs()
ok otto@ guenther@

Eliminate rabid semicolon

no need to swizzle load/store addresses for 32bit values on big endian systems

ok miod@

Updated fifotest.out

fifofs now respects the flags a descriptor was opened with.
Also add tests for when no FD events are specified.
A few tests still don't pass which will be addressed shortly.

Now that we have a_fflag in struct vop_poll_args we can handle
things like POLLOUT on a read-only fd more sensibly.  Previously,
any poll events could be used regardless of the actual file flags
of the descriptor.  The new fifo_poll() is effectively a pared down
soo_poll() with slightly different semantics to batter match FIFOs.
OK and feedback from guenther@

AI_ADDRCONFIG: skip loopback addresses, not loopback interfaces.

This is what RFC3493 suggests.  Fixes AI_ADDRCONFIG on setups where
global addresses are configured only on loopback interfaces.

Discussed with and ok eric@ gilles@

Move the AI_ADDRCONFIG setup to its own function.

Input from and ok gilles@ eric@

Remove man page links for crypto(9) key operations; reminded by jmc@

Less magic number

For PS/2 - status byte - bit 5:
The bit is '1' if the data on input port 60h came from the AUX port, or '0' if
the data came from the keyboard or from the keyboard controller itself.

KBS_TERR (kbd transmission error) was valid for AT-mode, but not for PS/2-mode.

ok ratchov@, miod@

Make argument name consistent and use .Fn rather than .Nm in section 3.
Patch from <Kaspars at Bankovskis dot net>.

add missing braces in _aucat_wmsg()

As ratchov@ notes:
"all _aucat_wmsg() callers set hdl->wtodo, so your diff can't break
things that used to work by accident."

ok ratchov@

Remove references to APIs that don't exist anymore.
Based on a diff from Vincent Gross <dermiste@kilob.yt>, thanks!

Implement If-Modified-Since.
From Kyle Thompson <jmp AT giga DOT moe>.
Tweaks by me.
OK benno@

WinSCP doesn't implement RFC4419 DH-GEX so flag it so we don't offer that
KEX method.  ok markus@

Huawei MU609 from Cristián Edwards.

ok dcoppa@

regen

Huawei MU609 from Cristián Edwards.

ok dcoppa@

Flag user specified routes with an AF_LINK gateway as RTF_LLINFO.

This makes ND6 and ARP layers happy since they both check for this flag,
but is was redundant to provide route(8)'s "-llinfo" option when adding
a l2 route.

RTF_LLINFO is clearly redundant for userland tools, so only allow the
kernel to set it.

ok benno@, claudio@

tweak;

Document that shm_open() accepts O_CLOEXEC and O_NOFOLLOW as extensions

ok miod@

use the sizeof the struct not the sizeof a pointer to the struct
in ssh_digest_start()

This file is only used if ssh is built with OPENSSL=no

ok markus@

emul_native is only used for kernel threads which can't dump core, so
delete coredump_trad(), uvm_coredump(), cpu_coredump(), struct md_coredump,
and various #includes that are superfluous.

This leaves compat_linux processes without a coredump callback.  If that
ability is desired, someone should update it to use coredump_elf32() and
verify the results...

ok kettenis@

fix a return path that lacked a rw_exit_write() call
ok kettenis@

use the sizeof the struct not the sizeof a pointer to the struct
ok claudio@

fix numbers for pppx, vscsi and diskmap

ok dlg@

Default daemon_flags are no longer in /etc/rc.conf,
but you can still find them in the rc.d(8) scripts.

Remove comments about default daemon_flags; most are empty, those
that aren't are redundant because they can be found in the rc.d(8)
scripts themselves, and they risk getting out of sync.
While here, sort the daemons alphabetically.
No functional change.

Triggered by a much smaller nameserver-only patch from stephan@.
OK ajacoutot@ rpe@ stephan@ and looks good to sthen@.

fix device numbering comments, no binary change

Document the disklabel(8) autopartitioning feature.
While here, reformat the documentation for the additional questions.

Feedback from schwarze@
OK henning@ on a similar diff
OK jmc@

Add the new template file based autopartitioning feature of disklabel(8)
to the OpenBSD installer. It is available during unattended installation.
The template file is fetched from an url, provided as answer to a new
question in the response file:

    URL to autopartitioning template for disklabel = url

Original diff from and OK henning@
'no objection' krw@

lightly document the posix option and POSIXLY_CORRECT; will sketch out
in more detail once i get my head round them...

Add SwissSign CA root certificates. Requested by robert@, ok dcoppa@ aja@ miod@

tweak previous;

sort the hardware list for urtwn, and update it's description in usb.4;

add missing .Vt macros

Use the right index when expanding/collapsing tree, from Thomas Adam.

Fix clipping bug in float to fixed point conversion, causing
audible glitches when samples smaller than the quantification
step are converted.

urtwn(4) man page updates for RTL8188EU support.
Patch by Mikhail <mp39590@gmail.com>, based on FreeBSD r264912 by kevlo.

Add support for RTL8188EU chips to urtwn(4).

Patch by Mikhail <mp39590@gmail.com>, based on FreeBSD r264912 by kevlo.

Tested with 8188EU (TP-Link TL-WN725N). No regression on 8188CUS.
There are performance problems with 8188EU devices, cause unknown.
These will hopefully be resolved soon.

ok mpi@

When writing a value to a field, don't read past the end of the source value
if the field is bigger.  Fixes crashes seen on HP bc2500 blades with MP
kernels.

ok kettenis@

Use ether_input() as default input packet handler and do the necessary
m_adj(9) to keep bridge(4) working while other pseudo-drivers are
converted to if_input().

Tested by mxb <mxb AT alumni DOT chalmers DOT se>, thanks!

ok henning@

reduce the scope of things that include uvm_swap_encrypt.h.

uvm_meter.c needs it to route the sysctl, uvm_swap.c needs it to
use the functionality, and uvm_swap_encrypt.c needs it to for obvious
reasons. userland sysctl already includes it explicitely.

everything else doesnt and shouldnt care.

ok miod@

Document usbd_open_pipe(9) and friends, from Sean Levy with some tweaks.

Use the size of the buffer not the pointer in a call to mem_free().
ok miod@ and guenther@ who both pointed out this argument is unused.

use the size of the buffer not the pointer in resume_randomness()
ok djm@ miod@ deraadt@

Print irq informations in pckbc_set_inputhandler().

ok miod@

Remove pattern length argument from match_pattern_list(),
we only ever use it for strlen(pattern).

Prompted by hanno AT hboeck.de pointing an out-of-bound read
error caused by an incorrect pattern length found using AFL
and his own tools.

ok markus@

Fix some misuse of the | operator.  In particular | has higher
precedence than ?:
ok guenther@ krw@ miod@

Clarify that hibernate_resume must be called with interrupts enabled.

Specify regression test host keys so that this test will work without
SUDO (because it can't read the real host keys) and in -portable on
systems where sshd hasn't been installed yet (because the keys don't
exist yet).

Use diff w/out -u for better portability

Preserve times to nanosecond precision instead of just microsecond.
Prefer to set attributes by fd for regular files, and not follwing
symlinks for others.

ok brynet@ millert@

Implement byte ranges.
From Sunil Nimmagadda <sunil At nimmagadda DOT net>
OK benno@

S was misplaced in r1.89, the optarg that was removed was actually needed
by R, so reintroduce it in the proper place.

spotted and diff by Sunil Nimmagadda

smtpd enqueue -S does not take an argument, fix optstring accordingly

fix by Nathanael Rensen

Remove subsections and start documentation of autoinstall only installer
features.

Feedback and OK jmc@
"Reads ok" krw@

struct usave is unused, and eproc is gone; delete the former and
stop referencing them in comments

add missing CISS_UNLOCK_SCRATCH/splx calls
ok krw@

Make sure we don't leak bytes from malloced memory in the padding of struct
dirent.  (The memset in previous commit was both wrong and insufficient.)

problem with memset noted by brad@ and jsg@
ok millert@

Eliminate the -U flag and make usage of DUID in /etc/dumpdates the default.
Correct old style entries so nothing has to be done for the admin.

diff from Manuel Giraud (manuel (at) ledu-giraud.fr)  Thanks!

fix build with option DEBUG

Die, damned distracting red space.

de-castify strtonum to make it consistent again.
Pointed out by bluhm@; no object change.
OK bluhm@, krw@

Use IPV6_MAXHLIM instead of 255; pointed out by bluhm@
No object change.

A ttl of 0 is valid. While here use MAXTTL instead of 255.
Input bluhm@, OK krw@

add missing splx calls
ok krw@

add missing splx calls
ok miod@

Hoist common assignments out of a series of if/if-else statements
in get_token(). Simplifies code and shrinks future diff.

No intentional functional change.