Merge pull request #30232 from keszybz/ukify-imports

Use exec() to import ukify

test: slacken plugged -> dead test

This test is for if devices transition from plugged -> dead -> plugged
on boot, but it is normal to see loop devices transition plugged -> dead
on first boot when systemd-repart adds the root partition.

test: Create analyze chroot with --rbind

The systemd-analyze integration test also tests chroots.
It builds its chroot by bind-mounting /,
but since /usr might be a separate mountpoint
it should create the chroot with --rbind.

core/unit: clean up unit_log_resources

* Use a unified struct to store accounting fields/suffixes
* Use strextendf_with_separator where appropriate
* Don't mix stack and heap allocation for one iovec array

core/unit: raise log level for unit_log_resources on certain memory thresholds

We already do this for all other types of accountings. Let's
make this nicer for memory accounting too.

macro-fundamental: add U64_{K,M,G}B

iovec-util: add missing assertion

test: test that delegation of some newer attrs that shall be delegated work

cgroup: bring list of delegated cgroup attributes up-to-date with current kernels

THis brings the list of attributes to delegate to managers of subcgroups
to the state of kernel 6.6.

We probably should unify this list, and maybe generate it automatically
from /sys/kernel/cgroup/delegate, but let's do that another time.

varlink: improve compat with varlink C reference implementation

The reference implementation seems to set the 'parameters' field for
method calls to 'null' if nothing is specified on its command line. We
so far only could deal if the parameters field was unset or set to the
empty object. Let's also accept the 'null' type.

test: add basic coverity tests for bootctl

test: create ESP and xbootldr partitions

test: mask mdmonitor when building image

Follow-up for 22e31655f3f9f54d932d4f48b92b36698e701729.

test: make install_mdadm() also install relevant kernel modules

Installing mdadm without kernel modules is mostly meaningless.

test: split out host_has_{btrfs,mdadm}() from TEST-64-UDEV-STORAGE

find-esp: do not skip fstype check even when --root= or --image= is specified

The check was introduced by 63105f33edad423691e2d53bf7071f99c83799ba,
but there is no reason to skip the check even in such cases.

find-esp: introduce verify_esp_flags_init() helper function

And split VERIFY_ESP_RELAX_CHECKS into two.

No functional change, just refactoring.

find-esp: do not fail when /boot on btrfs RAID on searching ESP or xbootldr

When /boot or friends is on btrfs RAID, btrfs_get_block_device_at() will
succeed with 0 and provide zero devnum. Then,
- if we are previleged, devname_from_devnum() maps the devnum to
  /run/systemd/inaccessible/blk, and the subsequent verification by blkid
  will fail,
- if we are unprevileged, sd_device_new_from_devnum() will fail.

This makes
- when find_esp() or find_xbootldr() is called without any paths, that
  is, called with the searching mode, then returns -ENOKEY, which should
  be handled gracefully by the caller,
- when they are called with an input path, then they provide the proper
  error message and suggestion.

Fixes RHBZ#2251262 (https://bugzilla.redhat.com/show_bug.cgi?id=2251262).

Merge pull request #30433 from yuwata/network-nexthop-cleanups

network: introduce ManageForeignNextHops=

Merge pull request #30453 from poettering/dissect-fixes

dissect-image: various fixes

test: mask the mdmonitor.service

It's pulled in by one of the udev rules (63-md-raid-arrays.rules) and it
fails every time, because there's no valid email address in
/etc/mdadm.conf:

[    5.778153] testsuite-64.sh[403]: mdadm: array /dev/md/mdmirror started.
[    5.819137] kernel: md/raid1:md127: not clean -- starting background reconstruction
[    5.819141] kernel: md/raid1:md127: active with 2 out of 2 mirrors
[    5.819159] kernel: md127: detected capacity change from 0 to 129024
[    5.821950] kernel: md: resync of RAID array md127
...
[    5.887192] mdadm[424]: mdadm: No mail address or alert command - not monitoring.
[    5.890772] systemd[1]: Starting mdmonitor.service...
[    5.891718] systemd[1]: Started mdmonitor.service.
[    5.892570] systemd[1]: mdmonitor.service: Main process exited, code=exited, status=1/FAILURE
[    5.892618] systemd[1]: mdmonitor.service: Failed with result 'exit-code'.

And as we (re)assemble the MD devices multiple times, this gets quite
noisy, especially since we later start hitting the service start rate
limit.

Fedora has the mdmonitor.service patched, so it won't start without
/etc/mdadm.conf being present, but Arch uses the upstream unit which
doesn't have such guard.

Let's just mask the service completely, which replaces all that noise
with one warning:

[    6.553583] testsuite-64.sh[294]: + udevadm wait --settle ...
[    6.580700] systemd[1]: sys-devices-virtual-block-md127.device: Failed to enqueue SYSTEMD_WANTS job, ignoring: Unit mdmonitor.service is masked.

man: make minor corrections to smbios-type-11(7)

- Correct initial version of io.systemd.credential* to 252
- Fix leftover copy-pasted heading

dissect-tool: right-align the partition number

The right-alignment was applied to the wrong column, because neither
ee8e497d249ab2e2df92aa024274f5b817270114 nor
1474d7ac2d308204e599a2502a8b5625bca76bcc updated the column count as
they should have.

openssl-util: avoid freeing invalid pointer

dissect-image: move comment to right place

The image name is extracted from the image path originally passed in,
i.e. not the contents of the image. And the image UUID is directly
retrieved from the partition table, hence also not from the contents.
Let's hence move the comment to separate out the stuff extract from the
file systems (and thus only available when mounting/with privs/with
block devices) from the data available without any of that.

dissect-image: fix fd leak in dissected_image_acquire_metadata()

We have to go through the "finish" label to properly close all pipes in
the error path, so that we don't leak them.

dissect-image: don't try to validate an extension release file with no image name

Otherwise we might validate the OS release file instead…

dissect-image: handle 'continue' event in metadata acquisition uniformly

Let's jump to the same label in all cases, that closes the associated
pipe, systematically.

dissect-image: path[] is now defined for all metadata fields

There's no point in handling a case that never can happen.

execute: improve log message about TTY ownership reset failures

test-network: add test for ManageForeignNextHops=no

test-network: reduce indent for verification

test-network: drop redundant call of tearDown()

network/nexthop: introduce ManageForeignNextHops= boolean setting

Closes #29034.

test: "never" is not a valid value for Restart=

[  154.140565] testsuite-07.sh[1014]: + systemctl start badbin_assert.socket
[  154.738606] testsuite-07.sh[1014]: + socat - ABSTRACT-CONNECT:badbin_assert.socket
[  154.768418] systemd[1]: Cannot find unit for notify message of PID 1021, ignoring.
[  154.812357] systemd[1]: /run/systemd/system/badbin_assert.service:3: Failed to parse service restart specifier, ignoring: never
[  155.347350] testsuite-07.sh[1014]: + timeout 10 sh -c 'while systemctl is-active badbin_assert.service; do sleep .5; done'
[  155.669695] (badbin)[1045]: badbin_assert.service: Failed to execute /tmp/badbin: Exec format error
[  155.676596] systemd[1]: badbin_assert.service: Failed with result 'exit-code'.
[  156.081953] testsuite-07.sh[1051]: failed
[  156.132018] testsuite-07.sh[1054]: ++ systemctl show -P ExecMainStatus badbin_assert.service
[  156.326583] (badbin)[1050]: badbin_assert.service: Failed to execute /tmp/badbin: Exec format error
[  156.343566] systemd[1]: badbin_assert.service: Failed with result 'exit-code'.
[  156.904658] (badbin)[1055]: badbin_assert.service: Failed to execute /tmp/badbin: Exec format error
[  156.913709] systemd[1]: badbin_assert.service: Failed with result 'exit-code'.
[  157.066900] testsuite-07.sh[1014]: + [[ 0 == 203 ]]
[  157.081588] testsuite-07.sh[618]: + echo 'Subtest /usr/lib/systemd/tests/testdata/units/testsuite-07.issue-30412.sh failed'

Follow-up for 1eeaa93de36.

Merge pull request #30445 from mrc0mmand/networkd-test-skip

Temporarily skip tests with known issues in the systemd-networkd test suite

execute: handle gracefully if we cannot lock /dev/console when resetting tty due to perms

This is the common case in --user instances, hence handle this
gracefully.

This should be safe since user instances won't get access to
/dev/console-related ttys anyway, but only their own ptys.

Merge pull request #30441 from poettering/nspawn-fixlets

nspawn: two trivial nspawn tweaklets

Merge pull request #30437 from YHNdnzj/job-start-msg

core/job: emit job start message if we're only waiting for unit state

hibernate-resume: don't wait forever if hibernate info is from EFI

test: temporarily skip checking NFT sets in test_address_static

Until https://github.com/systemd/systemd/issues/30427 is resolved.

test: temporarily disable test_sysctl

Until https://github.com/systemd/systemd/issues/30056 is resolved.

nspawn: drop redundant assignments

nspawn: suffix some paths in log messages with /, as per coding style

Merge pull request #30156 from yuwata/network-mtu

network: accept arbitrary size of MTU in .network

Merge pull request #30426 from mrc0mmand/nft-shenanigans

network: show the reason why NFT operation failed

network: actually show the unexpected flags

The original version would yield a slightly _unexpected_ message for
this [Address] section:

[Address]
Address=10.9.3.1/24
HomeAddress=yes
ManageTemporaryAddress=yes

systemd-networkd[68396]: /run/systemd/network/25-address-static.network: unexpected address flags "n/a" were configured. Ignoring [Address] section from line 144.

Let's instead show the unexpected flags:

systemd-networkd[69160]: /run/systemd/network/25-address-static.network: unexpected address flags "home-address,manage-temporary-address" were configured. Ignoring [Address] section from line 144.

core/job: emit job start message if we're only waiting for unit state

Currently, start/stop messages for device units are not used, since
job_perform_on_unit() does nothing and we simply wait for unit status
change. I think we still want some nice log messages explaining what
the start jobs for devices are doing, so let's fix this.

core/device: add stopping job message

The use case for stopping a device unit is indeed narrow,
but we still want to show a clear message.

Preparation for later commits.

po: Translated using Weblate (Turkish)

Currently translated at 100.0% (227 of 227 strings)

Co-authored-by: Oğuz Ersen <oguz@ersen.moe>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/master/tr/
Translation: systemd/main

network: drop redundant 'struct'

man/systemd.mount: tmpfs automatically gains After=swap.target dep

fstab-generator: disable default deps if x-systemd.{wanted,required}-by= is used

Fixes #30273

man/systemd.mount: update implicit deps on device unit

See also: 707ecf1423ba8797ccc3ced016cc9e0f23635838

Merge pull request #30363 from yuwata/analyze-find-template

analyze: also find template unit when an instance is specified

executor: don't duplicate FD array to avoid double closing

Just use ExecParam directly, as these are all internal to sd-exec now
anyway. Avoids double close when execution fails after FDs are set up
for inheritance and were already re-arranged.

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

Merge pull request #30422 from yuwata/network-tiny-fixes

network: tiny fixes

network: show the reason why NFT operation failed

network: add a missing space

kernel-install: Fix inspect with --root= when no version is specified

Using the kernel version from the host is incorrect in this case, so
fix the logic so it handles no version being specified correctly with
--root=.

bootctl: fix case-sensitive comparisons in reporting bootloader entries

Fixes #30159

Merge pull request #30389 from keszybz/test-ukify-du

Remove temporary directories created by test_ukify

kernel-install: Look for uki.conf in /usr/lib/kernel as well

network: drop unused Manager.routes_foreign

network: adjust log message

The address or neighbor processed here may not be foreign.

test: add test cases for issue #30357

analyze: also find template unit when a template instance is specified

Fixes a regression caused by 2f6181ad4d6c126e3ebf6880ba30b3b0059c6fc8.

Fixes #30357.

Co-authored-by: Jeff King <peff@peff.net>

network/neighbor: add missing OOM check

Adding Trekstor Primebook C13 rotation to 60-sensor.hwdb (#30415)

* Adding Trekstor Primebook C13 rotation to 60-sensor.hwdb

mkosi: Copy /boot into the ESP as well

Newer mkosi will start installing UKIs to /boot so prepare for that
by making sure we also copy /boot into the ESP.

Merge pull request #30406 from yuwata/resolve-clean-exit

resolve: make resolved exit cleanly

Merge pull request #30404 from yuwata/network-peer

network/route: fix reachability check when peer address is specified

Merge pull request #30407 from yuwata/network-clean-exit

network: make networkd exit cleanly

test: update log message and use SYNTHETIC_ERRNO()

Follow-up for ce5a6d5307568671cec7c253e984ffc0eafa726b.

Addresses https://github.com/systemd/systemd/pull/30049#discussion_r1395453408.

elf2efi: remove outdated comment mentioning linker script

Follow-up for 142f0c61a37091e233b80f02375cff1114dab24a

resolve: fix wrong error cause assignment to log_debug_errno()

Fixes #30392.

core/exec-invoke: sigwait() returns positive errno and never EINTR

Follow-up for 5b6319dceedd81f3f1ce7eb70ea5defaef43bcec (gosh this is
ancient), and effectively reverts 3dead8d925ea9db1fbd65b702b6b807e49ddeacf.

sigwait() is documented to "suspend execution of the calling thread
until one of the signals specified in the signal set becomes pending".
And the only error it returns is EINVAL, when "set contains an invalid
signal number". Therefore, there's no need to run it in a loop or
to check for runtime error.

test-network: check if networkd exits cleanly

network/ipv4ll: do not start sd-ipv4ll on exit

When assert_return() is critical, the following assertion is triggered
on exit:
---
 #0  0x00007f8b1f6b0884 in __pthread_kill_implementation () from target:/lib64/libc.so.6
 #1  0x00007f8b1f65fafe in raise () from target:/lib64/libc.so.6
 #2  0x00007f8b1f64887f in abort () from target:/lib64/libc.so.6
 #3  0x00007f8b208d02d6 in log_assert_failed (text=0x7f8b210009e0 "e->state != SD_EVENT_FINISHED", file=0x7f8b20fff403 "src/libsystemd/sd-event/sd-event.c",
     line=1252, func=0x7f8b21004400 <__func__.154> "sd_event_add_io") at ../src/basic/log.c:948
 #4  0x00007f8b208d0457 in log_assert_failed_return (text=0x7f8b210009e0 "e->state != SD_EVENT_FINISHED",
     file=0x7f8b20fff403 "src/libsystemd/sd-event/sd-event.c", line=1252, func=0x7f8b21004400 <__func__.154> "sd_event_add_io") at ../src/basic/log.c:967
 #5  0x00007f8b20c7d102 in sd_event_add_io (e=0x617000000080, ret=0x60c000000a20, fd=11, events=1, callback=0x7dfd85 <ipv4acd_on_packet>,
     userdata=0x60c000000a00) at ../src/libsystemd/sd-event/sd-event.c:1252
 #6  0x00000000007e3934 in sd_ipv4acd_start (acd=0x60c000000a00, reset_conflicts=true) at ../src/libsystemd-network/sd-ipv4acd.c:597
 #7  0x00000000007e72b9 in ipv4ll_start_internal (ll=0x6080000006a0, reset_generation=true) at ../src/libsystemd-network/sd-ipv4ll.c:278
 #8  0x00000000007e7462 in sd_ipv4ll_start (ll=0x6080000006a0) at ../src/libsystemd-network/sd-ipv4ll.c:298
 #9  0x00000000006047a1 in dhcp4_handler (client=0x617000000400, event=0, userdata=0x61a000000680) at ../src/network/networkd-dhcp4.c:1183
 #10 0x000000000075b1ed in client_notify (client=0x617000000400, event=0) at ../src/libsystemd-network/sd-dhcp-client.c:783
 #11 0x000000000075bf8d in client_stop (client=0x617000000400, error=0) at ../src/libsystemd-network/sd-dhcp-client.c:821
 #12 0x000000000077710f in sd_dhcp_client_stop (client=0x617000000400) at ../src/libsystemd-network/sd-dhcp-client.c:2388
 #13 0x000000000065cdd1 in link_stop_engines (link=0x61a000000680, may_keep_dhcp=true) at ../src/network/networkd-link.c:336
 #14 0x000000000041f214 in manager_free (m=0x618000000080) at ../src/network/networkd-manager.c:613
 #15 0x00000000004124e3 in manager_freep (p=0x7f8b1c800040) at ../src/network/networkd-manager.h:128
 #16 0x00000000004139f6 in run (argc=1, argv=0x7ffffe4522e8) at ../src/network/networkd.c:24
 #17 0x0000000000413b20 in main (argc=1, argv=0x7ffffe4522e8) at ../src/network/networkd.c:119
---
Prompted by https://github.com/systemd/systemd/pull/30049#issuecomment-1844087965.

test: check if resolved exits cleanly

resolve: do not trigger assertion on exit

By making assert_return() critical, we observe the following:
---
 Program received signal SIGABRT, Aborted.
 0x00007f01320b0884 in __pthread_kill_implementation () from /lib64/libc.so.6
 (gdb) bt
 #0  0x00007f01320b0884 in __pthread_kill_implementation ()
    from /lib64/libc.so.6
 #1  0x00007f013205fafe in raise () from /lib64/libc.so.6
 #2  0x00007f013204887f in abort () from /lib64/libc.so.6
 #3  0x00007f01338d02d6 in log_assert_failed (
     text=0x7f01340009e0 "e->state != SD_EVENT_FINISHED",
     file=0x7f0133fff403 "src/libsystemd/sd-event/sd-event.c", line=1399,
     func=0x7f01340045a0 <__func__.148> "sd_event_add_time")
     at ../src/basic/log.c:948
 #4  0x00007f01338d0457 in log_assert_failed_return (
     text=0x7f01340009e0 "e->state != SD_EVENT_FINISHED",
     file=0x7f0133fff403 "src/libsystemd/sd-event/sd-event.c", line=1399,
     func=0x7f01340045a0 <__func__.148> "sd_event_add_time")
     at ../src/basic/log.c:967
 #5  0x00007f0133c7ed83 in sd_event_add_time (e=0x617000022280,
     ret=0x610000007e98, clock=7, usec=24054941030, accuracy=0,
     callback=0x4625b4 <on_announcement_timeout>, userdata=0x610000007e40)
     at ../src/libsystemd/sd-event/sd-event.c:1399
 #6  0x00007f0133c7f725 in sd_event_add_time_relative (e=0x617000022280,
     ret=0x610000007e98, clock=7, usec=1000000, accuracy=0,
     callback=0x4625b4 <on_announcement_timeout>, userdata=0x610000007e40)
     at ../src/libsystemd/sd-event/sd-event.c:1462
 #7  0x0000000000464cac in dns_scope_announce (scope=0x610000007e40, goodbye=true) at ../src/resolve/resolved-dns-scope.c:1530
 #8  0x0000000000504d08 in link_free (l=0x612000023d40) at ../src/resolve/resolved-link.c:83
 #9  0x000000000052dbbd in manager_free (m=0x619000000a80) at ../src/resolve/resolved-manager.c:697
 #10 0x0000000000562328 in manager_freep (p=0x7f012f800040) at ../src/resolve/resolved-manager.h:198
 #11 0x000000000056315a in run (argc=1, argv=0x7fff22b06468) at ../src/resolve/resolved.c:25
 #12 0x0000000000563284 in main (argc=1, argv=0x7fff22b06468) at ../src/resolve/resolved.c:99
---
Prompted by https://github.com/systemd/systemd/pull/30049#issuecomment-1844087965.

systemctl: fix typo

Follow-up for 329050c5e2c7e9561699f87b5edb72edd0d54c96.

networkctl: fix typo

Follow-up for 2b98926f9809eb858a5abe4f64ebd067df5059d5.

core/executor: do destruct static variables and selinux before exiting

I was wondering why I couldn't trigger the assertion in safe_fclose()
when submitting #30251. It turned out that the static destructor was
not run at all :/

Replace main() with a minimized version of main-func.h. This also
prevents emitting negative exit codes.

Merge pull request #30399 from YHNdnzj/memory-accounting-always-peak

systemctl-show: always show memory peak if available

hibernate-util: de-duplicate clear_efi_hibernate_location

Merge pull request #30400 from bluca/coverity

Assorted coverity fixes

test-network: add test case for issue #30403

network/route: fix reachability check when peer address is specified

When an address with peer address is specified, the kernel by default
adds the prefix route for the peer address. When ManageForeignRoute=no
is set, then we also needs to check the prefix for the peer address.

Fixes #30403.

test-login: remove dead code

HANDLE_SLEEP is == _HANDLE_ACTION_SLEEP_LAST and the loop is bounded
by < _HANDLE_ACTION_SLEEP_LAST, so no need to check for it

CID#1529414

logind: use ASSERT_PTR to validate result before use

handle_action_lookup() in general can return NULL, but not
here as the handle is checked before, so add an assert.

CID#1529415
CID#1529416

systemctl: fix copy/paste

on_circle is set twice to the same value

CID#1529418

journal-remote: set upper length bound when parsing incoming headers

CID#1529420

systemctl-show: always show memory peak if available

core/cgroup: cache the last memory usage values before destroying cgroup

Currently, memory accounting values are only cached if it was queued
at least once before destroying cgroup. Let's always cache it like
what we already do for CPU usage.

Preparation for later changes.

mount: check that MountParameters is valid before use

Follow-up for 6c75eff6afd90

CID#1530430

NEWS: fix typo s/show/shown/

Merge pull request #30387 from mrc0mmand/TEST-17-tweaks

test: a couple of tweaks for TEST-17-UDEV

Merge pull request #30384 from YHNdnzj/rename-process

core/executor: save argv for later use by rename_process()

Merge pull request #30362 from mrc0mmand/cat-highlight-directives

shared: highlight directives when dumping configs