Use getaddrinfo to parse IP addresses and lookup names.

OK bluhm

Do not send zero sized vendor or client ids.

The frontend and engine do not like this due to improved error
checking.

Found by Josh Grosse and Renato Aguiar, thanks!

cms_att.c: tidy includes and add x509_local.h for upcoming change

calendars are so hard

replace atoi(3) usage with strtonum(3); ok/tweaks tb@

replace strtol(3) usage with strtonum(3); idea/ok/tweaks tb@

replace hand-rolled number parser with strtonum

original report by Collin Funk
ok bluhm, millert

fortune files are no longer being rot13d;

- remove the "real" dance. there is no longer a need for it.

- do not install the offensive files rotated 13 chars, as suggested by deraadt.
it hardly makes sense, and only one of the "potentially offensive" files
was being treated this way anyway

- simplify the makefile to catch up with all this

- update NOTES to catch up with all this

- the notes in README pertaining to not installing the offensive files is no
longer relevant, so remove it, as suggested by millert

virtio: Introduce dedicated attach args

Instead of abusing virtio_softc as attach args, create a separate
struct. Use it to pass the number of available interrupts. This will be
useful for vio(4) multi-queue support.

ok jan@

vio(4): Fix hardmtu without MRG_RXBUF

Without VIRTIO_NET_F_MRG_RXBUF, we cannot chain several buffers together
and we can only receive packets up to the length of the buffers we put
into the ring.

OK bluhm@

ts.c: typo flaf -> flag

Replace recallocarray() with a realloc() + memset() combo.

recallocarray(), with its guarantee that memory becoming unallocated is
explicitly discarded, is too slow. In rpki-client forming one particular
ibuf takes more then 4mins because every recallocarray() call ends up
doing a fresh malloc + memcpy + freezero call.
For sensitive data use ibuf_open() instead of ibuf_dynamic() to avoid
any memory reallocations.
OK tb@

Rearrange #ifdef TCP_SIGNATURE to keep braces balanced.

Use strtonum instead of atoi.

Test bitstring macro evaluation.

For all bitstring macros, add a test with side effects in the
arguments.  Also fix compiler warnings and wrap long line.  In
main() replace exit(0) with return(0) to check stack canary.  Create
expected test files with make target create-good.

OK florian deraadt@

Evaluate arguments of bitstring macros only once.

According to bit_alloc(3) man page the arguments to bitstring macros
are evaluated only once and may safely have side effects.  Fix the
implementation with temporary variables to fulfill this requirement.

OK florian@ deraadt@

style(9) fix. No functional changes.

C-h should not be treated specially and represented internally as \b but
as C-h like the other Ctrl keys. Backspace is already handled separately
if it VERASE.

Pass the screen_redraw_ctx struct into more functions instead of
individual arguments (for example for the pane status), from Michael
Grant.

Client flags was changed to uint64_t a while ago, fix a few cases where
it is still int (do not matter now but will with some new flags). From
Michael Grant.

Add window_pane_mode helper function to tell if a pane is in copy mode,
from Michael Grant.

Add copy-mode -d flag to scroll a page down if in copy mode already,
from Michael Grant.

Use i2s instead of hard coding the imsg type. Suggested by tb.

errno is unset, use fatalx(3) instead of fatal(3).

Be stricter in what we accept from the main process.

While here mention function where fatalx(3) occurred like everywhere
else.

Suggested by & OK tb

annoying whitespace found during an audit process

replace multiple '.arch armv8.3-a' with a pauth target attribute
'no objection' kettenis@

typo

make activate function confirm to the common idiom; ok miod

Do not peek inside of struct imsg.

input & OK tb

Since netstart r1.208 (2020), it no longer applies /etc/myname

Change Xr from netstart to rc.

From Christian Schulte, ok florian

sin6_to_str and i2s take a single argument.

Pointed out by tb

PASSTHROUGH -> FALLTHROUGH
ok ratchov@

More precision on what exactly OCSP_id_cmp and OCSP_issuer_id_cmp compare.
The existing description was lacking and incorrect, respectively.

Simplify engine_showinfo_ctl()

It only handles one imsg type these days, so it doesn't need to peek
into struct imsg at all.

pointed out by & OK tb

Stop peeking into struct imsg when relaying control messages.

pointed out by & OK tb

conf_def.c: add two trailing commas

Remove documentation for no longer existing or mostly unused allocators.

ok guenther@

Place uvm_km_zalloc(9) under #ifdef __i386__.

This allocator is only used by a single pmap which will be hopefully
converted.

Suggested by guenther@

Kill uvm_km_alloc(9) and uvm_km_alloc1(9).

ok guenther@

Do not peek inside of struct imsg.

While here use i2s helper function for error logging.

OK tb

Helper function for logging imsg type names.

OK tb as part of a larger diff

X509at_get_attr: zap trailing comma.

reminded by mandoc -Tlint

sync

LibreSSL no longer supports adding X.501 attributes to an EVP_PKEY

Remove the corresponding documentation.

sync

Change DHCP_SYNC_LEASE messages from log_info to log_debug, they result
in quite a lot of noise in a typical dhcpd sync setup.

From Michał Markowski, ok florian

Switch UI_UTIL_read_pw* to LCRYPTO_UNUSED()

ok jsing

Neuter the completely broken UI_UTIL_read_pw* API

Return 0 on success, return <= 0 on failure. Sigh. In particular, if an
allocation failed, the password that no one entered was considered valid.

ok jsing

Remove documentation of UI_UTIL_read_pw*

According to some, a fail-open password verification function is par for
the course for libcrypto. Unfortunately, we have been recommending its use
over similarly named EVP functions after what amounted to a coin toss a
few years back. Luckily enough, no one followed that advice and we can
soon remove this API for good.

Make clear it's about *ship* parts when you list the quiz(6) subjects.

ok jmc@

spelling

the index is alphabetically sorted;

sync

Enable per-cpu page cache; tested on Octeon.

Switch alpha to MI mplock code.

The greek quiz is so obscure that it is ridiculous -- noone can play
this.  Replace it with a new quiz about galley (ship) parts.  This
commit changes the *LAST UNMODIFIED ORIGINAL FILE* (meaning revision
1.1.1.1) from the original import that created OpenBSD on Oct 18,
1995.  With this commit, we have completed an amusing mission of
replacing the final parts of the original OpenBSD.

We have reached OpenBSD of Theseus.

ideas & assistance from mglocker, naval terminology help from jmc

Putting Xs into squares is all the rage with statisticians. We should
test that they can use bitstring macros for that.

with & OK deraadt
OK bluhm

Speed up script by using awk(1).

Discussed with deraadt@

Make sure pmap_kernel's mutex field gets correctly initialized rather than
bss initialized.

Generate tabset files using a script. ok deraadt millert

Fix some program names that were not expanded correctly when ncurses was
updated and were instead replaced by ?. ok millert

slight improvement to index file parsing; ok mlarkin

Ignore internal function keys if they have not got an entry in the key
table.

Remove unwanted trailing newlines from err/warn format strings.

Check for exact match for layout name before looking for a prefix match.

Remove use of CSP/LMK in pkcs12 create/verify tests

reminded by ... anton

As defined in the RFC, the SSH protocol has negotiable compression support
(which is requested as the name "zlib"). Compression starts very early in
the session.
Relative early in OpenSSH lifetime, privsep was added to sshd, and this
required a shared-memory hack so the two processes could see what was going
on in the dataflow.  This shared-memory hack was soon recognized as a tremendous
complexity risk, because it put libz (which very much trusts it's memory)
in a dangerous place, and a new option ("zlib@openssh.com") was added begins
compression after authentication (aka delayed-compression).  That change
also permitted removal of the shared-memory hack.
Despite removal from the server, the old "zlib" support remained in the
client, to allow negotiation with non-OpenSSH daemons which lack the
delayed-compression option.
This commit deletes support for the older "zlib" option in the client.
It reduces our featureset in a small way, and encourages other servers
to move to a better design.
The SSH protocol is different enough that compressed-key-material attacks
like BEAST are unlikely, but who wants to take the chance?
We encourage other ssh servers who care about optional compression support
to add delayed-zlib support.  (Some already do "zlib@openssh.com")
ok djm markus

add rcsid markers so that we can visually see the flurry of commits in
this area
ok mlarkin

add the dumbest #-comment parser for the Game_List parser, someone
else can improve this later.  My code is not substandard considering
what I saw.

add rcsid markers so that we can visually see the flurry of commits in
this area
ok mlarkin

fix spelling of sequence

ok mglocker

atc(6): update ORD marker beacon information

ok deraadt@

probition is over.  most people want want to laugh at lightly
offensive things.
suggested it is time by millert

these tests only print strerror result, which can be confusing to look up.
print errno also.

must use sh to run the script, because /usr/src may be noexec

Fix KERN_AUDIO broken in rev 1.440.

FALLTROUGH -> FALLTHROUGH

FALLTHROUHG -> FALLTHROUGH

cron: use strtonum() and tighter limits on step values

Using strtonum() instead of atoi() gives us an extra layer of bounds
checking for free while parsing an entry.  This is in addition to
the existing bounds checking in set_range().  The step value is now
limited to the maximum range for an entry.  If the field consists
of a range, the step must not be larger than the difference between
the high and low parts of the range.  OK deraadt@

Change SIGCHLD handler to just set a flag.
We already call reap_kids() in multiple event loops so there is no
need to call waitpid() inside the handler itself.
OK denis@ deraadt@

sntrup761x25519-sha512 now has an IANA codepoint assigned,
so we can make the algorithm available without the @openssh.com
suffix too. ok markus@ deraadt@

PKCS12_create(3): remove Xr to EVP_PKEY_add1_attr(3)

This API family has been neutered and will be removed in the next bump.
Further cross references will be untangled in the future.

Garbage collect unused attributes member from EVP_PKEY

ok miod

Remove copy_bag_attr()

It is no longer possible to set an attribute on an EVP_PKEY, so this
code is dead.

ok miod

Neuter EVP_PKEY_add1_attr_by_NID()

The last consumer in openssl(1) pkcs12 has been removed, so we no longer
need this function.

ok miod

openssl: adjust manual for LMK and CSP removal

openssl pkcs12: remove support for LMK and CSP attributes

Documentation on what the Microsoft-specific local machine keyset and the
cryptographic service provider are actually good for is hard to find. For
some reason (perhaps one million and two arguments for PKCS12_create() was
considered two too many) these hang off the EVP_PKEY in the attributes
member, which serves no other purpose.

Every use of EVP_PKEY (of which there are far too many) pays extra memory
taxes for this fringe use case. This complication is not worth it.

ok miod

Fix merge of bounce buffer segments in amd64 bus dma.

If the physical pages are contiguous, _bus_dmamap_load_buffer()
tries to merge the segments.  In case of mbuf chains, it can happen
that the physical bounce buffers are contiguous, but the virtual
addresses of mbuf m_data are not.  Then during transmit _bus_dmamap_sync()
tries to copy segments where it cannot access the virtual source
address which is mapped in a different mbuf.  So if bounce buffers
are used, physical and virtual buffer must be contigous, to merge
a segment.
While there, split check and decrement of variable i in a for loop
to make the code readable.

with and OK hshoexer@

Unlock unlock ipip_sysctl().

- IPIPCTL_ALLOW - atomically accessed integer;
- IPIPCTL_STATS - per-CPU counters;

In ipip_input() load `ipip_allow' value to `ipip_allow_local' and pass
it down to ipip_input_if() as `allow' arg.

ok bluhm

Use aes128-ctr for MAC tests since default has implicit MAC.
Also verify that the Cipher or MAC we intended to use is actually the one
selected during the test.

Introduce sysctl_securelevel() to modify `securelevel' mp-safe. Keep
KERN_SECURELVL locked until existing `securelevel' checks became moved
out of kernel lock.

Make sysctl_securelevel_int() mp-safe by using atomic_load_int(9) to
unlocked read-only access for `securelevel'.

Unlock KERN_ALLOWDT. `allowdt' is the atomically accessed integer used
only once in dtopen().

ok mpi

sync manual section numbers

Clear overlay when command prompt is entered. Also fix some spacing in
man page pointed out by jmc.

Fix answer.

2,$-1g/^/.,.1j does not combine every even-numbered line with the next
odd-numbered line. One correct way is 2,$-1g/^/.,+1j

Pointed out by ed1conf on mastodon.

ospfd: fix whitespace error introduced in previous

lldb: shut up the warning message on quit debugging kernel core file.

The diff from Yuichiro NAITO.

ok yasuoka

inet_pton returns 0 and -1 for error.

Adjust the error check that is now wrong after the inet_aton -> inet_pton
conversion.

Noticed by & OK bluhm.
OK tb

Mechanically change inet_aton to inet_pton.

npppd does not document that it would accept truncated or otherwise
not fully spelled out IPv4 addresses.

ok yasuoka