Merge pull request #10510 from poettering/uacess-brackets

coverity fixes

man: fix wording a bit on the cgroup accountings options (#10509)

The &MEMORY_ACCOUNTING_DEFAULT; resolves to "yes" or "no" while the rest
of the paragraph talked about "on" and "off". Let's adjust this and
stick to "yes" and "no"...

Quite frankly I think it's not a particularly good idea to change the
docs based configuration changes... THis can only be incomplete, and the
wording is still very awkward since we repeat the same sentence twice.

Merge pull request #10505 from poettering/have-namespace

skip various test-execute tests when we have no namespacing

units: drop systemd-user-sessions.service ordering dep on systemd-journal-flush.service (#10502)

THis dep existed since the unit was introduced, but I cannot see what
good it would do. Hence in the interest of simplifying things, let's
drop it. If breakages appear later we can certainly revert this again.

Fixes: #10469

string-util: introduce explicit_zero_safe()

The only real difference is that this wrapper can deal with NULL
pointer arguments, but only if the length is also zero.

CID 1396277

udev: use right error variable to make level decision

udev: fix bad if {}

CID 1396318
CID 1396319

test: skip various tests if namespacing is not available

Apparently on Debian LXC/AppArmor doesn't allow namespacing to container
payloads. Deal with it.

Fixes: #9700

tests: add helper call have_namespaces() to test whether Linux namespaces are available

A slighly sloppy test call for conditionalizing several tests.

tests: as per CODING_STYLE undefine basename() right after including libgen.h

tests: alloc-util.h and fs-util.h are our own headers

Hence include them with "" rather than <>. Also including them twices is
a pretty bad idea too.

execute: if we fail to do namespacing, explain why we refuse to continue in a debug message

Merge pull request #10484 from yuwata/udev-spawn-simplify

udev-event: cleanups about udev_event_spawn()

Merge pull request #10498 from yuwata/udev-builtin-log

udev-builtin: use log_device_*() macros

Merge pull request #10483 from yuwata/udev-cleanup-10

tree-wide: use log_device_*()

Merge pull request #10461 from yuwata/small-cleanups

Fixes recent defects reported by coverity

add ephemeral to nspawn-settings.

udevadm-info: fix memleak

Fixes CID#1396276.

udevadm-monitor: initialize variable

Fixes CID#1396278.

modules-load: return first error in the loop

Follow-up for a889e206a7434afe28039a1698e5ecf6a3fb7a9b.
Fixes CID#1396280.

reply-password: initialize 'length' variable

Follow-up for 1fd2786161feba4276a81c3468cfbfb118f56517.

This fixes the following warning when built by clang:
```
../src/reply-password/reply-password.c:76:21 warning: variable 'length' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
```

Fixes CID#1396281 and CID#1396277.

core: use ascii_toupper() instead of everytime judging whether it is the first message

udev-event: use sd_event_add_io() for reading stdout or stderr of spawned process

udev-event: unify two log_*() by log_full()

udev-event: Capitalize log messages

udev-event: fix condition to create pipe fds

The bug was introduced by feaa6db7ac287b34887f75c27a7efc2107a7b691.

udev-event: set O_CLOEXEC to pipe fds

udev-event: make failure in getting device properties fatal

test: create /dev/null in test-udev.pl

udev-event: use rearrange_stdio() and close_all_fds()

udev-event: set FORK_DEATHSIG flag

udev-event: drop redundant code

We have set FORK_RESET_SIGNALS in safe_fork(). So, calling
reset_signal_mask() after safe_fork() is redundant.

udev-event: move debug logging before safe_fork()

This fixes the following journal layout issue:
```
Oct 22 15:26:02 systemd-udevd[348]: RUN '/usr/sbin/setregdomain' /usr/lib/udev/rules.d/85-regulatory.rules:5
Oct 22 15:26:02 [439]: Starting '/usr/sbin/setregdomain'
Oct 22 15:26:02 systemd-udevd[348]: Process '/usr/sbin/setregdomain' succeeded.
```

udev-builtin-input_id: update log messages

udev-builtin-usb_id: update log messages

udev-builtin-uaccess: update log messages

udev-builtin-net_setup_link: update log messages

udev-builtin-keyboard: update log messages

sd_hwdb: update log messages

udev-builtin-btrfs: add debug log messages

udev-builtin-blkid: add more debug log messages

udev-builtin: fix return value of udev_builtin_hwdb()

This fixes a bug introduced by d354690e7d0429338cdd1c2cfe1fdd08fc53da9e.
Also, this adds more debug logs.

sd-device: fix argument name in log_device_*() macros

Follow-up for b0cba0ca526ed2d86e283a0fcfebdf0a4d4bea9b.

journald: use the "driver" transport for logging a refused connection

Otherwise, these messages can end up in `/dev/null` when journald is run
in containers as was shown in https://github.com/systemd/systemd/pull/10444#issuecomment-430927793.

sd-journal: escape binary data in match_make_string()

Fixes: #10383

udevadm: normalize exit status

After d354690e7d0429338cdd1c2cfe1fdd08fc53da9e, each udevadm command
returns negative errno on error. So, let's normalize the exit status
of udevadm.

Merge pull request #10493 from yuwata/parse-time-overflow

util: check overflow in parse_time() and parse_nsec()

link-config: use log_device_*()

udev-watch: use log_device_*()

udev-node: use log_device_*()

rfkill: use log_device_*()

mount-tool: use log_device_*()

login: use log_device_*()

sd-device-monitor: use log_device_*()

gpt-auto-generator: use log_device_*()

fsck: use log_device_*()

core/device: use log_device_*()

backlight: use log_device_*()

bash-completion: analyze: add timespan

Follow-up for 3f1c1287a96dfc146ce894bd6bfff388cba0ab55.

test: add tests for detecting overflow in parse_time() and parse_nsec()

util: check overflow in parse_nsec()

util: check overflow in parse_time()

analyze: Add "timespan" command to dump time span in usec

This is useful for a couple of cases, I'm mostly interested in case #1:

1. Verifying "reasonable" values in a trivially scriptable way
2. Debugging unexpected time span parsing directly

Test Plan:

```
% build/systemd-analyze timespan 20
Original: 20
      μs: 20
   Human: 20us
% build/systemd-analyze timespan 20ms
Original: 20ms
      μs: 20000
   Human: 20ms
% build/systemd-analyze timespan 20z
Failed to parse time span '20z': Invalid argument
```

Merge pull request #10395 from yuwata/udev-cleanup-9

udev-builtin: replace udev_device by sd_device

udev-builtin: propagate negative errno

udevadm-test-builtin: use find_device()

This also fixes minor memleak.

udevadm: use path friendly functions

udev: propagate original error from udev_builtin_add_property()

udev: logs error in udev_builtin_add_property()

sd-device: introduce log_device_*() macros

udev-builtin: modernize code a bit

- introduce _UDEV_BUILTIN_INVALID,
- use _UDEV_BUILTIN_MAX where applicable,
- add several assertions.

udevadm-test-builtin: replace udev_device by sd_device

udev: drop libudev-private.h from udev-builtin-{hwdb,net_id}.c

udev: make udev_builtin_run take sd_device

udev-builtin: simplify code a bit

udev: replace udev_device by sd_device from prototype of udev builtin commands

udev: replace udev_device by sd_device in udev-builtin-usb_id.c

udev: replace udev_device by sd_device in udev-builtin-uaccess.c

udev: replace udev_device by sd_device in udev-builtin-path_id.c

udev: replace udev_device by sd_device in udev-builtin-net_setup_link.c

udev: replace udev_device by sd_device in udev-builtin-net_id.c

udev: replace udev_device by sd_device in udev-builtin-keyboard.c

udev: replace udev_device by sd_device in udev-builtin-input_id.c

udev: replace udev_device by sd_device in udev-builtin-blkid.c

udev: replace udev_device by sd_device in udev-builtin-hwdb.c

udev: make udev_builtin_add_property() take sd_device instead of udev_device

tests: allow passing log_level via an environment variable (#10490)

tests: update networkd-test.py to use resolvectl instead of systemd-resolve (#10486)

The latter is merely a compat symlink nowadays.

Merge pull request #7411 from joergsteffens/tapechanger

udev-rules: extend tape and tape changer rules

Merge pull request #10477 from yuwata/fix-10474

network: include linux/fou.h if it exists

Merge pull request #10415 from poettering/boot-loader-spec-tweaks

docs: tweaks to the boot loader spec

systemd-nspawn: do not crash on /var/log/journal creation if not required

When running a read-only file system, we might not be able to create
/var/log/journal. Do not fail on this, unless actually requested by the
--link-journal options.

$ systemd-nspawn --image=image.squashfs ...

missing: support old linux/fou.h

linux/fou.h was introduced in 3.18.
FOU_ATTR_REMCSUM_NOPARTIAL was added in 4.0.
FOU_CMD_GET was added in 4.1.

Follow-up for 53cb501a1314740fa777f145067cefccda954487.
Fixes #10474.

Merge pull request #9437 from poettering/sd-boot-count

many sd-boot/bootctl fixes, and a new "boot counting" concept, for automatic fallback to older kernels on persistent failures

networkd: keep bond slave up if already attached

There is no need to disable an already correct enslaved interface.

relates to #10118

sd-dhcp: remove unreachable route after rebinding return NAK

Merge pull request #10470 from yuwata/log-message-updates

Log message updates

boot-loader-spec: drop link to obsolete patch

This is not what distros use, let's not point users to obsolete stuff.

update TODO

boot-loader-spec: elaborate on 'architecture' stanza a bit and use it in the example