test: add ASSERT_OK_POSITIVE() and ASSERT_OK_ZERO()

Merge pull request #33990 from yuwata/run-json-follow-ups

run: several follow-ups for recent change

Merge pull request #33957 from yuwata/network-generator-vlan

network-generator: fixlets for vlan interfaces

run: use sd_json_variant_set_field_id128()

sd-json: add sd_json_variant_set_field_id128() and _uuid()

run: drop unnecessary initializations

Follow-up for fe5a6c47af675bc0020c545d86fb103492e1d77c.

network-generator: drop wrong warning for rd.peerdns without value

network-generator: ignore kernel command line without value

And drop duplicated log messages.

network-generator: use extract_first_word()

Now, ip= with trailing colon is refused.

nspawn: Fix help typo

'an an' -> 'an'

man/systemd-sysusers, man/systemd-tmpfiles: fix sentence fragments

add "are executed" so that those two sentences have a verb.

nspawn: Assume unified cgroup hierarchy if there's no systemd in the image

If there's no systemd installation in the image, assume the unified
cgroup hierarchy.

docs: Mention the new mount API in the container interface doc

Let's mention that the new mount API may be used to establish new
mounts in a container without needing the /run/host/incoming directory.

Merge pull request #33979 from YHNdnzj/edit-util-no-duplicate-strip

edit-util: a few cleanups; support networkctl edit --stdin

systemd-run: add unit and invocation_id JSON output

networkctl: support edit --stdin

edit-util: several cleanups for --stdin handling

Follow-up for 329050c5e2c7e9561699f87b5edb72edd0d54c96

I don't particularly favor the duplicated strstrip()
and such, so let's ensure if we get fixed data it's
only trimmed once. Subsequently we can benefit more
by making all copies reflinks.

edit-util: do not try to recreate temp file if missing

We initially read from temp file, then strip it, and write
back to it. If the file suddenly disappeared during the process,
it indicates someone else is touching our temp file
behind our back. Let's not silently continue.

edit-util: clean up run_editor() a bit

- Add missing assertions
- Close all fds before spawning editor
- Use FOREACH_STRING() + empty_to_null() where appropriate
  Note that this slightly changes the behavior, in that
  empty envvars would be treated as unset and we'd try
  the next candidate. But the new behavior is better IMO.

mkosi: update debian commit reference

* 6e0f4f74ba Update changelog for 256.4-3 release
* 4b142f9c37 Depend on new linux-bpf-dev package where available
* f5fe5ecf4d autopkgtest: use hint-testsuite-triggers to ensure other packages changes trigger our testsuite
* 407932845d autopkgtest: run upstream test last
* 31458d03c2 Stop installing legaly pkla file in upstream CI too
* 484643291a Use d/not-installed instead of manual removals
* 752bb4c34c Stop shipping empty /etc/init.d directory
* 174603ffc2 Use debian/clean instead of override in d/rules
* 9a355e5a51 Drop redundant pot build
* 3d249c88cb Update changelog for 256.4-2 release

core/exec-invoke: call setpriority() after sched_setattr()

The nice value is part of struct sched_attr, and consequently invoking
sched_setattr() after setpriority() would clobber the nice value with
the default (as we are not setting it in struct sched_attr).

It would be best to combine both calls, but for now simply invoke
setpriority() after sched_setattr() to make sure Nice= remains effective
when used together with CPUSchedulingPolicy=.

terminal-util: do not query kernel cmdline for pty size

This is pointless and noisy even for debug level.

Merge pull request #33961 from bluca/busctl_exit

busctl: add support for --timeout and --num-matches for monitor verb

busctl: add --num-matches= for monitor verb

Useful in scripts when one wants to wait for a specific
signal before continuing

core/unit: do not use unit path cache in unit_need_daemon_reload()

When unit_need_daemon_reload() calls unit_find_dropin_paths() to check
for new drop-in configs, the manager's unit path cache is used to limit
which directories are considered. If a new drop-in directory is created,
it may not be in the unit path cache, and hence unit_need_daemon_reload()
may return false, despite a new drop-in being present. However, if a
unit path cache is not given to unit_file_find_dropin_paths() at all,
then it behaves as if the target path was found in the unit path cache.

So, to fix this, adapt unit_find_dropin_paths() to take a boolean
argument indicating whether or not to pass along the unit path cache.
Set this to false in unit_need_daemon_reload().

Fixes #31752

network-generator: use network_acquire() at more places

network-generator: introduce network_acquire() and netdev_acquire()

No functional change, just refactoring.

network-generator: trivial cleanups

network-generator: parse vlan ID from vlan interface name

Fixes #33954.

network-generator: allow to create bridge or friends without slave interfaces

network-generator: vlan= can be specified multiple times

test: add one more test case without hostname

test: modernize test-network-generator

test: note in README how to get full list of Ubuntu CI jobs

hwdb: Fix accelerometer mount matrix for ASUS T100TAS

busctl: add support for --timeout to monitor verb

Useful in scripts when you want to exit successfully after a certain time

Merge pull request #33959 from DaanDeMeyer/crash

Two crash handler improvements

nspawn: fix settings leak for init parameter

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

Follow-up for ce2291730d5f91190e97e7c515ac772ae4970062

crash-handler: Drop 10s sleep before we spawn the crash shell

It pointlessly delays getting to the crash shell so let's drop the
10s sleep.

crash-handler: Call vhangup on /dev/console before spawning crash shell

When pid 1 crashes, the getty unit for the console will happily keep
running which means we end up with two shells competing for the same
tty. Let's call vhangup on /dev/console to kill every other process
attached to the console before we spawn the crash shell. The getty
units have Restart=always but lucky for us, pid 1 just crashed in fire
and flames so it isn't actually able to restart the getty unit.

mkosi: Disable debuginfod

We generally don't care about library debuginfo so let's just disable
debuginfod so it doesn't get in the way when debugging.

We use /root/.gdbinit as the systemwide gdbinit location is distribution
specific.

meson: enable -Wunterminated-string-initialization

With af1a6db58fde8f64edcf7d27e1f3b636c999934c, now we can build with the
option.

Merge pull request #33951 from DaanDeMeyer/nspawn

nspawn: Allow specifying custom init program

basic|boot: silence Wunterminated-string-initialization gcc15 warnings

gcc15 has -Wunterminated-string-initialization in -Wextra and
warns about string constants that are not null terminated even though
the functions do do out of bounds access.
Silence the warnings by simply not providing an explicit size.

nspawn: Allow specifying custom init program

This allows for example forcing to use /sbin/init instead of always
using /usr/lib/systemd/systemd if it exists. Or it allows using a
different path altogether.

coredump: generate properly symbolized stacktrace for containerized processes

sysusers: check if requested group name matches user name in queue

When creating a user, check if the requested group name matches a user
name in the queue. If that matched user name is also going to be a group
name, then use it for the new user too. In other words, allow the
following:

 u foo -
 u bar -:foo

when both foo and bar are new users.

Fixes #33547

nspawn: Drop unused includes

Merge pull request #33942 from yuwata/udevadm-info-attribute-walk-json

udevadm-info: support json output for --attribute-walk

Merge pull request #33933 from yuwata/systemctl-bus-transport-and-runtime-scope

systemctl: fix bus transport and runtime scope handling

Merge pull request #33941 from yuwata/network-dhcp-pd-route-type

network/dhcp-pd: introduce UnassignedSubnetPolicy= to customize type of 'catch-all' route

systemctl: gracefully adjust bus transport and runtime scope when --boot-loader-entry=help

This fixes the following assertion:
===
SYSTEMD_LOG_LEVEL=debug systemctl --user -H foo --boot-loader-entry=help
Assertion 'transport != BUS_TRANSPORT_REMOTE || runtime_scope == RUNTIME_SCOPE_SYSTEM' failed at src/shared/bus-util.c:284, function bus_connect_transport(). Ignoring.
Failed to connect to bus: Operation not supported
===

Fixes a bug introduced by 97af80c5a7029c3f92e982dcf9338b9e67ad9cde.
Fixes #33661.
Fixes oss-fuzz#70153.

test-network: add test case for UnassignedSubnetPolicy=

network/dhcp-pd: allow to customize route type for delegated prefix

Closes #33929.

udevadm-info: support json output for --attribute-walk

Closes #33852.

src/pcrlock/pcrlock.c: Handle empty pcrlock.d directories

Running the following commands:

  # mkdir -p /var/lib/pcrlock.d/123-empty.pcrlock.d
  # /usr/lib/systemd/systemd-pcrlock predict --pcr=1+2+3+4+5+16

Will result in:

...
Floating point exception

Running the following commands:
  # mkdir -p /var/lib/pcrlock.d/123-empty.pcrlock.d
  # /usr/lib/systemd/systemd-pcrlock make-policy --pcr=1+2+3+4+5+16

Will result to this (partial) log:
...
Predicted future PCRs in 133us.
[]
...
Written policy digest 0000000000000000000000000000000000000000000000000000000000000000 to NV index 0x1921da6
...

So, add missing checks to handle gracefully cases where there's no variant
inside the component.

Signed-off-by: Arnaud Patard <arnaud.patard@collabora.com>

logind: add PreparingForShutdownWithMetadata property

The PrepareForShutdownWithMetadata signal was added via
e4aab5cf1a00bbb73f325f4f785dd4171ccdae77 but a corresponding property
was not. A property has to be a single type, so the bool needs to be
one of the key/value pairs as 'ba{sv}' is not a valid property.

Merge pull request #33912 from DaanDeMeyer/mkosi

test: Add a way to quickly iterate on an integration test

mkosi: Prevent busybox from getting pulled into opensuse images

OpenSUSE's busybox has a bunch of Provides for basic tools that cause
it to get pulled into images unless the corresponding tool is explicitly
installed so let's add explicit tools to make sure we don't get busybox.

systemd-analyze: Add svg scaling options

+ Scale the x-axis of the resulting plot by a factor (default 1.0)
+ Add activation timestamps to each bar

Signed-off-by: rajmohan r <rajmohan.r@kpit.com>

test: Add a way to quickly iterate on an integration test

Rebuilding the integration test every time is very slow. Let's
introduce a way to iterate on an integration test without rebuilding
the image every time. By making a btrfs snapshot before we run the
integration test, we can then systemctl soft-reboot after running
the test to restore the rootfs to a pristine state before running
the test again.

As /run/nextroot will get nuked on reboot or soft-reboot, we introduce
a tmpfiles snippet to make sure it is recreated every (soft-)reboot
and adapt the existing tests to deal with this new symlink.

mkosi: Switch back to btrfs

The next commit will introduce a way to iterate on integration
tests which depends on btrfs specific features.

We leave CentOS on ext4 as its kernel does not support btrfs.

mkosi: Enable Autologin= again on Debian

Debian uses /usr/bin/login from the shadow package instead of util-linux
which doesn't support credentials. Let's enable autologin the old
fashioned way for now.

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=833256

test: Implement TEST_SHELL for mkosi based test runner

TEST_SHELL can be used to get a shell in the integration test
environment without actually immediately starting the test.

test: Rename INTERACTIVE_DEBUG to TEST_SHELL

docs: Update upgrade commands in HACKING.md

- Add the required options to make the package managers non interactive
- Use apt-get instead of apt
- Remove --reinstall from apt-get command so we only install newer packages
- Add --needed to pacman command so we only install newer packages

mkosi: Drop locale tmpfiles snippet

Not required anymore as we don't do /usr only anymore and the symlink
will be created by the debian systemd package.

Merge pull request #33939 from yuwata/resolve-varlink

resolve: type field in result of ResolveService may be NULL

Merge pull request #33944 from yuwata/journal-max-retention-sec

journal: do not rotate journal when MaxRetentionSec= is set

journal: set flushed flag even if we fail to open runtime journals

As at this stage, a persistent journal file has been already opened, and
saved seqnum has been reset, and any later journal entries will be stored
to the file. Hence we should not open the runtime journal file by
server_system_journal_open() again.

journal: comment the default value in journald.conf

journal: do not rotate journal when MaxRetentionSec= is set

The setting is about vacuuming archived journal files. It is not
necessary to rotate the current journal. Note, journal file rotation is
controlled by MaxFileSec=.

Fixes #31315.

udevadm-info: trival cleanups

- drop trailing an empty line,
- use FOREACH_ARRAY() macro,
- add short comment for boolean argument.

resolve: refuse invalid service without type field

Fixes Fixes #33935.

resolve: voidify unused result of function call

network: split out core logic route_type_is_reject()

Preparation for later change.

sleep: add HibernateOnACPower= option (#33846)

* Add HibernateOnACPower= systemd-sleep configuration option

Merge pull request #33918 from YHNdnzj/exec-cred-cleanup

core/exec-credential: several cleanups

Merge pull request #33930 from yuwata/update-syscall-tables-and-linux-headers

Update syscall tables and linux headers

sd-event: change error code -EINVAL -> -EIO

EINVAL should be used when a function is called with an invalid
argument. Here, the signal is not a function argument.

Follow-up for 7a64c5f23efbb51fe4f1229c1a8aed6dd858a0a9.

basic/linux: update kernel headers from v6.11-rc1

seccomp: list fstatat, newfstat, and llseek

syscall-list: update syscall tables

This adds fstatat (and its friends), llseek, and uretprobe.

systemctl: refuse --capsule=foo with --system

Fixes the following assertion:
===
systemctl --capsule=hoge --system reboot
Assertion 'runtime_scope == RUNTIME_SCOPE_USER' failed at src/shared/bus-util.c:479, function bus_connect_transport(). Aborting.
Aborted (core dumped)
===

Follow-up for 56cb74c3cd1358d7d0b3f613feaf2eeab601a6bd.

core/exec-credential: do not use unlink_and_free for relative path under dfd

core/exec-credential: emit correct error on invalid cred source

The (!source && !search_path) branch is effectively not reached,
but rather confusing. Let's return -EINVAL properly.

core/exec-credential: use struct load_cred_args everywhere

We currently duplicate the same set of params for every funcs.
Let's unify this, and make things more manageable.

core/exec-credential: use maybe_decrypt_and_write_credential() for SetCred= too

core/exec-credential: drop misleading comment regarding EEXIST

Follow-up for 2c2ed3272b992b6eb13864e49847c943e42be00f

As explained in the referenced commit, we shall never get EEXIST
in the first place.

core/exec-credential: trivial coding style cleanup

core/dbus-execute: normalize "rename" arg properly

Follow-up for 40dd2a1c24b886d9e635fef8f7b5336700df03c3

Addresses https://github.com/systemd/systemd/pull/33916#discussion_r1702616229

po: Translated using Weblate (Croatian)

Currently translated at 82.8% (193 of 233 strings)

Co-authored-by: Marin Kresic <marinjurekresic@gmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/main/hr/
Translation: systemd/main

po: Translated using Weblate (Greek)

Currently translated at 37.7% (88 of 233 strings)

Co-authored-by: Dimitrys Meliates <demetresmeliates+fedora@gmail.com>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/main/el/
Translation: systemd/main

base-filesystem: do not attempt to create a /lib64 -> /usr/lib/<tuple> symlink

In multi-arch distributions (debian and derivatives) multiarch tuples under
/usr/lib are used, such as /usr/lib/x86_64-linux-gnu/ but the /lib64 symlink
should never point there, it should always point to /usr/lib64, as that's
how they are set up by distribution-specific tools.

https://packages.debian.org/bookworm/amd64/libc6-i386/filelist
https://packages.debian.org/bookworm/mipsel/libc6-mips64/filelist
https://salsa.debian.org/md/usrmerge/-/blob/master/convert-usrmerge?ref_type=heads#L295
https://salsa.debian.org/md/usrmerge/-/blob/master/convert-usrmerge?ref_type=heads#L517
http://bugs.debian.org/1076491

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

meson: Use -fstrict-flex-arrays=3

Let's explicitly pass the value to -fstrict-flex-arrays. This does
not change behavior but it does (selfishly) make my error not bug
out with an error saying -fstrict-flex-arrays does not exist.

hwdb: fix auto rotate on Asus Q551LB (#33921)

core/service: drop redundant flush_n_restarts indicator

Now that we track auto-restarts with a dedicated state,
there's no need for a separate variable for this.

I also took the chance to reorder some struct members.

Merge pull request #33925 from YHNdnzj/exec-serialize-path-escape

core/execute-serialize: two fixes

core/execute-serialize: use serialize_item_escaped() for external paths

Otherwise, read_stripped_line() would spuriously drop trailing spaces.

Fixes #33924

core/execute-serialize: drop extraneous '=' in ip-{in,e}gress serialization