Merge pull request #22516 from keszybz/bit-and-pieces

Post-review fixups and other tiny improvements

Revert "man: fix description of --force in halt(8) (#7392)"

This reverts commit 5d9adb5b60b815b477ba9e6b19ef0fd7e1854a38.

Merge pull request #22412 from yuwata/sd-dhcp6-client-cleanups

sd-dhcp6-client: several fixes and cleanups

homework: avoid runtime formatting of a fixed string

Follow-up for fe9bd5ad3670f6a34f9ea9b4e2c16bec6000ce11.
I checked that the formatted string has the same dashes as
before fe9bd5ad3670f6a34f9ea9b4e2c16bec6000ce11.

share/gpt: add stringified defines for GPT partition types

NEWS: minor grammar adjustment

Follow-up for e41dcb822167116c8924e56b39640536df894bbd.

systemctl: expand error message

docs: more internal links

test: set `ReadWritePaths=$BUILD_DIR` when running w/ coverage

otherwise units using `DynamicUser=yes` won't be able to write the
coverage stats (currently affecting TEST-20-MAINPIDGAMES).

`DynamicUser=yes` implies `ProtectSystem=strict` and
`ProtectHome=read-only` and can't be overridden hence we need to
utilize `ReadWritePaths=` to work around that.

Merge pull request #22487 from poettering/bootspec-source-flags

bootctl: show all discovered entries, but show state + type in details

Merge pull request #22508 from poettering/stat-ino-compare

stat-util: add new stat_inode_same() helper

Merge pull request #22507 from poettering/id128-compare-tweaks

sd-id128: comparison tweaks

docs: make clear that if you use threaded cgroups you need to do that two levels down from your delegated cgroup

Prompted by: #22486

sd-daemon: use path_join() instead of manual path concat

coredump: fix inode check

When checking if we look at the root directory we actually need to
compare both st_dev *and* st_ino. The existing check only checked the
latter. Fix that.

fd-util: use ERRNO_IS_XYZ() macros where appropriate

tree-wide: port various places over to new stat_inode_same() helper

stat-util: add helper stat_inode_same() for comparing stat's st_dev/st_ino in one

We do this all over the place, hence let's add a simple helper that does
this and particularly carefully and thoroughly.

Merge pull request #22506 from poettering/devnum-zero-btrfs-block-dev

gpt-auto: some (primarily cosmetic) fixes to backing block device detection in gpt-auto-generator/sd-device

Merge pull request #22445 from lnussel/logind

logind: post review fixes

bootspec: avoid zero size VLA

apparently some checkers don't like that. Let's be entirely safe here,
and use malloc() based allocation, given that the entries are user
controlled.

boot: suppress XBOOTLDR if same device as ESP when enumerating entries

On my local system I linked up the ESP and XBOOTLDR partitions, and
ended up with duplicate entries being listed. Try hard to detect that
and only enumerate entries in the ESP if it turns out that both dirs
have the same dev_t.

This should detect both bind mounted and symlinked cases and should make
our list output less confusing.

bootspec: make sure all return values are initialized on return of find_esp_and_warn()

THis makes sure that find_esp_and_warn() + find_xbootldr_and_warn()
follow our usual coding style that on success all return values are
initialized. We got that right in most successful codepaths out of these
functions, but missed the one where the paths are manually overwritten
via env vars.

bootctl: show more information about boot entry state in list

Let's improve display of boot entries and show what type they have (i.e.
boot loader spec type 1, or type 2, or auto-discovered or reported by
boot loader), and in particular mark entries the boot loader discovered
but we can't find (i.e. that likely vanished, or possibly couldn't be
found due to a misconfiguration) and that the boot loader didn't find
but we see (which are new, or possibly also the result of
misconfiguraiton).

This is supposed to be a replacement for #22161, but instead of hiding
vanished entries, highlights them, which I think is more appropriate for
a low-level tool such bootctl.

Replaces: #22161 #22398

bootspec: also collect/mark the "selected" boot entry (i.e. the one currently booted)

it's helpful and easy, so let's do it

bootspec: also parse new 'beep' loader.conf variable

bootspec: parse/show devicetree-overlay field too

It has been defined in the boot loader spec, and is the only field we
currently don't parse, hence fix that.

update TODO

bootspec: port one more use of basename() to path_extract_filename()

Revert "bootctl: Ignore boot entries (continue #22041)"

veritysetup: whitespace fix

homed: use SD_ID128_TO_UUID_STRING() at one more place

tree-wide: use id128_equal_string() at various places

id128-util: add new helper id128_equal_string()

Quite often we compare uuids/id128 formatted as strings with specific
values. So far we usually used streq() for that. let's add a new
explicit helper for this in id128_equal_string() that compares a string
with an sd_id128_t and is more robust than a simple string comparison.
Moreover, we can mroe easily reuse the various defines we have for
specific UUIDs, for example those from gpt.h.

sd128: export sd_id128_to_uuid_string()

We expose various other forms of UUID helpers already, i.e.
SD_ID128_UUID_FORMAT_STR and SD_ID128_MAKE_UUID_STR(), and we parse
UUIDs, hence add a high-level helper for formatting UUIDs too.

This doesn't add any new code, it just moves some helpers
id128-util.[ch] → sd-id128.[ch], to make them public.

Merge pull request #22505 from mrc0mmand/more-coverage-tweaks

A couple of coverage-related tweaks

test: check if running in container earlier

Merge pull request #22496 from yuwata/network-cleanups-keep-configuration

network: cleanups for KeepConfiguration= setting

gpt-auto: properly handle case where we can't determine devno of /usr/ fs

get_block_device_harder() returns == 0 if the fs is valid, but it is not
backed by a single devno. (As opposed to returning > 0 if the devno is
valid). Let's catch this case and log a clear message, and don't bother
open the device in that case.

This is mostly cosmetical, as either way, systemd-gpt-auto-generator
doesn't work in scenarios like that.

Prompted-by: #22504

sd-device: refuse opening device mit major/minor of zero early

device nodes with 0 dev_t are not real (and for that reason such inodes
are used as whiteouts in overlayfs, for example), hence refuse them
early. It seems wrong going to sysfs for something we know can't exist
anyway.

test: check for (possibly) missing test coverage

If the test logs contain lines like:

```
...systemd-resolved[735885]: profiling:/systemd-meson-build/src/shared/libsystemd-shared-250.a.p/base-filesystem.c.gcda:Cannot open
```

it means we're possibly missing some coverage since gcov can't write the stats,
usually due to the sandbox being too restrictive (e.g. ProtectSystem=yes,
ProtectHome=yes) or the $BUILD_DIR being inaccessible to non-root users.

test: set ACLs for the build dir when built with coverage

Otherwise unprivileged processes (like systemd-resolved) can't write
coverage data.

do not call __register_atfork directly

this way it is cleaner and more portable and systemd links against libpthread anyway

systemctl: fix halt -f

Commit adefc8789b always asks logind for shutdown first. So I broke halt
-f which is supposed to issue a direct syscall in that case.

logind: more verbose struct initialization

logind: increase max wall message length to 4096

logind: style fixes

logind: add handle enum to struct

Avoid pointer arithmetic at the expense of repeating the enum value
in the struct.

test-dhcp6-client: add test for rapid commit

test-dhcp6-client: cleanups

This does many cleanups. Most prominently,
- check the sents packet more strictly,
- make the packets used in the test more readable,
- unify callbacks.

fuzz-dhcp6-client: add prefix hint and vendor option to sent message

fuzz-dhcp6-client: merge with fuzz-dhcp6-client-send

fuzz-dhcp6-client: test multiple states

Then, now all three paths client_process_information(),
client_process_reply(), and client_process_advertise_or_rapid_commit_reply()
in client_receive_message() are fuzzed.

sd-dhcp6-client: append extra options before elapsed time option

The value of elapsed time option is determined in runtime and not
deterministic. It is hard to test the sent packets if it is located
in the intermidiate of the packet.

sd-dhcp6-client: logs invalid NTP option

Returning negative errno triggers to produce debugging log in
dhcp6_lease_parse_message().

sd-dhcp6-client: split dhcp6-internal.h into two

Also, this moves string tables to dhcp6-protocol.c.

network: do not restart DHCPv6 client when it is already running in managed mode

Otherwise, if a network has two router and one provides M flag and the
other provides O flag, then the DHCPv6 client switches the running mode
repeatedly. That makes the IPv6 network configuration of the host
unstable.

sd-dhcp6-client: reset Information Refresh Time on stop

sd-dhcp6-client: add log about Information Refresh Time

And store IRT only when the client is running in information
requesting mode.

sd-dhcp6-client: use memcmp_nn() at one more place

dhcp-identifier: generate static and constant DUID-EN when the client is running in test mode

Follow-up for 9216fddc5a8ac2742e6cfa7660f95c20ca4f2193.

dhcp-identifier: introduce dhcp_identifier_set_duid()

dhcp-identifier: introduce duid_type_to_string()

sd-dhcp6-client: do not expose set_transaction_id()

This is mostly for tests or fuzzers. Hence, this makes the function
requires that the client is running in the test mode.

Also, now the function mask the value for message type.

sd-dhcp6-client: use OrderedSet for vendor option

This also fixes memleak.

ordered-set: introduce ordered_set_clear()

sd-dhcp6-client: use struct hw_addr_data

sd-dhcp6-client: stop IO event source when client entered bound state

Also when the client receives a reply when running in information
requesting mode.

sd-dhcp6-client: adjust assertions

sd-dhcp6-client: log state transition

sd-dhcp6-client: rename client_set_state() -> client_start_transaction()

Previously, it obeys state transition, but now it is only used when
(re-)starting transaction. Let's rename again in more explanative name.

sd-dhcp6-client: use proper type for string table lookup funcs

sd-dhcp6-client: voidify client_set_state()

sd-dhcp6-client: log message is processed before state is changed

sd-dhcp6-client: move client_timeout_resend() and client_set_state()

These are deeply related to client_message_send(). Hence, move them near
the function.

sd-dhcp6-client: move client_ensure_iaid()

No functional change.

sd-dhcp6-client: merge client_start() and client_reset()

sd-dhcp6-client: use event_reset_time_relative() at one more place

sd-dhcp6-client: max_retransmit_count is only used when client is in DHCP6_STATE_REQUEST

sd-dhcp6-client: drop unnecessary event_source_disable()

The event source will be soon re-enabled in the same function.

The function client_timeout_resend() may return earlier without
re-enabling the timer souce. However,
- the timer event source is one shot by default. Hence, it is not
  necessary to disable in the callback function,
- when it returns early, then client_set_state() or client_stop() is
  called before return, and they re-ernable or disable the timer.

sd-dhcp6-client: set lease expiration timer in client_enter_bound_state()

The timer must be active until the client get re-enter bound state,
and the timeout must be determined by the lease acquired when entering
the bound state.

sd-dhcp6-client: drop unnecessary assignment

sd-dhcp6-client: initialize IO event source in sd_dhcp6_client_start()

sd-dhcp6-client: make state transition stricter

sd-dhcp6-client: prohibit spurious state transition

Previously, `client_set_state(client, DHCP6_STATE_STOPPED)` might make
the client enter the SOLICIT state.

sd-dhcp6-client: disable T1 timer on T2

Otherwise, the client may be enter RENEW state after REBIND state.

sd-dhcp6-client: call client_notify() in client_enter_bound_state()

sd-dhcp6-client: fix lifetime handling

This fixes the following issues:
- if T1 is finite and T2 is infinite, then the timer for T1 was not
  enabled,
- after T1 and T2 are randomized, T1 may be longer than T2.

sd-dhcp6-client: use event_reset_time_relative()

sd-dhcp6-client: introduce client_enter_bound_state()

To make client_set_state() smaller. This should not change any behavior.

sd-dhcp6-client: fix overflow in calculating timeout value

sd-dhcp6-client: disable event source when client is stopped or freed

sd-dhcp6-client: use in6_addr_to_string()

sd-dhcp6-client: stop client on error in client state transition

sd-dhcp6-client: add missing one more error handling

sd-dhcp6-client: rename client_start() -> client_set_state()

sd-dhcp6-client: introduce client_process_*()

This is mostly for shortening `client_receive_message()`.

This also fixes the following:
- do not trigger SD_DHCP6_CLIENT_EVENT_INFORMATION_REQUEST notification
  when non-reply message,
- add several missing error handlings caused in client_start().

sd-dhcp6-client: fix possible unaligned read or write

sd-dhcp6-client: voidify client_reset()

sd-dhcp6-client: introduce dhcp6_lease_new_from_message()