update TODO

notify: add support for MANAGERPIDFDID

pid1: pass pidfdids to invoked services in $MAINPIDFDID and $MANAGERPIDFDID

random-util: include pidfdid in fallback RNG buffer

This doesn't make the RNG cryptographic strength, but if we have it
easily accessible, why not include the pidfd id. It is after all not
vulnerable to reuse.

pidref: now that we have the cached pidfdid of our own process, use it

Note that this drops a lot of "const" qualifiers on PidRef arguments.
That's because pidref_is_self() suddenly might end changing the PidRef
because it acquires the pidfd ID.

We had this previously already with pidfd_equal(), but this amplifies
the problem.

I guess we C's "const" doesn't really work for stuff that contains
caches, that is just conceptually constant, but not actually.

pidfd-util: add helper for getting our own pidfdid

let's start caching our own pidfd id, since it's somewhat involved to
get but a much better identifier for things than the classic PID is.

pidfd-util: dont open a new pidfd unnecessarily in pidfd_check_pidfs()

build: fail the build if we accidentally drop a "const" qualifier on a parameter

let's be strict here, our codebase allows it.

machine-id-setup: move code to use chase() (#35979)

I just wanted to switch the machine id setup code to use chase() or its
changes, given it supports --root=/--image= operation. That turned out
to be a rabbit hole, and became much bigger...

cryptsetup and friends: use dispatch_verb() (#36072)

nspawn: downgrade log message about usrquota to debug

This is shown every time nspawn is started, which is annoying
and there's nothing a user can do about it, since it depends on
an extremely new kernel. Downgrade to debug.

Follow-up for 611ae598889471830b2f1d7251c271b79884b1c4

resolved: empty "ipv4hint" and "ipv6hint" SvcParams are invalid

According to RFC 9460 "An empty list of addresses is invalid."

https://www.rfc-editor.org/rfc/rfc9460.html#section-7.3

integritysetup: use dispatch_verb()

veritysetup: use dispatch_verb()

cryptsetup: use dispatch_verb()

cryptsetup: drop doubled space

cryptsetup: use mangle_none() at one more place

tree-wide: drop merely used _cleanup_ macros, and introduce several new ones (#36071)

udev: cleanups around sd_listen_fds() (#36030)

Fixes a bug introduced by 2f0aa9a80445ef18086260a60fad71920ad9486c.

In the offending commit, `sd_varlink_server_listen_auto()` is used. That
may add file descriptors of netlink socket for uevent and legacy control
socket varlink server. Then, udevd may fail to receive uevents.

tree-wide: several follow-ups for varlink methods (#36080)

Follow-ups for #35914.

udev-rules: introduce OPTIONS="dump" (#36069)

split out of #35968.

sbsign: Make sure output file respects umask (#36083)

update TODO

chase: move appending of trailing slash out of loop

let's move final processing of the filename out of the loop, and apply
it in all cases, uniformly, even if we are asked to only return the
final filename.

chase: introduce flags that verify that chased inode is regular file or dir

This also implies the new CHASE_MUST_BE_DIRECTORY flag in case the
specified path ends in a slash. This makes the rules stricter, it means
we'll be closer to how this is handled in kernel: if a path ends in a
slash it can never refer to a non-directory.

machine-id-setup: rework writing of /etc/machine-id around chase()

Let's make sure we properly stay inside of the root fs if one is
provided, i.e. stop using prefix_roota() at one more place.

machine-id-setup: remove unnecessary 'else'

machine-id-setup: rework --commit based on chase()

machine-id-setup: do sync() stuff only when operating on the real rootfs

If we operate relative some explicit rootfs the reason for the sync()ing
is not really given, we are not really acting as a milestone at boot,
but "offline".

machine-id-setup: rearrange --help to match how we do them these days

chase: use streq() not path_equal() to compare filenames

chase: allow using chase() as mkdir_p() replacement

This allows using CHASE_MKDIR_0755 without CHASE_NONEXISTENT or
CHASE_PARENT, so that it will create the final component of the path
too should it be missing.

This is really useful as a mkdir_p() replacement that returns an fd to
the final component, and knows how to operate relative to a root fs.

Kinda reverts 4ea0bcb9229fe12e0c428659d76934351b821872 (which only
refused the flags combination which didn't work, instead of making it
work, which is what this commit does.)

This also corrects behaviour if CHASE_MKDIR_0755 is used in one more
way: we'll now always open the dir as O_PATH. This is generally the
better idea, but matters in particular once with allow using
CHASE_MKDIR_0755 to create the final component: we should uniformly
return an O_PATH dir that must be converted to a proper fd first before
using it.

mount-util: introduce umountat_detach_verbose()

This new helper allows unmounting paths by fd.

sbsign: Make sure output file mode respects umask

sbsign: Make two debug logs into error logs

udev-varlink: add io.systemd.service.GetEnvironment

There is no reason to not support the method in udevd.

Follow-up for c8127075ae22ad0f64d3072f400818f6e1a9ab48.

tree-wide: rely on sd_varlink_dispatch() for validating zero-argument method calls, too

Follow-ups for a7cc7038ac06d2c02cbaa5a749aa47b4f7cbe786.

udev,network: drop unnecessary check for result of sd_listen_fds_with_names()

udev-manager: rework initialization of device monitor

Let's determine the socket for device monitor based on its name.

udev-ctrl: use -EBADF for invalid file descriptor

udev: split out legacy control socket handling to udev-manager-ctrl.c

Mostly no functional change, except for that now the socket for device
monitor is determined by its name.

capability-util: rename output argument of capability_gain_cap_setpcap()

core: use usec_add() at one more place

Follow-up for 8af1b296cb2cec8ddbb2cb47f4194269eb6cee2b.
Fixes CID#1590317.

Tweaks to generic "io.systemd.service" Varlink interface, and hook it up in all long-running Varlink daemons (#35914)

pid1,nspawn: enable usrquota on /tmp/ + /dev/shm/ (#36036)

Split out of #36010.

This simply enables the "usrquota" mount option on the two dirs if they
are backed by tmpfs.

#36010 contains a test that actually verifies that the mount option does
its thing.

tree-wide: coding style fixlets done by coccinelle (#36074)

man/systemd.special: Add suspend-then-hibernate.target to sleep.target documentation

tree-wide: coding style fixlets done by coccinelle

coccinelle: add .gitignore for cache files

tree-wide: use log_level_is_valid() more

test: add superficial test for generic per-service varlink API

tree-wide: implement generic io.systemd.service Varlink interface in all long-running services

Given we have the generic interface, let's hook it up everywhere.

This doesnt bother with the Reload() call usually, since that's more
involved, but hooks up the other relevant functions where applicable.

varlink: add generic GetEnvironment() call to the Varlink "service" interface

It's just so useful being able to retrieve the current env block from
our services. Add a concept for that. It's really simple, and dumb.

In many ways it's like /proc/$PID/environ, but shows the actual
environ[] array visible to the app, not just some memory that was
originally used for the env block passed in, but might have been rearranged.

varlink: tweak introspection description for Reload()

varlink: drop redundant check

sd_varlink_dispatch() alredy validates all this, no need to check this
manually.

varlink: rely on sd_varlink_dispatch() for validating zero-argument method calls, too

json-util: fine tune json_dispatch_log_level()

Let's give the user control on how to handle JSON "null" assignments of
the log level. As one of three cases: as failure, as LOG_NULL (i.e. to
turn off logging) or as LOG_INFO (as our usual default log level).

Let's then use that in the generic SetLogLevel() call, so that callers
can use it to explicitly turn off logging in a service.

Note that this was (probably accidentally) already implemented, except
that the introspection enforcement blocked it. Let's clean this up and
make this officially a thing, since it's generally useful to turn off
logging I think.

nspawn: enable usrquota support on /tmp/ and /dev/shm/

units: enable usrquota support on /tmp/

pid1: enable usrquota support on /dev/shm

man: replace "-" with "none" in cryptsetup commands and crypttab

`none` is more explicit and harder to overlook than `-` which in certain
contexts could be misunderstood to mean standard input.

TODO: fix typo

Follow-up for fb8d46cffeac5a9607f94022978ae6ce95e6b8dc.

process-util: fix typo

Also rebreak comment.

Follow-up for 03b89cf213d5d1a60601ff158a6366718461fa4b.

login: fix typo

Follow-up for a6ad410ffa5d63647ea9048932b7b459cbe55662.

boot: introduce _cleanup_strv_free_

boot: introduce _cleanup_file_close_

selinux-util: drop _cleanup_context_free_

It is used at only one place, not necessary.

capability-util: drop _cleanup_cap_free_charp_

Also moves cap_free_charpp() to test-cap-list.c, as it is used only
there.

tree-wide: use _cleanup_hashmap_free_ and friends

udev-rules: introduce OPTIONS="dump" token

Should be useful for debugging.

udev-dump: split out dump_event() from udevadm-test.c

pid1: when a password is requested during PAMName= processing, query it via the ask-password logic

udev: add --verbose option for 'udevadm test' and add more verbose logs (#36021)

split out of #35968.

tree-wide: switch various uses of mkdtemp() over to mkdtemp_malloc() (#36057)

escape: make 'bad' parameter optional

Treat a NULL bad parameter just like an empty one: do not escape any
additional characters except for the CC chars.

meson: bump C std to gnu17

GCC 8.4 (our baseline) defaults to gnu17 already.

udev-rules: add trace logs for GOTO and parent conditions

udev-rules: update log messages

This also adds trace logs about token result, and skipping assignment
caused by final value already assigned.

udev-rules: ignore whole command result if it is too long and does not contain newline

udev-rules: introduce udev_replace_chars_and_log()

And logs about replacement only when trace logging is enabled.

udev-rules: add more trace logs for string match

udev-rules: logs result of format substitution

This also drops redundant token string in log message on truncation.

No functional change, but should improve debuggability.

udev-rules: show original token string in log_event_error() and friends

This also makes
- make many loggers refactored,
- log_event_error() and friends UdevEvent object,
- introduces log_event_trace().

No functional change, just refactoring and preparation for later
commits.

udevadm-test: introduce -v/--verbose option to show verbose log messages

Currently this does not show any extra log messages. In later commits,
more verbose log messages will be added.

hexdump: if size is SIZE_MAX, use strlen()

Similar how we do this as various places: if SIZE_MAX is specified as
size determine the size automatically via strlen().

nspawn: assorted coding style fixes

tree-wide: switch various uses of mkdtemp() over to mkdtemp_malloc()

mkosi: update fedora commit reference

* c7fecc18eb Skip test_sysusers_defined on upstream builds
* f257050ddd Add a .editorconfig file
* aa5546e9ed Add a few mkosi artifact directories to .gitignore
* 01e5bc1c93 Set SBAT in sd-boot when building on OBS
* a7eec4d33d Run upstream integration test suite with Fedora CI

Several fixlets for GCC warnings (#36051)

Replaces #36043 and #36039.

Two readme tweaks (#36053)

README: add sgx to list of required groups

Follow-up for c9c4899f4444d9586e541b5e72597a37f949433a.

docs/HACKING: use 'run0' to indicate which commands require privileges

update TODO

README: update requirements

udev: set clock group for PTP and RTC devices

Add a new group for clock devices to enable applications like linuxptp
to open clocks without root privileges.

fundamental-macro: conditionalize several gcc warning pragmas

This fixes the following error:

In file included from ../src/basic/macro.h:13,
                 from ../src/basic/env-util.h:9,
                 from ../src/nss-systemd/nss-systemd.c:7:
../src/fundamental/macro-fundamental.h:61:9: error: option ‘-Wzero-as-null-pointer-constant’ is valid for C++/ObjC++ but not for C [-Werror=pragmas]
   61 |         _Pragma("GCC diagnostic ignored \"-Wzero-as-null-pointer-constant\"")
      |         ^~~~~~~
../src/nss-systemd/nss-systemd.c:106:1: note: in expansion of macro ‘DISABLE_WARNING_ZERO_AS_NULL_POINTER_CONSTANT’
  106 | DISABLE_WARNING_ZERO_AS_NULL_POINTER_CONSTANT;
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

nss-systemd: work around -Werror=zero-as-null-pointer-constant issue with PTHREAD_MUTEX_INITIALIZER

This fixes builds on Fedora:

../src/nss-systemd/nss-systemd.c:105:18: error: zero as null pointer constant [-Werror=zero-as-null-pointer-constant]
  105 |         .mutex = PTHREAD_MUTEX_INITIALIZER,
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~
../src/nss-systemd/nss-systemd.c:105:18: error: zero as null pointer constant [-Werror=zero-as-null-pointer-constant]
../src/nss-systemd/nss-systemd.c:109:18: error: zero as null pointer constant [-Werror=zero-as-null-pointer-constant]
  109 |         .mutex = PTHREAD_MUTEX_INITIALIZER,
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~
../src/nss-systemd/nss-systemd.c:109:18: error: zero as null pointer constant [-Werror=zero-as-null-pointer-constant]
../src/nss-systemd/nss-systemd.c:113:18: error: zero as null pointer constant [-Werror=zero-as-null-pointer-constant]
  113 |         .mutex = PTHREAD_MUTEX_INITIALIZER,
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~
../src/nss-systemd/nss-systemd.c:113:18: error: zero as null pointer constant [-Werror=zero-as-null-pointer-constant]
../src/nss-systemd/nss-systemd.c:117:18: error: zero as null pointer constant [-Werror=zero-as-null-pointer-constant]
  117 |         .mutex = PTHREAD_MUTEX_INITIALIZER,
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~
../src/nss-systemd/nss-systemd.c:117:18: error: zero as null pointer constant [-Werror=zero-as-null-pointer-constant]
cc1: all warnings being treated as errors

fundamental: unify gcc warning pragmas at one place

This also drops HAVE_WSTRINGOP_TRUNCATION as the corresponding option
was introduced in GCC 8, and our baseline is GCC 8.4.

test: add link to Ubuntu autopkgtest status page in README

Known issues will be listed there by the team managing the infra

process-util: port pid_from_same_root_fs() to pidref + more (#35975)

Let's continue our move to PidRef.