journal-gateway: use localStorage["cursor"] only when it has valid value

Discovered by LGTM.

journal-gateway: explicitly declare local variables

Suggested by LGTM.

sleep: fix printf format of fiemap fields

Use PRIu64 and PRIu32 constants to also get the format right on LP-64
architectures.

For the 64-bit fields, we need a cast to (uint64_t), since __u64 is
defined as a `long long unsigned` and PRIu64 expects a `long unsigned`.
In practice, both are the same, so the cast should be OK.

README.md: add an LGTM badge

Now that @yuwata has fixed a lot of alerts, the badge looks
much better than before, so let's show it :-)

tree-wide: use localtime_r() instead of localtime()

Follow-up for e46acb7950a9f07ac60d772309de842c444ad2bd.

netdev: do not assign unused values

crypt-util: Translate libcryptsetup log level instead of using log_debug()

This makes sure that errors reported by libcryptsetup are shown to the
user instead of getting swallowed up by log_debug().

timedate: use gmtime_r() and localtime_r()

gmtime() and localtime() operate on a static buffer. let's avoid this,
as we never know whether some library might use these calls in some
backrgound thread.

Discovered by lgtm:

https://lgtm.com/projects/g/systemd/systemd/

Merge pull request #9411 from poettering/systemctl-is-running-fixes

tiny systemctl fixes

man: add comments about MTUBytes= and MACAddress= for tuntap devices

Closes #9407.

Merge pull request #4522 from lucaswerkmeister/unescape-template

escape: support --unescape with --template

busctl: make parameter const

it's initialized from an argv[] entry, hence let's be make it
non-writable by default.

meson: check whether gnutls supports TCP fast open

Fixes #9403

systemctl: if we can't detect the system-is-running state, assume it's not running

systemctl: log errors to LOG_DEBUG rather than eating them up entirely

systemctl: add 'static' to constant state array

bls: allow firmware-supported file systems

This copies the text from Matthew Garrett's version verbatim. At
least MacOS uses HFS+ for their ESP.

[hwdb] Add support for HP ProBook 650 wifi switch key (#9260)

Add support for HP ProBook 650 wifi switch key

resolved: assert t->server is set in dns_transaction_emit_tcp.

Uncovered by Coverity. Fixes CID 1393390.

core/job: add check for return of job_type_merge_and_collapse()

Using an assertion is fine, since calls to job_merge_into_installed()
are protected by a check for job_type_is_conflicting().

Uncovered by Coverity, fixes CID 996307.

hwdb: added accelerometer quirk for Aspire Switch 10 (sw5-011)

Merge pull request #9398 from yuwata/lgtm-fixes

fixes error handlings and several cleanups suggested by LGTM

core/manager: Make sure jobs statistics are not double after daemon-reload

We add n_installed_jobs and n_failed_jobs to our inner state after
deserialization. This is fine during daemon-reexec when we start with clear
Manager (and some jobs possibly queued before deserialization), however,
daemon-reload works with the same manager and adding the values would
effectively double the counters. Reset the counters before we deserialize and
add their values again.

man: SystemMaxUse= clarification in journald.conf(5)

Similar semantics applies to {System,Runtime}MaxUse= as to
{System,Runtime}MaxFiles=, i.e. active files aren't vacuumed and the limit
is not thus strictly honored.

escape: add --instance option

Suggested by @keszybz in #4522.

boot: do not assume the last character in cmdline is a newline

Fixes #9401.

escape: support --unescape with --template

tmpfiles: specify access mode for /run/systemd/netif

This partially reverts 2af767729489f6baa98a2641b2007acab44ed353.
As the directories are certainly readable and not-writable by
non-privileged users.

network: fix log message

timesync: update log message

Fixes #9388.

timesync: changes type of drift_freq to int64_t

drift_freq is used for storing timex.freq, and is a 64bit integer.
To support x32 ABI, this changes the type of drift_freq to int64_t.

Fixes #9387.

hwdb: Fix secondary backslash on MSI GS65 laptop

The MSI GS65 Stealth Thin series laptop contains an extra physical backslash key next to the spacebar which does not yield the correct keycode.

This pull request fixes the issue

Add resolution of Trust Flex Graphics Tablet to hwdb

Product:
https://www.trust.com/en/product/21259-flex-design-tablet-black
http://trust.com/21259

Support in libinput is already in place since libinput 1.11 release (I tested it)
https://gitlab.freedesktop.org/libinput/libinput/issues/38

So only missing part is definition in hwdb.

docs: put LGTM on the list of QA tools and fix a couple typos

install: fix error handling in is_symlink_with_known_name()

portable: fix error handling

import: drop unnecessary condition

sd-dhcp: drop unnecessary condition

machine-pool: drop unnecessary condition

resolve: drop dead code

resolve: drop dead code

bootctl: drop dead code

journal: drop redundant condition

resolve: fix return value type of dns_answer_has_dname_for_cname()

resolve: dns_scope_network_good() does not returns negative errno

core/cgroup: drop unnecessary condition

bus-util: fix error handling

tree-wide: add multiple inclusion guard

meson: drop unused option

Follow-up for add384dd4d2b96db6ace5ad9c52b1dd7553ebec2.

core: drop unused log message

temporary_filesystem_add() does not parse mount options.

core: Actually use the resolved path for TemporaryFileSystem= (#9385)

The code already resolves specifiers using unit_full_printf() but then uses the
unresolved version again for temporary_filesystem_add().

tree-wide: drop copyright lines for more authors

Acks in https://github.com/systemd/systemd/issues/9320.

meson: bump package and library version

final touches to NEWS

man: small fixups for systemd-boot(8)

- "UEFI firmware" is a name, no need for an article
- The specification is more than "recommendation"

man: correct the meaning of TimeoutStopSec= (#9325)

Fixes: #9325

tmpfile: do not specify mode and owner to /run/systemd/netif

Fixes #9369.

journalctl: handle inotify exhaustion nicer

Instead of failing, log that inotify wds
are exhausted and fallback to -n switch.

Fixes #1296.

udev: exclude DRBD from block device ownership event locking

It does not make sense for udev to even open DRBD block devices
(/dev/drbdX). It is on one hand not necessary as DRBD is controlled by
something else in the stack (e.g., pacemaker), and it even can get
cumbersome in various scenarios (e.g., DRBD9 auto-promote).

Closes: #9371
Signed-off-by: Roland Kammerer <roland.kammerer@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>

man: systemd.offline-updates: Document system-update-pre.target usage

Merge pull request #9368 from yuwata/bash-completion-timedate

follow-ups for 'timedatectl show'

NEWS: mention 'timedatectl show'

bash-completion: support 'timedatectl show'

NEWS: mention %E

Merge pull request #9366 from poettering/udev-wants-dep-fix

udev SYSTEMD_WANTS property fix

core: explicitly trigger changing udev SYSTEMD_WANTS properties

This compensates for the unsynchronized reload cycles of systemd and
udev: we manually trigger the deps listed in SYSTEMD_WANTS properties if
they change for device units that are already up. That way all deps
defined that way will be triggered at least once: the first time the
unit goes up by the usual dependency logic, and if it already is up by
the device.c specific logic.

Fixes: #9323

analyze: use _cleanup_ for freeing the manager object

analyze: use log_unit_error_errno()'s return value where we can

tree-wide: pass NULL arguments to manager_startup() directly, avoid declaring unneeded variables

units: let's use two ExecStart= lines instead of ;

core: use fflush_and_check() where appropriate

update TODO

Merge pull request #9365 from keszybz/condition-security-followup

ConditionSecurity= followup

test-condition: extend tests to all ConditionSecurity= values

Also print out what we detect, for manual verification.

NEWS: mention ConditionSecurity=uefi-secureboot

Follow-up for be405b909e5d78b43e3af47e0d10cd84c714e2f3.

condition: add new conditon ConditionSecurity=uefi-secureboot

We have the detector call for this anyway, and it's useful for
conditioning out dbxtool.service, hence let's add this tiny new option.

Merge pull request #9363 from yuwata/specifier-config-dir

core: add unit specifier for configuration directory root

meson: make DNS-over-TLS support optional

This adds dns-over-tls option to meson. If set to 'false',
systemd-resolved is not linked with libgnutls.

Merge pull request #9350 from poettering/bls-fixes

various fixes to the systemd-boot documentation

systemctl: mask always reports the same unit names when different unknown units are passed

Before this patch:

  # systemctl --runtime mask abuild.mount does-not-exist.mount does-also-not-exist.mount
  Unit abuild.mount does not exist, proceeding anyway.
  Unit abuild.mount does not exist, proceeding anyway.
  Unit abuild.mount does not exist, proceeding anyway.
  Created symlink /run/systemd/system/abuild.mount → /dev/null.
  Created symlink /run/systemd/system/does-not-exist.mount → /dev/null.
  Created symlink /run/systemd/system/does-also-not-exist.mount → /dev/null.

After this patch:

  # systemctl --runtime mask abuild.mount does-not-exist.mount does-also-not-exist.mount
  Unit abuild.mount does not exist, proceeding anyway.
  Unit does-not-exist.mount does not exist, proceeding anyway.
  Unit does-also-not-exist.mount does not exist, proceeding anyway.
  Created symlink /run/systemd/system/abuild.mount → /dev/null.
  Created symlink /run/systemd/system/does-not-exist.mount → /dev/null.
  Created symlink /run/systemd/system/does-also-not-exist.mount → /dev/null.

test: add test for specifier of configuration directory root

core: add unit specifier for configuration directory root

Follow-up for 14068e17f32a06e6a1f8f72416018bc653b8ea2d.

units: fix typo in After=

Followup for c7668c1ce04fa85370432d197d2ccd9411e85649.

test-sizeof: another aproach to _Float128 availability detection

NEWS: document that the BLS is now part of our tree

man: update systemd-boot(7) man page in many ways

Let's fully document where the list of entries come from, including unified
images and such.

Let's add a "Files" section (replacing the "Configuration" section), and
let's move it after they keybinding section (why? because keybinds are
primary UI material, while configuration is one level more complex than
that).

Also, reword lot's of stuff to make it more precise.

Fixes: #5127

man: fix URL to BLS

Let's refer to our own version now.

doc: document the `architecture` setting

doc: allow multiple initrd entries per BLS snippet

sd-boot already supports that anyway, and the Fedora folks asked for
this on the fedora mailing list, hence let's simply add this.

doc: update BLS links list

let's make these proper links

doc: add a second indentation level to partition list, to separate MBR and GPT partitions

tree-wide: drop MSG_NOSIGNAL flag from recvmsg() invocations

MSG_NOSIGNAL is only defined for sendmsg(), not for recvmsg(), hence
let's drop it's use, in particular as it appears to create problems on
older kernels. See:

https://lists.freedesktop.org/archives/systemd-devel/2018-June/040869.html

update NEWS contributors list again

Revert "mention alias for poweroff in machinectl --help"

This reverts commit bebd5382ba4c820377f05e1e518c6dfd96b278be.

For the reasoning, see: https://github.com/systemd/systemd/pull/9342#issuecomment-398715417

units: make system-update-pre.target a passive unit (#9349)

This is an additional synchronization point normally not needed. Hence,
let's make it passive, i.e. pull it in from the unit which wants to be
ordered before the update service rather than by the update service
itself.

Merge pull request #9348 from keszybz/copyright-removal

Copyright removal

Drop copyrights tags for "systemd authors"

basic/bpf-program: drop obsolete comment

As far as I can see, nothing in that particular file is from the
kernel. Let's just drop the comment.

tree-wide: drop copyright headers from frequent contributors

Fixes #9320.

for p in Shapovalov Chevalier Rozhkov Sievers Mack Herrmann Schmidt Rudenberg Sahani Landden Andersen Watanabe; do
  git grep -e 'Copyright.*'$p -l|xargs perl -i -0pe 's|/([*][*])?[*]\s+([*#]\s+)?Copyright[^\n]*'$p'[^\n]*\s*[*]([*][*])?/\n*|\n|gms; s|\s+([*#]\s+)?Copyright[^\n]*'$p'[^\n]*\n*|\n|gms'
done

shared/path-lookup: remove unnecessary 'if' to help gcc

Fixes #9343.