resolved: do not keep dns_server to dns_stream ref if tls connection failed (#9855)

The references to the dns_server are now setup after the tls connection is setup.
This ensures that the stream got fully stopped when the initial tls setup failed
instead of having the unref being blocked by the reference to the stream by the server.
Therefore on_stream_io would no longer be called with a half setup encrypted connection.

Fixes the issue reported in #9838.

resolve: do not hit CNAME or DNAME entry in NODATA cache (#9836)

Fixes #9833.

fsck: use our usual syntax for defining bit masks

Merge pull request #9848 from yuwata/fix-9835-9844

core: namespace fixes

core/execute: fix dump format for Limit*=

Fixes #9846.

journal: do not remove multiple spaces after identifier in syslog message

Single space is used as separator.
C.f. discussions in #156.

Fixes #9839 introduced by a6aadf4ae0bae185dc4c414d492a4a781c80ffe5.

util: bind_remount_recursive_with_mountinfo(): ignore submounts which cannot be accessed

Fixes #9844.

core/namespace: add more log messages

Suggested by #9835.

meson: actually honor pkgconfig*dir options (#9841)

both were silently ignored leading to some of the pkg-config files
ending up in the wrong place

timedate: emit property changed signal after all jobs are completed

Follow-up for 3af0a96c0fcc623bd16649fc3640396a657cf9ef (#9684).

Merge pull request #9827 from yuwata/fix-9795-9820

journal: fixes issues reported by ASan

tmpfiles: don't adjust qgroups on existing subvolumes

The qgroup logic (types 'q' and 'Q') only has an effect if there's no previous
setup at all, and any explicitly configured subvolumes with their qgroups are
left entirely unmodified.

The idea is that if users want a different logic than the one we set up by
default, then by all means they should do that before hand, and tmpfiles won't
override their logic.

resolve: do not compress target names in SRV records

Fixes #9793.

Merge pull request #9830 from yuwata/journalctl-help

journal: do not hide options in help message

syslog: fix segfault in syslog_parse_priority()

kernel-install: don't try to run depmod when kernel doesn't support modules

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

journal: fix size of buffer

resolve: use memcmp_safe() and memcpy_safe()

As the length of salt in NSEC3 may be zero.

Fixes #9757.

util: introduce memcmp_safe()

core: use memcpy_safe()

Fixes #9738.

journal: fix syslog_parse_identifier()

Fixes #9829.

journal: do not use newa() or strjoina() for message

Fixes another issue reported in #9795.

journal: do not hide options in help message

Even if built without gcrypt, show the relevant options in help message.
Otherwise, the help message diverges from the man page or suggestions
by the shell completion.

bash-completion: journalctl: add --grep and --case-sensitive

journal: do not pass a negative value to memcpy()

The message may contains only whitespaces.

Fixes #9795.

link: fix type for link-config's "features" array of tristates

The "features" fields is parsed as a tristate value. The values
are thus not of type NetDevFeature enum but int. The NetDevFeature
enum is instead the index for the features array.

Adjust the type. In practice, this had no impact because NetDevFeature
enum commonly has size of int.

Also, don't use memset() 0xFF to initilize the int with -1. While
it works correctly in practice, it feels ugly.

test: cast values in proper type to suppress warnings

Follow-up for #9789.

Merge pull request #9817 from yuwata/shorten-error-logging

tree-wide: Shorten error logging and several code cleanups

Merge pull request #9744 from yuwata/fix-9737

Make RootImage= work with PrivateDevices=

hwdb: redefine Lenovo ThinkPad X140e touchpad dimensions (#9818)

The default setup for the Lenovo ThinkPad X140e is 104x104mm, and the kernel
claims that it's 64x21. The default 104x104mm dimensions causes the vertical
axis to act oddly, causing random vertical jitters and higher vertical
sensitivity.

Measuring it showed that it was 74x32, and these touchpad dimensions provide
a better (if a little bit slower) experience but a consistent sensitivity
in all directions.

These values were obtained using the `touchpad-edge-detector` tool.

Merge pull request #9789 from filbranden/cmp1

Add new CMP(a, b) macro

systemctl: add support for --wait to is-system-running

This makes it possible to wait until boot is finished without having to poll
for this command repeatedly, instead using the syntax:

  $ systemctl is-system-running --wait

Waiting is implemented by waiting for the StartupFinished signal to be posted
on the bus.

Register the matcher before checking for the property to avoid race conditions.

Tested by artificially delaying startup with a oneshot service and calling this
command, checked that it emitted `running` and exited with a 0 return code as
soon as the delay service completed startup.

Also tested that booting to degraded state unblocks the command.

Inserted a delay between getting the property and waiting for the signal and
confirmed this seems to work free of race conditions.

Updated the --help text (under --wait) and the man page to document the new
feature.

tree-wide: use returned value from log_*_errno()

dns-domain: use CMP() in dns_name_compare_func

resolve: use CMP() in dns_resource_record_compare_func

This function doesn't really implement ordering, but CMP() is still fine to use
there. Keep the comment in place, just update it slightly to indicate that.

tree-wide: Convert compare_func's to use CMP() macro wherever possible.

Looked for definitions of functions using the *_compare_func() suffix.

Tested:
- Unit tests passed (ninja -C build/ test)
- Installed this build and booted with it.

network: Use CMP() macro for comparison.

Follow up for PRs #9764 and #9760.

macros: add CMP(a, b) macro.

Macro returns -1, 0, 1 depending on whether a < b, a == b or a > b.

It's safe to use on unsigned types.

Add tests to confirm corner cases are properly covered.

macro: drop __extension__, reformat and reindent

Drop __extension__, since we don't use gcc -Wpedantic or -ansi.

Reformat code for spacing. Add spaces after commas almost everywhere.
Reindent code blocks in macro definitions, for consistency.

tree-wide: shorten error logging a bit

Continuation of 4027f96aa08c73f109aa46b89842ca0e25c9c0e9.

resolve: use _cleanup_ attribute

machine: use free_and_replace() and TAKE_PTR()

login: use free_and_replace() and TAKE_PTR()

initctl: do not ignore errors in function

machinectl: shorten error logging a bit

smack-util: use 'path' paremeter as is if it's already absolute in mac_smack_fix_at()

Merge pull request #9792 from poettering/hashmap-mempool

minor hashmap fixes

resolve: treat some icmp errors as disconnected

Fixes #9773.

resolve: sort headers

Merge pull request #9775 from yuwata/follow-up-9766

namespace: implicitly adds DeviceAllow= when RootImage= is set

RootImage= may require the following settings
```
DeviceAllow=/dev/loop-control rw
DeviceAllow=block-loop rwm
DeviceAllow=block-blkext rwm
```
This adds the following settings implicitly when RootImage= is
specified.

Fixes #9737.

core: introduce cgroup_add_device_allow()

core/namespace: drop mount points outside of root even if RootDirectory= is not set

core/namespace: drop conditions depends on `root` is empty or not

After 0722b359342d2a9f9e0d453875624387a0ba1be2, the variable `root`
is always set.

resolve: fix error handling of dns_name_is_valid()

man: mention that Hostname= for DHCP must be a valid DNS domain name

network: DHCP: ignore error in setting hostname when it is given by uname()

C.f. #9759.

network: use free_and_replace()

network: also check that Hostname= is a valid DNS domain name

Merge pull request #8822 from fbuihuu/rfc-tmpfiles-safe-upstream

Make tmpfiles safe

man: include libsystemd-pkgconfig.xml in a few more man pages

for some reason this was forgotten for a couple of sd_bus man pages,
let's fix that.

test-resolved: follow coding style, use fopen() with 'e'

hashmap: add an explicit assert() for detecting when objects migrated between threads

When clients don't follow protocol and use the same object from
different threads, then we previously would silently corrupt memory.
With this assert we'll fail with an assert(). This doesn't fix anything
but certainly makes mis-uses easier to detect and debug.

Triggered by https://bugzilla.redhat.com/show_bug.cgi?id=1609349

hashmap: add an environment variable to turn off the memory pool used by hashmaps

Triggered by https://bugzilla.redhat.com/show_bug.cgi?id=1609349

man: be more explicit about thread safety of sd_journal

Triggered by https://bugzilla.redhat.com/show_bug.cgi?id=1609349

This adds two generic paragaphs we include via xinclude. One is the
"strict" version, which contains wording saying that we are thread
agnostic and what that means. And the other is the "safe" version, for
the cases we provide fully safety.

Let's then change most man pages to use either of these generic
paragraphs. With one exception: man/sd_journal_get_catalog.xml contains
both kinds of function, we hence use manual wording.

update TODO

Merge pull request #9791 from poettering/user-runtime-dir-fixes

user-runtime-dir@.service fixes

Merge pull request #9787 from yuwata/dns-over-tls-log

DNS-over-TLS(openssl): fix error handling

units: assign user-runtime-dir@.service to user-%i.slice

This service won't use much resources, but it's certainly nicer to see
it attached th the user's slice along with user@.service, so that
everything we run for a specific user is properly bound into one unit.

units: order user-runtime-dir@.service after systemd-user-sessions.service

We use systemd-user-sessions.service as barrier when to allow login
sessions. With this patch user@.service is ordered after that too, so
that any login related code (which user-runtime-dir@.service is) is
guaranteed to run after the barrier, and never before.

units: make sure user-runtime-dir@.service is Type=oneshot

We order user@.service after it, hence we need to properly know when it
finished starting up.

user-runtime-dir: downgrade a few log messages to LOG_DEBUG that we ignore

As the comments already say it might be quite likely that
$XDG_RUNTIME_DIR is not set up as mount, and we shouldn't complain about
that.

Moreover, let's make this idempotent, so that a runtime dir that is
already gone and is removed again doesn't cause failure.

resolve: openssl: make dnstls_stream_{write,read}() may return zero

resolve: fix error handling of SSL_shutdown()

resolve: fix typo and coding style cleanups

resolve: clear error queue before calling SSL_*()

man: document that 'nofail' also has an effect on ordering

Prompted by #9765

Merge pull request #9776 from keszybz/sd-bus-docs

More man pages for sd-bus and related changes

resolve: fix error handling of dns_stream_read()

Merge pull request #9764 from yuwata/hash_ops-cleanups

Hash ops cleanups

man: add sd_bus_message_rewind(3)

man: document sd_bus_slot_get_bus in sd_bus_slot_ref(3)

Similar reasoning as for sd_bus_message_get_bus().

man: add sd_bus_slot_set_description(3)

man: add sd_bus_slot_set_userdata(3)

man: add sd_bus_message_set_expect_reply(3)

man: document sd_bus_message_get_bus() in sd_bus_message_new(3)

It's not a particularly obvious place, but it's a trivial function that isn't
worth a man page of its own, and it doesn't fit anywhere else either.

man: add sd_bus_message_verify_type(3)

sd-bus: verify destination and sender values when setting

We would verify destination e.g. in sd_bus_message_new_call, but allow setting
any value later on with sd_bus_message_set_destination. I assume this check was
omitted not on purpose.

man: add sd_bus_message_get_type(3)

sd_bus_message{get_type,is_signal,is_method_call,is_method_error} get one man
page.

sd_bus_message_{set,get}_{destination,path,interface,member,sender} are put in
the second one.

man: add sd_bus_slot_ref(3)

man: document sd_bus_message_new_method_return

meson: print stats in check-api-docs

man: document *_with_description functions

man: add sd_bus_message_new(3)

man: move more examples to stand-alone files and use 2-space indentation consistenty

Moving them out makes it easier to run them through a compiler, use automatic
indentation, and opens the possibility to provide a download link in the
future. I verified that all examples compile cleanly.

(2-space indentation is used because the examples are already significantly
indented in the man page, and we need to keep them narrow so that they display
well on standard terminals.)

man: clarify what can be NULL in an sd_bus_unrefp call

Confusingly, the argument is called 'bus' in all cases. Let's not give people
the idea to call sd_bus_unrefp(NULL).

man: add sd_bus_message_new_call(3)

man: add sd_bus_message_new_signal(3)

basic: use FAMILY_ADDRESS_SIZE() macro in hash_ops for in_addr_data

network: make compare functions safe

As the variable 'line' is unsigned.