udev: if a tablet has BTN_0, label it as ID_INPUT_TABLET_PAD

Graphics tablet devices comprise multiple event nodes, usually a Pen, Finger
and Pad node (that's how the kernel postfixes them). Pen and Pad are labeled
as ID_INPUT_TABLET but the pad doesn't actually send stylus events - it
doesn't usually have BTN_TOOL_PEN, merely BTN_STYLUS.

For the last several years, libwacom has set ID_INPUT_TABLET_PAD for all pad
devices known to it based on vid/pid and a "* Pad" name match. That does not
cover devices not in libwacom. libinput relies on ID_INPUT_TABLET_PAD to
initialize the pad backend.

We can't drop ID_INPUT_TABLET without breaking userspace, but we can add
ID_INPUT_TABLET_PAD ourselves - where a device has BTN_0 in addition to
BTN_STYLUS, let's add it as a pad.

There are some devices (notably: bamboos) that use BTN_LEFT instead of BTN_0
but they are relatively rare and there's a risk of mislabeling those devices,
so let's just stick with BTN_0 only.

Fix compilation without libcryptsetup (#17071)

Merge pull request #17059 from yuwata/network-mdb-follow-ups

network: follow ups for bridge MDB support

Merge pull request #17080 from poettering/repart-umask-fix

repart: don't mark image files generated with --empty=create executable

Merge pull request #17067 from keszybz/rc-local-path

Expose the path to rc.local in the man page

repart: don't mark image files generated with --empty=create executable

Merge pull request #17076 from poettering/dissect-cleanup

minor cleanups to the dissector code

test-execute: kill Waldo and Quux

Retire /var/lib{,/private}/{quux,waldo} after they have served their purpose.

test-network: add test for [BridgeMDB] section

network: add debug message for configuring MDB entries

network: configure bridge MDB entries after bridge has carrier

network: drop [BridgeMDB] entries if Bridge= is not set

network: check MulticastGroupAddress= is neither a local multicast address nor all nodes address

in-addr-util: introduce in4_addr_is_local_multicast()

in-addr-util: introduce in6_addr_is_link_local_all_nodes()

network: drop unnecessary headers

network: move functions in networkd-mdb.c

Merge pull request #17066 from keszybz/allow-loopback-addresses

Allow loopback addresses

Merge pull request #16675 from topimiettinen/exec-syscall-error-action

SystemCallFilter, SystemCallErrorAction error actions kill and log

man: reorder autogenerated dbus api lists

3e5f04bf6468fcb79c080f02b0eab08f258bff0c was trying to do the right thing, but
the resulting list does not match the autogenerated order (which is the same as
the order in vtable definition). I assume the addition was done manually. Fix
the order so that dbus-docs-fresh test is not unhappy.

gpt: move GPT_LINUX_GENERIC definition next to the user GPT partition types

No code changes, just some reordering.

dissect: introduce PartitionDesignator as real type

dissect: rename mount_options_from_part() → mount_options_from_designator()

After all, it actually takes a designator argument, not a partition
index or so.

man: substitute path to rc.local in the man page

Different systems use different paths for it and users are confused when the
man page has a path different than the one on the local system.

https://bugzilla.redhat.com/show_bug.cgi?id=1876905

test: remove test-dissect-image

It does pretty much exactly what systemd-dissect does and is a manual
test, hence let's remove it as redundant code.

meson: RC_LOCAL_SCRIPT_PATH_START to RC_LOCAL_PATH

RC_LOCAL_SCRIPT_PATH_START and RC_LOCAL_SCRIPT_PATH_STOP were was originally
added in the conversion to meson based on the autotools name. In
44508946534eee032927c263b79464832656dd6e RC_LOCAL_SCRIPT_PATH_STOP was dropped.
We don't need to use such a long name.

socket-proxy: allow localhost addresses

With this change, only manager_connect() in timesync.d uses AI_ADDRCONFIG. There
we are connecting to a remove server, so the flag is appropriate.

sd-bus: allow localhost addresses

By settings AI_ADDRCONFIG in hints we cannot for example resolve "localhost"
when the local machine only has a loopback interface. This seems like an
unnecessary restriction, drop it.

Inspired by https://bugzilla.redhat.com/show_bug.cgi?id=1839007.

fstab-generator: add 'nofail' when  NFS 'bg' option is used

Otherwise a 'Before=remote-fs.target' dependency is added to the mount unit
which defeats the purpose of 'bg' option.

tests: try to allocate a delegated scope unit for our cgroup tests

Prompted-by: #16632

exec: SystemCallLog= directive

With new directive SystemCallLog= it's possible to list system calls to be
logged. This can be used for auditing or temporarily when constructing system
call filters.

---
v5: drop intermediary, update HASHMAP_FOREACH_KEY() use
v4: skip useless debug messages, actually parse directive
v3: don't declare unused variables with old libseccomp
v2: fix build without seccomp or old libseccomp

exec: Add kill action to system call filters

Define explicit action "kill" for SystemCallErrorNumber=.

In addition to errno code, allow specifying "kill" as action for
SystemCallFilter=.

---
v7: seccomp_parse_errno_or_action() returns -EINVAL if !HAVE_SECCOMP
v6: use streq_ptr(), let errno_to_name() handle bad values, kill processes,
 init syscall_errno
v5: actually use seccomp_errno_or_action_to_string(), don't fail bus unit
parsing without seccomp
v4: fix build without seccomp
v3: drop log action
v2: action -> number

busctl: add missing shortopt -l

core: use strv_free_and_replace() at one more place

Merge pull request #16998 from zonque/networkd/mdb

network: add support for setting MDB entries

socket-util: reset length argument for second getsockopt call in fd_[gs]et_rcvbuf

In case the first getsockopt() call changes the value.

Merge pull request #16976 from keszybz/systemctl-service-log-levels

systemctl: add service-log-{level,target} verbs

Merge pull request #17055 from keszybz/two-coverity-fixes

Two coverity fixes

Merge pull request #17049 from mrc0mmand/code-and-spell-check

tree-wide: assorted cleanups/fixes

basic/copy: vodify unlinkat() call

Not much we can do at this point.

Coverity CID#1432670.

test: use pclose() for popen()

Coverity CID#1432701.

systemctl: give a nice hint about org.freedesktop.LogControl1 when applicable

Merge pull request #17031 from poettering/path-start-limit

core: propagate start limit hit from triggered unit to path unit

Merge pull request #17039 from poettering/dbus-default-dep

tweak when we synthesize dbus deps for service units

man: rework UMask= explanation

Mention the JSON user record stuff. Mention pam_umask explicitly.
Mention that UMask= of the per-user user@.service instance can be used
too.

Fixes: #16963

tree-wide: don't needlessly negate error number passed to bus_error_message()

Like it's customary in our codebase bus_error_message() internally takes
abs() of the passed error anyway, hence no need to explicitly negate it.
We mostly got this right, but in too many cases we didn't. Fix that.

bootctl: handle if LoaderSystemToken is invalid for some reason

This points to some firmware issue, but we should still be able to
correct things if this happens. Hence log about this and fix it.

Fixes: #17041

Merge pull request #17027 from poettering/env-escape-fix

make sure we our env file writer, parser and shell agree on things

shared: drop a redundant if statement

libsystemd: drop a redundant if statement

home: return SYNTHETIC_ERRNO() when appropriate

basic: drop a redundant if statement

core: consolidate alloc & put operations into one statement

tree-wide: fix typos found by codespell

Reported by Fossies.org

unit-def: drop pointless 0 initialization of first enum value

This is implied in C and we generally don't bother with this, so don't
bother with this here either.

core: propagate unit start limit hit state to triggering path unit

We already do this for socket and automount units, do it for path units
too: if the triggered service keeps hitting the start limit, then fail
the triggering unit too, so that we don#t busy loop forever.

(Note that this leaves only timer units out in the cold for this kind of
protection, but it shouldn't matter there, as they are naturally
protected against busy loops: they are scheduled by time anyway).

Fixes: #16669

core: propagate triggered unit in more load states

In 4c2ef3276735ad9f7fccf33f5bdcbe7d8751e7ec we enabled propagating
triggered unit state to the triggering unit for service units in more
load states, so that we don't accidentally stop tracking state
correctly.

Do the same for our other triggering unit states: automounts, paths, and
timers.

Also, make this an assertion rather than a simple test. After all it
should never happen that we get called for half-loaded units or units of
the wrong type. The load routines should already have made this
impossible.

test-fileio: test test strings with shell, too

test-env-file: add test that tests our env file read + writer + shell against each other

Should hopefully make regressions on this unlikely.

env-file: bring our decoding of double-quoted strings in env files in line with shell

In shell, inside of double quotes only a select few chars should be
escaped. If other chars are escaped this has no effect. Correct the list
of chars that need such escaping.

Also, make sure we can read back the stuff we wrote out without loss.

Fixes: #16788

test-fileio: use test strings that are actually valid in shell

service: add implicit dbus deps only for Type=dbus units

We want to be able to use BusName= in services that run during early boot
already, and thus don't synthesize deps on dbus there. Instead add them
when Type=dbus is set, because in that case we actually really need
D-Bus support.

Fixes: #17037

core: don't warn if BusName= is used for non-Type=dbus services

It's useful for more than just Type=dbus now, given #16976. Hence, let's
drop the warning.

man: document that RemainAfterElapse=no means that also the triggered unit needs to deactivate

Fixes: #16378

systemctl: immediately reject invalid log levels

Symbolic names and number in the appropriate range are allowed
(log_level_from_string() DTRT already).

The target names are more messy, so we leave the verification to the service.

systemctl: merge log_target(), log_level(), service_log_setting()

systemctl: add service-log-{level,target} verbs

Heavily inspired by #15622. This adds:
  systemctl service-log-level systemd-resolved
  systemctl service-log-level systemd-resolved info
  systemctl service-log-target systemd-resolved
  systemctl service-log-target systemd-resolved console

We already have systemctl verbs log-level, log-target, and service-watchdogs.
Those two new verbs tie nicely into this scheme.

Merge pull request #17009 from poettering/rootprefix-noslash

remove duplicate slashes in systemd-path output if rootprefixdir is "/"

hashmap: make sure to initialize shared hash key atomically

if we allocate a bunch of hash tables all at the same time, with none
earlier than the other, there's a good chance we'll initialize the
shared hash key multiple times, so that some threads will see a
different shared hash key than others.

Let's fix that, and make sure really everyone sees the same hash key.

Fixes: #17007

socket: downgrade log warnings about inability to set socket buffer sizes

In containers we might lack the privs to up the socket buffers. Let's
not complain so loudly about that. Let's hence downgrade this to debug
logging if it's a permission problem.

(This wasn't an issue before b92f350789e33942be0cf85af22a580c1fd483d6
because back then the failures wouldn't be detected at all.)

test-acl-util: skip test if /tmp doesn't do ACLs

Fixes: #17013

mkosi: include libfido2 in fedora mkosi image

We want to build this image with as many deps turned on as we can make
happen, hence include libfido2 in the image.

Merge pull request #17028 from poettering/logind-replace-job

logind: make TerminateSession() count

network: do not add prefix to RA if radv is not configured

Fixes #17017.

cryptsetup: generate the unit to umount keydev filesystem

Previously we would call umount from ExecStartPost= of
systemd-cryptsetup instance in order to get rid of the keydev
mount (i.e. filesystem containing keyfile). Let's generate unit to
handle umount. Making this symmetrical (both mount and umount of keydev
are handled by units) fixes the problem with lingering keydev mounts.

Motivation for the change is the issue where keydev mount would stay
around even if device was successfully unlocked and mount is no longer
needed. That could happen previously because when generator options are
not prefixed with "rd." we run generators twice (e.g. rd.luks.key=...).

In such case disk is unlocked in initramfs phase of boot (assuming the
initrd image contains the generator and is able to handle unlocking of
LUKS devices). After switchroot we however enqueue start job for
systemd-cryptsetup instance (because units are regenerated second time)
and that pulls in its dependencies into transaction. Later the main
systemd-cryptsetup unit not actually started since it is already active
and has RemainaAfterExit=yes. Nevertheless, dependencies get activated
and keydev mount is attached again. Because previously we called umount
from ExecStartPost= of systemd-cryptsetup instance the umount is not
called second time and keydev filesystem stays lingering.

logind: make sure when we are explicitly asked to terminate session/user/seat to use "replace" job mode

Otherwise our request will possibly fail if something else is already
enqeued, but given this is an explicit user request, let's not allow
things to fail.

Fixes: #16702

logind: clarify what the second argument of session_stop() means when calling it

logind: minor simplification

man: document the random delay of persistent timers

The manual states that a persistent timer triggers it's service
immediately on activation to catch up with missed invocations, but since
PR #11608 it is no longer the case if RandomizedDelaySec= is set to a
non-zero value.

test-network: add test for ENOBUFS issue #17012

The issue seems already fixed by PR #16982 and its follow-up commit
4934ba2121d76229659939e19ab7d70a89446629.

unit: declare BusName= in all our units that are on the bus, event if they don't use Type=dbus

This information is always useful, so let's always declare it,
particular in the light of #16976.

Merge pull request #17020 from yuwata/backlight

backlight: validate sysattr value

hostnamed: add various debugging log messages when we determine chassis type

No actual code flow changes, just a bunch of debug messages.

Merge pull request #17005 from poettering/sockopt-ipv4-ipv6

rework how we call per-ipv4/per-ipv6 sockopts that conceptually exist on both

pkg-config: prefix is not really configurable, don't pretend it was

We generally don't support prefix being != /usr, and this is hardcoded
all over the place. In the systemd.pc file it wasn't so far. Let's
adjust this to match the rest of the codebase.

sd-path: use ROOTPREFIX without suffixed slash

We already have rootprefix_noslash as meson variable, export it so that
we can also use it in C code.

Fixes: #16773

backlight: validate read sysattr value

If actual_brightness is larger than max_brightness, then fall back to
use brightness attribute.

Also, if the saved value is invalid, then this makes remove the file in
/var/lib/systemd/backlight.

Hopefully fixes #17011.

backlight: do not claim that ID_BACKLIGHT_CLAMP= property is not set

sd-device: introduce sd_device_set_sysattr_valuef()

man: always document both the ipv4 and the ipv6 sockopt

tree-wide: add helper for IPv4/IPv6 sockopts

A variety of sockopts exist both for IPv4 and IPv6 but require a
different pair of sockopt level/option number. Let's add helpers for
these that internally determine the right sockopt to call.

This should shorten code that generically wants to support both ipv4 +
ipv6 and for the first time adds correct support for some cases where we
only called the ipv4 versions, and not the ipv6 options.

timesyncd: don't attempt to call IP_TOS sockopt on IPv6 sockets

hostnamed: map 'all-in-one' DMI chassis type to desktop

Merge pull request #16933 from poettering/copy-hardlinks

copy: optionally recreate hardlinks when copying file trees

test: add BridgeMDB to fuzz-network-parser/directives.network

man: document new Network.BridgeMDB config options

network: add support for setting MDB entries

Multicast snooping enabled bridges maintain a database for multicast
port memberships to decide which mulicast packet is supposed to
egress on which port.

This patch teaches networkd to add entries to this database manually
by adding `[BridgeMDB]` sections to `.network` configuration files.

Merge pull request #16947 from keszybz/socket-parsing-rework

Socket parsing rework

Merge pull request #17003 from yuwata/conf-parser-downgrade-log-level

tree-wide: downgrade log level in conf-parsers

sd-netlink: add MDB types

These are used to configure multicast membership entries of bridge ports.