Sync variable names with netstart version of stripcom().
Use $_file variable instead of $1 with the wile-loop.

Align comments of stripcom() function in netstart and install.sub.

Extend the information shown by diskinfo() to help to identify
disks. Extract the disk information enclosed in <> and the NAA
from the last matching dmesg line. Useful if there are multiple
identically sized disk of the same type.

Suggested by jirib at devio dot us
Discussed with deraadt@

In ssl.h TLS 1.0 is called TLSv1.  Adapt name in test to make it pass.
OK jsing@

Adapt test to new syslogd's local message priority.

Incoming TCP connections from clients fill the files of a log server.
A log client reconnects at every SIGHUP.  Write these accept and
close messages with debug priority, then they can be turned on in
syslog.conf.  Default is off.
While there, move a debug message and set the priority of the exit
message explicitly to error.
OK mpf@ millert@

Simply decoding of domain search option. No intentional functional
change.

Add prid for CN72xx/CN73xx.

Make SoC version available in a simple form.

Use uint8_t instead of u_int8_t - for consistency and to make things easier
for portable.

From Raphael Hittich.

si_addr should be of type void *, not char *

From Andrew Aldridge (i80and(at)foxquill.com)
ok millert@ mpi@

Simplify the size calculation for memset()

From Anton Lindqvist (anton.lindqvist(at)gmail.com)

Inspired by recent dhcrelay(8) changes add some stricter CFLAGS. No code
changes needed (yet).

"kernel" got renamed to "boot" in vm.conf but I didn't commit the parse.y bits.

Found by Pontus Lundkvist

OK deraadt

replace some long if/else chains with a switch

a little const here and there to prevent rules from changing

prepenv can take a const rule

Remove the CD9660MAXPATH define. It was used to construct the path for
opening the file, so PATH_MAX makes more sense here. While there change
the code to do two less allocations per file.

ok deraadt

less chatty EXAMPLES; from anton lindqvist

trailing ; on end of macro definition is wrong; ok guenther

Do not expose vmm(4) VM IDs to the user, use vmd(8)'s IDs instead.

Each VM has two IDs: one from the kernel (vmm) and a different one
from userland (vmd).  The vmm ID is not consistent and incremented on
every boot during runtimg of the host system.  The vmd ID remains the
same during the lifetime of a configured VM, even after reboots.
Configured VMs will even get and keep their IDs when the configuration
is loaded.  This is more what users expect.

Pointed out and tested by otto@

OK deraadt@

Convert bcopy to memcpy where the memory does not overlap, otherwise,
use memmove.  While here, change some previous conversions to a simple
assignment.

ok deraadt@

shuffle for a saner read; help/ok bluhm

document -=; while here, simplify things so that we don;t add a separate
usage every time we think of a new way to use this;

-= bits from anton lindqvist
ok tb tom

Sizes for the remaining free() calls, and use mallocarray()
ok visa

Consistentcy between nmembers and size order. From Christopher Hettrick;
ok deraadt@

Check that local syslog messages have the the expected priority.

fix missing \ on \n, spotted by Edgar Pettijohn

Use the correct interrupt disestablish routine.

Extend the fdt interrupt API a little.

Add the LOG_SYSLOG facility to local messages.
OK millert@

Replace bcopy with a simple assignment where both variables are
properly aligned and sockaddr_union fields, or with memcpy when
the memory doesn't overlap.

OK bluhm@

use memset() instead of bzero()
OK jmatthew

fix format string found by clang -Wformat-security
OK deraadt millert

bump version for new development branch

Compile C code with -funwind-tables.  Fixes exception handling on architectures
that don't generate .eh_frame info by default such as arm64.

ok jsg@

first print size in meta-data then supplied arg size when an inconsistency is
detected wrt recallocarray()

remove stray character in previous that broke the build

Unify a few attach-time patterns between usb *com drivers, and remove
extra zero'ing of variables not needed because softc is pre-zero'd.
ok jsg

add sizes to free() calls
ok dhill

Convert bcopy to memcpy as the memory does not overlap.

ok deraadt@

3 more mallocarray() uses

Syslogd's debug and error messages have changed a bit, adapt test.

Replace 2 bcopy()'s with a simple assignment as both variables are
properly aligned malloc(9)d data and sockaddr_union fields.  While here,
convert the remaining bcopy() to memmove().

with and ok @bluhm

The global variable Startup is not used anymore and can be deleted.
The functionality has moved into log_setdebug().
OK millert@

As we did not have nice log functions before, ttymsg() had to return
the error string.  Log the message when the error happens and make
the function void.
OK millert@

The function logmsg() was used to generate local messages and to
process incoming messages.  Split this functionality into log_info()
and logline().  Sort the parameters like they appear in the syslog
line.
OK millert@

Replace logerror() functions with generic log_warn() from log.c.
Make messages a bit more consistent.  Note that the new function
supports format strings.  Replace some log_debug() with log_warn().
OK millert@

Shuffle sendhup() into dispatch.c, the only place it is used.

Not all devices support mmap, document EINVAL in this case too.
OK deraadt@

Clamp size to length if snprintf ever indicates overflow
w/ millert

Replace all combinations of logerror() and die() with log.c fatal().
Internally syslogd's fatal() calls die() to do cleanup if necessary.
Also replace all err(3) after log_init() with fatal().
OK millert@

Unify variables used for recvmsg/sendmsg

ok millert@ deraadt@

sendmsg returns a signed value, fix error check

ok millert@ deraadt@

We can enable many -W compiler warnings now.

OK rzalamena@

Rework the packet.c code for decoding and assembling DHCP messages.

This code suffered from many years of incremental improvements, fix it to:
- verify available buffer space in all cases and don't just trust the caller
- have an API where you always pass the full buffer length and absolute offset
- use consistent types for lengths and buffer offsets (int vs. size_t, ssize_t)
- don't just ignore errors, return and fail when something is wrong

OK rzalamena@

Don't rely on asprintf setting pointer to NULL on failure.

ok millert@ deraadt@ tom@

When building counter memory in preparation to copy to userland, always
zero the buffers first.  All the current objects appear to be safe,
however future changes might introduce structure pads.
Discussed with guenther, ok bluhm

Add Home and End for copy mode.

For a non-interactive attempt to pkg_delete a firmware package, print a
message advising use of "fw_update -d" rather than saying "ok" and doing
nothing. (pkg_delete intentionally makes it hard to uninstall firmware
to prevent surprises when cleaning a machine).

aja and I were surprised by the "ok" leaving the fw package installed.
suggestion from espie to mention fw_update -d in the message.

Replace logdebug() with generic log_debug() from log.c.  Implement
log_debugadd() to construct debug message incrementally.
OK deraadt@

Try again to resolve problems with mistaking sessions for windows: now
do not look up windows as sessions (and panes as windows) when they are
qualified with a ':' or a '.'. So 'foo' as a window target will look for
windows and sessions called 'foo', but ':foo' will only look for
windows, and 'foo:' only for sessions. This means the common case of
using an unadorned session as a window target (send -tfoo) should
continue to work, but an explicit window will not get confused with a
session (send -t:foo).

Give each client a name. This defaults to the tty name as before but
falls back to an alternative if the tty name is not available. This is
clearer than overloading the client ttyname member and allows us to
remove the path stored in the tty struct, it should always be the same
as the client.

cfg_file can be static.

Change tty_repeat_space to write large sets of spaces together instead
of writing them individually.

various improvements, from klemens nanni;
i've omitted hunk 3 of his diff, as what's there now is correct;

ok mikeb

- -Z before -z in options list
- add -Z to help and usage()

an annoying whitespace glitch

timeval has trailing padding on powerpc and m88k, so memset it before
copyout to avoid leaking kernel stack

ok deraadt@

Here at OpenBSD we change ABIs at the fling of a hat.  Just in case a
future disk info sysctl has pads in the structures, use M_ZERO when
allocating the storage to avoid leaking kernel memory.

clear structure on the stack before copying out for sysctl.  At present
it has no pads, but a future ABI change could add something and leak
kernel memory.

sync

Allow nc to save the peer certificate and chain in a pem file specified
with -Z
ok jsing@

Add tls_peer_cert_chain_pem - To retreive the peer certificate and chain
as PEM format. This allows for it to be used or examined with tools
external to libtls
bump minor
ok jsing@

Internal changes to allow for relayd engine privsep. sends the hash of the
public key as an identifier to RSA, and adds an function for relayd to
use to disable private key checking when doing engine privsep.
ok jsing@

Do not print a warning if closing the control socket fails.  It
cannot happen and there is nothing that could be done about it.
OK deraadt@

add a config test flag to sasyncd

ok reyk@ deraadt@ jmc@

Also remove non syspatch hidden files/dirs.

Do not purge the CONFIG_SERVERS config in the parent. The ticket code uses
the servers config for its rekeying handling. Without this no rekeying happens
and httpd stops working. Learned the hard way by me and beck@
OK reyk@

struct vfsconf is tightly packed, but let's M_ZERO it in case that ever
changes to avoid exposing userland memory.

Ensure, that during upgrades, the default for the "Location of sets"
question is set to http if /etc/installurl exists and override the
install method information from the cgi server.

'makes sense' deraadt@

The unsigned result cannot be -1, turn it into a signed variable.

OK rzalamena@

constify pc_remote and pc_circuit

OK rzalamena@

Send a RTM_PROPOSAL to clear out other dhclient instances on startup.

Replaces forcing interface link state down and up to generate
RTM_IFINFO messages.

document comments in table files; help/ok gilles

tweak previous;

Add missing newline in fprintf error message.

OK jsg@

Treat SSID's like the 0 to 32 bytes of uint8_t data that they really
are. Track and use the actual lengths and use memcpy()/memcmp()
instead of strcmp()/strdup().

Cleanup the code further.  Get rid of the remaining C++-style comments,
remove some debug printfs, get rid of STATIC and __inline defines and
remove a bit more unused code.

ok drahn@, visa@

Issue memory barrier before lock release, not after. This ensures
the release write becomes globally visible only after any writes
of the critical section are globally visible. In practice, the
reordering has not happened because the kernel runs in the total
store order mode.

Tested by and OK kettenis@

add a free() size near microcode loading

Rewrite tohexstr() to use the common idiom of converting a byte
string to hex nybble by nybble.  This avoids using the return value
of snprintf() unchecked.  OK deraadt@

Add 8265 and 3168 support. Both cards need new firmware files which currently
need to be installed by hand until the iwm-firmware package has been updated.
This includes handling (by ignoring) of IWM_UCODE_TLV_FW_MEM_SEG and
IWM_DEBUG_LOG_MSG firmware commands. Mostly stolen from dragonflybsd.
With this in the X270 has working ethernet and wifi.
Commit! deraadt@

disallow creation (of empty files) in read-only mode; reported by
Michal Zalewski, feedback & ok deraadt@

cp -p the bootblocks to RELEASEDIR; ok tb

warn format string should be literal.

OK deraadt@, millert@

Change parse_string() to take an optional integer pointer that can
be used to return the final size of the parsed (i.e. un-vis'ed)
string. Use same, plus memcpy() to ensure entire final string is
copied to intended destination even if there are embedded NULs.

Add caveat telling it's better to always read a data byte when passing
control messages.
Problem noted by Luke Small; ok deraadt@

Send and receive 1 byte of data in addition to the control message. This is
needed to make the code work in more cases.
Sending actually is not really needed, but do it anyway, to avoid confusing
mismatch between producer and consumer.
Problem noted by Luke Small; ok deraadt@

WSDISPLAY_MAXFONTCOUNT macro was introduced in sys/dev/wscons/wsconsio.h
(revision 1.75) to limit the number of fonts that can be loaded.

Reflect that in the man page as well.

OK jcs@