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()

sd-dhcp6-client: use structured initializer

sd-dhcp6-lease: unify lease lifetime calculation

sd-dhcp6-client: always use ENODATA when a lease does not have requested data

sd-dhcp6-client: unify dhcp6_option_append_{ia,pd}()

sd-dhcp6-client: store PD prefix hint in ia_pd

And allows to specify multiple hints.

sd-dhcp6-client: introduce dhcp6_ia_free()

sd-dhcp6-client: unify IA option header

This also fixes possible unaligned read of message.

sd-dhcp6-lease: reset client or server ID when length is zero

Addresses https://github.com/systemd/systemd/pull/22406#issuecomment-1029391091.

sd-dhcp6-lease: convert assert_return() -> assert() in non-public functions

sd-dhcp6-client: rename ia -> ia_na, pd -> ia_pd in sd_dhcp6_lease

The previous naming is quite misleading.

sd-dhcp6-client: fix typo ia_pd -> ia_na

Fortunately, currently both ia_pd and ia_na in sd_dhcp6_client are
equivalent now.

sd-dhcp6-client: introduce two helpers to create message

sd-dhcp6-client: trigger assertion whn invalid IA type is provided

If the condition hits, then it is an error in coding, instead of a user
misconfiguration or invalid receivd message.

Merge pull request #22499 from mrc0mmand/ci-tweaks

A couple of Packit/Super-Linter tweaks

ci: re-enable s390x Packit builds

The s390x Copr builders are finally[0] 'native' instead of QEMU-emulated,
which makes them pretty quick to be used in PRs.

[0] https://lists.fedoraproject.org/archives/list/copr-devel@lists.fedorahosted.org/message/AR3ZDKET3EXZHV3MSU3UHMO7EIKBGAN2/

ci: explicitly disable multi-status for Super-Linter

to, hopefully, get rid of the following error:

```
 2022-02-13 13:32:12 [ERROR]   Failed to get [GITHUB_TOKEN]!
 2022-02-13 13:32:12 [ERROR]   []
 2022-02-13 13:32:12 [ERROR]   Please set a [GITHUB_TOKEN] from the main workflow environment to take advantage of multiple status reports!
```

ci: use the 'slim' version of Super-Linter

The 'slim' version drops certain storage-heavy linters[0] which we don't
use anyway, so let's make the job a bit faster by downloading and using
a smaller image.

[0] https://github.com/github/super-linter#slim-image

NEWS: clarify that ForceDHCPv6PDOtherInformation= is removed

Closes #22493.

network: drop managed configs on reconfigure when KeepConfiguration=yes

Otherwise, the managed configs, that is addresses, routes and so on
configured by the previously assigned .network file will not be dropped
on reconfiguring the interface.

network: drop redundant condition

The function `link_drop_foreign_addresses()` is only called in
`link_drop_foreign_config()`, and the same condition is located in the
caller.

cgroup-util: refuse the case that both path and suffix are empty strings

Fixes CID#1322378.

boot: Correctly check the return value of CheckEvent

Fixes: #22428

Merge pull request #22485 from poettering/cgroup-id-not-support

cgroup: downgrade warnings of "cgroup ID" can't be read (on old kernels)

ci: run all fuzz targets on CIFuzz

CIFuzz has been kind of broken for a couple months because
coverage reports downloaded from OSS-Fuzz contain absolute
paths while paths to files changed in PRs are relative and they
don't match. It makes it kind of hard for CIFuzz to figure out
what it should run so it runs either all fuzz targets or just new
fuzz targets. Until that issue is fixed let's just always predictably run
all fuzz targets.

pid1: export cgroup ID among per-unit cgroup information

It's really interesting for debugging purposes and we have it already,
hence expose it as dbus property.

cgroup-util: minor modernizations

Rename return parameters to "ret", use ternary op without second
argument, rebreak comments, use isempty() more.

cgroup: downgrade warning if we can't get ID off cgroup

The cgroupid feature was not available in old cgroupvs2 kernels, hence
try to get it but if we can't because it's not supported, then only
debug log about it and proceed.

(We only needs this for cgroup bpf stuff, but that isn't available on
such old kernels anyway)

Fixes: #22483