Merge pull request #10967 from evverx/check-directives

travis: make sure that *.perf and directives.* files are in sync

tests: update test/fuzz/fuzz-netdev-parser/directives.netdev

This is a follow-up to 2266864b04257d7a.

travis: make sure that *.perf and directives.* files are in sync

New features are constantly added to networkd. Apparently, not everybody
knows that the "directives" files should be updated too to make
the fuzzers aware of them.

tree-wide: more IOVEC_MAKE() conversions

test: make TEST-27 non-racy

Not sure how I missed this, but we of course need to wait for the
"systemd-run" commands to finish before we can check the output files
this generated.

Merge pull request #10952 from evverx/keep-fuzz-udev-rules-going

tests: make fuzz-udev-rules work also in the environment created by run_minijail

tests: suppress "unwanted log lines" in several fuzzers

According to https://oss-fuzz.com/fuzzer-stats/by-fuzzer/fuzzer/libFuzzer/job/libfuzzer_asan_systemd,
fuzz-network-parser, fuzz-netdev-parser and fuzz-journal-remote produce
a lot of unwanted log lines. Let's set the maximum log level to LOG_CRIT
as we do in the other fuzzers.

tests: make fuzz-udev-rules work also in the environment created by run_minijail

This should close https://oss-fuzz.com/testcase?key=5642013043589120.

See also https://github.com/google/oss-fuzz/issues/1983.

Merge pull request #10951 from thom311/network-dhcp-route-option

add accessor for sd_dhcp_route's "option"

man,factory: update factory config for nsswitch.conf to match the man pages

Also add a note in the man pages to remind people to adjust the factory config
and other man pages at the same time.

Merge pull request #10955 from keszybz/rc-local-more-info

Provide more information in rc-local-generator

rc-local-generator: provide more debugging information

C.f. https://bugzilla.redhat.com/show_bug.cgi?id=1516188.

rc-local-generator: use macro to define main()

Merge pull request #10944 from poettering/redirect-file-fix

StandardOutput=file: fixes

Merge pull request #10902 from poettering/highlight-status

Highlight status

tests: keep going even if setup_mount_namespace has failed

This should address https://github.com/google/oss-fuzz/issues/1983.

network: use sd_dhcp_route_get_option() accessor in link_set_dhcp_routes()

network: add sd_dhcp_route_get_option() accessor

Since sd_dhcp_lease_get_routes() returns the list of all routes,
the caller may need to differenciate whether the route was option
33 (static-routes) or 121 (classless-static-route).

Add an accessor for the internal field.

Merge pull request #10813 from poettering/cgroup-exec-start-pre

make sure ExecStartPre= and Delegate=yes mix well

tree-wide: use IOVEC_MAKE() at many places

test: add a test for StandardError=file:…

This deserves a test of its, given how broken on so many levels this
previously was.

core: be more careful when inheriting stdout fds to stderr

We need to compare the fd name/file name if we inherit an fd from stdout
to stderr. Let's do that.

Fixes: #10875

bus-unit-util: properly accept StandardOutput=append:… settings

dbus-execute: generate the correct transient unit setting

dbus-execute: fix indentation

core: drop references to 'StandardOutputFileToCreate'

This property never existed, let's drop any reference to it.

Merge pull request #10949 from yuwata/man-improve-unset

man: slightly update man pages of network related units

man: update explanation for boolean options

man: improve the explanation of "Defaults to unset"

test: add test for setting service manager environment variables

This is a follow-up for #10594. I somehow forgot to commit this when I
worked on that.

Merge pull request #10947 from poettering/sd-radv-fixes

sd-radv fixes

journald: don't try to rotate user journals in /var/log/journal/ if we are still in log-to-runtime-journal mode

Fixes: #10879

networkd: Add support for ERSPAN tunnel

Please see: https://patchwork.ozlabs.org/patch/800327/
```
[NetDev]
Name=erspan-test
Kind=erspan

[Tunnel]
Independent=true
ERSPANIndex=123
Local = 172.16.1.200
Remote = 172.16.1.100
Key=101
SerializeTunneledPackets=true
```

First restructuration of Logitech mice in 70-mouse.hwdb

Untested, just carefully copy and pasted under the created sections. First approach to #10941

sd-radv: use struct initializers

sd-radv: per our CODING_STYLE don't use 'bool' in public headers

sd-radv: no need to initialize r to zero

sd-radv: make sure we can deal with in_addr_to_string() failing

sd-radv: when receiving a too short packet, make sure we dequeue it

sd-radv: propagate OOM, it's reason to fail

sd-radv: follow our usualy rule that destructors can take NULL

sd-radv: simplify IPv6 address is-null check

sd-radv: drop const from non-ptr argument

sd-radv: use ether_addr_is_null() where appropriate

units: make fsck/grows/makefs/makeswap units conflict against shutdown.target

They are the only units we shipped/generated where this was missing really. Let's fix these.

Follow-up for: #10933

tree-wide: use SWAP_TWO a bit more

test-network: add tests for #5430 and GatewayOnLink=

Merge pull request #10928 from yuwata/fix-9940

network: also load foo.netdev.d/*.conf

man: link Delegate= documentation up with the markdown docs

docs: document the .control/ subcgroup contract in the docs

core: when Delegate=yes is set for a unit, run ExecStartPre= and friends in a subcgroup of the unit

Otherwise we might conflict with the "no-processes-in-inner-cgroup" rule
of cgroupsv2. Consider nspawn starting up and initializing its cgroup
hierarchy with "supervisor/" and "payload/" as subcgroup, with itself
moved into the former and the payload into the latter. Now, if an
ExecStartPre= is run right after it cannot be placed in the main cgroup,
because that is now in inner cgroup with populated children.

Hence, let's run these helpers in another sub-cgroup .control/ below it.

This is somewhat ugly since it weakens the clear separation of
ownership, but given that this is an explicit contract, and double opt-in should be acceptable.

Fixes: #10482

unit: tweak status output a bit

Let's highlight the unit description string in the status updates, to
separate them a bit more the english sentence they are part of, and thus
make the different casing less surprising.

show-status: use free_and_replace() where we can

show-status: fold two bool flags function arguments into a flags
parameter

Merge pull request #10919 from yuwata/sd-device-monitor-fixes

sd-device-monitor: several fixes

generator.c: systemd-fsck-root.service conflict with shutdown.target

Otherwise a "reboot" or "poweroff" in the initramfs will have to wait
until systemd-fsck-root.service has completed, which might never happen
if the root device never shows up.

test-network: add one more test for netdev dropin .conf files

For netdev, config files are loaded twice, and the first time,
only Match and NetDev setions are read. So, the test given by
the previous commit covers only the second loading.
This adds another test that also covers the first loading.

Merge pull request #10901 from poettering/startswith-list

add new STARTSWITH_SET() macro

Merge pull request #10927 from yuwata/udev-ctrl-cleanup

udev: update log messages and fix coding style issues

Merge pull request #10925 from yuwata/sd-boot-fixes

sd-boot: fixes trailing whitespace trimming and updates document

Merge pull request #10910 from yuwata/list-cleanups

basic/list: tiny fixes

Merge pull request #10909 from yuwata/import-cleanups

Import: cleanups

Merge pull request #10894 from poettering/root-cgroup-fix

A multitude of cgroup fixes

terminal-util: use PATH_STARTSWITH_SET() instead of plain startswith() where appropriate

After all, we deal with paths here.

journal-upload: use static initialization

journal-remote: shorten code a bit by using strcspn()

tree-wide: port more code over to PATH_STARTSWITH_SET()

path-util: beef up PATH_STARTSWITH_SET() macro a bit

Let's remove duplicate evaluation, and let's return the result of the
succesful path_startswith() call, i.e. the suffix to the matching
prefix.

tree-wide: port over more cases to STR_IN_SET()

tree-wide: port various places over to STARTSWITH_SET()

strv: add new macro STARTSWITH_SET()

This is to startswith() what PATH_STARTSWITH_SET() is to
path_startswith().

Or in other words, checks if the specified string has any of the listed
prefixes, and if so, returns the remainder of the string.

update TODO

tests: add a fuzzer for the udev rules parser (#10929)

test-network: add tests for dropin .conf files

network: also load foo.netdev.d/*.conf

Fixes #9940.

udevd: do not call udev_ctrl_enable_receiving() if the socket is passed from pid1

Before c4b69e990f962128cc6975e36e91e9ad838fa2c4, if the socket fd is
passed from pid1, `udev_ctrl_enable_receiving()` was not called.
Let's preserve the original logic.

udev: drop duplicate whitespaces

udev-ctrl: update log messages

udev-rules: include device sysname in log messages

udev-ctrl: fix coding style isuues

Merge pull request #10926 from AsciiWolf/copyright

Drop my copyright

mkosi: drop my copyright

po: drop my copyright (cs.po)

man: systemd-boot does not read loader.conf.d/*.conf

Fixes #10923.

sd-boot: fix trailing whitespace trimming

Fixes #10922.

hwdb: Add ACCEL_MOUNT_MATRIX for Trekstor T13B tablet

udevd: do not set buffer size if the socket is passed from pid1

Before c4b69e990f962128cc6975e36e91e9ad838fa2c4, if the socket fd is
passed from pid1, `udev_monitor_set_receive_buffer_size()` (now it is
a wrapper of `sd_device_monitor_set_receive_buffer_size()`) was not
called. Let's preserve the original logic.

sd-device: do not call device_monitor_enable_receiving() for passed fd from pid1

Before c4b69e990f962128cc6975e36e91e9ad838fa2c4, if the socket fd is
passed from pid1, `udev_monitor_enable_receiving()` (now it is a wrapper
of `device_monitor_enable_receiving`) was not called.
Let's preserve the original logic.

Fixes #10754.

sd-device: check whether filter is updated or not in sd_device_monitor_filter_update()

sd-device: update log message

sd-device: fix return value on error path

units: order systemd-rfkill.socket after /var/lib/systemd/rfkill (#10904)

Otherwise we might install the socket unit early, but the service
backing it late, and then end up in strange loops when we enter rescue
mode, because we saw an event on /dev/rfkill but really can't dispatch
it nor flush it.

Fixes: #9171

tests: skip test-execute on TravisCI under ASan

Unfortunately, f5f9a580dd4a3cf487b didn't help much and now
the next subtest gets stuck from time to time. Let's skip
test-execute altogether so as not to bother anybody with
spurious failures.

https://github.com/systemd/systemd/issues/10696 is still open.
Everybody is welcome to share ideas :-)

Fix a few docs typos (#10907)

Found with [codespell](https://github.com/codespell-project/codespell)

basic/list: fix minor coding style issue

basic/list: add missing header

Some macros use assert() which is (re)defined in macro.h

import: use structured initializers

import: use _cleanup_ attribute for CURL object

Fix typo in hacking guide: "no possible" -> "not possible" (#10905)

analyze: Fix build without seccomp (#10899)