TEST-67-INTEGRITY: several fixes (#35366)

Fixes #35363.

Chores for RC3 (#35383)

nsresourced: log about correct errno (#35386)

Split out from #35242

network/queue: do not increase reference counter when remove request is not queued

Fixes a bug introduced by ef45f5c8d0bfbeda2a57b28bba9831f044b23b21.
Fixes #35164.

nsresourced: log about correct errno in userns_destroy_cgroup()

cgroup-util: use RET_NERRNO where appropriate

TEST-67-INTEGRITY: blkid should not provide the underlying loopback block device

Fixes #35363.

TEST-67-INTEGRITY: modernize test code

- make udevd generate debugging logs for loopback and DM devices,
- insert 'udevadm wait' at several places to make the device processed
  by udevd,
- cleanup generated integritysetup service before moving to next
  algorithm,
- drop unnecessary exit on command failure,
- also test data splitting mode for all algorithms.

nspawn: make sure --private-users-ownership=no and =off work the same way

We usually want to use "extended booleans" for cases like this, i.e.
that "off", "no" and "0" can be used interchangably for turning
something off.

sysext: coding style fixes & fix a memory leak

Follow-up for: bbec1c87d3bf8d14eeb1ee3b4df973a53cca2e58

po: update translations

ninja -C build systemd-update-po

hwdb: update

ninja -C build update-hwdb

NEWS: update contributors list

Minor follow-ups for recent PRs (#35381)

service: don't second guess invocation mode again

let's just check the debug invocation boolean, and not recheck the
restart mode again. It's mostly redundant (because the boolean should
not have been become true if the restart mode was not set accordingly).

Moreover, i think we might want to eventually allow a manual way to
enable debug invocation mode, and hence this pointless checking would
become a problem.

Also, we never check the restart mode again in other cases, hence we
shouldn't here either.

nspawn: improve error message when we cannot look into a container tree due to perms

tests: fix access mode of root inode of throw-away container images

Otherwise the root inode will typically have what mkdtemp sets up, which
is something like 0700, which is weird and somewhat broken when trying
to look into containers from unpriv users.

nspawn: don't try to unregister a machine we never registered

When registering we condition this on "arg_register". Let's do the same
when unregistering, otherwise we might end up trying to unregister a
machine we never registered.

shared/bootspec: mark _to_string funcs as _const_

Addresses https://github.com/systemd/systemd/pull/34959#discussion_r1860451777

shared/bootspec: use FOREACH_ELEMENT where appropriate, avoid unneeded memzero()

shared/bootspec: move boot_entry_addons_done() up, drop separate prototype

Follow-up for e2501a851e10f5279862b2bccbdd9184572cce6a

basic/fileio: minor coding style cleanup

Follow-up for bbec1c87d3bf8d14eeb1ee3b4df973a53cca2e58

man: split systemd.conf(5) into multiple sections

No changes in wording, let's just make a very long man page a bit more
digestable by adding sections, and then reordering settings to fit into
them.

Update TODO

bootspec fixups (#34959)

ukify: Switch to JSON HWID description format (#35208)

Fixes #35176

bootspec: Look at /loader/addons in XBOOTLDR

The bootspec util-lib's handling of global addons didn't previously
match the behavior of sd-stub, and this commit corrects that.

First, bootspec didn't load global addons from the XBOOTLDR dir, but the
stub does. So, bootspec now enumerates addons in XBOOTLDR, not just ESP

Second, the stub only loads resources (including addons) from the
partition that it was found on. Thus, we must keep track of which
partition the global addons come from, and which partition each boot
entry comes from. In other words: global addons found on the ESP will
NOT apply to UKIs found in XBOOTLDR, and bootspec now reflects that.

bootspec: Fixup loading of local addons for UKIs

Follow-up for 59b3df9

bootspec: Fixup memory leak

This would previously leak memory: the array was deleted but contents
inside of the array were not

sd-stub: Fixup typo & measurement order

A previous commit accidentally reversed the measurement order of the
confext initrds and sysext initrds via a minor typo. This commit fixes
the typo and restores the original measurement order

Follow-up: ac32323

sysext: set SELinux context for hierarchies and workdir

man: several more assorted fixes

Continuation of 4ebbb5bfe88ac3d793c395472648660c33251546.
Closes #35307.

machine: increase timeouts in attempt to fix #35115 (#35117)

An attempt to fix https://github.com/systemd/systemd/issues/35115

sd-varlink: fix bug when enqueuing messages with fds asynchronously

When determining the poll events to wait for we need to take the queue
of pending messages that carry fds into account. Otherwise we might end
up not waking up if such an fd-carrying message is enqueued
asynchronously (i.e. not from a dispatch callback).

man/systemd-system.conf: Correct "struct" to "strict" (#35364)

cryptsetup: convert pkcs11/fido2 to iovec for key handling

key-data might be NULL. Fixes crash:

0  0x0000559c62120530 in attach_luks_or_plain_or_bitlk (cd=0x559c6b192830, name=0x7ffd57981dc4 "root", token_type=TOKEN_FIDO2, key_file=0x0, key_data=0x0, passwords=0x0, flags=524296, until=0)
    at ../src/cryptsetup/cryptsetup.c:2234
        pass_volume_key = false
        r = 1469577760
        __func__ = '\000' <repeats 29 times>
1  0x0000559c6212279c in run (argc=6, argv=0x7ffd5797fe98) at ../src/cryptsetup/cryptsetup.c:2597
        discovered_key_data = {iov_base = 0x0, iov_len = 0}
        key_data = 0x0
        token_type = TOKEN_FIDO2
        destroy_key_file = 0x0
        flags = 524296
        until = 0
        passphrase_type = PASSPHRASE_NONE
        volume = 0x7ffd57981dc4 "root"
        source = 0x7ffd57981dc9 "/dev/disk/by-uuid/8372fb39-9ba4-461a-a618-07dcaae66280"
        status = CRYPT_INACTIVE
        tries = 0
        key_file = 0x0
        config = 0x7ffd57981e05 "luks,discard,fido2-device=auto,x-initrd.attach"
        use_cached_passphrase = true
        try_discover_key = true
        discovered_key_fn = 0x7ffd5797fa70 "root.key"
        passwords = 0x0
        cd = 0x559c6b192830
        verb = 0x7ffd57981dbd "attach"
        r = 0
        __func__ = "\000\000\000"
2  0x0000559c621231e6 in main (argc=6, argv=0x7ffd5797fe98) at ../src/cryptsetup/cryptsetup.c:2674
        r = 32553
        __func__ = "\000\000\000\000"

Follow-up for 53b6c99018f918a5d2c9000ac5fe3a2440115ea7

updatectl: fix DBus method signature for SetFeatureEnabled

The signature was changed to 'sit' in sysupdated during review, but updatectl
kept using 'sbt'

Revert "Revert "man: use MIT-0 license for example codes in daemon(7)""

This reverts commit 7a9d0abe4df81a020e2e457405f8509b2afb7c04.

Revert "man: use MIT-0 license for example codes in daemon(7)"

This reverts commit 6046cc3660810efcc6fe50b1c850ea642218245b.

man: use MIT-0 license for example codes in daemon(7)

This page contains many short example codes. I do not think we should
add SPDX-License-Identifier for all codes.

Closes #35356.

man: assorted fixes (#35326)

Closes #35307.

test: Use env in testsuite readme

Let's make sure we use env when we're setting environment variables
to rely less on shell specifics.

core/device: ignore ID_PROCESSING udev property on enumerate (#35332)

Fixes #35329.

Check inode number to see if we are in init namespace (#35306)

This is a more comprehensive fix compared to #35273. Also adds a minimal
test only.

Based on Luca's #35273 but generalizes the code a bit.

In v258 we really should get rid of the old heuristics around userns and
cgroupns detection, but given we are late in the v257 cycle this keeps
them in.

test: Dump coredumps from journal in the integration test wrapper

Fixes #35277

networkd-test.py: disable IPv6AcceptRA= if not necessary

To speed up the test. Otherwise, it takes about few seconds interfaces
to enter the configured state. And may networkd-wait-online timeouts.

TEST-17: add reproducer for issue #35329

Without the previous commit, the test case will fail.

core/device: ignore ID_PROCESSING udev property on enumerate

This partially reverts the commit 405be62f05d76f1845f347737b5972158c79dd3e
"tree-wide: refuse enumerated device with ID_PROCESSING=1".

Otherwise, when systemd-udev-trigger.service is (re)started just before
daemon-reexec, which can be easily happen on systemd package update, then
udev database files for many devices may have ID_PROCESSING=1 property,
thus devices may not be enumerated on daemon-reexec. That causes many
units especially mount units being deactivated after daemon-reexec.

Fixes #35329.

test: mask tmpfiles.d file shipped by selinux policy package in containers

This tmpfiles.d wants to write to sysfs, which is read-only in containers,
so systemd-tmpfiles --create fails in TEST-22-TMPFILES when ran in nspawn
if the selinux policy package is instealled. Mask it, as it's not our
config file, we don't need it in the test.

mkosi: Use shared extra tree between initrd and main image

Let's share more between initrd and main system and use a shared
extra tree to achieve that.

mkosi: Make sure mkosi.clangd always runs on the host

If the editor that invokes mkosi.clangd is a flatpak, let's make sure
that mkosi is run on the host and not in the flatpak sandbox since it
won't be installed there.

Various mkosi and integration test fixes (#35336)

test: Lint integration-test-wrapper.py

test: Fix typing errors in integration-test-wrapper.py

test: Format integration-test-wrapper.py

ukify: Fix typing error

Move mypy.ini and ruff.toml to top level

This allows reusing them for integration-test-wrapper.py as well.

integration-test-wrapper: Remove unneeded format strings

mkosi: Add github CLI to tools

mkosi: Add dnf and dnf5 to sanitizer workaround list

mkosi: Install clangd everywhere

mkosi: Use bash to execute command -v

command is only an executable on Fedora due to a downstream patch,
on Arch for example it's only a builtin so we have to use bash to
execute command -v to get proper results on Arch.

mkosi: Add shellcheck to tools

mkosi: update to latest

NEWS: add blurb thanking Nick Owens

Nick's largely responsible for nerd-sniping me into fixing #34516
and did most of the testing.

hwdb: add entry for Chuwi Hi10 X1 (#35331)

https://www.chuwi.com/product/items/chuwi-hi10-x1.html
Rotated -90 degrees in the Z axis.

curl-util: do not configure new io event source when the event loop is already dead

Similar to c5ecf0949460dd0bf3211db128a385ce6375252e, but for io event source.

Fixes #35322.

man: update documentation about basic .netdev file handling

Follow-up for #34909 and later PRs.

man: asorted fixes

Closes #35307.

measure: add 'dtbauto' option in help message

'dtbauto' command line was missing from the help string. Add it.

po: Translated using Weblate (French)

Currently translated at 100.0% (257 of 257 strings)

Co-authored-by: Léane GRASSER <leane.grasser@proton.me>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/main/fr/
Translation: systemd/main

networkd-test.py: fix interface state checker

After 259125d53d98541623b69e83000b5543f2352f5e, network interfaces
declared by .netdev files are created after systemd-networkd sends READY
notification. So, even when networkd is started, the netdevs may not
be created yet, and 'ip' command may fail. Let's also check the return
code of the command.

This also
- drops never worked stdout checks,
- makes the test fail if the interface is not created within the timeout.

nspawn: improve log message on bad incoming sd_notify() message

It's the PID that is wrong, not the UID/GID, be precise.

nspawn: fix userns_mkdir() invocation

The wrong error code was logged.

But actually given that userns_mkdir() is fine with existing dirs, let's
drop the redundant conditionalization.

Follow-up for: a1fcaa1549d86098d0ba75254b6afc96c786b3b6

shutdown: propagate one more error from sync_making_progress()

No functional change, just refactoring, as anyway all errors will be
ignored by the caller.

namespace-util: handle -ENOSPC by userns_acquire() gracefully in is_idmapping_supported() (#35313)

Follow-up for edae62120f13b24d51812d1d7c0ab24acb420305.
Fixes #35311.

shutdown: close DM block device before issuing DM_DEV_REMOVE ioctl

Otherwise, the ioctl() may fail with EBUSY.

Follow-up for b4b66b26620bfaf5818c95d5cffafd85207694e7.
Hopefully fixes #35243.

basic/linux: update kernel headers from v6.12

Undeprecate commandline params forcequotacheck, fastboot, and forcefsck

Those are historical names, but there is nothing wrong with them. The files on
/ (/fastboot, /forcefsck, and /forcequotacheck) are problematic because they
require a modification of the root file system. But the commandline params work
fine. They have the obvious advantage compared to our "modern" option that they
are much easier to type without looking up the spelling in the docs. Undeprecate
them to avoid unnecessary churn.

test-namespace: tweak log message a bit

virt: make use of ns inode check in running_in_userns() and running_in_cgroupns() too

detect-virt: check the inode number of the pid namespace

The indoe number of root pid namespace is hardcoded in the kernel to
0xEFFFFFFC since 3.8, so check the inode number of our pid namespace
if all else fails. If it's not 0xEFFFFFFC then we are in a pid
namespace, hence a container environment.

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

[Reworked by Lennart, to make use of namespace_is_init()]

namespace-util: add generic namespace_is_init() call

namespace-util: update log messages

namespace-util: handle -ENOSPC by userns_acquire() gracefully in is_idmapping_supported()

Follow-up for edae62120f13b24d51812d1d7c0ab24acb420305.
Fixes #35311.

link README.logs from tmpfiles.d/legacy.conf only if available

The file README.logs is installed only if SysVInit support is enabled.
Thus the link should depend on it as well.

varlink: apparently on old kernels SO_PEERPIDFD returns EINVAL

hwdb: add support for PineTab2 to 60-sensor.hwdb (#35304)

Add accelerometer support for PineTab2

userdbctl: two trivial fixlets (#35296)

Fixes: #35294

tpm2-util: fix parameter name

man: split cryptenroll man page into sections (#35297)

man: Document ukify --hwids= and --devicetree-auto= options

userdbctl: respect selected disposition also when showing gid boundaries

Follow-up for: ad5de3222f7

userdbctl: fix counting

Fixes: #35294

userbdctl: show 'mapped' user range only inside of userns

Outside of userns the concept makes no sense, there cannot be users
mapped from further outside.

cryptenroll: it's called PKCS#11, not PKCS11

In the --help text we really should use the official spelling, just like
in the man page.

man: add enrollment type sections to cryptenroll man page

We have the same sections in the --help text, hence we even more so
should have them in the man page.

bash-completion: add --list-devices to systemd-cryptenroll

And also use it to list suitable block devices.

core/service: service_add_fd_store() consumes passed fd

Without this change, the fd is closed twice on failure.

Fixes a bug introduced by dff9808a628c31b7ecb1f1aba8fdc3be06ce8372.

Fixes #35288.

ukify: Switch to JSON HWID description format