nspawn,vmspawn: improve errors for unknown users and groups

login: use STREROR_USER helper

creds: improve message about unknown user

Before:
$ build/systemd-creds --uid=asdf
Failed to resolve user 'asdf': No such process
Now:
$ build/systemd-creds --uid=asdf
Failed to resolve user 'asdf': Unknown user

core: improve messages about unknown users and groups

$ sudo build/systemd-run --uid=asdf whoami
$ journalctl -e
(whoami)[1007784]: run-p1007782-i5200512.service: Failed to determine user credentials: No such process
(whoami)[1007784]: run-p1007782-i5200512.service: Failed at step USER spawning /usr/sbin/whoami: No such process
systemd[1]: run-p1007782-i5200512.service: Main process exited, code=exited, status=217/USER
systemd[1]: run-p1007782-i5200512.service: Failed with result 'exit-code'.

Now:
(whoami)[1013204]: run-p1013202-i5205932.service: Failed to determine credentials for user 'asdf': Unknown user
(whoami)[1013204]: run-p1013202-i5205932.service: Failed at step USER spawning /usr/sbin/whoami: Invalid argument
systemd[1]: run-p1013202-i5205932.service: Main process exited, code=exited, status=217/USER
systemd[1]: run-p1013202-i5205932.service: Failed with result 'exit-code'.

run: improve log message for unknown user/group

Before:
$ sudo build/systemd-run --scope --uid=asdf whoami
Failed to resolve user asdf: No such process
Now:
$ sudo build/systemd-run --scope --uid=asdf whoami
Failed to resolve user 'asdf': Unknown user

tmpfiles: improve error message for missing user/group

From a boot with a dracut initrd:
systemd-tmpfiles[242]: /usr/lib/tmpfiles.d/tpm2-tss-fapi.conf:2: Failed to resolve user 'tss': No such process
systemd-tmpfiles[242]: Failed to parse ACL "default:group:tss:rwx", ignoring: Invalid argument
systemd-tmpfiles[242]: /usr/lib/tmpfiles.d/tpm2-tss-fapi.conf:4: Failed to resolve user 'tss': No such process
systemd-tmpfiles[242]: Failed to parse ACL "default:group:tss:rwx", ignoring: Invalid argument
systemd-tmpfiles[242]: /usr/lib/tmpfiles.d/tpm2-tss-fapi.conf:6: Failed to resolve group 'tss': No such process
systemd-tmpfiles[242]: /usr/lib/tmpfiles.d/tpm2-tss-fapi.conf:7: Failed to resolve group 'tss': No such process

udev: define a generic helper to print messages about unknown users and groups

We cannot just use %m, because strerror returns a confusing error message
for ESRCH or ENOEXEC. udev code was doing a good job, but the error handling
was very verbose. Let's encapsulate the customized error messages in a
helper.

No functional change, except that the error messages have a slightly different
form now. The old messages were a bit better, but we don't have as much
flexibility in the new scheme. "Failed to resolve user 'foo': Unknown user"
should be good enough.

repart: prefix LUKS superblock label with "luks-" by default, and add explicit VolumeLabel= setting to control it (#39713)

Alternative-to: #39536

NEWS: menton future removal of SysV support and requirement bump

Split out script for musl builds (#39758)

docs: Update MEMORY_PRESSURE to mention recent improvements in GLib

See https://gitlab.gnome.org/GNOME/glib/-/issues/2931 for the changes in
GLib upstream. Using `GMemoryMonitor` is now more compliant with the
systemd recommended approach, but it needs further work to read the
recommended environment variables rather than unconditionally accessing
the per-cgroup PSI kernel file directly.

Signed-off-by: Philip Withnall <pwithnall@gnome.org>

fs-util: enable automatic access mode logic in xopenat()

test: modernize test-log and add test cases for log_format_iovec() (#39750)

ci: whitespace fix

musl: split out script to setup build

This makes it easier to set up a local build with musl:
$ tools/setup-musl-build.sh build-meson
$ ninja -C build-meson

musl: drop unused temporary pkgonfig directory

tmpfile-util: make sure errno is left untouched by CLEANUP_TMPFILE_AT()

This mimics how rm_rf_physical_and_free() operate, and we should do the
same here, and make clean-up error paths not touch errno needlessly.

nspawn: move oci code to use json_dispatch_strv_environment()

Various comment and whitespace cleanups (#39740)

tree-wide: drop space after shell redirection ops

Those are all unary operators.

core/namespace: tidy up comments and debug logs

Saying "table" everywhere is not needed. Everybody can see that the table
is a table is a table. Also tweak the grammar in various places to make
reading nicer.

test: add testcase for VolumeLabel=

dissect-tool: accept encryption password via $PASSWORD

nspawn: fix flags type for chase()

discover-image: support reading metadata from verity enabled DDI images, too

hwdb: Fix keyboard backlight keys on Acer Nitro 5 AN515-58 (#39769)

Pressing Fn+F10 on Acer Nitro 5 AN515-58 incorrectly triggers display
brightness down (scancode 0xef) instead of keyboard backlight control,
causing the screen to go completely dark. Similarly, Fn+F9 (scancode
0xf0) has no function explictily stated in hwdb causing unknown keycode
debug messages.

Both keys should control the keyboard backlight as labeled on the
keyboard. Map scancodes 0xef and 0xf0 to kbdillumup and kbdillumdown
respectively to enable proper keyboard backlight control.

Drop remaining CGroup v1 specific code (#38969)

NEWS: mention the LUKS superblock default labelling change.

This is a (weak) compat break, hence document it in NEWS.

NEWS: insert hyphen

repart: avoid label string clashes between LUKS superblocks and the filesystems on them

Let's make sure that by default /dev/disk/by-label/ symlinks avoid
ambiguities, and the LUKS volume carries a different one than the file
system inside it.

Alternative-to: #39536

repart: fix typo

Follow-up for cfb6c543245be0a1908c270941b7881ac7395d1b.

NEWS: cleanups and rewordings, extend the section about musl

I think we should make it clear that the "incomplete musl support" does not
mean that it'll for certain be completed later. The feedback from users will be
an important consideration.

test-log: add test cases for log_format_iovec()

test-log: migrate to use ASSERT_OK() and friends

log: replace format string in LOG_ITEM() to a dummy one when analyzed by Coverity

Hopefully silence many false-positives.

log: make each string generated in log_format_iovec() NUL terminated

Nowadays, we append an extra NUL for each data if possible for safety.
We already do the same for example at write_to_kmsg(), log_do_context(),
write_to_journal(), log_struct_iovec_internal(), and so on.
This does not change any behavior, as the iov_len field is unchanged.

repart: adjust "partno" column

In a typical output from systemd-repart, the output is very wide any any wasted
space is bad because it pushes the interesting information even further to the
right. We usually need at most one or two digits to express the partition
numbers, so let's shorten the title of the column to effectively remove two
columns in the output.

In JSON output, the old field name is retained. This follows the pattern
already used for field "drop-in_files".

Also right-align the columns with numbers always to the right. I doesn't make
sense to align the columns which are only used for JSON output, so stop setting
alignment for those.

coredumpctl: remove unnecessary line break

hwdb: Add Elecom IST Pro trackball (#39762)

Added entries for the Elecom IST Pro via its three connection methods- a
USB cable, the included G1000 USB receiver, and Bluetooth.

The G1000 USB receiver _may_ have to be removed in the future depending
on the input devices that can connect to it. According to Elecom, the
receiver can have up to three different input devices connected such as
trackballs, mice, keyboards, etc. That said, as far as I can tell, the
IST Pro is the only released Elecom device that uses the receiver. The
non-pro model and the upcoming Elecom Huge Plus might use the same
receiver, but that should not matter as both devices are trackballs.

NEWS: fix typos

NEWS: more adjustments for v259~rc1

shell-completion: drop deprecated cgroup controller suggestion

cgroup-util: drop outdated comments

cgroup-util: drop unused SYSTEMD_CGROUP_CONTROLLER_{LEGACY,HYBRID}

cgroup-util: do not check validity of controller in cg_split_spec()

Now the controller part is always ignored, hence let's skip check for
the controller part of the spec. This also make it acceppt unnormalized
path. Previously paths were checked by path_is_normalized(), but now
checked by path_is_safe(). Also, now this mapps an empty path to NULL.

cgroup-show: drop unnecessary call of cg_mangle_path()

As the caller already dropped controller part.

cgroup-util: drop 'controller' argument from cg_get_path()

The argument is not used anymore. Let's drop it.

cgroup-util: drop 'controller' argument from cg_get_keyed_attribute() and cg_is_empty()

The argument is completely unused. Let's drop it.

cgroup-util: drop unused cg_unified() and friends

cgls: drop cgroup v1 support and warn when cgroup v1 controller is specified

limits-util: drop cgroup v1 support from physical_memory()

cgroup-util: drop cgroup v1 support in cg_get_path()

We have dropped cgroup v1 support in v258. Let's assume we are running
on cgroup v2 and drop unused legacy code.

cgroup-util: use cg_is_available() in cg_kill_supported()

cgroup-util: drop cgroup v1 support from cg_mask_supported_subtree()

We have dropped cgroup v1 support in v258. Let's drop unused legacy code.

tree-wide: replace cg_get_path_and_check() with cg_get_path()

We have dropped cgroup v1 support in v258. When running on cgroup v2,
cg_get_path_and_check() with SYSTEMD_CGROUP_CONTROLLER as controller is
equivalent with checking if we are running on cgroup v2 and then
cg_get_path(). As we can assume we are running on cgroup v2, then the
check is not necessary anymore, thus we can replace
cg_get_path_and_check() with cg_get_path().

cgroup-util: drop cgroup v1 support from cg_pid_get_path()

We have dropped cgroup v1 support in v258. Let's drop legacy code.
Then, we can drop 'controller' argument from cg_pid_get_path() and
cg_pidref_get_path().

cgroup-util: drop 'controller' argument from cg_path_open()

Nowadays it always takes SYSTEMD_CGROUP_CONTROLLER as controller.
Let's drop it.

No functional change, just refactoring.

cgroup-util: drop 'controller' argument from cg_set_attribute(), cg_get_attribute() and friends

Non-null controller arguments are always ignored when running on cgroup v2.
Let's drop the argument.

No functional change, just refactoring.

cgroup-util: drop 'controller' argument from cg_enumerate_subgroups()

Nowadays it always takes SYSTEMD_CGROUP_CONTROLLER as controller.
Let's drop it.

No functional change, just refactoring.

cgroup-util: drop 'controller' argument from cg_enumerate_processes()

Nowadays it always takes SYSTEMD_CGROUP_CONTROLLER as controller.
Let's drop it.

No functional change, just refactoring.

cgroup-show: drop 'controller' argument from show_cgroup() and show_cgroup_and_extra()

Nowadays these always take SYSTEMD_CGROUP_CONTROLLER as controller.
Let's drop it.

No functional change, just refactoring.

cgroup-util: introduce cg_is_available() and check it in tests

Now most of our code does not support cgroup v1. Let's skip test cases
if we are running on cgroup v1.

test: skip TEST-64-UDEV-STORAGE-simultaneous_events on Debian

It consistently fails with the new 6.17 kernel and nobody can
figure out why, so just disable it for now as it's just noise

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

NEWS: first prep for v259

Chores for RC1 (#39757)

Add experimental musl support (#38825)

This adds experimental support of building systemd with musl. This
allows to build systemd with musl and run most of the unit tests.
Running integration tests is not yet supported, however. Hopefully, this
should be a good starting point to support musl.

This requires musl-1.2.5 with
https://github.com/kraj/musl/commit/fde29c04adbab9d5b081bf6717b5458188647f1c.
The patch is already backported to Alpine/postmarketOS's musl package
since musl-1.2.5-r11. See
https://gitlab.alpinelinux.org/alpine/aports/-/commit/333e50c2052c74bb372f821fdeea2e6814f2fe7a.

man: regenerate man/rules/meson.build

ninja -C build update-man-rules

Translations: update systemd-po

ninja -C build systemd-update-po

Translations: update systemd-pot

ninja -C build systemd-pot

Update syscalls table

ninja -C build update-syscall-tables update-syscall-header

NEWS: update contributors list

Update hwdb

ninja -C build update-hwdb

update TODO

keymap: Ignore brightness keys on Dell Inspiron 3505 to avoid double events

On the Dell Inspiron 3505 both the atkbd and acpi-video input devices report
an event for pressing the brightness up / down keys, resulting in user
space seeing double events and increasing / decreasing the brightness 2 steps
for each keypress.

Fix this by adding the device to the already existing list of Dell
devices that suffer from the same problem.

Signed-off-by: Armin Wolf <W_Armin@gmx.de>

musl: ci: add build test and unit tests

musl: glob-util: filter out . and .. even if GLOB_ALTDIRFUNC is not supported

musl neither support GLOB_ALTDIRFUNC nor GLOB_BRACE.
Let's make safe_glob() work even when GLOB_ALTDIRFUNC is not supported.
Currently, GLOB_BRACE is simply ignored when it is not supported.

musl: add fallback parse_printf_format() implementation

musl does not provide parse_printf_format(). Let's introduce a fallback
method.

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

musl: meson: add libutmps support

musl only provides fake functions, hence even with -Dutmp=true,
utmp/wtmp functionalities do not work. Let's allow to build with
libutmps.

musl: utmpx: add several missing definitions

iovec-util: drop redundant ()

dissect-image,execute: switch root hash/root hash sig storage to struct iovec

let's go one step further on our iovec'ification journey.

No change in behaviour, just rework to make struct iovec used
everywhere.

test-network: several tweaks (#39752)

test-network: drop two overly verbose outputs

Follow-up for 7f9c0c31d2f00f472c361868ac2184d77113db72.

test-network: set TimeoutStopFailureMode=abort when running with sanitizers

Hopefully this provides useful data for issue #39631.

Revert "test-network: disable several more sandbox features when running with sanitizers"

This reverts commit 454ce423050890bdc8c3c90add3713592a3cab38.

Unfortunately, it does not work.

logind-dbus: avoid assertion on HandlePowerKey=factory-reset

The factory-reset action is handled together with the various
sleep/shutdown actions, but its not either one.

Fixes #39385

tree-wide: several cleanups (#39746)

network/dhcp: several follow-ups for recent change (#39744)

Follow-ups for 7f9c0c31d2f00f472c361868ac2184d77113db72

resolve: voidify one function call

Fixes CID#1643235.

tree-wide: swap arguments for streq()

We usually set a variable first, then constant.

test-network: migrate to use ASSERT_OK() and friends

network-internal: do not save partially parsed values

resolve: fix typo

Follow-up for 8209f4adcde08d225f56269e608ccd5f6704cd70.

efivars: fix typo

Follow-up for ab69a04600fd34c152c44be6864eb3bc64568e17.

sd-dhcp-lease: use the last hostname option when multiple hostname is specified

sd-dhcp-lease: drop ret_ prefixes from the parsers

network: unify log messages

sd-event: several follow-ups for recent change (#39743)

test: add missing assertion

Follow-up for 5a5cb6ba50594355734ff58487d2272a86b741b1.
Fixes CID#1643217.

sd-event: do not ignore result of callbacks

Follow-up for 4c8b6d636c92e84f4b40db5656db58f71f397a1d and
6aff6d3ffc8a481bb663ac4dd0cf479845a6e24f.
Fixes CID#1643218 and friends.