test: avoid using pipes in certain cases

`command -v <bin> | grep ...` can under certain conditions cause the
`command` to exit with SIGPIPE, which in combination with `set -o
pipefail` means that the tests sometimes randomly die during setup.
Let's avoid using pipes in such cases.

test: use set -o pipefail

This breaks some existing loops which previously ignored if the piped
program exited with EC >0. Rewrite them to mitigate this (and also make
them more robust in some cases).

test: shell code cleanup

Just some basic stuff, like double quotes around strings, using
predictable values for possibly unset variables, formatting, etc.

fileio: use take_fdopen_unlocked()

This fixes maybe-uninitialized warning:
```
../src/basic/fileio.c: In function ‘chase_symlinks_and_fopen_unlocked’:
../src/basic/fileio.c:1026:19: warning: ‘f’ may be used uninitialized in this function [-Wmaybe-uninitialized]
 1026 |         *ret_file = f;
      |         ~~~~~~~~~~^~~

```

Merge pull request #19332 from yuwata/log_xxx_yyy_errno-follow-ups-19317

core,timedate,network: make log_unit_error_errno() or friends refuse zero errno

Merge pull request #19325 from sainAk/patch-1

hwdb: 60-keyboard:: merge Micro Star in MSI and  merge MSI Prestige And MSI Modern FnKeys and Special keys definitions

network: update log message

In some optimization level with gcc, including netdev->ifname in log
message causes error.

Merge pull request #19290 from yuwata/network-dhcp-do-not-configure-twice

network: refuse to configure DHCP client more than once

Merge pull request #19310 from yuwata/network-dhcp-anonymize

network: dhcp4: several fixes and cleanups for Anonymize=

man: correct requirements for systemd-oomd.service

Fixes: #19331

hwdb: merge MSI Prestige And MSI Modern FnKeys and Special keys definitions

'A10' was the model name for prestige series with 10th gen intel processors and 'SC' was the gpu config for a specific model
but now msi has refreshed these series with 11th gen intel processors so the model name becomes 'A11X' where x is gpu config name more info [here](https://www.msi.com/Business-Productivity/Prestige-14-A11X/Specification) and [here](https://www.msi.com/Business-Productivity/Prestige-15-A11X/Specification)
[this was the link for model comparison of 10th gen cpus](https://www.msi.com/Business-Productivity/Prestige-15-A10X/Specification) (Dead link now)
complete list of Prestige series laptops [here](https://www.msi.com/Business-Productivity/Products#?tag=Prestige-Series)

And modern series follows the same naming convention, full list of laptops [here](https://www.msi.com/Business-Productivity/Products#?tag=Modern-Series).

examples

Prestige series
Prestige 14 A10x ( 14inch Intel 10th gen )
Prestige 15 A11x ( 15inch Intel 11th gen )

First gen Modern Series
Modern 15 A10x  ( 15inch Intel 10th gen)
Modern 15 A4x ( 15inch AMD 4000 series)

Second gen Modern Series
Modern 15 B10x  ( 15inch Intel first gen modern laptop)
Modern 14 B4x ( 14inch AMD second gen modern laptop)

all these laptops have the same keyboard layouts with same functionality on windows (can be reviewed from their support manuals).

hwdb: merge Micro Star in MSI

network: make log_netdev_xxx_errno() refuse zero errno

timedate: make log_unit_xxx_errno() refuse zero errno

core: make log_unit_xxx_errno() refuse zero errno

core: use UNIT_TRIGGER at more places

Just some minor simplification.

Merge pull request #19326 from keszybz/reorder-meson-status-output

Reorder meson status output

Merge pull request #19318 from yuwata/udev-ignore-additional-newline-on-verify-19314

udev: ignore additional newline at the end on verify

Merge pull request #19317 from keszybz/check-return-values-from-log_errno-functions

Check return values from log_errno functions

po: Translated using Weblate (Korean)

Currently translated at 100.0% (189 of 189 strings)

Co-authored-by: simmon <simmon@nplob.com>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/master/ko/
Translation: systemd/main

Assert zero is not passed to log_{device,rule,token}_*_errno()

Voidify log_{device,token,rule}_debug()

See analogous change for log_debug() for discussion.

core: drop pointless assert()

We checked this literally two lines earlier, no point in checking so
quickly again.

update TODO

core/selinux: fix wrong assertion when 0 is passed to log_debug

https://github.com/systemd/systemd/pull/19317#issuecomment-820245680

meson: output one "item" for DNS-over-TLS

meson: reorder the "features" list

Try to make this more manageable by reording:
- dependencies / inputs
  (with subcategory of compression libraries)
- major components / outputs
- optional features / conditionals that don't fit into the two above categories

The division isn't well defined, because libraries often correspond one-to-one
to feature, but not always.

meson: fold static libsystemd and libudev into the generic status line

It's a very specialized feature and we don't need to devote a separate line to it.

meson: reindent

test: use systemd-run --wait in TEST-44-LOG-NAMESPACE

The test appears to be occasionally failing. It uses systemd-run to echo
'hello world' into a namespaced journal and then uses journalctl to look for it,
but it doesn't wait.
In the failed runs it can't find it, but the automated journal dump shows
the message at the end.

Use --wait to avoid races.

udev/cdrom_id: drop unnecessary cleanup and simplify loop

udev/cdrom_id: do not pass ioctl return value to log_debug_errno()

While at it, let's print the tray status in human readable form.

libsystemd-network: fix dhcp option buffer confusion

We were writing to the wrong buffer with a wrong offset :(
Bug present since the original introduction of the code in
04b28be1a306fd2ba454d3ee333d63df71aa3873.

meson.build: typo

libsystemd-network: check that errno==0 is not passed to log functions

libsystemd-network: use macro for definitions of log_{lldp,dhcp,…}_errno

No functional change.

Check that errno passed log_{interface,link}_*_errno() is non-zero

sd-dhcp-client: introduce sd_dhcp_client_is_running()

Voidify log_link_debug

See analogous change for log_debug() for discussion.

Merge pull request #19315 from yuwata/network-wait-online-address-family-follow-ups

network: several follow-ups for #19069

journald: fix %m usage

udev: ignore additional newline in sysfs attribute on verify

Some driver may automatically add additional newline at the end.
E.g., portno attribute for qeth driver.

Fixes #19314.

fileio: introduce a new flag to make write_string_file() ignore trailing newline

When writing a sysfs attribute, kernel may automatically append newline
at the end, e.g., 'portno' attribute for qeth driver.

Do not try to return 0 from log_debug()

As @yuwata correctly points out, this became broken when log_debug()
started returning -EIO. I wanted to preserve this pattern, but it turns
out it is not very widely used, and preserving it would make the whole
thing, already quite complicated, even more complex.

log_debug() is made like log_info() and friends, and returns void.

shared/module-util: fix errno value passed to log function

If r == 0, no harm done. But if r > 0, this would be interpreted as an
errno value, wrongly.

basic/log: assert that 0 is not passed as errno, except in test code

Let's assert if we ever happen to pass 0 to one of the log functions.
With the preceding commit to return -EIO from log_*(), passing 0 wouldn't
affect the return value any more, but it is still most likely an error.
The unit test code is an exception: we fairly often pass the return value
to print it, before checking what it is. So let's assert that we're not
passing 0 in non-test code. As with the previous check for %m, this is only
done in developer mode. We are depending on external code setting
errno correctly for us, which might not always be true, and which we can't
test, so we shouldn't assert, but just handle this gracefully.

I did a bunch of greps to try to figure out if there are any places where
we're passing 0 on purpose, and couldn't find any.
The one place that failed in tests is adjusted.

About "zerook" in the name: I wanted the suffix to be unambiguous. It's a
single "word" because each of the words in log_full_errno is also meaningful,
and having one term use two words would be confusing.

Merge pull request #19302 from bluca/uninit

tree-wide: avoid uninitialized warning on _cleanup_ variables

basic/log: assert that %m is not used when error is not set

This is only done in developer mode. It is a pretty rare occurence that we
make this kind of mistake. And even if it happens, the result is just a misleading
error message. So let's only do the check in non-release builds.

basic/log: force log_*_errno() to return negative

This silences some warnigns where gcc thinks that some variables are
unitialized. One particular case:

../src/journal/journald-server.c: In function 'ache_space_refresh':
../src/journal/journald-server.c:136:28: error: 'vfs_avail' may be used uninitialized in this function [-Werror=maybe-uninitialized]
  136 |         uint64_t vfs_used, vfs_avail, avail;
      |                            ^~~~~~~~~
../src/journal/journald-server.c:136:18: error: 'vfs_used' may be used uninitialized in this function [-Werror=maybe-uninitialized]
  136 |         uint64_t vfs_used, vfs_avail, avail;
      |                  ^~~~~~~~
cc1: all warnings being treated as errors

which is caused by

   d = opendir(path);
   if (!d)
           return log_full_errno(errno == ENOENT ? LOG_DEBUG : LOG_ERR,
                                 errno, "Failed to open %s: %m", path);
   if (fstatvfs(dirfd(d), &ss) < 0)
           return log_error_errno(errno, "Failed to fstatvfs(%s): %m", path);

For some reason on aarch64 gcc thinks we might return non-negative here. In
principle errno must be set in both cases, but it's hard to say for certain.
So let's make sure that our code flow is correct, even if somebody forgot to
set the global variable somewhere.

partition, random-seed, logind: fix log messages with %m

We would print "...: Success", which is not too terrible, but not pretty
either.

sd-device: improve log message and tweak style

We shouldn't say the attribute is missing right after ruling out ENOENT.

libsystemd-network: fix dhcp option buffer confusion

We were writing to the wrong buffer with a wrong offset :(
Bug present since the original introduction of the code in
04b28be1a306fd2ba454d3ee333d63df71aa3873.

meson: simplify the BUILD_MODE conditional

Using a enum is all nice and generic, but at this point it seems unlikely that
we'll add further build modes. But having an enum means that we need to include
the header file with the enumeration whenerever the conditional is used. I want
to use the conditional in log.h, which makes it hard to avoid circular imports.

bpf-devices: update comment

wait-online: update debug log messages

network: shorten code a bit

Merge pull request #19266 from mrc0mmand/testsuite-shellcheck

test: make the test scripts shellcheck-compliant

tree-wide: avoid uninitialized warning on _cleanup_ variables

With some versions of the compiler, the _cleanup_ attr makes it think
the variable might be freed/closed when uninitialized, even though it
cannot happen. The added cost is small enough to be worth the benefit,
and optimized builds will help reduce it even further.

meson: build tests with -Wno-maybe-uninitialized if -O2/-flto are used

We intentionally do not inline initializations with definitions for
a bunch of _cleanup_ variables in tests, to ensure valgrind is triggered.
This triggers a lot of maybe-uninitialized false positives when -O2 and
-flto are used. Suppress them.

util: shorten allow_listed_char_for_devnode()

Merge pull request #19287 from yuwata/network-manage-foreign-routing-policy-rule-19106

network: add ManageForeignRoutingPolicyRules= boolean setting

Merge pull request #19069 from LetzteInstanz/waiting_for_address_family

systemd-networkd-wait-online: wait for specific address family

rfkill: add some casts to silence -Werror=sign-compare

man: update explanation for Anonymize=

network: dhcp4: warn when Anonymize=yes and ClientIdentifier= is not mac

network: drop unnecessary explicit initializations

network: dhcp4: do not request any additional options when Anonymize=yes

This makes networkd can use recieved options we do not request.

network: move and rename network_apply_anonymize_if_set()

test-network: test waiting for address family

wait-online: wait for address family

This introduce -4 and -6 commandline options.

sd-network: read IPv4/IPv6 address states from state files

network: save IPv4/IPv6 address states into state file

This also introduces RequiredFamilyForOnline= setting to .network file,
and IPv4AddressState/IPv6AddressState DBus properties.

network: move AddressFamily into network-util for the use by wait-online later

udev: delete useless codes

It seems no one will touch queue.bin

dhcp: Implemented BindToInterface= configuration option

test: bunch of assorted tweaks to make shellcheck happy

test: use arrays to make things a bit cleaner

test: replace the obsolete `` syntax with $()

test: use an explicit no-op for file truncation

test: tidy up arithmetic expressions

test: use quotes where necessary

to avoid possible word splitting.

test: use set -eux and set -o pipefail everywhere

This should make the scripts more robust.

Merge pull request #19303 from yuwata/fix-typo

tree-wide: fix typo

network: fix indentation

network: lldp: update mac address

network: refuse to configure engines such as DHCP client more than once

tree-wide: update comment about unnecessary initialization

man: fix typo

man: update description for ManageForeignRoutes=

network: introduce ManageForeignRoutingPolicyRules= boolean setting in networkd.conf

The commit 0b81225e5791f660506f7db0ab88078cf296b771 makes that networkd
remove all foreign rules except those with "proto kernel".

But, in some situation, people may want to manage routing policy rules
with other tools, e.g. 'ip' command. To support such the situation,
this introduce ManageForeignRoutingPolicyRules= boolean setting.

Closes #19106.

network: add missing default setting in networkd.conf

load-fragment: parse specifiers in CPUAffinity= (#19281)

meson: do not fail if rsync is not installed with meson 0.57.2

https://github.com/mesonbuild/meson/issues/8641

Our CI started to fail. Even if the change is reverted in meson,
we need a quick workaround here.

packit: drop the 'sources' file after cloning the Fedora repo

Otherwise rebase-helper thinks we're are a dist-git repository,
replacing the generated git archive with PR changes with the tarball
found in the 'sources' file.

Merge pull request #18701 from bugaevc/mdns-unicast

resolved: reply using unicast mDNS when appropriate

Merge pull request #13496 from wat-ze-hex/custom-bpf-progs-parameterized-3

bpf: extend bpf cgroup program support

man: fix discriptions for --exec-delay

There are two ambiguity in the original description:
1. It will delay all RUN instructions, include builtin.
2. It will delay before running RUN, not each of RUN{program} instructions.

sd-dhcp-client: constify argument in sd_dhcp_lease_get_xxx()

networkctl: drop unused variables

test-repart: use cryptsetup and losetup autoclose

The test occasionally fails as the umount is not yet completed when
cryptsetup close is invoked.
Both cryptsetup and losetup have supported deferred cleanup for some
time now, so use it instead to avoid races.

++ losetup -P --show --find /tmp/test-repart.dMOfYQ8UUF/zzz
+ LOOP=/dev/loop6
+ VOLUME=test-repart-11882
+ touch /tmp/test-repart.dMOfYQ8UUF/empty-password
+ cryptsetup open --type=luks2 --key-file=/tmp/test-repart.dMOfYQ8UUF/empty*** test-repart-11882
+ mkdir /tmp/test-repart.dMOfYQ8UUF/mount
+ mount -t ext4 /dev/mapper/test-repart-11882 /tmp/test-repart.dMOfYQ8UUF/mount
+ diff -r /tmp/test-repart.dMOfYQ8UUF/mount/def /tmp/test-repart.dMOfYQ8UUF/definitions
+ umount /tmp/test-repart.dMOfYQ8UUF/mount
+ cryptsetup close test-repart-11882
Device test-repart-11882 is still in use.
+ rm -rf /tmp/test-repart.dMOfYQ8UUF