ci: limit permissions for `differential-shellcheck`

ci: trigger differential-shellcheck workflow on push

Also update `differential-shellcheck` to latest version - https://github.com/redhat-plumbers-in-action/differential-shellcheck/releases

Fixes: https://github.com/redhat-plumbers-in-action/differential-shellcheck/issues/215

Merge pull request #26828 from DaanDeMeyer/tmpfile-at

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

fd-util: make sure fd_reopen() works with AT_FDCWD systematically

Prompted by:

https://github.com/systemd/systemd/pull/26827#pullrequestreview-1341171981

Merge pull request #26866 from DaanDeMeyer/chase-fix

chase-symlinks: Add CHASE_FILENAME and more at() helpers

execute: Add kernel cmdline arguments for tty term, rows and columns

Let's allow configuring tty term and size using kernel cmdline arguments
so that when running in a VM we can communicate the terminal TERM and size
from the host via SMBIOS extra kernel cmdline arguments.

docs: Fix vscode debugging section in HACKING.md

Let's account for the recent changes in mkosi in the debugging
with vscode section.

chase-symlinks: Add more chase_symlinks_at() helpers

Copies of the corresponding chase_symlinks() helpers.

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

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

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.

test: wrap mkfs.*/mksquashfs/mkswap binaries when running w/ ASan

journald-console: Add colors when forwarding to console

Let's color output when we're forwarding to the console. To make this
work, we inherit TERM from pid 1 and use it to decide whether we should
output colors or not.

terminal-util: Don't assume terminal is dumb if connected to /dev/null

If we're connected to /dev/null, we're likely going to be writing
colors to something that's not stdout/stderr, so let's fall back to
checking TERM if that's the case.

doc: update ELF_PACKAGE_METADATA to use linker flag instead of manual script

bfd/gold/mold/lld support this flag, so document it explicitly, and drop the
manually written linker script which is no longer necessary

test: add tests for summary output of udevadm verify

test: reindent testsuite-17.11.sh

udevadm-verify: add a short summary