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.

sd-dhcp6-client: add missing address existence check

This adds similar check as 58da18251f468de9de4cc7b36804c924e2fd4421.

Merge pull request #22471 from yuwata/network-bridge-vlan-protocol

network: bridge: fix vlan protocol

Merge pull request #22452 from yuwata/network-ipv6ll

network: use sysctl to set IPv6LL address generation mode when the interface is already up

test-network: add missing tests for bridge properties

network: bridge: fix endian of vlan protocol

Fixes #22469.

test-network: add testcase for re-generating IPv6LL address

network: attempt to trigger kernel IPv6LL address generation

Try to ensure kernel IPv6 link local address generation occurs by
setting the per-if addr_gen_mode sysctl when the link is already up,
instead of the netlink interface (IFLA_INET6_ADDR_GEN_MODE).

The netlink setting is sufficient in cases where the interface is not
yet up when networkd configures an interface - bringing the interface
up will trigger in-kernel address generation.

If the interface is already up, yet the interface has no IPv6LL assigned
setting IFLA_INET6_ADDR_GEN_MODE has no effect.

Writing the addr_gen_mode sysctl is a best effort attempt at triggering
address generation regardless of interface state because it also works
in cases where the interface is already up.

Fixes #22424.

network: move link_set_ipv6ll_stable_secret() to networkd-ipv6ll.c

network: skip to set IPv6LL address generation mode if the requested mode is already set

network: use log_link_warning_errno() or friends where applicable

network: monitor current IPv6LL address generation mode

network: introduce two helper functions for setting IPv6LL address generation mode

udev/net: support Match.Firmware= in .link files (#22462)

In cbcdcaaa0ec5 ("Add support for conditions on the machines firmware")
a new Firmware= directive was added for .netdev and .network files.
While it was also documented to work on .link files, in actual fact the
support was missing. Add that one extra line to make it work, and also
update the fuzzer directives.

Merge pull request #22467 from mrc0mmand/TEST-64-lvm-static-autoactivation

test: lvm 2.03.15 dropped the static autoactivation

test: lvm 2.03.15 dropped the static autoactivation

so install the respective generator only if we're running with older
lvm versions.

See: https://sourceware.org/git/?p=lvm2.git;a=commit;h=ee8fb0310c53ed003a43b324c99cdfd891dd1a7c

Clarify protocol used in systemd-journal-upload (#22465)

* Clarify protocol used in systemd-journal-upload

Merge pull request #22458 from poettering/parse-safe-string

some safety tweaks to conf-parser.[ch]

meson: disable export-dbus-interfaces target when cross-compiling

ERROR:
Cannot use target systemd as a generator because it is built for the
host machine and no exe wrapper is defined or needs_exe_wrapper is
true. You might want to set `native: true` instead to build it for
the build machine.

ci: remove MULTI_STATUS from superlinter

Judging by
ERROR! Failed to call GitHub Status API!

it doesn't seem to work. Even if it did it would just clutter the status
checks I think so let's just remove MULTI_STATUS along with
GITHUB_TOKEN.

Merge pull request #22248 from yuwata/network-tc-use-request-queue

network: tc: use request queue to configure QDisc or TClass

conf-parser: add specific parser for PID values

tree-wide: use config_parse_safe_string() at various places

conf-parser: update config_item_*_lookup() to follow modern coding style

Let's rename the return parameters ret_xyz, and always initialize them
if we return >= 0, as per our current coding style.

conf-parser: refuse section headers with control characters early, just in case

conf-parser: minor refactorings/modernizations

conf-parser: use _cleanup_free_ where appropriate

conf-parse: add generic config_parse_safe_string() helper

This helper is just like config_parse_string() but does some superficial
checks for control characters and quotes.

In most cases we currently use config_parse_string() we probably want to
use config_parse_safe_string() for safety reasons.

Add support for `isolated` parameter

Add the "Isolated" parameter in the *.network file, e.g.,

[Bridge]
Isolated=true|false

When the Isolated parameter is true, traffic coming out of this port
will only be forward to other ports whose Isolated parameter is false.

When Isolated is not specified, the port uses the kernel default
setting (false).

The "Isolated" parameter was introduced in Linux 4.19.
See man bridge(8) for more details.
But even though the kernel and bridge/iproute2 recognize the "Isolated"
parameter, systemd-networkd did not have a way to set it.

ci: validate actions and fix actionlint warnings

some actions like Coverity and CFLite aren't run on every PR so to make
sure they are more or less fine when they are changed it makes sense to
at least check them with superlinter/actionlint: https://github.com/rhysd/actionlint

The following warnings were fixed along the way:
```
.github/workflows/mkosi.yml:55:7: shellcheck reported issue in this script: SC2086:info:6:14: Double quote to prevent globbing and word splitting [shellcheck]
   |
55 |       run: |
   |       ^~~~
.github/workflows/mkosi.yml:55:7: shellcheck reported issue in this script: SC2046:warning:6:40: Quote this to prevent word splitting [shellcheck]
   |
55 |       run: |
   |       ^~~~
.github/workflows/mkosi.yml:55:7: shellcheck reported issue in this script: SC2006:style:6:40: Use $(...) notation instead of legacy backticked `...` [shellcheck]
   |
55 |       run: |
   |       ^~~~
```
```
.github/workflows/coverity.yml:31:9: shellcheck reported issue in this script: SC2086:info:1:93: Double quote to prevent globbing and word splitting [shellcheck]
   |
31 |         run: echo "COVERITY_SCAN_NOTIFICATION_EMAIL=$(git log -1 ${{ github.sha }} --pretty=\"%aE\")" >> $GITHUB_ENV
   |         ^~~~
```

network: tc: use hashmap to store traffic control settings in .network files

As now each qdisc or tclass is configured when it is ready.

network: tc: introduce order dependency of traffic control

network: tc: use request queue to configure traffic control

But no dependency resolution is implemented.

network: tc/teql: set tca_kind in verify()

Then, qdisc_configure() can be simplified.

network: tc: monitor qdisc and tclass

sd-netlink: allow to dump qdisc and tclass