core: undo the dependency inversion between unit.h and all unit types

core: Break circular dependency between unit.h and cgroup.h

udev: create /dev/disk/by-label symlink for LUKS2 (#8998)

LUKS2 header supports to device label and blkid since 2.32 version
already supports this option.

Persistent udev storage rules should create symlink for this label.

For older devices this value is not set so changed rule should be compatible.

sysusers: drop an unused variable

Follow-up for ec0327d69c260d9a3770e9d722dbdcde996651cc.

service: FileDescriptorStoreMax should also imply NotifyAccess

Commenting out "WatchdogTimeout=3min" in systemd-logind.service causes
NotifyAccess to go from "main" to "none", breaking support for logind
restart.  Let's fix that.

bash-completion: add missing options and commands of timedatectl

Follow-up for 6129ec852ee470a3682d55f87852ee7ccabb5520.

Merge pull request #8947 from yuwata/meson-0.44

meson: bump minimum required version to 0.44

man: fix reference in StandardOutput=

Since StandardOutput=file:path is more similar to StandardInput= than
StandardInputText=, and only StandardInput= is actually documented above
StandardOutput= whereas StandardInputText= is documented below it, I
assume the intention was to refer to the former.

Merge pull request #8986 from yuwata/sd-bus-free-and-strdup

several cleanups

test: use _cleanup_hashmap_free_ instread of _cleanup_(hashmap_freep)

core: use _cleanup_set_free_ instread of _cleanup_(set_freep)

sd-bus: use free_and_strdup()

Merge pull request #8968 from yuwata/bus-macro-2

bus-util: add several macros for defining functions of getting dbus properties

man: drop a trailing space

test: add tests for unit_name_from_dbus_path()

Merge pull request #8962 from floppym/issue8905

basic: timezone_is_valid: check for magic bytes "TZif"

dbus-manager: introduce property_get_{hashmap,set}_size()

dbus-unit: check userdata before obtaining data

dbus-unit: use BUS_DEFINE_PROPERTY_GET* macros

dbus-swap: use BUS_DEFINE_PROPERTY_GET* macros

dbus-socket: use BUS_DEFINE_PROPERTY_GET* macros

dbus-mount: use BUS_DEFINE_PROPERTY_GET* macros

dbus-manager: use BUS_DEFINE_PROPERTY_GET* macros

dbus-execute: use BUS_DEFINE_PROPERTY_GET* macros

bus-util: add several macros for defining functions of getting dbus properties

basic: add log_level argument to timezone_is_valid

Merge pull request #8959 from yuwata/bash-completion-analyze

bash-completion: update completion for systemd-analyze

Merge pull request #8966 from yuwata/bus-resolve-fallback-dns

resolve: add bus property for fallback DNS servers

core: use automatic cleanup more

po: uk: fixed missed word

updated uk-translation for 'network time synchronization' construction.

Updated uk translation

basic: timezone_is_valid: check for magic bytes "TZif"

Fixes: https://github.com/systemd/systemd/issues/8905

systemd-analyze: make dump work for large # of units

If there is a large number of units, the size of the generated dump
string can overstep DBus message size limit. So let's pass that string
via a fd.

man: Fix a minor typo in systemctl(1)

Signed-off-by: Philip Withnall <withnall@endlessm.com>

Merge pull request #8924 from yuwata/fix-3682

resolve: allow whitespaces in the digest

Merge pull request #8961 from yuwata/fix-8960

network: make route_update() accept NULL

resolvectl: show current DNS server only when it is set

resolvectl: show fallback DNS servers by status command

resolve: introduce FallbackDNS bus property

This also makes DNS bus property not include the fallback DNS servers.

network: make route_update() accept NULL

This also fixes a wrong argument for route_configure().

Fixes #8960.

network: fix indentation

tmpfiles: remove newline that shouldn't be there

We should really keep "else" branches together with their "if", hence
drop the double newline here...

terminal-util: add a function that shows a pretty separator line

Follow-up for #8824

test: do not call alloca() inside the list of arguments

Merge pull request #8953 from yuwata/bus-macro

core: simplify dbus properties

test-execute: remove state directories before running tests

Fixes #8679.

units: set RestrictNamespaces=yes to networkd and resolved

Closes #8949.

sd-dhcp: drop deserialize_dhcp_option() as it is equivalent to unhexmem()

resolve: allow whitespaces in the DS digest or DNSKEY key data

Fixes #3682.

hexdecoct: drop an unnecessary check

basic: fix comment style

hexdecoct: ignore whitespace within the input hexadecimal text of unhexmem()

test: add testcase for environment file variables with backslashes

systemd silently strips out backslashes in variables from environment
files. Add a testcase that explicitly tests for this behaviour.

Merge pull request #8964 from MikhailKasimov/patch-2

man: systemd-journal-upload: rsa 2048

Update systemd-journal-upload.service.xml

Security note on 2048 bits is added.

man: systemd-journal-upload: rsa 2048

rsa:2048 is currently in use as minimal recommended key length on cert/key generating. Specifying rsa:1024 is not recommended for security reasons.

bash-completion: analyze: support cat-config verb

Follow-up for 854a42fb2e9db1b9eaa381559d7671f2e9b3a0f1.

bash-completion: analyze: add --no-pager for some verbs

Merge pull request #8824 from keszybz/analyze-show-config

systemd-analyze show-config

mtd: some basic code cleanups

While looking at our exit() invocations I noticed that the mtd_probe
stuff uses 'exit(-1)' at various places, which is not really a good
idea, as exit codes of processes on Linux are supposed to be in the
range of 0…255.

This patch cleans that up a bit, and fixes a number of other things:

1. Let's always let main() exit, nothing intermediary. We generally
   don't like code that invokes exit() on its own.

2. Close the file descriptors opened.

3. Some logging for errors is added, mostly on debug level.

Please review this with extra care. As I don't have the right hardware
to test this patch I only did superficial testing.

Networkd tests: add bridge link properties to test

Test the "[Bridge]" section keys

```
[Bridge]
UnicastFlood=true
HairPin=true
UseBPDU=true
FastLeave=true
AllowPortToBeRoot=true
Cost=555
Priority=23
```

```
test_bridge_init (__main__.BridgeTest) ... ok
test_bridge_port_priority (__main__.BridgeTest) ... ok
test_bridge_port_priority_set_zero (__main__.BridgeTest)
It should be possible to set the bridge port priority to 0 ... ok
test_bridge_port_property (__main__.BridgeTest)
Test the "[Bridge]" section keys ... ok

```

man: journal-gatewayd.service: links on {rem,upl}

While set of systemd-journal-{gatewayd,remote,upload}.service services presents single subsystem on journald logs network transmission, systemd-journal-gatewayd.service description should also contain links to other parts of this subsystem: systemd-journal-remote.service and systemd-journal-upload.service.

string-util: rename strdash_if_empty() to empty_to_dash()

path-util: introduce empty_to_root() and use it many places

core: merge duplicated functions

core: send NULL instead of empty string

core: use offsetof() for Syslog{Level,Facility} dbus properties

Merge pull request #8950 from dtardon/cleanup

use automatic cleanup more

github: use multiple issue templates

Github now has issue templates in the web interface, and allows
more than one to be specified. Let's split our single template
in two: bug report and RFE.

Merge pull request #8943 from keszybz/coverity-fixes

Coverity fixes

core: use bus_property_get_*() functions instead of NULL

sd-bus: use automatic cleanup more

core: drop property_get_syscall_errno()

core: drop property_get_{capability_bounding_set,ambient_capabilities}()

core: use BUS_DEFINE_PROPERTY_GET_ENUM() macro

core: drop 'bus_' prefix from bus_property_get_protect_{home,system}()

core: simplify property_get_cpu_affinity()

basic: use automatic cleanup more

basic: use automatic cleanup more

basic: use automatic cleanup more

journal: use automatic cleanup more

doc: mention the required minimum version of meson and python

meson: use array type option

Array type option is supported since 0.44.0.

meson: use warning() method

This bumps the required minimum version of meson to 0.44, as
`warning()` method is supported since 0.44.

meson: use get_supported_arguments()

This bumps the required minimum version of meson to 0.43, as
`get_supported_arguments()` is supported since meson-0.43.

man: systemd-networkd-wait-online: systemd.service (#8945)

* man: systemd-networkd-wait-online: systemd.service

While service type is mentioned (is a oneshot system service), link on systemd.service is added. 'See Also' section is also updated with link on systemd.service man-page.

man: systemd-escape: add missed short keys (#8944)

Added short keys -u and -m for --unescape and --mangle respectively. These short keys are present in systemd-escape --help output and are absent in man systemd-escape page.

conf-parser: accept trailing backslash at the end of the file (#8941)

This makes it behave the same whether there is a blank line or not at
the end of the file.  This is also consistent with the behavior of the
shell on a shell script that ends on a trailing backslash at the last
line.

Added tests to test_config_parse(), which only pass if the corresponding
change to config_parse() is included.

add journal-upload.conf refentrytitle (#8942)

Add journal-upload.conf refentrytitle to have the same format to systemd-journal-remote.service description, which contains refentrytitle on journal-remote.conf in 'See Also' section.

core/mount-setup: remove part of check which is always true

k was set to join_controllers at this point and only incremented, so
it cannot be null at this point.

CID #1390949.

basic/fs-util: remove logically dead code

We can jump to chase_one from two places. In the first 'todo' is set to
'buffer', which comes from path_make_absolute_cwd() and is nonnull In the
second 'todo' is set to 'joined' which is checked to be nonull a few lines
above the jump. So let's kill the code that deals with null todo there.

CID #1390941.

udevadm: fix null dererefence on allocation error

CID #1390936.

localed: fix memleak in error path

CID #1390929.

logind: fix borked r check

CID #1390947, #1390952.

basic/format-table: add missing va_end()

CID #1390930, #1390940.

Introduce _cleanup_(strbuf_cleanupp) and use it to fix null deref on error

catalog_update() would call strbuf_cleanup(NULL) on allocation error.
CID #1390928.

shared/sleep-config: fix memleak of strv, add test

CID #1390921, #1390951.

Merge pull request #8689 from davide125/static

meson: add support for building static libsystemd and libudev

Merge pull request #8939 from yuwata/fix-comment

several cleanups

timedate: use free_and_strdup()