load-fragment: simplify list insertion logic

LIST_FIND_TAIL and LIST_INSERT_AFTER can work for empty list.

cgroup: IODeviceWeight= or friends can take device node files in /run/systemd/inaccessible/

systemd creates several device nodes in /run/systemd/inaccessible/.
This makes CGroup's settings related to IO can take device node
files in the directory.

core/socket: shorten socket_fdname()

core/socket: dump more settings

dbus-cgroup: merge several blocks which operate almost same tasks

dbus-execute: use empty_to_null() where it can be applicable

doc: add {Condition,Assert}ControlGroupController= to TRANSIENT-SETTINGS.md

Follow-up for e16647c39d195804711a006667d5bce49c0ef73d.

cgroup: move path checking logic to dbus-cgroup.c

bus-unit-util: simplify bus_append_cgroup_property()

bus-unit-util: make dependency settings can take multiple units

This allows people to specify multiple units in dependency settings
e.g. `Requires=foo.service baz.service`.

dbus-socket: move truncation check to bus_socket_set_transient_property()

basic: introduce socket_protocol_{from,to}_name()

And use them where they can be applicable.

doc: update TRANSIENT-SETTINGS.md

man: add explanation about transient path or socket units in systemd-run

run: add support to create transient path and socket unit

bus-unit-util: add socket unit related options

Also, split bus_append_unit_property_assignment().

core: implement transient socket unit

socket-util: add socket_address_type_{from,to}_string()

core/socket: add socket_port_type_from_string()

core:socket: fix string in socket_exec_command_table

core,seccomp: fix logic to parse RestrictAddressFamilies= in dbus-execute.c

If multiple RestrictAddressFamilies= settings, some of them are
whitelist and the others are blacklist, are sent to bus, then parsing
result was corrupted.
This fixes the parse logic, now it is the same as one used in
load-fragment.c

core,seccomp: fix logic to parse syscall filter in dbus-execute.c

If multiple SystemCallFilter= settings, some of them are whitelist
and the others are blacklist, are sent to bus, then the parse
result was corrupted.
This fixes the parse logic, now it is the same as one used in
load-fragment.c

core: move path_kill_slashes() to manager

core: fix wrong validity check

tree-wide: use !strv_isempty() instead of strv_length() > 0

analyze: use normal bus connection for "plot" verb (#7685)

We need to connect to hostnamed, so a private bus connection is no good.
It'd be simpler to use the normal bus connection unconditionally, but
that'd mean that e.g. systemd-analyze set-log-level might not work in
emergency mode. So let's keep trying to use the private connection except
for "plot".

Fixes #7667.

Merge pull request #7687 from hbakken/work/add-missing-features2

Kernel 3.10 compat

build-sys: install TRANSIENT-SETTINGS.md and UIDS-GIDS.md (#7690)

cryptsetup-generator: Don't mistake NULL input as OOM (#7688)

Since systemd v236, several Arch users complained that
systemd-cryptsetup-generator exits with an OOM error and that it
prevents the boot from continuing.

Investigating the diff of cryptsetup-generator between v235 and v236 I
noticed that create_disk allowed for the `password` and `filtered`
variables to be NULL (they're handled with `strempty()`) but not their
`*_escaped` versions, and returned OOM errors in those cases.

Fix this by checking that the input string is non-NULL before deciding
that `specifier_escape` had an OOM error.

I could not test this fix myself, but some users have reported success.

Downstream bug: https://bugs.archlinux.org/task/56733

missing: Add DM_DEFERRED_REMOVE

Also include missing.h in dissect-image.c to pick it up.

missing: Define SMACK_MAGIC if it's missing

nspawn: Include missing.h

missing: Define EFIVARFS_MAGIC if missing

missing: Add MAX_HANDLE_SZ

catalog: update Russian translation (#7678)

Translated taint message.

Also added a blank line before "Current system is tagged" for better
visual separation between current system state and tags description.

missing: Add PR_SET_MM_{ARG,ENV}_{START,END}

missing: Add some more btrfs structs and constants

btrfs: Include missing.h in btrfs-ctree.h

condition: Create AssertControlGroupController (#7630)

Up until now, the behaviour in systemd has (mostly) been to silently
ignore failures to action unit directives that refer to an unavailble
controller. The addition of AssertControlGroupController and its
conditional counterpart allow explicit specification of the desired
behaviour when such a situation occurs.

As for how this can happen, it is possible that a particular controller
is not available in the cgroup hierarchy. One possible reason for this
is that, in the running kernel, the controller simply doesn't exist --
for example, the CPU controller in cgroup v2 has only recently been
merged and was out of tree until then. Another possibility is that the
controller exists, but has been forcibly disabled by `cgroup_disable=`
on the kernel command line.

In future this will also support whatever comes out of issue #7624,
`DefaultXAccounting=never`, or similar.

Merge pull request #7665 from poettering/main-cleanup

let's split up main() into more functions, to make it digestable

catalog: update Polish translation (#7660)

Merge pull request #7664 from poettering/fix-integration-tests

Fix integration tests

man: generalize "binary" to "program" (#7668)

Systemd services are permitted to be scripts, as well as binary
executables.

The same also applies to the underlying /sbin/mount and /sbin/swapon.
It is not necessary for the user to consider what type of program file
these are.  Nor is it necessary with systemd-nspawn, to distinguish between
init as a "binary" v.s. a user-specified "program".

Also fix a couple of grammar nits in the modified sentences.

meson: libudev_core and udevadm should have LOG_REALM=LOG_REALM_UDEV (#7666)

Otherwise, setting udev_log=debug in /etc/udev/udev.conf has no effects since
systemd-udevd is built with LOG_REALM=LOG_REALM_UDEV.

However using LOG_REALM_UDEV (for libudev_core) reveals another similar bug for
udevadm which should also define LOG_REALM_UDEV.

Merge pull request #7661 from keszybz/slice-cleanups

Slice cleanups and systemd-mount --owner

main: add some more comments for the early initialization phase

main: reorder variable declarations a bit

Let's remove a bit redundancy, and list variables of the same
type/category in one declaration line.

main: split out 'skip_setup' check into its own functions

And let's optimize it a tiny bit, by only iterating through the argument
list once, instead of twice.

main: do bother with uid_to_name() unless we do debug logging

main: conditionalize fixup_environment() internally

This code is executed before we parse command line/configuration
parameters, hence let's not use arg_system to figure our how to clean up
things, but instead PID == 1. Let's move that check inside of the
function, to make things a bit more robust abstract from the outside.

Also, let's add a log message about this, that was so far missing.

main: let's move ACTION_RUN test into initialize_runtime()

Let's hide this check inside the function and make it easier to follow
the general control flow of main().

mount-setup: fix MNT_CHECK_WRITABLE error handling, and log about the issue

Let's correct the error handling (the error is in errno, not r), and
let's add logging like the rest of the function has it.

main: tweak timerslack message a bit

Let's clarify that this is a non-issue, by downgrading it to LOG_WARN
and saying "ignoring" in the message.

main: move install_crash_handler() and mount_cgroup_controllers() invocations

Let's place them in initialize_runtime(), where they appear to fit best.
Effectively this is just a move a little bit down, swapping places with
log_execution_mode(), which should require neither call to be done
first.

Note that changes the conditionalization a bit for these calls, from
(PID == 1) to (arg_system && arg_action == ACTION_RUN). At this point this is pretty much the same
however, as we don't allow PID 1 without ACTION_RUN and without
arg_system set, safety_checks() ensures that.

core: move arg_show_status fix-up into load_configuration()

It's part of finalizing our runtime parameters, hence let's move this
into load_configuration() after we loaded everything else. This is safe,
since we don't use it between the location where it was and where we
place it now yet.

main: split out code that sets up the console/terminal and stuff

More refactoring to make main() more digestable.

main: split out code that collects passed fds

More refactoring to make main() more digestable

main: slight modernizations for status_welcome()

There's no point in duplicating the complex parse_env_file() invocation,
hence let's not do it.

main: let's move the arg_show_status check into status_welcome()

It's kinda nice to hide this check inside of status_welcome() itself, so
that it handles all this on its own.

main: slightly rearrange serialization fdset, and logging/console setup

Let's merge two if blocks, and move log_close()/log_open() out of the
testing codepath, as there's no reason to have it there.

main: move chdir("/") a bit earlier

There's no need to do this within the block where logging is closed,
hence move it earlier, so that this block can be kept as small as
possible.

main: move initialize_join_controllers() invocation into load_configuration()

This just sets up some variables the loaded configuration will then
modify. Let's invoke it hence right before loading the configuration.

This moves the initialization just a tiny bit later, but that shouldn't
matter, since we never access it in-between.

main: split out taint string logging into its own function

It's sufficiently complex now, let's add our own function for this too.

core: add more safety check

Let's make sure that if we are PID 1 we are invoked in ACTION_RUN mode,
and in arg_system mode, as well as the opposite.

Everything else is untested and probably not worth supporting hence
let's bail out early if people try anyway.

core: split out test summary output into its own function

More refactoring to make main() shorter.

main: split out security policy loading into its own function

More refactoring to make things more digestable.

main: invoke pager_open() only at one place

Let's just move this up a few lines and unify the invocation.

core: split out various startup safety checks from main() into its own function

No functional changes, just some refactoring to make main() more
digestable.

main: split out all parsing of command line arguments/kernel arguments/configuration files

Let's shorten main() a bit, and split out everything that loads our
configuration and runtime parameters into a function of its own.

No changes in behaviour.

main: minor optimization

Let's remove one memory allocation in the common path.

test: fix TEST-13-NSPAWN-SMOKE

We need to specify a full path to the "ip" binary and busybox "ip" has a
slightly different output than the normal ip, and won't show "DOWN".
hence instead ensure that at lest not "UP" is in there.

test: make sure "make" in the test directories works again

Fixes: #7648

generator: add helper function for writing unit files

It doesn't save too much, but it's a common pattern so I think it's worth
to factor this out.

Fix logical error in meson.build (#7658)

sysvinit_path and sysvrcnd_path have to be set both to activate Sysv compatibility.

mount: add option to specify uid= and gid=

The kernel needs two numbers, but for the user it's most convenient to provide the
user name and have that resolved to uid and gid.

Right now the primary group of the specified user is always used. That's the most
common case anyway. In the future we can extend the --owner option to allow a group
after a colon.

[I added this before realizing that this will not be enough to be used for user
runtime directory. But this seems useful on its own, so I'm keeping this commit.]

logind: use free_and_replace in one spot

No functional change.

tree-wide: use SPECIAL_ROOT_SLICE

core: reuse slice_build_parent_slice

logind: fix misleading message

This message would also be emitted at boot for any user with linger
enabled, so "logged in" is the wrong term to use.

logind: simplify one conditional

Don't bother with removing the directory if we didn't create it.

man: Clarify when OnFailure= activates after restarts (#7646)

tree-wide: add DEBUG_LOGGING macro that checks whether debug logging is on (#7645)

This makes things a bit easier to read I think, and also makes sure we
always use the _unlikely_ wrapper around it, which so far we used
sometimes and other times we didn't. Let's clean that up.

Merge pull request #7637 from yuwata/transient-path

core/path: implement transient path unit

Merge pull request #7631 from cgwalters/systemctl-offline

systemctl,verbs: Introduce SYSTEMCTL_OFFLINE environment variable

Merge pull request #7644 from poettering/memzero

add memzero()/zero() coccinelle scripts and use them

CODING_STYLE: provide better explanation why /* */ over // (#7647)

Let's provide a real reason why /* */ should be used for commenting,
rather than //, beyond mere taste.

(This ultimately simply codifies how I use // vs. /* */ comments, and I
think this is useful as an explanation and reason hence.)

TODO: update

doc: update TRANSIENT-SETTINGS.md

bus-unit-util: add path unit related options

core/path: implement transient path unit

core/dbus-timer: improve error messages

bus-unit-util: improve error messages

bus-unit-util: remove unnecessary oom check

We cannot detect error by alloca().

final v236 update (#7649)

Merge pull request #7608 from poettering/more-news-v236

more v236 news improvements

Merge pull request #6598 from kyle-walker/shutdown-limit

core: Limit the time and attempts in shutdown remount/umount efforts

Merge pull request #7640 from keszybz/tainting-updates

Tainting updates

Merge pull request #7469 from kinvolk/dongsu/nspawn-netns

nspawn: introduce an option for specifying network namespace path

Update TODO

meson: restore --update-catalog call after install

This got dropped by mistake in 72cdb3e783174dcf9223a49f03e3b0e2ca95ddb8.