Rename ucc_intr_to_usage() to ucc_bits_to_usage(), no functional change.

Add missing leading colon to error message.

There are some races in the hardware which thus require a few more tries
to get ownership.

See Linux commit 501fd9895c1d7d8161ed56698ae2fccb10ef14f5

ok stsp@

Two more Capt. Obvious comments go away.

Don't use log_procname before it is initialized. In particular
set_iff_up() is called from initialize_interface() and was using
log_procname instead of ifi->name in its fatal()'s.

Diagnosed by tb@ from a recent reddit post by zielonykid1234.

Consolidate '-e' edit command parsing into ask_cmd(), simplifying
the main '-e' loop.

No functional change.

Improve some logging.

Ignore client creating session when working out size if it is a control
client.

bind-key needs to allow commands for any argument for the moment.

Validate command argument types (string or command list) and give more
useful error messages.

setupterm needs char * not const char * on some platforms.

Fix up some printflike attributes.

Start inputs as NULL so not freeing random stack garbage, GitHub issue 2852.

Make all conversion routines consistent by returning a struct
ucc_keysym. No functional change.

Remove the KB_DEFAULT flag from the wskbd keymap layout. Presence of
this flag will cause wskbd to discard the given keymap layout and
instead favor the layout of the associated wsmux. This is not a problem
while attaching ucc(4) during boot as the wsmux uses the default layout
at this point. However, if the layout is changed using /etc/kbdtype from
rc(8) during boot, attaching ucc(4) at this point would cause
wskbd_attach() to get stuck in an infinite loop:

wskbdX: cannot load keymap, falling back to default

... which in turn is caused by ucc(4) only providing a us layout and
using anything else in /etc/kbdtype would not work.

I missed this as I don't use /etc/kbdtype but cwen@ and Mazzurco
Riccardo <mazzurco dot riccardo at protonmail dot com> reported the same
problem.

Prevent ucc(4) from matching report IDs with an empty input report.

Fixes cwen@'s Creative BT-W3 audio device.

If the buffer given to the ucc interrupt handler is too long, do not
discard it but instead just inspect the first bytes that can make up a
usage. This is a work around as the correct solution is to only inspect
a limited number of bits as given by the report count and size
associated with the Consumer usage page.

Final piece to get florian@'s Microsoft Sculpt keyboard working.

Determine if a hid item is an array by solely inspecting presence of
associated min/max boundaries. Assuming that every usage array starts
with the Control usage is incorrect.

Fixes claudio@'s GMMK keyboard and partial fix for florian@'s Microsoft
Sculpt keyboard.

Fix automatic upgrade after fetching response file with dhcp.

When the auto upgrader tried to fetch the sets, sometimes ftp did
not find any.  The interface was down and had no address.  As
autoconf was not cleared, the upgrade's static netstart and dhcpleased
both tried to configure the interface, which cannot work.

Disable dhcp with "ifconfig inet -autoconf" in get_responsefile()
after it is not used anymore.

OK florian@

- guiness -> guinness
- heros -> heroes
- kill a dup

from tom

Fix ping group offsets.

ok jcs@

Add an #ifdef DEBUG/#endif chunk that dumps the entire GPT header when
printing a GPT.

No functional change outside of DEBUG.

Fix various read buffer overflow when printing ASN.1 strings (which are
not necessarily NUL terminated). Same as schwarze's fix in t_x509a.c r1.9.

From David Benjamin and Matt Caswell (part of the fixes in OpenSSL 1.1.1l)

ok inoguchi

When an interface disappears, e.g. when a usb dongle gets unplugged,
we get a RTM_IFANNOUNCE message not a RTM_IFINFO message.
Handle this message to not accumulate "unknown" interfaces.
While here fix a bug where we would remove an interface only in the
engine process but not in the frontend when an interfaces gets
unplugged while we process a RTM_IFINFO message for it.
OK benno

When an interface disappears, e.g. when a usb dongle gets unplugged,
we get a RTM_IFANNOUNCE message not a RTM_IFINFO message.
Handle this message to not accumulate "unknown" interfaces.
OK benno

use Pa for resolv.conf.tail, since there is no longer a man page
to Xr;

vini vidi vici -> veni vidi vici

make SYNOPSIS match usage to indicate -l is optional in:

swapctl [[-l] | -s] [-k]

from martin vahlensieck

A couple more Capt. Obvious comments can go.

Trim leading/trailing whitespace from -e input to make command
parsing more robust (e.g. 'edit 0 ' is now accepted) and strict
(e.g. 'reinit gptx' is now rejected).

Input which may want that whitespace is not trimmed (e.g. GPT
partition names).

Use consistent definition of whitespace corresponding to that
used by isspace().

Fix compilation with debug enabled.

ok kettenis@

Some consumer control descriptor reports uses an array of usages instead
of enumerating all supported ones. This representation is more compact
as all supported usages are given by the array min and max boundaries.

In enumerating mode, the interrupt report has one level indirection as
it includes the bit offset of the usage requiring the driver to map bits
to usages in order to resolve the key press.
In array mode, the interrupt report includes the usage without any
indirection, removing the need map bits to usages.

ucc(4) can now operate either in enumerating or array mode depending on
the descriptor report. This should make more keyboards usable.

Thanks to Alessandro De Laurenzis <just22 at atlantide dot mooo dot com>
for testing.

fix outdated comments, from Emil Engler, thanks.

We no longer have resolv.conf.tail.
From Scott Bennett, thanks!

Say which read failed, helps debugging.

We don't have a config file on the ramdisks; prevents a warning.

Since we have a config file now we should Xr its manpage.
From Scott Bennett, thanks!

I missed that wskbd_rawinput() is hidden behind WSDISPLAY_COMPAT_RAWKBD
as discovered by deraadt@; should hopefully make ucc(4) work on alpha.

args_make_commands_now needs to take an extra reference to the returned
command list since it will be freed already.

Move command argument parsing common functions and don't bother to parse
again if given a command rather than a string.

Key bindings steal a reference to the command instead of adding their
own, it was correct not to add a reference when parsing, but the
bind-key then needs to add one.

Revert one of previous, for some reason it is being freed.

Fix a few memory leaks.

Fix the initial length check in iwx_fill_probe_req() to match iwm(4).

Patch by zxystd from OpenIntelWireless.

Limit width and height to tty correctly, GitHub issue 2843.

Fix handling of leading #s when working out width.

Do not double free expanded path in source-file, also remove some
unnecessary assignments.

Insert alias in the right place, GitHub issue 2842.

Since the raw keymap already has a capacity of holding all present keys,
use it as a lookup table using the bit as the index. Removes the need to
linearly scan the same keymap during handling of a key press, not that
performance matters here since the length of the keymap is small but the
code gets a bit simpler and one struct becomes redundant.

While here, fix two incorrect conditionals in ucc_parse_hid() which
compared two different quantities.

ucc also fails to work here, because there is no wskbd_rawinput

Free value properly.

Preserve argument type in command and convert to string on demand.

Pass typed arguments out of the parser into the arguments list and let
it convert them into strings.

ioc(4) disappeared with sgi removal...

escape two instances of the word "An" in Rs/Re blocks: although mandoc
can handle it, groff does not like it;

Use new syntax for default key bindings.

Stop caring about empty commands, just treat as a null command.

octeon has no wskbd, so ucc cannot link.  the files.usb parts should have
handled this, but they don't.  disable until this is revisited.

Remove some members of struct cmd which are no longer used.

Preserve command group when moving temporary list to current list being
buit.

Rename a member to match what it will be in future.

make the description for ucc match exactly;

Add args parsing callback for some future work, currently unused.

Wrap command argument definitions in their own struct.

fix indent in previous

Switch macppc to ld.lld

Add powerpc to LLD_ARCH.  Remove -Wl,-relax flag; ld.bfd needed it,
but ld.lld can't take this flag.

To avoid linker errors on macppc, you need a snapshot where
/usr/bin/ld is LLD.

ok kettenis@ deraadt@

refactor to resolve unused variable on non-WSDISPLAY_COMPAT_RAWKBD arch

A couple more spacing fixes.

Spacing tweaks.

Hide struct args behind a couple of accessor functions.

Add a couple of const and fix some warnings.

Remove some unnecessary blank lines.

Expose args_value struct (will be needed soon) and add some missing frees.

Add a way to create an empty arguments set.

Remove stray spaces after function names.

Fill colour palette correctly from option for new panes, GitHub issue
2831.

Actually parse contents of {} as a command and then convert to a string
instead of just copying arguments around as strings.

Add a helper function for actually parsing the command.

sync

add ucc(4) manual, looks good to jmc@

Add ucc(4), a driver for USB HID Consumer Control keyboards. Such
keyboard is a pseudo device which is used to expose audio and
application launch keys. My prime motivation is to get the volume mute,
increment and decrement keys to just work on my keyboard without the
need to use usbhidaction(1).

Looks reasonable to kettenis@ mpi@ and ok jcs@

add consumer control hid usages, will be used by ucc(4)

Back out last commit.

deraadt@ pointed out that the operation is correct.
It attaches, but by then it is being pulled.  The dying flag is set to
avoid half the kinds of failure conditions.  The other half of the failure
conditions are detect on-the-spot.

And it seems when compiled for debug, this on-the-spot check which notices
the device isn't there, has a printf.

openssh-8.7

When plugging and unplugging an adapter continuously, it waits for a timeout
over and over again.  The problem is that the device has been unplugged,
but the attach handler still issues USB requests via ure_ctl().

The easiest solution is to deactivate the device proactively in
ure_ctl() when we see that error there.

From Christian Ludwig

In the attach routine, the hardware bug workaround is only for legacy PCI
interrupts.  Move it to the right place.

Revert previous; Turns out the start_hw() functions don't always
work right during the resume path. To be revisited later.

Run more device init code when resuming iwm(4) and iwx(4) devices.

deraadt@ pointed out that the resume code path only ran a small part
of the entire hardware init sequence which runs when the device is
first attached at boot time.

In particular, we didn't wait for device stabilization, and MSI-X
initialization was attempted too early, and the "persistence bit"
workaround was skipped (which, ironically, is supposed to address
a known hardware bug during resume).

Tested by jcs on AX201 and by myself on 8265 with no regressions seen.

do not crash when a manpath directory contains a symbolic link
that points to a directory rather than to a regular file;
bug reported by Lukas Epple <sternenseemann at systemli dot org>,
and my patch also tested by him on NixOS

fix the section number in the <title> element for preformatted pages;
minibug reported by Ian <Ropers at gmail dot com> on misc@

Purge the cached nodes when we switch back to SCAN.  In case we lost
the connection because an AP is gone away, this makes sure we don't
end up picking a now non-existant AP over and over.

Another problem this works around is that when trying to re-join that
AP, the attempt to connect would not yield any event message from the
firmware, and then we would get stuck.  Since we now definitely choose
a different AP, this issue does not show up.

ok stsp@

Consistently use ieee80211_begin_scan() to switch to SCAN.
React to deauth/disassoc and link state events if we're
already past the SCAN stage.

ok stsp@

implement reception of "VLAN 0 priority tagged" packets.

according to 802.1Q, vlan 0 on the wire is special and should be
interpreted as if it was a packet received on the parent interface,
but you get the packet priority information encoded in the vlan
header.

historically we drop vlan tagged packets that don't have a vlan
interface configured for the received tag number. historically we
have deviated from 802.1Q by allowing for the configuration of a
vlan subinterface with the vnetid "unset". this works just like any
other vlan interface, but it uses tag 0 on the wire. however, if
you're in a situation where you're receiving vlan tagged 0 packets
that really are part of the same layer 2 ethernet domain as the
parent inteface, this doesnt work well.

landry@ is in such a situation at work where the network is sending
his OpenBSD boxes packets with VLAN tag 0. sometimes. most of the
time the packets are untagged, which is expected, but sometimes
they have a VLAN header set. this causes problems, particularly
with arp.

this diff does the smallest possible change to enable reception of
these vlan 0 priority tagged packets. if an "unset" vlan interface
is not configured on the parent, then vlan 0 tagged packets get
their header stripped and continue stack processing as if they didnt
have the tag at all.

landry has been running this for months.
ok sthen@ claudio@

Fix an off-by-one in bwfm(4) SSID length checks.
SSIDs are binary data, not NUL-terminated strings.

ok patrick@

Hide detailed firmware error reports from iwn/iwm/iwx by default.

These drivers will now report "fatal firmware error" on a single line
by default and provide the full trace when debugging has been enabled
with e.g. 'ifconfig iwm0 debug'.

ok sthen@ kevlo@

Pull roots out of the trust store in the legacy xsc when building chains
to handly by_dir and fun things correctly. - fixes dlg@'s case and
by_dir regress in openssl-ruby

ok jsing@

Add veb(4) to the list of pseudo devices that can be created.

ok deraadt@

Import regress tests for SM2. Not yet linked to the build.

Part of Github PR #105

Import initial code for the SM2 cipher

This adds the SM2 algorithm defined in the Chinese standards
GB/T 32918.1-2016, GB/T 32918.2-2016, GB/T 32918.3-2016,
GB/T 32918.4-2016 and GB/T 32918.5-2017.

This is an ISC licensed implementation contributed by Ribose.inc, based
on the same code that was contributed to OpenSSL by Jack Lloyd. The port
to LibreSSL was done by Ronald Tse and Nickolay Olshevsky.

Github PR #105

I made quite a few cleanup passes on this, but more is needed, some
of which will happen in-tree before this is linked to the build.

ok deraadt inoguchi (a long time ago), jsing

Add a check_trust call to the legacy chain validation on chain add, remembering
the result in order to return the same errors as OpenSSL users expect to override
the generic "Untrusted cert" error.

This fixes the openssl-ruby timestamp test.

ok tb@