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

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()

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

docs: tweaks to the boot loader spec

A couple of changes:

1. Clearly name the drop-in entry files "Type #1", and the unified
   kernel images "Type #2", and be clearer that the latter is specific
   to UEFI.

2. Suffix all directory paths with a trailing "/" to clarify that these
   are directories. Also, enclose them all in ``.

3. Add introductory paragraph that explains that there is Type #1 and
   Type #2 and what they are about.

4. Explain that Type #2 is about signed UEFI SecureBoot.

5. Don't claim that $BOOT/loader/ contains really all files defined by
   the spec, because that's not true, Type #2 images are not located there
   after all.

Fixes: #10399

man: use proper <keycap> and <keycombo> docbook tags for key bindings

doc: add a markdown document introducing the boot assessment logic

man: document boot counting logic in systemd-boot

man: document systemd-bless-boot-generator

man: document systemd-boot-check-no-failures.service

man: document systemd-bless-boot

man: document the various EFI vars sd-boot sets

units: add simple boot check unit

This is might be useful in some cases, but it's primarily an example for
a boot check service that can be plugged before boot-complete.target.

It's disabled by default.

All it does is check whether the failed unit count is zero

man: update kernel-install(8) documentation

Many general updates, but most importantly, document the
/etc/kernel/tries logic briefly.

kernel-install: teach bot loader spec entry generator to generate entries with "tries"

This makes two changes:

1. When called for "remove" any drop-ins with "+" suffix are removed
   too, so that the logic works for entries with boot counting enabled
   too and we don't lose track of configuration snippets created that
   way.

2. When called for "add" we optionally generate a "+" suffix, based on
   the data in /etc/kernel/tries if it exists.

   This basically means after "echo 5 > /etc/kernel/tries" any installed
   kernels will automatically set up for 5 boot tries before older
   kernels will be tried.

add "systemd-bless-boot-generator"

This generator automatically pulls in "systemd-bless-boot.service" if
a boot with boot counting is detected.

add new systemd-bless-boot.service that marks boots as successful

This is the counterpiece to the boot counting implemented in
systemd-boot: if a boot is detected as successful we mark drop the
counter again from the booted snippet or kernel image.

man: document new "boot-complete.target" unit

units: add generic boot-complete.target

update TODO

tree-wide: replace 'unsigned int' with 'unsigned'

Merge pull request #10462 from yuwata/test-sd-device-monitor

Add tests for sd_device_monitor and small cleanups

nspawn: do not include '%m' in log message if errno is zero

import: include error cause in log message

gpt-auto-generator: include error cause in log message

firstboot: include error cause in log message

dissect: include error cause in log message

coredump: include error cause in log message

core: include error cause in log message

cgtop: include error cause in log message

busctl: include error cause in log message

bootctl: include error cause in log message

fs-util: include error cause in log message

analyze: include error cause in the log message

analyze: update log message

tree-wide: some automatic coccinelle fixes (#10463)

Nothing fancy, just coccinelle doing its work.

test: add tests for sd_device_monitor

Merge pull request #10457 from yuwata/fuzz-11019

sd-dhcp: fixes oss-fuzz#11019 and several cleanups

core: stop ignoring errors in connect_logger_as

When journald reaches the maximum number of active streams, it,
basically, starts to decline new connections. On the client
side it can be detected by getting EPIPE and, if the writing
process isn't lucky enough, getting SIGPIPE soon afterwards.
systemd has always ignored EPIPE, which makes it very hard
to keep track of services losing logs. This patch should make
it easier to detect such services by just staring at the logs
carefully.

In case anyone is interested, the following one-liner run as any user
can be used to paralyze all the stream logging on a machine:

for i in {1..4096}; do systemd-cat -t HEY-$i & done

Merge pull request #10460 from yuwata/setsockopt_int-more

tree-wide: use setsockopt_int() more

sd-device-monitor: include error cause in log message

sd-device-monitor: drop unnecessary condition

core: beautify per-unit consumed resources log message a bit. (#10390)

Shorten message to say "no IP traffic" if there is no IP traffic, rather
than "received 0B IP traffic, sent 0B IP traffic".

Fixes: #9816

Merge pull request #10437 from poettering/env-util-love

some env-util.c love

Merge pull request #10389 from poettering/nspawn-path-fix

nspawn $PATH execvpe() fix

Merge pull request #10450 from poettering/foreach-line-excorcism

FOREACH_LINE excorcism

Merge pull request #10447 from poettering/fgets-excorcism

let's get rid of fgets()

sd-dhcp6: drop empty 'error' label

test-dhcp6-client: normalize logs

fuzz: add testcase of oss-fuzz#11019

sd-dhcp6: make dhcp6_option_parse_domainname() not store empty domain

This improves performance of fuzzer.
C.f. oss-fuzz#11019.

sd-dhcp6: do not update serverid when ENOMEM

sd-dhcp6: coding style cleanups

sd-dhcp6: fix argument and error handling of dhcp6_option_parse_status()

resolved-mdns: use TAKE_FD()