contoller -> controller

phsyaddr -> physaddr

spelling

addres -> address

unhandle -> unhandled

slignment -> alignment

preferense -> preference

plaform -> platform

Don't use *ENTRY_NB() with END_BUILTIN(), at least yet

Problem noted by naddy@

establush -> establish

Add ENTRY_NB() macro for doing an ASM function entry without setting
the binding to global (NB == "no binding"), as clang 13 is now
warning about changing the binding from global to weak.

This first pass does amd64 and sparc64 and pulls DEFS.h out of the
per-arch directory to a common directory; others to follow

ok kettenis@

Revert the hack for Apple M1 systems.

ok patrick@

Rewrite the kernel FPU handling code.  The new code saves the FPU state
in cpu_switch() instead of at the kernel edge and gets rid of the FPU
state tracking in struct cpu_info and struct pcb.  This fixes the random
crashes seen with SMP kernels on Apple M1.

ok patrick@

Use correct defines for random seed magic/length.

Spotted by Andreas Schnebinger

uniq(1): miscellaneous style(9)-ish stuff

- Include what you use:
  + Add <strings.h> for strcasecmp(3).
  + Drop <errno.h>, unused.
- Pull the "errstr" declaration out of the getopt(3) loop
  up to the top of main().
- Align the strtonum(3) error messages with the argument
  names in the uniq.1 manpage and the usage string.  They
  are referred to as "chars" and "fields".
- Trim some whitespace.
- Return from main() instead of exit(3).
- De-(void) fprintf(3).
- Prefer getprogname(3) to __progname.

uniq(1): bump numchars, numfields from int to long long

Also bump repeats from int to unsigned long long.

While here, don't cast the result of strtonum() and unwrap some lines.

Remove unused function prototype.

Add missing locking to pmap_extract(9) and pmap_unwire(9).

ok patrick@, mpi@

Multiprocol -> Multiprotocol

Identifiier -> Identifier

copyright++;

spelling

fix memleak in process_extension(); oss-fuzz issue #42719

eascape -> escape

approciate -> appropriate

failured -> failed

uniq(1): freopen(3) stdin, stdout

uniq(1) defaults to stdin and stdout, has exactly one input and one
output, and permits the user to replace the default input and/or the
default output with a given file.

freopen(3) is the right idiom in this context.  Get rid of file() and
the extra FILE pointers.  Switch from fprintf(3) to printf(3).

While here, const the "str" argument to show().

Thread: https://marc.info/?l=openbsd-tech&m=164078603312936&w=2

ok millert@

contibutions -> contributions

identfier -> identifier

spelling
ok dtucker@

spelling
ok ratchov@

excpetions -> exceptions

deafult -> default

Try to turn on less-capable mouse modes when turning on more-capable, to
increase the chances we get something even if the terminal doesn't support
the one we really want. GitHub issue 3020.

Interrups -> Interrupts

Add missing locking to pmap_extract(9) and pmap_unwire(9).

ok patrick@, mpi@

avaialble -> available

specifed -> specified

Newer Apple firmware on chipsets without a hardware RNG require the host to
provide a buffer of random bytes to the device on initialization.

Fix up handling of IFF_ALLMULTI and call uaq_iff() from uaq_init().

from Brad

typo: p5-Socket-SSL -> p5-IO-Socket-SSL
ok kn@

relayd(8): don't create sockets between CAs and RELAYs.

CA and RELAY process types don't need to communicate with other CA
or RELAY processes respectively, so don't create and distribute ipc
socketpairs.

Tested by and ok denis@

Add Xiaomi MiniWif to list of devices supported by mtw(4).
Works for me on amd64.

ok hastings@

Use a distinct variable while iterating the list of existing devices.

ok mvs@

Reported-by: syzbot+e2d1df67f742a5a47938@syzkaller.appspotmail.com
Reported-by: syzbot+72298724beda82ec8e7f@syzkaller.appspotmail.com

Sync from libc: Use _MAX_PAGE_SHIFT, rather than #ifdef mips64

ok otto@

Sync rcexec with reality.

Add back support for -B net -b bsd.rd which emulates a PXE install and
results in an autoinstall. This can be used to quickly create new OpenBSD
installs.
OK dv@

minor tweak

Prevent a double free by assigning the new keymap and corresponding size
after the allocation and initialization is done. Otherwise, a race is
possible if malloc ends up sleeping.

ok sashan@

Reported-by: syzbot+7f8224e9f1a3487caf25@syzkaller.appspotmail.com

Delete dead (duplicate) test

Prevent concurrent access to incomplete or dying `sc' caused by sleep
points in pppacopen() and pppacclose() paths. Use the same "sc_ready"
logic we use for 'pppx_if' structure.

Reported-by: syzbot+a7ac144b48f7f471f689@syzkaller.appspotmail.com
ok anton@ dlg@

combine ifdef blocks
from Ted Bullock

Plug memleaks

CID 345150

Plug memleak

CID 345156

More EVP_* return value checks

CID 345152

Remove redundant NULL checks

CID 345154

Minor cleanup after EC_POINT_get_affine_coordinates() unified the
_GFp and _GF2m variants.

CID 345155

More return value checks for EVP_* API

CID 345159

Check return values of EVP_* API

CID 345158

knfmt makes this slightly nicer.

One more leak of the same kind

Plug memleak

CID 345160

Set failed in test_random_points()

CID 345141

make the argument name for veb's "child-iface" consistent;
from caspar schutijser

Consistently name page argument `pg'.

Reduce differences with NetBSD, no functional changes.

Add regress tests for grep -A, -B and -C flags

ok otto@

PHYs that we want to talk to over SGMII need to be switched into SGMII
mode for the link to go up.  This is relevant on RJ45 SFPs.

ok sthen@

Mention attachment and reference to acpi(4).

Enable ohci(4) and ehci(4) at acpi(4).

Allow ohci(4) and ehci(4) to attach to acpi(4).

ok kettenis@

acpi_intr_disestablish() is available on all our ACPI platforms which
support interrupts via ACPI, so there's need for an ifdef around it.

ok kettenis@

Cleanup struct entity. Remove the unneeded has_data field, the same
information can be figured out by looking at the data pointer itself.
It is NULL when there is no data and not-NULL if there is data.
OK benno@

Merge multiple similar code bits together into repo_done(). Which currently
finds all repos that point to the passed backend and flushes their queues.
Also make sure that REPO_FAILED is set in cases where no sync happened and
also fall back from rrdp to rsync in any case of failure (including errors
merging the rrdp repo itself).
OK tb@

Ensure file descriptor send/receive is not allowed.

Do not allow send/receive of kcov descriptors as the file descriptor can
be kept alive longer than expected causing syzkaller to no longer being
able to enable remote coverage.

ok visa@

Reported-by: syzbot+ab2016d729cda7b0d003@syzkaller.appspotmail.com

Tweak some DEBUG output.

Always put '0x' prefix in front of displayed hex values,
i.e. the checksums.

Add missing letoh32() so bad endian checksum is clear.

Be consistent when displaying 'expected' and 'got' checksums,
i.e. 'expected' == calculated checksum and 'got' == checksum
found in GPT.

Add unp_dispose() threads to 'unsopassgc' test.

Tweak some DEBUG output.

'EFI PART' reads better than "TRAP IFE" and
checksums are easier to read/compare when shown
in hex rather than decimal.

Prompted by recent GPT vs disklabel misc@ thread
concerning  Rob Whitlock's problematic GPT.

whitespace tweak, no functional change.

it doesnt make sense to configure a vport as a span port.

move away from using the M_PROTO1 flag to prevent loops with vports

if a vlan interface is configured on a vport interface, vlan(4)
will take the packet away from ether_input before the veb bridge
input handler gets to clear M_PROTO1. this leaves the flag on the
mbuf as it goes through the l3 stacks. if it goes back out a vport
into a veb, the presence of M_PROTO1 means the packet ends up getting
dropped, which is unexpected.

this diff specialises vport handling by veb even more to avoid the
problem the flag was handling. vports get their own bridge input
handler that skips veb processing completely because a packet being
received on a vport can only occur if a veb has decided to forward
it there and has already processed it. when the stack sends a packet
out a vport interface, then we do actual veb bridge input handling.

bug reported on misc@ and the fix tested by Simon Baker

Fix typo in comment

lld: restore upstream default for --[no-]allow-shlib-undefined

ok patrick@ kettenis@

lld: do not report undefined weak references in shared libraries

https://github.com/llvm/llvm-project/commit/52bfd2c1ccd86ff813ee6df5a6700690acdd912f

This fixes an issue introduced in D101996.

A weak reference in a shared library could be incorrectly reported if
there is another library that has a strong reference to the same symbol.

Differential Revision: https://reviews.llvm.org/D115041

ok patrick@ kettenis@

Use lowercase letters for hexadecimal constants, as both jsing and I
prefer this.

Rewrite X509v3_addr_canonize() with new accessors

This is again a straightforward conversion and leads to something which
matches our usual style more.

ok jsing

Validate AFIs before sorting in X509v3_adr_canonize()

Again, we're dealing with necessarily not fully validated data here,
so a check up front seems prudent.

ok jsing

Rewrite/simplify X509v3_addr_is_canonical()

This is a more or less straightforward conversion using the new
IPAddressFamily accessor API. As a result, some checks have become
a bit stricter, which is only desirable here.

ok jsing

Check AFI/SAFI before comparing them in X509v3_addr_is_canonical()

As mentioned in a previous commit, IPAddressFamily_cmp() can't really
check for trailing garbage in addressFamily->data. Since the path
validation and hence the X.509 validator call X509v3_addr_is_canonical(),
this deals with only partially validated data.

ok jsing

Make IPAddressFamily_cmp() more pleasing on the eye

Define and use MINIMUM() instead of a ternary operator and separate
the code from the declarations. Also, we can spare a line to make the
return legible instead of squeezing it into another ternary operator.

addressFamily->data contains a two-bytes AFI and an optional one-byte
SAFI.  This function currently also compares any trailing garbage that
may be present. Since comparison functions can't really error, this
needs to be checked bofore it is used. Such checks will be added in
subsequent commits.

ok jsing

Style improvements in X509v3_addr_add_range()

ok jsing

Style improvements in X509v3_addr_add_prefix()

ok jsing

The initial kd_lookup() in kcovopen() is redundant as kcov is a cloning
device. Turn it into an assertion instead to better state the expected
semantics of D_CLONE.

Prompted after a discussion with visa@

Another small readability tweak in X509v3_addr_inherits()

Declare IPAddressFamily before using it.

Use an accessor in X509v3_addr_inherits()

Add a comment to i2r_IPAddrBlocks that we may want/have to deal with
unknown address family types.

Pointed out by jsing during review.

Add a few accessors for IPAddressFamily and make first use of them

One reason why this file is hard to read are endless repetitions of
checks and assignments reaching deep inside structs. This can be made
much more readable by adding a bunch of accessors. As a first step,
we deal with IPAddressFamily, where we want to check the type of the
ipAddressChoice member, check whether the inheritance element is present
or access the addressOrRanges field.

This diff already makes minimal use of these accessors to appease -Werror.
More use and additional accessors will follow in later passes.

ok inoguchi jsing

Now that -A prints -- markers as it is supposed to filter them out when
doing the consistency check for locore.o; ok millert@

Fix printing of -- marker when using -A as noted by gnezdo@
ok millert@

Simplify and explain expand_addr() a bit

RFC 3779 section 2.1.2 does a decent job of explaining how IP addresses
are encoded in. What's stored amounts to a prefix with all trailing zero
octets omitted. If there are trailing zero bits in the last non-zero octet,
bs->flags & 7 indicates how many. addr_expand() expands this to an address
of length 4 or 16 depending on whether we deal with IPv4 or IPv6.

Since an address can be the lower or the upper bound of a prefix or
address range, expansion needs to be able to zero-fill or one-fill the
unused bits/octets. No other expansion is ever used, so simplify the
meaning of fill accordingly. There's no need to special case the case
that there are no unused bits, the masking/filling is a noop.

ok jsing