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

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.

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

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()

tree-wide: use strv_free_and_replace() macro

time-util: fix indentation for comments

Merge pull request #8923 from yuwata/resolvectl-drop-funcs

resolvectl: drop service_family_{from,to}_string()

Merge pull request #8938 from keszybz/sd-bus-automatic-cleanup

Use automatic cleanup more in sd-bus

use max. message size allowed by DBus spec (#8936)

C.f. https://dbus.freedesktop.org/doc/dbus-specification.html#message-protocol-messages.

Merge pull request #8934 from yuwata/fix-8913

 bus-util: print a friendly message when PID1 is not systemd

bus-util: print a friendly message when PID1 is not systemd

Follow-up for 861f16d2679eeda79e8185057cef24653913e300.

Fixes #8913.

analyze: merge acquire_full_bus() and acquire_systemd_bus()

Follow-up for 5c69b31c137ffa24bca5a4628f9b4879d0f4e84a.

sd-bus: add bus_freep and use _cleanup_

sd-bus: use automatic cleanup more

sd-bus: trivial simplification

man: fix invalid option name --devpath in 'udevadm info' (#8935)

man: --debug option is implied in udev test and test-builtin commands (#8933)

Merge pull request #8902 from yuwata/link-multiple-mac

network,link: make MACAddress= in [Match] section can take multiple MAC addresses

network,udev: sort included headers

man: document that MACAddress= in [Match] section can take multiple MAC addresses

network,udev: make MACAddress= in [Match] section take multiple MAC addresses

test: add tests for config_parse_hwaddrs()

libsystemd-network: introduce config_parse_hwaddrs()

ether-addr-util: add hash_ops functions for struct ether_addr

ether-addr-util: make ether_addr_from_string() stricter

meson: recompile all sources for install_libudev_static and install_libsystemd_static

This means that when those targets are built, all the sources are built again,
instead of reusing the work done to create libbasic.a and other convenience static
libraries. It would be nice to not do this, but there seems to be no support in
our toolchain for joining multiple static libraries into one. When linking
a static library, any -l arguments are simply ignored by ar/gcc-ar, and .a
libraries given as positional arguments are copied verbatim into the archive
so they objects in them cannot be accessed.

https://stackoverflow.com/questions/2157629/linking-static-libraries-to-other-static-libraries
suggests either unzipping all the archives and putting them back togather,
or using a linker script. Unzipping and zipping back together seems ugly.
The other option is not very nice. The linker script language does not
allow "+" to appear in the filenames, and filenames that meson generates
use that, so files would have to be renamed before a linker script was used.
And we would have to generate the linker script on the fly. Either way, this
doesn't seem attractive. Since those static libraries are a niche use case,
it seems reasonable to just go with the easiest and safest solution and
recompile all the source files. Thanks to ccache, this is probably almost as
cheap as actually reusing the convenience .a libraries.

test-libsystemd-sym.c and test-libudev-sym.c compile fine with the generated
static libs, so it seems that they indeed provide all the symbols they should.

meson: only build test-lib{systemd,udev}-static-sym for pic

meson: add test-lib{systemd,udev}-static-sym

This is the same as test-lib{systemd,udev}-sym, but linked to the static
variants of those libraries.

meson: add support for building static libsystemd and libudev

tools/oss-fuzz: install private shared library non-executable (#8927)

Apparently oss-fuzz's "bad build check" is confused by the library.
Let's make it non-executable, so the checker ignores it.

Should fix https://github.com/google/oss-fuzz/issues/1330.

man/systemd.special: fix network v.s. LSB scripts (#8930)

$network is converted to network-online.target, not network-target.

See https://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/
and the implementation at `src/sysv-generator/sysv-generator.c`.

Merge pull request #8915 from yuwata/fix-8904

 login: do not wall message on cancelling shutdown when Manager.enable_wall_messages is false

link: Add support to configure channels of the specified network device (#8882)

closes #8856

Merge pull request #8898 from poettering/nspawn-mount-block

some nspawn cgroup and mount lock-down fixes

Merge pull request #8926 from keszybz/man-journal-remote

Small additions to sd-journal-{remote,upload}.service man pages

man: fix title of systemd-networkd-wait-online(8) (#8925)

resolvectl: drop service_family_{from,to}_string()

These functions are used only for checking the validity of input string.
Hence, this drops them and introduces a simple checking function.

man: add missing 'query' command in examples of resolvectl

man: add a bit more text about systemd-journal-upload.service

man: add a bit more text about systemd-journal-remote.service

Merge pull request #8922 from yuwata/fix-8920-8921

man: update references to systemd-journal-{remote,upload}

man: fix typo

man: mention that --url option of journal-upload can specify the port number

man: mention the default port number of journal-remote

man: update references to systemd-journal-{remote,upload}

Fixes #8920 and #8921.

Eliminate config_dirs vars which hold a static strv