test-network: show systemd-networkd.service

As there may exist some other drop-in configs.

test-network: stop service before editing unit file

test-network: add --lsan-options option

Merge pull request #12705 from keszybz/varlink-json-fix-and-two-cleanups

Varlink json fix and two cleanups

sd-journal: voidify call

To appease coverity, CID#1400674.

networkd: shorter two error statements a bit

Also let's not use a cast if we don't have to.

basic/memory-util: do not "return" anything from memzero() macro

The macro is not used in expressions, so we don't need the ternary statement.

systemctl: process all units matched by a glob in the cat verb by default

Originally, `systemctl cat` would match only active units, for example:

$ systemctl cat sshd.service

would cat the sshd.service unit file even if the service was inactive.
However:

$ systemctl cat ssh*

would show it only if it was active.

Let's unify the behavior and cat all unit files regardless of a state,
if no state was given explicitly to filter.

shared/varlink: add missing va_end()

Coverity CID#1401347.

Revert "varlink: initialize Varlink with 0"

This reverts commit 8688c29b5aece49805a244676cba5bba0196f509, but leaves the
reproducer. Structured assignment should be enough to fully initialize the
variable and new0 is not necessary.

Merge pull request #12699 from yuwata/test-network-argparse

test-network: make path to networkd or friends configurable

shared/varlink: add missing terminator in json strings

Should finally fix oss-fuzz-14688.
8688c29b5aece49805a244676cba5bba0196f509 wasn't enough.

The buffer retrieved from memstream has the size that the same as the written
data. When we write do write(f, s, strlen(s)), then no terminating NUL is written,
and the buffer is not (necessarilly) a proper C string.

networkd: rewrite condition to make it easier to understand

test-user-util: add simple test for make_salt()

bash-completion: correctly react to an unescaped unit name

Merge pull request #12636 from mbroz/master

Cryptsetup: call crypt_load() once for LUKS and support LUKS2 tokens

networkctl: fix memleak

Closes #12701.

test-network: make the test take ASAN and UBSAN options

test-network: make path to networkd or friends configurable

Merge pull request #12628 from keszybz/dbus-execute

Rework cpu affinity parsing

nspawn: also support ifindex when specifying network interface

Merge pull request #12646 from yuwata/networkctl-use-table

networkctl: use Table

dhcp4: use memdup_suffix0() instead of strndup() for binary data

C.f. ac0a94f7438b49a0890d9806db1fa211a5bca10a.

Fixes #12693.

pager: Fix deadlock when using built-in pager

The parent is waiting for an EOF on the pipe transmitting the pager name
before starting to send data. With external pagers this happens due to
execlp() CLOEXEC'ing the pipe, so the internal pager needs to close it
manually.

cryptsetup: Add dependency for detached header
Fixes #12650

cryptsetup: Extract dependency check to a separate function

cryptsetup: style improvements, error checking
Return early on failure/nothing to do
Removed braces from single-line ifs
Check return value of  fstab_filter_options

cryptsetup: code style

tests: skip test-bpf only when we're 100% sure it's run in containers

This is just a follow-up to https://github.com/systemd/systemd/pull/12617.

Merge pull request #12617 from mbiebl/skip-test-bpf-containers

test-bpf: skip test when run inside containers

shared/cpu-set-util: only force range printing one time

The idea is to have at least one range to make the new format clearly
distinguishable from the old. But it is enough to just do it once.
In particular, in case the affinity would be specified like 0, 2, 4, 6…,
this gives much shorter output.

systemctl: present CPUAffinity mask as a list of CPU index ranges

shared/cpu-set-util: introduce cpu_set_to_range()

test-cpu-set-util: add test for dbus conversions

shared/cpu-set-util: make transfer of cpu_set_t over bus endian safe

Merge pull request #12690 from yuwata/test-network-and-man-netdev

test-network,man: trivial cleanups

hwdb: add axis range corrections for the Asus N53SV

test-network: bump time limit for wait_address()

Maximum time interval of DHCPv4 request is 63sec.
So, setting 100sec is enough.

Merge pull request #12449 from ljmf00/hwdb-accel-location-patch

hwdb: Add support for Teclast F6 Pro base sensor

shared/cpu-set-util: drop now-unused cleanup function

test-execute: use CPUSet too

cpu_set_malloc() was the last user. It doesn't seem useful to keep
it just to save the allocation of a few hundred bytes in a test, so
it is dropped and a fixed maximum is allocated (1024 bytes).

pid1: when reloading configuration, forget old settings

If we had a configuration setting from a configuration file, and it was
removed, we'd still remember the old value, because there's was no mechanism to
"reset" everything, just to assign new values.

Note that the effect of this is limited. For settings that have an "ongoing" effect,
like systemd.confirm_spawn, the new value is simply used. But some settings can only
be set at start.

In particular, CPUAffinity= will be updated if set to a new value, but if
CPUAffinity= is fully removed, it will not be reset, simply because we don't
know what to reset it to. We might have inherited a setting, or we might have
set it ourselves. In principle we could remember the "original" value that was
set when we were executed, but propagate this over reloads and reexecs, but
that would be a lot of work for little gain. So this corner case of removal of
CPUAffinity= is not handled fully, and a reboot is needed to execute the
change. As a work-around, a full mask of CPUAffinity=0-8191 can be specified.

pid1: don't reset setting from /proc/cmdline upon restart

We have settings which may be set on the kernel command line, and also
in /proc/cmdline (for pid1). The settings in /proc/cmdline have higher priority
of course. When a reload was done, we'd reload just the configuration file,
losing the overrides.

So read /proc/cmdline again during reload.

Also, when initially reading the configuration file when program starts,
don't treat any errors as fatal. The configuration done in there doesn't
seem important enough to refuse boot.

pid1: parse CPUAffinity= in incremental fashion

This makes the handling of this option match what we do in unit files. I think
consistency is important here. (As it happens, it is the only option in
system.conf that is "non-atomic", i.e. where there's a list of things which can
be split over multiple assignments. All other options are single-valued, so
there's no issue of how to handle multiple assignments.)

test-cpu-set-util: add a smoke test for test_parse_cpu_set_extend()

test-cpu-set-util: add simple test for cpus_in_affinity_mask()

Move cpus_in_affinity_mask() to cpu-set-util.[ch]

It just seems to fit better there and it's always better to have things
in shared/ rather than basic/.

Rework cpu affinity parsing

The CPU_SET_S api is pretty bad. In particular, it has a parameter for the size
of the array, but operations which take two (CPU_EQUAL_S) or even three arrays
(CPU_{AND,OR,XOR}_S) still take just one size. This means that all arrays must
be of the same size, or buffer overruns will occur. This is exactly what our
code would do, if it received an array of unexpected size over the network.
("Unexpected" here means anything different from what cpu_set_malloc() detects
as the "right" size.)

Let's rework this, and store the size in bytes of the allocated storage area.

The code will now parse any number up to 8191, independently of what the current
kernel supports. This matches the kernel maximum setting for any architecture,
to make things more portable.

Fixes #12605.

man: re-indent systemd.netdev.xml

test-network: drop 25-nlmon.network

man: add vrf section

tree-wide: replace if_indextoname() with format_ifname()

util: introduce format_ifname()

networkctl: use Table for 'lldp' command

networkctl: use Table for 'label' command

networkctl: use Table for 'status' command

networkctl: use Table for 'list' command

table: add TABLE_IFINDEX type

table: add more basic types

table: add teble_add_string_cell()

Merge pull request #12687 from yuwata/parse_ifindex_or_ifname

util: introduce parse_ifindex_or_ifname() and use it in networkctl and resolvectl

networkctl: use parse_ifindex_or_ifname()

resolvectl: use parse_ifindex_or_ifname()

util: add assertions

util: introduce parse_ifindex_or_ifname()

networkctl: add support to delete virtual netdevs

We now don't have the support to delete netdevs and dependent
upon iproute to delete. With this we can delete via networkctl
and use in our test cases too.

Note that it supports deleting multiple links at once.

```
 sudo ./networkctl delete test1 test2 test3 test4
```

Merge pull request #12635 from yuwata/nlmon-support

network: add nlmon support

Merge pull request #12573 from yuwata/network-dhcp-issue-12490

network: set preferred source in removing route entry on address lost

systemd-sleep: replace rtc wakealarm with CLOCK_BOOTTIME_ALARM s2h

refactor to use timerfd in place of rtc wakealarm

confirm CLOCK_BOOTTIME_ALARM support in can_s2h

Remove CLOCK_BOOTTIME_ALARM task from TODO

remove unnecessary check on clock_supported return

seccomp: add new 5.1 syscall pidfd_send_signal() to filter set list

test: exit cleanly on success

cryptsetup: Add LUKS2 token support.

LUKS2 supports so-called tokens. The libcryptsetup internally
support keyring token (it tries to open device using specified
keyring entry).
Only if all token fails (or are not available), it uses a passphrase.

This patch aligns the functionality with the cryptsetup utility
(cryptsetup luksOpen tries tokens first) but does not replace
the systemd native ask-password function (can be used the same in
combination with this patch).

test-network: add a basic test for nlmon

network: add nlmon support

nlmon is a Netlink monitor device.

network: add NetDevVTable::generate_mac flag

cryptsetup: call crypt_load() for LUKS only once

The crypt_load() for LUKS2 can read a quite big area of disk
(metadata area size is configurable and can increase up to megabytes).

This initialization is not needed to be repeated, just use the existing context.

(This patch is also required for the following change.)

cryptsetup: Do not fallback to PLAIN mapping if LUKS data device set fails.

If crypt_load() for LUKS succeeds, we know that it is a LUKS device.
Failure of data device setting should fail in this case; remapping
as a PLAIN device late could mean data corruption.

(If a user wants to map PLAIN device over a device with LUKS header,
it should be said explicitly with "plain" argument type.)

Also, if there is no explicit PLAIN type requested and crypt device
is already initialized (crypt_data_type() is set), do not run
the initialization again.

network: do not send DHCPRELEASE when CriticalConnection= is enabled

test-network: use wait_online() in NetworkdDHCPServerTests or friends

test-network: move test_domain() to NetworkdNetworkTests class

test-network: rename needlessly long class names

test-network: use wait_online() instead of waiting for 10 seconds

But some tests requires both IPv4 and IPv6 addresses are configured.
Only in that case wait for addresses to be configured.

test-network: drop unused link

test-network: rename link_remove() to remove_links()

test-network: remove routes even on failure

test-network: use setUp() and tearDown() to delete routing policy rule tables

test-network: remove fou ports on failure

test-network: stop dnsmasq before removing links

test-network: warn when firewalld.service is running

test-network: add tests for issue #12490

network: lower log level about critical connection

network: split dhcp_lease_lost() into small pieces

network: set dhcp4_configured flag false before configuring address

Otherwise, even if dhcp_lease_acquired() fails, the flag is still on.

network: reset Link::dhcp4_configured flag earlier

network: set preferred source in removing route entry on address lost

Closes #12490.

Merge pull request #12437 from poettering/chmod-and-chown-rewrite

chmod_and_chown() rewrite

Merge pull request #12390 from poettering/string-file-mkdir

fileio: add a WRITE_STRING_FILE_MKDIR_0755 flag to write_string_file() that creates parent directories if needed

Merge pull request #12392 from poettering/firstboot-salt

generalize crypt_r() salt generation code

Merge pull request #12597 from keszybz/analyze-timestamp

Analyze timestamps

Merge pull request #12682 from keszybz/pull-in-user-targets

rules: enable hardware-related targets also for user instances

NEWS: add some entries

For #12680, #12571.

capability-util: be more careful with types

Merge pull request #12679 from yuwata/journal-issue-12400

journal: do not trigger assertion when journal_file_close() get NULL