sd-dhcp-server: use sd_dhcp_client_id

fundamental: prefer byte swap builtins over byte swapping manually

This builtin reduces complexity and GCC/Clang have supported these builtins for a long time.

Merge pull request #30702 from yuwata/sd-dhcp-client-id

dhcp: introduce sd_dhcp_client_id and relevant functions

backlight: supprt ID_LEDS_CLAMP udev property for leds subsystem devices

Closes #30507.

udev/dmi-memory-id: update table with latest SMBIOS specification

Closes #30699.

Merge pull request #30700 from yuwata/storagetm-fixlets

storagetm: several trivial fixlets

network/route: drop TTL propagate support for MPLS routes

This effectively reverts 9b88f20aba6a78baf18d89e99ff31d5ee40856b8.

We do not support MPLS routes, only IPv4 or IPv6 routes are supported.

network/queue: fix potential double-free on oom

Currently, link_queue_request_safe(), which is a wrapper of
request_new(), is called with a free function at
- link_request_stacked_netdev() at netdev/netdev.c,
- link_request_address() at networkd-address.c,
- link_request_nexthop() at networkd-nexthop.c,
- link_request_neighbor() at networkd-networkd.c.

For the netdev case, the reference counter of the passed object is increased
only when the function returns 1. So, on failure (with -ENOMEM)
previously we unexpectedly dropped the reference of the NetDev object.
Similarly, for Address and friends, the ownership of the object is moved to the
Request object only when the function returns 1. And on failure, previously
the object was freed twice.

Also, netdev_queue_request(), which is another wrapper of request_new()
potentially leaks memory when the same NetDev object is queued twice.
Fortunately, that should not happen as the function is called only once
per object.

This fixes the above issue, and now the ownership or the reference
counter of the object is changed only when it is succeeded with 1.

Merge pull request #30691 from yuwata/resolve-ipv6

resolve: do not listen to IPv6 when disabled by sysctl

test: rewrite test-exec-deserialization.py

Rewrite the test in bash and make it part of our integration test suite,
so it's actually executed in all our upstream CI environments.

The original test is flaky in environments where daemon-reload might
occur during the test runtime (e.g. when running the test in parallel
with the systemd-networkd test suite). Also, it was run only in CentOS
CI in limited way (i.e. without sanitizers), since it tests the host's
systemd, instead of the just built one.

Resolves: #29943

various: unexport a few internal structs

We don't expose destructors for these internal structs already.
Let's make the struct itself implementation detail too.

sd-dhcp-lease: use sd_dhcp_client_id

dhcp: move sd_dhcp_client_id_to_string() to sd-dhcp-client-id.[ch]

Then, this makes it take sd_dhcp_client_id object, and introduce
sd_dhcp_client_id_to_string_from_raw().

dhcp: introduce sd_dhcp_client_id and relevant functions

This splits out client ID handling from sd-dhcp-client.c to
sd-dhcp-client-id.[ch]. This will be used later in other places.

storagetm: ensure we pass dev_t* to sd_device_get_devnum

On MIPS32 OABI, st_rdev is unsigned long, not dev_t. Use a temporary
variable to avoid an incompatible pointer.

Bug: https://bugs.gentoo.org/920576
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=21278
Fixes: https://github.com/systemd/systemd/issues/30626

storagetm: fix use of wrong stat element

storagetm: always hash stat.st_mode

To make the hash function consistent with the compare function.

unit: order systemd-resolved after systemd-sysctl

Otherwise, IPv6 enable/disable setting may be changed after resolved is
started.

resolve: do not listen to IPv6 when disabled by sysctl

Fixes #30669.

Merge pull request #30689 from mrc0mmand/even-more-cocci-tweaks

coccinelle: drop a couple of FIXMEs

coccinelle: add a rule for in_addr_hash_func()

Follow-up for c01a5c0.

coccinelle: drop a couple of FIXMEs

Turns out Coccinelle can handle compound literals just fine, the parsing
errors were caused by incorrectly parsed macros in code before the
literals, so let's just provide simplified versions for such macros.

The parsing error in `Type *foo[ELEMENTSOF(bar)] = {};` is actually
harmless; it occurs only when creating an array of pointers for a type
that's in an external header and it occurs only on the first parser's
pass, subsequent passes resolve the type correctly.

Also, unset ENABLE_DEBUG_HASHMAP, so Coccinelle doesn't expand the
hashmap debug macros.

As for the remaining FIXMEs, I opened a couple of issues in the
Coccinelle upstream to see if they can be fixed there (or at least
properly analyzed).

Merge pull request #30684 from systemd/update-labeler-configuration

ci: migrate labeler configuration to the new format

ci: allow testing changes made to labeler configuration

ci: use a boolean value for the boolean field

The issue[0] behind this workaround has been resolved[1], so we can set it
to a proper boolean field.

[0] https://github.com/systemd/systemd/issues/18671
[1] https://github.com/actions/labeler/pull/480

ci: migrate labeler configuration to the new format

Turns out updating the labeler action is a bit annoying[0], so the
breaking change wasn't detected in the version bump PR.

[0] https://github.com/actions/labeler/#notes-regarding-pull_request_target-event

Follow-up to f88c9b0728.

sd-journal: introduce cleanup function and hash ops for Directory

This makes the folloing:
- Each Directory object now has a reference to sd-journal.
- Hence, directory_free(), which is renamed from remove_directory(), can
  be called without sd-journal as an argument.
- Introduces hash ops for Directory, so the finalization becomes
  slightly simpler.
- Allocate hashmaps that store Directory objects when necessary.
- Split out add_directory_impl().

No functional changes, just refactoring.

Merge pull request #30591 from yuwata/device-util

device-util: introduce device_in_subsystem() and device_is_devtype() helper functions

Merge pull request #30593 from yuwata/sd-dhcp-duid

dhcp: introduce sd_dhcp_duid and relevant functions

Merge pull request #30614 from yuwata/udev-event-take-worker

udev: make UdevEvent take UdevWorker object

utmp-wtmp: check actual value of bool instead of pointer

logind: use handle_action_to_string where appropriate

Since 138224fc807091d31f19a3b22f066d6044626001, HandleActionData
records the corresponding HandleAction. Let's use it instead of
relying on inhibit_what when mapping to string.

Merge pull request #30674 from YHNdnzj/bus-wait-for-cleanup

bus-wait-for-{jobs,units}: some cleanups/modernizations

build(deps): bump github/codeql-action from 2.22.8 to 3.22.12

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.8 to 3.22.12.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/407ffafae6a767df3e0230c3df91b6443ae8df75...012739e5082ff0c22ca6d6ab32e07c36df03c4a4)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

build(deps): bump actions/labeler from 4.3.0 to 5.0.0

Bumps [actions/labeler](https://github.com/actions/labeler) from 4.3.0 to 5.0.0.
- [Release notes](https://github.com/actions/labeler/releases)
- [Commits](https://github.com/actions/labeler/compare/ac9175f8a1f3625fd0d4fb234536d26811351594...8558fd74291d67161a8a78ce36a881fa63b766a9)

---
updated-dependencies:
- dependency-name: actions/labeler
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

build(deps): bump actions/upload-artifact from 3.1.2 to 4.0.0

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3.1.2 to 4.0.0.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/0b7f8abb1508181956e8e162db84b466c27e18ce...c7d193f32edcb7bfad88892161225aeda64e9392)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

build(deps): bump meson from 1.3.0 to 1.3.1 in /.github/workflows

Bumps [meson](https://github.com/mesonbuild/meson) from 1.3.0 to 1.3.1.
- [Release notes](https://github.com/mesonbuild/meson/releases)
- [Commits](https://github.com/mesonbuild/meson/compare/1.3.0...1.3.1)

---
updated-dependencies:
- dependency-name: meson
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

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

bus-wait-for-units: rearrange function

bus-wait-for-units: correctly report state if disconnected from bus

bus-wait-for-units: rephrase error message

bus-wait-for-units: use hashmap_remove_value where appropriate

bus-wait-for-jobs: remove unneeded strna and refactor code a bit

bus-wait-for-jobs: a few modernizations

bus-wait-for-jobs: rephrase error message

bus-wait-for-jobs: reorganize functions

bus-wait-for-jobs: sort includes

Merge pull request #30656 from mrc0mmand/dfuzzer-shenanigans

test: avoid starting/stopping unwanted units during fuzzing

resolve: DnsTransaction.scope may be NULL when dns_transaction_close_connection() is called

Follow-up for ac1b7b9e1933c14bc7bf36d4f32a888afb3f2f4d.

Fixes CID#1533020.

resolve: add several comments for DNS type table

Also update compile time checks.

Follow-up for 818bb6f4825b57c2cd2783fbffe2b2ef82a31573.

Merge pull request #30658 from jnohlgard/udevadm-lock-return-code

udevadm: Propagate return code from verb result

test: add simple coverage tests for 'udevadm lock'

udevadm: Propagate return code from verb result

udevadm lock did not propagate the return code from the child process
because all positive values were treated as success.

v2:
Now 'udevadm test-builtin' ignores all positive return values from the
builtin commands. Otherwise, as the hwdb builtin returns an positive value
when a matching entry found, 'udevadm test-builtin hwdb' will fail.

v3:
Initialize partition table before calling 'sfdisk --delete'.

Co-authored-by: Yu Watanabe <watanabe.yu+github@gmail.com>

test: show journal entries matching the test's syslog identifier as well

test: avoid starting/stopping unwanted units during fuzzing

Skip calling start and stop methods on unit objects, as doing that is
not only time consuming, but it also starts/stops units that interfere
with the machine state. The actual code paths should be covered (to some
degree) by the respective method counterparts on the manager object.

fd-util: don't eat up errors in fd_cloexec_many

Follow-up for ed18c22c989495aab36512f03449222cfcf79aa7

Before this commit, a successful fd_cloexec() call would
discard all previously gathered errors.

test: wrap adding a suppression in a method

sd-journal: check sd-event state before setting up post change timer

The similar check already exists in schedule_post_change().

The function is currently called at two places.
- journal_file_open() in sd-journal:
  In this case, if the timer is not set up, then journal_file_post_change()
  will be called at the end of journal_file_append_entry(). So, the necessary
  task will be done sequentially when an journal entry is stored to the opened
  journal file. That is desired when the function is called at outside of the
  event loop.
- server_open_journal() in journald:
  This is not called after we exit the event loop.

So, we can safely do nothing in the function if the event loop is being
finished or already finished.

Fixes #30644.

coccinelle: re-indent comments

coccinelle: fix typo

Follow-up for b25d3b36a21cd7a68039b85f887cc849ff7664a1.

dns: update record type enum to match iana

Adds some new dns record types. Also, some types were inserted into the
middle of the enum — this corrects an error where the enum constants for
some of the record types previously held an incorrect value.

Merge pull request #30649 from YHNdnzj/close-nointr-unnecessary

Some modernization/cleanup for fd-util

units: add ConditionSecurity=tpm2 to systemd-tpm2-setup units

ConditionSecurity=measured-uki can be true even with TPM 1.2 which we
don't support, so add an explicit check for TPM 2.0.

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

Follow-up for 2e64cb71b9c0160c3

po: Translated using Weblate (Punjabi)

Currently translated at 26.4% (60 of 227 strings)

Co-authored-by: A S Alam <aalam@users.noreply.translate.fedoraproject.org>
Translate-URL: https://translate.fedoraproject.org/projects/systemd/master/pa/
Translation: systemd/main

various: don't use close_nointr if retval is not checked anyway

logind-session-device: use _cleanup_close_

shared/async: use safe_close where appropriate

fdset: use FOREACH_ARRAY at one more place

fd-util: modernization

machinectl: add `restart` convenience alias (#30625)

network/address: unconditionally check if address is ready

Previously, we checked if the address is ready only when it is an IPv6
address. But, IPv4 address never enter tentative state, hence the check
for an IPv4 address is effectively equivalent to the one for IPv6.
So, let's unify the check. This does not change any effective behavior.

Merge pull request #30643 from bluca/completion

bash completion additions for systemctl/systemd-analyze

Merge pull request #30641 from YHNdnzj/hibernation-device

Follow-ups for hibernate-util

Merge pull request #30634 from dtardon/docbook-valid-3

DocBook validation fixes (part 3)

bash completion: add systemd-analyze architectures

Follow-up for fb8cc599ed863502d67

analyze: allow --no-legend with architectures verb

It prints a table with a header, allow omitting it

Follow-up for fb8cc599ed863502d67

bash completion: make systemctl mount-image/bind autocomplete on active services

The verb works only on running service units, so complete on that as the first
parameter, and a local file as the second. The other parameters are inside the
service namespace so we can't autocomplete from the outside, return early.

bash completion: add systemctl service-log-level/target

sleep: don't log duplicate error

write_resume_config() logs error on its own.

hibernate-util: remove unused code

All callers of write_resume_config() pass non-NULL device.

hibernate-util: make sure we use blockdev path for HibernationDevice.path

Before this commit, this field could spuriously contain the path of the
swapfile.

Merge pull request #30639 from mrc0mmand/more-cocci-shenanigans

Another batch of Coccinelle tweaks

tpm2-util: declare the cleanup attribute first

As we do everywhere else (apart from it being incosistent, the switched
order also makes Coccinelle unhappy).

bootctl: add a missing space

test: use timestamp_is_set()

test: use set_isempty() in one more place

test: use IN_SET()/ERRNO_IS_NEG_*() more

tree-wide: use IN_SET() more

coccinelle: dial back warnings about performance

Turns out I _really_ underestimated the impact of
--include-headers-for-types, as it significantly reduces both runtime
and storage penalties. For example, on my machine the runtime of
uncached run goes down from ~15 minutes to ~2 minutes, and similarly the
total storage needed by the cache goes from ~15 GiB down to ~3 GiB.

coccinelle: help Coccinelle with some more complex macros

Drop the original macro file, since it's not needed anymore thanks to
resolving includes properly, but introduce a similar file -
parsing_hacks.h - that helps Coccinelle in some specific corner cases.

This eliminates most of the outstanding parsing errors in source files.
The remaining ones are limitations of the parsing engine (see the FIXMEs
in pasing_hacks.h) and need further investigation.

coccinelle: search the system include path for header files as well

Since Coccinelle is originally a kernel tool, it doesn't search the
system include path by default for header files. Without this we're
missing a lot of types provides by stdlib (and other libraries we make
use of).

Merge pull request #30631 from yuwata/network-address-free

network/address: tiny cleanups for address_free()

sleep: connect to correct bus when locking homed-managed homes

Partially reverts 122f6f1eaa4447449c7054793f6497eb9e4d03c6

Fixes https://github.com/systemd/systemd/issues/29938#issuecomment-1869508708

man: rewrite as <group>

man: fix markup

<arg> is not allowed inside <title>.

man: move </variablelist> to the right place

man: fix <cmdsynopsis> content

Inline text is not allowed.

man: fix markup

<option> is not allowed in <cmdsynopsis>.

man: restrict <command> to the actual command

man: move macro decls. out of <funcsynopsis>

<funcsynopsis> doesn't allow block content.

man: fix markup

<arg> is not allowed inside <term>.