Do not dereference ``ifa_ifp'' when we already have an ``ifp'' pointer.

Rename shutdown to exit.

The eigrpe process also needs to pledge "cpath" for unlinking the
control socket.

After some consideration, simply allow TIOCSCTTY in the "tty" pledge.
Discussion with nicm.

Do not dereference ``ia_ifp'' when we already have an ``ifp'' pointer.

Use rt_ifp as intended.

During s2k15 we fixed this ugly 20+ years loopback hack of having a
rt_ifp different than rt_ifa->ifa_ifp.

ok millert@, bluhm@

Kill dead code missed in per-ifp counter removal.

ok millert@, bluhm@

document "id" request; from Gregor Best

delivery to maildir needs pledge fattr

from Gregor Best <gbe@unobtanium.de>

halex removed the -p restriction, so do not document it;
from kirill bychkov

at present the setpriority() syscall is considered fairly low risk and
placed in pledge "proc".  pledge "stdio getpw proc", from Theo Buehler

Add a regress for libc handling of SIGTHR

Pledge "stdio rpath tty" for hangman(6).

Patch submitted by Ricardo Mestre <serial@helheim.mooo.com>

ok semarie@

rename ml_join to ml_enlist and expose it to the rest of the kernel.

pledge in doas.  startup pledge "stdio rpath getpw proc exec id".  4
more times after that more attributes are dropped: "proc" after bsd
auth has spawned/received result from the login_* program; "getpw"
after the final getpwent lookup, "id" after the final uid changing,
and "rpath" after constructing getcwd.  leaving only "exec", for the
ride into execve().

Until we understand the sitaution better, we should pledge() after
setlocale(), not before.  Not just here, but probably everywhere?

Fix typo in comment. From Theo Buehler.

Move all mdoc(7) node validation done before child parsing
to the new separate validation pass, except for a tiny bit
needed by the parser which goes to the new mdoc_state() module;
cleaner, simpler, and surprisingly also shorter by 15 lines.

Reject too small bits value in BN_generate_prime_ex(), so that it does not risk
becoming negative in probable_prime_dh_safe(). Reported by Franck Denis who
noticed `openssl gendh 0' would segfault.
Fix adapted from OpenSSL RT#2701.

ok beck@ jsing@

Use SSL_CTX_set_ecdh_auto() instead of rolling our own version.

ok gilles@

Only enable SSL_VERIFY_PEER when the verify option is set on a listener.

Always enabling SSL_VERIFY_PEER unnecessarily increases the number of
messages/bytes in the TLS handshake and increases our attack surface,
since we request and then process client certificates.

ok gilles@

In the case where len is not a multiple of sizeof(RC4_CHUNK) the RC4 code
will end up doing a read and write of up to 7 bytes beyond the specified
length. This is effectively a non-issue since we read and write back the
same data and due to alignment it is within a page boundary.

Regardless, avoid this by removing the "special" handling for the remaining
length and allow the standard (non-chunk) code to process the remaining
bytes, which does not result in overrun.

Reported by Pascal Cuoq <cuoq at trust-in-soft.com> - thanks!

ok beck@ miod@

Setting fcntl(F_SETOWN) for a pipe failed with inappropriate ioctl
for device.  In sys_fcntl() the ioctl(TIOCSPGRP) is called, but the
pipe expects SIOCSPGRP.  Sockets have a specal case for the same
reason, so adapt the special code for pipes.
OK millert@

Style fixes; from Ilya Kaliman

Remove a couple of unhelpful defines.

ok nicm@

Don't bother casting NULL.

ok nicm@

Assign pointer to NULL rather than 0.

ok nicm@

Penultimate commit to remove EXTERN.

ok nicm@

Do some cleanup in syslogd ttymsg().  Add a debug message when the
syslogd child calls fork(2) to delay blocked output.
OK benno@

client_key_table was missing.

fix memory leak in error path
ok djm@

add missing rcsid
ok renato@

By popular demand add a default binding for mouse wheel up to scroll
into history (if the mouse is, on of course).

make sure ProgressMeter gets set up. Fixes some weirdness in package installs,
as it now must contain a proper linkback to its own state.

No longer mention rtable_get(), it's a private function now.

While here use C99 types in function definitions.

Return the correct error code when a table already exists.

u_short -> unsigned int for rtableid.

Do not call uvm_swap_finicrypt_all() a second time in dumpsys().

ok tedu@, deraadt@, miod@

Remove Diffie-Hellman moduli entries below 2048.

OpenSSH requires a 2048 minimum for DH in the client and server.

input and ok sthen@
ok dtucker@, djm@

Add support for route summarization.

Working great but need more testing, especially with ipv6. For now
we don't validate if one configured summary is inside another or the
presence of duplicates. Will address these issues in a future commit.

Minor fixes and code cleanup.

Compare pointers to NULL rather than 0.

ok djm@

tweak previous;

Use client pointer not file descriptor in logging.

add a new getsockopt option IP_IPDEFTTL to retrieve the default ttl.
this can be used as an alternative to sysctl net.inet.ip.ttl, in
programs that use pledge().
ok reyk@, "Like this" deraadt@

Fix write to other user's tty.  The device has to be opened with
O_WRONLY, but without O_CREAT.  So freopen(3) has to be replaced
with open(2) and dup2(2).
from deraadt@

believe this will work with "stdio rpath wpath cpath inet" now that
SO_RCVBUF is permitted.  It may even be possible to lose the "inet" a
bit later in the code.  Anyone want to test?

Document SOCK_DNS flag
ok guenther tedu semarie

At guenther's suggestion replace dnssocket() with a SOCK_DNS flag on
socket().  Without pledge, all other socket behaviours become permitted,
except this one case: connect/send* only works to *:53.  In pledge mode,
a very few are further restricted.  Some backwards compatibility for
the dnssocket/dnsconnect calls will remain in the tree temporarily so
that people can build through the transition.
ok tedu guenther semarie

Add SIOCGIFGMEMB to "route" (returns a list of all interfaces who are
member of the given group).  This is used by some parse.y.

OK deraadt@

Fix 802.1p VLAN priority code points for VLAN_HWTAGGING.
Our in-kernel ether-vtag has a different layout to the vr TXSTAT register.

ok sthen@

... but keep the previous logic for sparc, which is the other user of this
Makefile, until it switches to MI installboot, too.

Missing commit of vax switch to MI installboot(8), forgotten when it went in.

Lob a style(9) grenade in here.

add "proc" to pledge for script(1)

it needs to "forward" SIGWINCH to subprocesses.

ok deraadt@

The table could change when retrying so don't save it at start of
server_client_handle_key.

After pledge "dns" has been refactored and setsockopt(SO_RCVBUF)
has been added to it, the syslogd privsep parent does not need
pledge "inet" anymore.
discussed with deraadt@

Add an explicit check for a malformed AS segment with (segment length 0),
avoiding division by zero when deciding whether it contains 2- or 4-byte ASNs.
Refactor TCHECK calls to ensure proper coverage.

From Kevin Reay, ok canacar with wording tweak (I used "malformed" rather
than canacar's suggested "invalid size" or Kevin's original "empty").

fix a use after free found by clang using an approach suggested by renato
ok renato@

On xmalloc failure, stop trying to determine the total amount of allocated
memory up to this point by using sbrk(2).  This is of course wildly incorrect
for any mmap-based malloc(3).

This also makes it possible to bring pledge(2) to gnu/.

comments kettenis@, ok miod@

update regress for pledge after whitelisted-path view inclusion

Fix ocsp by adding a missing TAILQ_INIT().

Confirmed by markus@ with an identical diff

clear whitelisted-paths view in pledge.

the following diff adds a clear view of whitelisted-paths in pledge.

before, whitelisting "/usr/local/bin" path would make only "/usr/local/bin"
VNODE was present and let "/usr/local", "/usr", and "/" been ENOENT. It was a
somehow odd filesystem hierarchy, and it breaks realpath(3).

with this diff, the directories that are one of the parents of a
whitelisted-directory become visible to stat(2) related syscalls, but only
with restricted permissions: stat(2) will lie a bit, and saying they owned by
root:wheel and mode is --x--x--x. Note that only stat(2) is affected by this
"view", and the owner/mode aren't effectively changed: it is just a "lie".

while here, refactor a bit pledge_namei() in order to avoid multiple for-loop
on whitelisted-path array.

ok deraadt@

allow SO_ERROR all the time

sync

Replace a function-local allocation with stack memory.

ok djm@

In order to become able to generate syntax tree nodes on the roff(7)
level, validation must be separated from parsing and rewinding.
This first big step moves calling of the mdoc(7) post_*() functions
out of the parser loop into their own mdoc_validate() pass, while
using a new mdoc_state() module to make syntax tree state handling
available to both the parser loop and the validation pass.

Always allow the setsockopt & getsockopt system calls... however, in the
default case only allows SOL_SOCKET SO_RCVBUF which is very common in
network-facing daemons.  Many of them manage this on a socket after
dropping abilities which can get them _new_ sockets.. syslogd, bgpd,
relayd, etc etc.  Other sockopts still require specific pledges.
Tested by bluhm.

Remove old tame() stub

Add pledge(2) for radiusctl(8) and radiusd(8).
- radiusd: "stdio inet"
- radiusd_radius: "stdio inet"
- radiusd_bsdauth:
  - "stdio proc" for the non-priviledged process
  - "stdio getpw rpath proc exec" for the priviledged process
- radiusctl: "stdio dns inet"

"go ahead" deraadt

break long lines in examples; ok jmc@

Test syslogd logging to a tty which belongs to a user.  This is
done with a utmp entry for a pty fake login.  All messages are read
from the pty and written into a log file.

style cleanup, no functional change

Enable some code that does power management magic.

Simplify, no functional change:
Delete the outmdoc, outman, and outfree function pointers.

Properly restore the prefetchable memory window upon resume.

ok deraadt@

Add parenthesis gcc wants and remove defined but unused label gcc
complains about.

ok deraadt@ on parethesis

Delete the empty example file "ftpchroot"; no example is needed.
Move the one useful bit of information contained in the file ("one
user name per line") to the ftpd(8) manual page where it belongs.
OK deraadt@ sthen@

LABELSECTOR is a DEV_BSIZE quantity. So multiply by DEV_BSIZE and not
d_secsize when calculating disklabel location.

ok jsing@

Nuke unused variable.

including <ohash.h> requires including <stdint.h> beforehand;
noticed by Svyatoslav Mishyn <juef at openmailbox dot org>

pledge "stdio exec proc" early on, setpriority, then pledge "stdio exec"
from Theo Buehler

free rbio before wbio
ok jsing@

missing deref on char check. from David Binderman

Remove the define NOT, replace it with '!'. No binary change.

"The ^ is used in regular expressions and many versions of fnmatch(3)
accept both ! and ^. However, we are never going to accept ^ instead of
! so I think this makes sense" -millert@

"go for it" -nicm@

A little style(9) for sh.h includes

ok nicm@

Stop supporing "legcay" time formats that OpenSSL supports. Rewrite the
utctime and gentime wrappers accordingly. Along with some other cleanup.
this also removes the need for timegm.
ok bcook@ sthen@ jsing@

Change regress to not believe legacy times are valid

Allow setpriority in "proc" as well, since a few shells have "nice"
builtin.  make a note that setpriority() should be weakened, unless "id"
is also present.
pointed out by Theo Buehler

Move stddef.h include from sh.h to the file that uses it.

ok nicm@

Move string.h include from sh.h to the files that use it.

ok nicm@

fix memory leaks in error paths
ok renato@

Revert transfer submission to r1.85.  Seems to make okan@'s scanner
work properly.

Move limits.h include from sh.h to the files that actually need it. No
binary change.

ok nicm@

More removal of EXTERN.

ok nicm@

Print vnode type for sendfd/recvfd not file type, ok semarie

Stop checking for RTF_UP directly, call rtisvalid(9) instead.

While here add two missing ``rtableid'' checks in in6_selectsrc().

ok bluhm@

Stop checking for RTF_UP directly, call rtisvalid(9) instead.

While here add a missing ``rtableid'' check in in_selectsrc().

ok bluhm@

Sync rtisvalid(9) check for local route entries with r1.257 of
net/ip_input.c

Remove superfluous NULL checks.

ifa are refcounted to ensure that rt_ifa is always valid.