shared/bootspec: use FOREACH_ARRAY more

shared/bootspec: use isempty where appropriate

shared/bootspec: use path_make_absolute where appropriate

shared/bootspec: inline iterator var

Also, do not bump 'line' until the end of the loop.
Otherwise, log_syntax() below logs about the wrong
line number.

shared/bootspec: add missing assertions

bootctl: fix crash when parsing addon without .cmdline section

Follow-up for 706ca67d3074b2a405ee8fe5de307416e4915b9f

update TODO

network: IPv6 Compliance RFC4862: Address Lifetime Expiry (Hosts Only) [v6LC.3.2.2]

RFC 4862 Section 5.5.3, bullet e, sub-bullet 3 applies to existing
addresses, i.e. when address_get() returns success. If the address is
new (i.e. address_get() fails), then we should not be adding 2 hours to
the lifetime_valid_usec. Instead, use the valid_lifetime from the RA's
Prefix Information Option.

This change allows v6LC.3.2.2 to pass. Also verified all v6LC3.2.* tests
pass. This covers all the v6LC tests from Group2: Router Advertisement
Processing and Address Lifetime.

Fixes #32652.

network: IPv6 Compliance: Router Advertisement Processing, Reachable Time [v6LC.2.2.15] (#32792)

Previously, RA option fields were being ignored when the Router Lifetime
value was zero. Remove this logic to be compliant with RFC4861.

Extract from: https://www.ietf.org/rfc/rfc4861.html#section-4.2, p.21,
first paragraph:

    The Router Lifetime applies only to
    the router's usefulness as a default router; it
    does not apply to information contained in other
    message fields or options.

This affected IPv6 Conformance test:
    v6LC.2.2.15: Router Advertisement Processing, Reachable Time.

Fixes #31842.

Co-authored-by: Matt Muggeridge <Matt.Muggeridge@hpe.com>

curl-glue: catch libcurl attempting to change timeout handler when we destroy a curl context on exit

If we destroy both an event loop and a curl contect object at the same
time, then we get into this weird situation where curl wants us to
reconfigure a timout event source right before destruction, which
sd-event will refuse however, since it is already being shutdown.

Hence, catch that and simply don't bother adjusting the timeout, since
we cannot get back from there anyway.

machined: return recognizable error when we try to register the same machine name twice

core/service: remove unnecessary reset of notify_access_override

This already gets reset in service_enter_dead()
and service_start().

varlink: rename vsockCid → vSockCid field

hwdb: ASRock LED Controller classified incorrectly as joystick due to buttons and axis (#32775)

The function `builtin-input_id` incorrectly identifies the ASRock LED Controller
as an input device due to the presence of buttons and axis. To fix this we add
a new rule in `hwdb.d/60-input-id.hwdb`.
The properties have been set to empty instead of `0` because some programs
might check if the value is set at all instead of checking its value, as discussed
in #32773.

The device has no real keys. The devices is controlled by i2c interface and some
settings in UEFI, and it provides an header to connect LED strips and similar devices.
I suppose it's possible that ASRock intended to connect devices with buttons for
controlling LEDs to it, but: (i) the controller itself does not have key, (ii) to my
knowledge no such device exists. So I think we can unset that property as well.
On a sidenote, unsetting those properties does not affect the i2c interface,
OpenRGB still interacts normally with the device.

Fixes #32773.

TEST-38-FREEZER: Relax regex a little

The state might be "freezing-by-parent" as well so let's take that
into account.

Fixes #32746

hwdb: ID_INPUT_XYZ allows an empty string

This is for making CIs for #32775 green.

Merge pull request #32758 from yuwata/journal-ratelimit

journal: modernize journal ratelimiting

build(deps): bump pkg/arch from `b13e94a` to `b578e90`

Bumps pkg/arch from `b13e94a` to `b578e90`.

---
updated-dependencies:
- dependency-name: pkg/arch
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

build(deps): bump pkg/debian from `7cc898c` to `824c52f`

Bumps pkg/debian from `7cc898c` to `824c52f`.

---
updated-dependencies:
- dependency-name: pkg/debian
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

man: add v257 tag

Even though v256-final is not released yet, let's v257 tag now to make
not PRs for v257 conflict with each other.

journal-rate-limit: replace in-house management of JournalRateLimitGroup with OrderedHashmap

No functional change, just refactoring.

hwdb: updated Librem 11 accelerometer (#32772)

hwdb: update Dere N12 / Juno Tablet 3 accelerometer (#32765)

po: Translated using Weblate (Hindi)

Currently translated at 100.0% (233 of 233 strings)

Co-authored-by: Scrambled 777 <weblate.scrambled777@simplelogin.com>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/main/hi/
Translation: systemd/main

journal-rate-limit: split out journal_ratelimit_group_acquire()

No functional change, just refactoring.

journal-rate-limit: introduce cleanup function for JournalRateLimitGroup

This also makes journal_ratelimit_group_new() use structured initializer.

journal-rate-limit: rename variables

'r' is usually used for storing return value.

journal-rate-limit: use FOREACH_ELEMENT() and usec_add()

journal-rate-limit: make journal_ratelimit_group_free() return NULL

To make it follow our recent coding style.

journal-rate-limit: add trailing comma in table

test: introduce test cases for journal_ratelimit_test()

tree-wide: use LOG_PRI() and LOG_FAC()

debug-generator: Allow specifying name of unit-dropin credential

A fixed name is too rigid, let's give users the ability to define
custom drop-in names which at the same time also allows defining
multiple dropins per unit.

We use ~ as the separator because:
- ':' is not allowed in credential names
- '=' is used to separate credential from value in mkosi's --credential
  argument.
- '-' is commonly used in filenames
- '@' already has meaning as the unit template specifier which might be
  confusing when adding dropins for template units

Merge pull request #32755 from yuwata/test-network-cleanups

test-network: several cleanups

test-network: simplify and unify waiting loop

Also,
- drop unused fail_assert arguments,
- use wait_links() and check_networkd_log() in wait_activated().

test-network: make link_exists() take multiple arguments

This also improves performance of test_delete_links().

Fix Positivo N14EPE and N15EPE key toggle touchpad and search key

libcrypt-util: fix wrong errno value assignment

Follow-up for 9de324c3c919f20fd49e1d25579f5a66cac0eaa0.

Merge pull request #32747 from YHNdnzj/tmpfiles-ret-gather

tmpfiles: some cleanups/follow-up

Merge pull request #32748 from yuwata/test-network-improvements

test-network: several more improvements

test-network: check existence of kernel bug

This adds checks for the kernel bug caused by
https://github.com/torvalds/linux/commit/3ddc2231c8108302a8229d3c5849ee792a63230d,
it will be fixed by
https://patchwork.kernel.org/project/netdevbpf/patch/20240510072932.2678952-1-edumazet@google.com/

test-network: introduce --no-journal option

This should be useful when the test run as a service, e.g.
running on a mkosi image.

Merge pull request #32738 from yuwata/trivial-cleanups

Trivial follow-ups for recent changes

NEWS: update for latest changes

Merge pull request #32491 from yuwata/journalctl-fix-boot

journalctl: several fixes and cleanups for --boot= option handling

tmpfiles: use RET_GATHER more, add missing assertions

Note that item_do() now aborts on OOM, since it's
pretty pointless to iterate further if memory allocation
doesn't work.

tmpfiles: don't compare errno with negative value

Follow-up for 677430b3c7fcd1b352eb66f19b8746741459b91a

tmpfiles: clean up hardlinks_vulnerable a bit

dangerous_hardlinks() -> hardlinks_protected(),
and the meaning of the function is now in line
with fs.protected_hardlinks value.

Plus, We ship 50-default.conf where the sysctl
is enabled. Mention it in the comment.

Merge pull request #32741 from yuwata/network-dhcp4-route-to-dns

network/dhcp4: fix assignment of routes to DNS or NTP servers

Merge pull request #32739 from yuwata/vmspawn-journal-forward

vmspawn: fix journal forwarding

mkosi: Update centos submodule to latest

Merge pull request #32743 from yuwata/test-network-for-mkosi

test-network: several improvements for running with mkosi

core/unit: use FOREACH_ELEMENT() to add dependencies for journal namespace instances

Follow-up for 06c0f569e94cc7406886079d241f1c1f82711937.

test: wait for network interface reconfigured after updating config

Otherwise, at this stage, the interface may be in e.g. initialized or
pending state, and the drop-in file introduced by the previous command
may not be registered to the state file for the interface.

Fixes #32685.

test-network: do not fail when /etc/protocols does not exist

Also this makes several checks more strict.

test-network: do not fail test_macvlan() with old kernel or ip command

test-network: do not fail if macvlan module is not available

network/dhcp4: do not set gateway if DNS server or friends is in the acquired prefix

Previously, even if a DNS server is in the acquired prefix, the route to the
server might have gateway address.
This makes the prefix route, which is always configured, is also handled
as same as static routes, and do not use any gateway if the prefix route
is the most suitable route to access the destination.
The same change is also applied to route to NTP servers and semi-static
routes.

Fixes a regression introduced by 0ce86f5eeb0921b44a9782260a8c88aafb15ffde.

Fixes #32715.

test: add basic tests for in_addr_prefix_covers_full()

journalctl: make --list-boots support -n/--lines= option

Also mention that -r/--reverse is supported by the command.

journalctl: fail and show error message when no boot ID found

No boot ID in journal should be definitly spurious.
Let's warn about that and exit with failure.

logs-show: use GREEDY_REALLOC_APPEND()

logs-show: fix stored timestamp when advance_older is true

Currently, the parsed timestamp is only used when advance_older is
false. Hence, this does not change any behavior. But, let's fix it anyway.

journalctl: fix support of --boot=ID±offset format

Fixes a regression introduced by e44f06065bf20e8d0e4adacff61350ebd36f299e.

After the offending commit, if a boot ID suffixed with an offset is
specified to --boot=, the boot ID was ignored.
This fixes the issue.

To fix the issue, this merges journal_find_boot_by_id() and
journal_find_boot_by_offset().

logs-show: flush matches before and after finding boots

Otherwise, if several matches already set, then the first seek to head
or tail may move the cursor to an invalid place, hence they provide
wrong ID(s). Also, reading journal after calling these function may
provide unexpected data.

Currently, the caller does not install any matches before calling the
functions, and does not read any journal entry after journal_get_boots()
succeeds or journal_find_boot_by_offset() succeeds with 0. Hence, this
should not change any behavior. Just for safety.

journalctl: fix --boot=0 with --file=- (from stdin)

Follow-up for 592855c3189549fed93b1060b72299910c6ab1d0.

journalctl: split out journal_acquire_boot() from add_boot()

No functional change, just refactoring and prepration for later changes.

vmspawn: fix finding systemd-journal-remote binary

Follow-up for 258d26940fb6bfa1b5b8acf03cc4a37f2657e4ac.

Fixes https://github.com/systemd/systemd/pull/31218#discussion_r1596168449.

vmspawn: trivial cleanups for start_systemd_journal_remote()

- prefix argument for storing result with 'ret_',
- align arguments for strv_new().

core/service: shorten code a bit

Follow-up for 026335e9aab5ccf1fc9477215beadd0acef94073.

time-util: define TIMESPEC_OMIT

Follow-up for 34c3d574742e867ef97e79509e4051a82f1b7d9b.

journalctl: several cleanups for parse_boot_descriptor()

- rename to parse_id_descriptor(), to make it usable for other kind of
  ID later.
- add missing assertions,
- prefix arguments for storing results with 'ret_',
- drop unnecessary 'else'.

logs-show: add missing strempty()

Follow-up for 8e976dc9209853d5d4d2db3016289f2a5ab99fd9.

logs-show: drop uid argument from add_matches_for_user_units()

It is always equivalent to getuid(). Let's call getuid() in the
function instead.

basic/linux: Copy netfilter.h to the source tree

This fixes build with old toolchains prior to Linux < 4.2 which do not
have a definition for NFPROTO_NETDEV.

shell-completion: on zsh, systemd-cat is a precommand

_precommand lets zsh complete other commands and their arguments
e.g. it can complete grep with "systemd-cat gr" and complete grep options
with "systemd-cat grep -"

network: Add missing IPv6AcceptRA to list of config sections

Merge pull request #32689 from YHNdnzj/cred-missing

core/exec-credential: complain louder if inherited credential is missing

Merge pull request #32694 from weblate/weblate-systemd-main

Translations update from Fedora Weblate

Merge pull request #32733 from YHNdnzj/core-some-cleanup

core: minor cleanups

po: Added translation using Weblate (Hindi)

Co-authored-by: Scrambled 777 <weblate.scrambled777@simplelogin.com>

po: Translated using Weblate (German)

Currently translated at 97.8% (228 of 233 strings)

Co-authored-by: Jarne Förster <fedora@mymailclient.de>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/main/de/
Translation: systemd/main

mkosi: Drop usage of updates-testing for Fedora 40

The util-linux update we need has migrated to the stable repositories.

TEST-46-HOMED: Ignore "Disk Usage" field as well

This can change between the call to homectl inspect and userdbctl
user so let's ignore it along with the other disk fields.

Fixes #32727

core/namespace: take char* const* for strv, use FOREACH_ARRAY

path-util: take char* const* for strv where appropriate

core/unit: use FOREACH_ARRAY at one more place

core/unit: cast unused retval to void

core/dbus-unit: correct argument alignment for SD_BUS_METHOD_WITH_ARGS

Merge pull request #32729 from DaanDeMeyer/kernel-command-line

mkosi: Extend kernel command line

Merge pull request #32701 from bluca/vmspawn/machinectl-shell

vmspawn: implement machinectl shell for vm class machines and send termination signal via D-Bus-over-SSH

mkosi: Make sure the kernel fails loudly on oops/panic/softlockup

Helps with noticing kernel related failures.

mkosi: Add raid=noautodetect to kernel command line

Should speed up boot times a little.

vmspawn: forward signals to VM PID 1 via D-BUS when available

Allow to gracefully shutdown by initiating the operation from
the guest

vmspawn: register with io.systemd.Machine.Register

When available, use varlink to register the VM, so that we can talk
over SSH to the guest. Enables 'machinectl shell' for vmspawn guests.

vmspawn: add a dropin override to sshd-vsock@.service

systemctl-show: make show_memory_available cover memory_available itself

Follow-up for f380473edfa899706d630bb64750ab50c5c04961

This cleans up the code a bit. Also, before this commit,
if MemoryAvailable is set but show_memory_available
is false, and we have nothing else to output, empty
parenthesis is shown. This can be easily reproduced
on -.slice:

> systemctl status -- -.slice
> ...
> Memory: 1.8G ()
> ...

Merge pull request #32717 from keszybz/very-important-fixes

Whitespace adjustments and comments

docs,man: Avoid some ambiguous uses of "may not"

Like much English text, the systemd documentation uses "may not" in the
sense of both "will possibly not" and "is forbidden to".  In many cases
this is OK because the context makes it clear, but in others I felt it
was possible to read the "is forbidden to" sense by mistake: in
particular, I tripped over "the target file may not exist" in
systemd.unit(5) before realizing the correct interpretation.

Use "might not" or "may choose not to" in these cases to make it clear
which sense we mean.

pidfd: properly detect if libc offers pidfd syscalls and make use of them then

We never updated the meson checks when glibc finally learned about these
syscalls, address that.