chase-symlinks: Return zero from access() and stat() helpers

We never check if r > 0 when using these helpers, so let's just
return zero like we usually do.

chase-symlinks: Add CHASE_EXTRACT_FILENAME flag

Useful in combination with CHASE_PARENT to get a directory file
descriptor to the parent directory of the resolved path and the
filename of the resolved path in the parent directory (which might
not be the same as the filename of the input path because of symlinks).

udev-rules: fix nvme symlink creation on namespace changes

The nvme by-id symlink changes to the latest namespace when a new namespace gets
added, for example by connecting multiple NVMe/TCP host controllers via nvme
connect-all.
That is incorrect for persistent device links.
The persistent symbolic device link should continue to point to the same NVMe
namespace throughout the lifetime of the current boot.
Therefore the namespace id needs to be added to the link name.

tmpfile-util: Add open_tmpfile_linkable_at() and link_tmpfile_at()

fs-util: Add open_parent_at()

lock-util: Add make_lock_file_at()

unit: Add LOG_CONTEXT_PUSH_UNIT()

A helper macro to push all unit related fields onto the log context.
We also modify exec_spawn() to use it.

log: Add key/value support to the log context

Now that we have reference counting, it's useful to be able to push
single key values onto the log context separately, so that we don't
have to allocate new storage to join the separate string together into
a single field which means we won't be able to reuse a context containing
the same field.

log: Avoid pushing the same fields more than once on the log context

Let's try to optimize against pushing the same fields multiple times
onto the log context. To achieve this we make the log context reference
counted and return an existing context object if it's using the same
fields.

A consequence of this is that we have to make sure attaching/detaching
is coupled to the lifetime of the context object, so we make the attach
and detach functions private for now. If we need independent attach/detach
in the future, we can make that work with some extra complexity but since
we don't need it yet, let's not support it for now.

Merge pull request #26893 from yuwata/uki-util-update-log

util: rename uki-util.[ch] -> kernel-image.[ch]

mkosi: Default to debug log level for udev as well

Unlike CI, the debug output just goes to the journal, so there's no
harm in enabling it, even if it's noisy.

man/network-generator: replace dracut.kernel reference with dracut.cmdline

`dracut.kernel.7` is just a symlink to `dracut.cmdline.7`, so the web reference
points to a non-existent URL
(https://man7.org/linux/man-pages/man7/dracut.kernel.7.html).

gpt-auto-generator: fix typo

Merge pull request #26867 from dtardon/list-dependencies-circular

Fix printing of circular dependencies

util: rename uki-util.[ch] -> kernel-image.[ch]

As the file contains functions not only for UKI.

uki-util: rename KernelType -> KernelImageType

The name KernelType is slightly ambiguous. The type is about kernel
image. Let's rename the enum to clarify that.

uki-util: rename uki -> f

As the function is not only for UKI.

uki-util: update log messages

udev-rules: add another check for conflicting expressions

Log an error when a rule line contains the following kind of conflicting
match expressions:

  KEY=="foo", KEY=="bar"

compare-operator: added unit tests

Merge pull request #26875 from yuwata/core-transaction

core/transaction: several fixes for merge_unit_ids()

strv: introduce strv_print_full()

ci: drop clang-13, add clang-16

bootctl: fix wrong type comparison

Merge pull request #26886 from ldv-alt/udevadm-verify

udev-rules: fix matching of token types that support alternative patterns

loop-util: add more debugging logs in loop_configure()

udev-rules: fix check for conflicting and duplicate expressions

Fix check for conflicting and duplicate expressions of types that
support alternative patterns.

Fixes: 3ec58d0cd8f6 ("udev-rules: check for conflicting and duplicate expressions")

Revert "udev: prepare memory for extra NUL termination for NULSTR"

This reverts commit cd3c8a117ccf3505e49d34324473e2175ef0a9ce which was
papering over the bug instead of a proper fix made by the previous
commit.

udev-rules: fix matching of token types that support alternative patterns

For those token types that support matching of alternative patterns,
their token values are interpreted as nulstr, so make sure the parser
does the right thing and makes these token values terminated by two
subsequent NULs so they could be safely interpreted as nulstr.

Before this fix, the following rules would result to "echo foo" invocation:
  ENV{foo}=", RUN"
  ENV{foo}=="bar", RUN+="echo foo"
because the value of `ENV{foo}` is treated as nulstr, and it used to match
against alternative patterns, in this case `bar`, `, RUN`, and `="echo foo`.

Fixes: 25de7aa7b90c ("udev: modernize udev-rules.c")

hwdb: fix Wifi toggling for Haier 7G-Series/JWU (#25293) (#26878)

Merge pull request #26884 from YHNdnzj/tmpfiles-unit-name-update

Update references to obsolete systemd-tmpfiles.service

Merge pull request #26880 from yuwata/test-tmpfiles

tmpfiles: cleanups

Merge pull request #26879 from mrc0mmand/cocci

tree-wide: simplify x ? x : y to x ?: y where applicable

docs: update unit name for sd-tmpfiles-setup

unit: sysext: update unit name for sd-tmpfiles-setup

Fixes #26882

test-systemd-tmpfiles.py: create global temporary directory

For some unknown reasons, the temporary directory created by the test
below is not removed:
```
 # Test the case that a valid symlink is in the path.
 label = 'valid_symlink-deep'
 test_content('f= {} - - - - ' + label, label, user=user, subpath='/deep/1/2', path_cb=valid_symlink)
```
To keep /tmp clean, let's create the global temprary directory.

test-systemd-tmpfiles.py: use test_content() at one more place

tmpfiles: show file type in octal, instead of hex

tree-wide: simplify x ? x : y to x ?: y where applicable

coccinelle: add a transformation for GNU conditionals

i.e. x ? x : y => x ?: y

coccinelle: respect spacing from the semantic patch

man: explicitly list three command syntax at the beginning

fuzz-journal-remote: fix potential fd-leak

fuzz-journal-remote: remove temporary files on exit

Merge pull request #26759 from medhefgo/boot-stack-protector

boot: Add support for stack protector/trapping math/ubsan

units: Order user@.service after systemd-oomd.service

The user manager connects to oomd over varlink. Currently, during
shutdown, if oomd is stopped before any user manager, the user
manager will try to reconnect to the socket, leading to a warning
from pid 1 about a conflicting transaction.

Let's fix this by ordering user@.service after systemd-oomd.service,
so that user sessions are stopped before systemd-oomd is stopped,
which makes sure that the user sessions won't try to start oomd via
its socket after systemd-oomd is stopped.

Merge pull request #26862 from yuwata/uki-util

uki-util: export functions of inspecting kernel

Merge pull request #26861 from yuwata/exec-util

exec-util: introduce execute_strv() and EXEC_DIR_SKIP_REMAINING

Merge pull request #26855 from DaanDeMeyer/repart-fixes

Repart fixes

core/transaction: do not log "(null)"

As we ignores the failure in merge_unit_ids(), so unit_ids may be NULL.

core/transaction: make merge_unit_ids() return non-NULL on success

core/transaction: make merge_unit_ids() always return NUL-terminated string

Follow-up for 924775e8ce49817f96df19c2b06356c12ecfc754.

The loop run with `STRV_FOREACH_PAIR()`, hence `if (*(unit_id+1))` is
not a good way to detect if there exist a next entry.

Fixes #26872.

Merge pull request #26863 from yuwata/kernel-install-cleanups

kernel-install: cleanups

sleep: fix default values unmatched with manual

Merge pull request #26871 from weblate/weblate-systemd-master

Translations update from Fedora Weblate

po: Translated using Weblate (Bulgarian)

Currently translated at 100.0% (193 of 193 strings)

Co-authored-by: Velislav Ivanov <velislav.bg@gmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/master/bg/
Translation: systemd/main

systemctl: move printing of state to the printing func.

No functional change, just refactoring.

kernel-install: check all entry candidates first, then check existence of $pref/loader/entries only once

If we have multiple entries in $ENTRY_TOKEN_SEARCH, and $pref/loader/entries
exists, then previously we would always exit after the first candidate and
ignore the second and later candidates.

Follow-up for 1b43f868934e971480249a6e0fa2f45da906ea2e.

Revert "kernel-install: also try to find $BOOT by partition GUID"

This reverts commit 41f39e2144c935868059e894c1a1e13803e3568f.

From the post merge comment in #26648:

> Hmm, this is highly problematic, no? if I boot from my ssd and then
> plug in a fedora live usb stick, then there will be two ESPs around,
> the one from my ssd and the one from the live usb one, and this code
> might find the wrong one and bad things will happen

uki-util: move functions for inspecting kernel to uki-util.[ch]

bootctl: introduce inspect_kernel()

Then, now verb_kernel_identify() and verb_kernel_inspect() can share
most of the code.

bootctl: introduce uki_read_pretty_name()

No functional change, just prepration for later commits.

bootctl: find matching section in read_pe_section()

No functional change, just refactoring.

bootctl: return 0 instead of EXIT_SUCCESS

bootctl: move verb_kernel_identity()

No functional change, just for making beautify later diffs.

os-util: add missing headers

For 'bool' and 'usec_t'.

exec-util: introduce EXEC_DIR_SKIP_REMAINING flag

Will be used in later commits.

exec-util: extract the core logic of execute_directories() as execute_strv()

Then, we can use it with a custom enumerator of executables.

No functional change, preparation for later commits.

exec-util: enumerate executables earlier

Then, return earlier if no executable found.

exec-util: drop meaningless casts

exec-util: tighten variable scope a bit

Merge pull request #26849 from DaanDeMeyer/fwd-colors

journald-console: Add colors when forwarding to console

systemctl: drop stray empty line

systemctl: always print circular deps. at the end

Before:
a.target
○ └─b.target
    ├─...
●   └─paths.target

After:
a.target
○ └─b.target
●   ├─paths.target
    └─...

boot: Add undefined sanitizer support

Sadly, no stack traces, but this is better than nothing.

meson: Share more C flags

boot: Add support for -ftrapv

boot: Add support for -fstack-protector

boot: Detect nested assertions

systemctl: fix formatting of circular dep.

Before:
a.target
○ └─b.target
    └─...
●   └─paths.target

After:
a.target
○ └─b.target
    ├─...
●   └─paths.target

systemctl: fix indentation of circular deps

We're still at level `level` here.

Before:
b.target
○ ├─a.target
  │   └─...
● └─paths.target

After:
b.target
○ ├─a.target
  │ └─...
● └─paths.target

Fixes #26052.

Merge pull request #26783 from yuwata/loop-ref-follow-up

udev: slightly extend comment and add more tests

test: add a testcase that dir_fd_is_root() is not confused by bind mount

See the comment in dir_fd_is_root() and
https://github.com/systemd/systemd/pull/26820#issuecomment-1469554966.

Merge pull request #26803 from yuwata/udevadm-verify-summary

udevadm-verify: print a short summary

mkosi: Enable some debugging options by default

man: fix misspelled executable name (#26858)

repart: Report better errors if partition sizes are too small

repart: Zero full verity signature partition size

systemd-dissect requires the entirety of the partition following the
signature to be zeroed, so let's do just that.

repart: Make sure we seek to beginning of partition target before copy

mkosi: Use default timeout of 10s

Let's override the default timeout to something more reasonable for
mkosi builds.

udev/ata_id: use unliagned helpers

The array is a union, aligned as uint16_t, so we were accessing fields at
offsets of two, so we didn't do actually do unaligned access. But let's make
this explicit.

udev/v4l_id: use O_CLOEXEC|O_NOCTTY

This is the usual set of flags. O_CLOEXEC doesn't matter because we don't spawn
anything, but O_NOCTTY could possibly make a difference if the helper is called
on a wrong device type.

meson: add udev builtins to dist-check

They now pass, and we might as well test them to avoid unexpected
regressions.

udev: implement --version in all builtins

Those are separate binaries, and occasionally people will get a misplaced
binary that doesn't match the rest of the installed system and be confused, so
it good to be able to check the version. It is also nice to have the same
interface in all binaries.

Note that we usually use a separate 'enum ARG_VERSION = 0x100' for an option
without a short name. We can use a less verbose approach of simply taking any
unused letter, which works just as well and even the compiler would warn us
if we tried to use the letter in another place. This way we avoid a few lines
of boilerplate.

The help texts are adjusted to have an empty line between the synopsis and
option list, and no empty lines after the option list.

udev/cdrom_id: do not abort on unknown options

Copy-pasta is good, but you have to pick a reliable source.

udev/mtd_probe: convert to run()

udev/mtd_probe: implement --help

udev/fido_id: implement --help

udev/ata_id: stop using errno, fix logic

The code was setting errno, which we don't do, and which is hard to get right…
Rework the code to use the usual negative-errno convention and add some
debug logging.

disk_scsi_inquiry_command() was working partially by accident: the v3 fallback
would enter the v4 check path, and pass it because the v4 data would still be
initialized to 0.

udev/ata_id: drop unused output parameter