catalog: (void)ify all things

catalog: use structured initialization more

catalog: let's use mkdir_parents() and use same mode as tmpfiles.d for this dir

let's use mkdir_parents() (because its shorter), and 0755 as access
mode, so that things have the access mode tmpfiles.d also suggests.

Prompted by: #11903

catalog: be a bit less aggressive with line breaking. After all 109ch is what we going for

catalog: don't make catalog_entry_lang() clobber output params on failure

A minor fix to bring this in-line with our coding style

units: fix systemd.special man page reference in system-update-cleanup.service

Merge pull request #11916 from yuwata/pid1-id-renaming-handling

core: handle ID_RENAMING= udev property

Merge pull request #11918 from keszybz/clang-8

Two trivial changes to build with clang 8

nspawn: move payload to sub-cgroup first, then sync cgroup trees

if we sync the legacy and unified trees before moving to the right
subcgroup then ultimately the cgroup paths in the hierarchies will be
out-of-sync... Hence, let's move the payload first, and sync then.

Addresses: https://github.com/systemd/systemd/pull/9762#issuecomment-441187979

po: update Polish translation

test-execute: avoid undefined variable warning when compiled w/o seccomp

build-sys: disable -Wstring-plus-int

clang-8 warns about constructs like "foobar"+3 (to get "bar"). We
like to use patterns like this, so let's just disable the warning.

remove "." path components from required mount paths

unit_require_mounts_for may be passed path arguments that contain "."
components like for user's home directories where "." is sometimes used
to specify some form of anchor point.

This change stops considering such path as an error and removes the "."
components instead.

Closes: #11910

hwdb: Add support for Alienware touchpad toggle

BugLink: https://bugs.launchpad.net/bugs/1605433

Merge pull request #11911 from keszybz/bootspec-handle-invalid-efi

Bootspec handle invalid efi

TODO: update

test: add test for ID_RENAMING= udev property handling by pid1

man: mention possible actions supported by "udevadm trigger"

udevadm trigger: make --action option support all possible actions

This also make udevadm abort operations if invalid action is specified.

sd-device: move device_action_from_string() and friends to device-private.h

core/device: make devices with ID_RENAMING= property be considered not ready

shared/bootspec: do not fail on errors when reading EFI vars

It seems that my EFI storage is corrupted. The kernel reports the file as present, but returns
an error when reading. Nevertheless, this shouldn't prevent me from reading the entry
list.

Fixes #11909.

shared/bootspec: treat empty EFI vars as missing

We shouldn't really make any fuss about this. Also, change 'var' to 'variable'
for consistency with systemctl.c.

shared/bootspec: minor simplification

Merge pull request #11898 from keszybz/meson-conv-lib

Use a convenience library for the sources shared between core/ and shutdown/

json: log about the right line number

We mean the line number in the json data, not the line number in our
C source code.

Addresses: https://github.com/systemd/systemd/pull/9762#issuecomment-441183987

meson: drop "_la_" from variable name

A left-over from libtool times.

meson: use a convenience library for the sources shared between core/ and the outside

This avoids double compilation. Those files are tiny, so it doesn't save time,
but we avoid repeated warnings and errors, and it's generally cleaner to it
this way.

The number of commands in 'ninja -C build clean && ninja -C build' drops from
1462 to 1455 for me.

Merge pull request #10495 from poettering/login-boot-hookup

add "systemctl reboot --boot-loader-menu=" and "systemctl reboot --boot-loader-entry="

pkgconfig: avoid double slash with split-usr configuration

By defining rootprefix= we avoid a double slash in $systemdsystemunitdir and
other variables. This fixes a regression introduced in
1c2c7c6cb3d92315624f3711114e86b0acfbce63 where the variables using rootprefix=/
would start with a double slash. This should be interpreted the same, but is
certainly ugly.

The rootprefix variable was added to systemd.pc in
1c2c7c6cb3d92315624f3711114e86b0acfbce63, so there is no question of backwards
compatiblity. If people try to "override" the prefix and specify
--define-variable=rootprefix=/, they will get a double slash, which should be
OK, and is the same as --define-variable=rootprefix=/something/, which also
results in a double slash somewhere in the strings.

update TODO

docs: document the new environment variables logind groks

logind: refactor logic of returning polkit challenge result into its own function

Let's share some code.

systemctl: add support for booting into boot menu/entry

(This also removes support for booting into the EFI firmware setup
without logind. That's because otherwise the non-EFI fallback logind
implements can't work.)

Fixes: #9896

logind: add support for booting into the boot menu or a specific boot menu entry

This behaves similar to the "boot into firmware" logic, and also allows
either direct EFI operation (which sd-boot supports and others might
support eventually too) or override through env var.

bootspec: validate bootpsec entry ids before we use them

bootspec: get rid of find_default_boot_entry() entirely

Now only two operations are left. Let's just move this into the caller,
since it should make things simpler, clearer and shorter, in particular
as there's only a single user for this.

bootspec: rework find_default_boot_entry() to use boot_entries_load_config_auto()

it's pretty much the same code now, hence use it.

bootspec: introduce new helper boot_entries_load_config_auto()

It's a simple wrapper around boot_entries_load_config(), but determines
the ESP/XBOOTLDR paths automatically at first. Also, it looks for a path
/run/boot-loader-entries/ and loads the entries from there if it
exists. This is supposed to be a hook for other boot loaders to make our
tools aware of their own entries.

bootspec: correct error code shown in log msg

bootspec: move augmentation of loader-discovered entries into bootspec.c

Previously, bootctl would show boot loader entries discovered by the
boot loader which couldn't found locally separately in the output.

Let's move this code into bootspec.c, and beef it up a bit. This way we
can use it later on for logind, and correctly show automatically
discovered windows/macos entries too.

bootspec: simplify find_default_boot_entry()

If the only caller passes NULL for the two paths, let's remove the
parameter altogether.

systemctl: let's FORK_WAIT where we can

systemctl: when something is not supported use EOPNOTSUPP

systemctl: be careful with boot loader entries lacking a 'linux' line

A boot loader entry might not have a 'linux' line, but an 'efi' line or
something else. Let's handle that case nicely.

bootspec: move log msg from systemctl.c to bootspec.c

find_default_boot_entry() is only used by systemctl.c, and currently
handles one log message in the caller instead of the callee. Let's
simplify that and move it over, too

bootctl: tweak 'list' output a bit

Let's suppress the final newline in the list if it's the last entry we
are outputting.

logind: optionally support non-EFI reboot-to-firmware

This extends the reboot-to-firmware logic in logind, so that other than
EFI firmwares could be theoretically support. The scheme is like this:
if you want to support this, set the $SYSTEMD_REBOOT_TO_FIRMWARE=1 env
var for logind. If so, this will override the EFI logic, and cause a
file /run/systemd/reboot-to-firmware file to be created when
reboot-to-firmware is requested. This file has no contents, it's mere
existance indicates a reboot with reboot-to-firmware set.

The idea is that for alternative firmwares a drop-in for logind is added
that sets the env var, in combination with some code run during shutdown
that checks for the file and does the right thing.

efivars: extend character set that may be used in boot loader entry ids a bit

See https://github.com/systemd/systemd/pull/10495#discussion_r233992570

Merge pull request #11896 from poettering/shutdown-fixes

move src/core/shutdown.c and helpers to src/shutdown/

docs: add a note about compilation options

Closes #6371.

core: warn if people use the undocumented/depreacted ConditionNull=

Triggered by:

https://github.com/systemd/systemd/issues/11812

udev: restore debug level when logging a failure in the external prog called by IMPORT{program}

It was already the case before commit a75211421fc9366068e6d9446e8e567246c72feb,
which upgraded the log to warning.

This seems an unintended side effect as the commit message doesn't mention it
and the old behavior looks more appropriate.

shutdown: rearrange shutdown sources in source tree

Let's move the shutdown binary into its own subdirectory in
src/shutdown, after all it is relatively isolated from the normal PID 1
sources, being a different binary and all.

Unfortunately it's not possible to move some of the code, since it is
shared with PID 1, that I wished we could move, but I still think it's
worth it.

shutdown: (void)ify more stuff

shutdown: fix up return type of sync_making_progress()

We shouldn't return negative errnos as "bool", hence fix the type of the
function to "int".

update TODO

Merge pull request #11881 from yuwata/networkd-vs-interface-renaming

Networkd vs interface renaming

udev: run programs in the specified order

This fixes bugs introduced by 29448498c724da7ade1b5efb20d7472c1b128d2c
and d838e14515c82b05a07f2bf393cce057b45b2b53.

Previously, RUN and SECLABEL keys are stored in udev_list with its unique
flag is false. If the flag is false, then udev_list is just a linked
list and new entries are always added in the last.
So, we should use OrderedHashmap instead of Hashmap.

Fixes #11368.

hwdb: add touchpad resolutions for 2015-2017 MacBook(Pro)'s. (#11874)

These are all models using an SPI keyboard and touchpad and using the
same applespi kernel driver.

travis: use /bin/systemd instead of /usr/bin/systemd

Apparently systemd is in /bin now.

dhcp: refuse to configure DHCP IAID if the interface is under renaming

systemd-networkd itself does not start dhcp client, but the code
may be used in other projects. So, check that the interface is under
renaming or not.

network: do not configure interfaces under renaming

util: introduce device_is_renaming()

It will be used in the later commit.

network: always drop configs when interface is renamed

Before the renaming, wrong .network file may be assigned to the link.
So, let's always drop link configuration.

udev: set ID_RENAMING property when interface renaming is requested

And drop the property on the corresponding 'move' uevent.

udev: do not read UdevEvent object before checking it is non-NULL

udev: drop unnecessary copy of new interface name

netlink: check new interface name is valid or not before sending request

Merge pull request #11841 from keszybz/dns-packet-speedup

DNS packet speedup

Merge pull request #11871 from yuwata/systemctl-show-format-unprintable

systemctl: format many entries in 'show' command

Merge pull request #11888 from keszybz/non-atomic

Drop unused atomic operations

fuzz-dns-packet: add test case with lots of labels

https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=13422

resolved: when adding RR to an answer, avoid comparing keys twice

We'd call dns_resource_record_equal(), which calls dns_resource_key_equal()
internally, and then dns_resource_key_equal() a second time. Let's be
a bit smarter, and call dns_resource_key_equal() only once.

(before)
dns_resource_key_hash_func_count=514
dns_resource_key_compare_func_count=275
dns_resource_key_equal_count=62371
4.13s user 0.01s system 99% cpu 4.153 total

(after)
dns_resource_key_hash_func_count=514
dns_resource_key_compare_func_count=276
dns_resource_key_equal_count=31337
2.13s user 0.01s system 99% cpu 2.139 total

resolve: split the RR comparison function in two

No functional change.

resolved: use a temporary Set to speed up dns question parsing

This doesn't necessarily make things faster, because we still spend more time
in dns_answer_add(), but it improves the compuational complexity of this part.
If we even make dns_resource_key_equal_faster, this will become worthwhile.

systemctl: format LogExtraFields= in 'show' command

systemctl: format BindPaths= or TemporaryFileSystems= in 'show' command

systemctl: print RestrictAddressFamilies= in 'show' command

systemctl: show nothing if no LoadError=

systemctl: format IPIngressBytes= or friends nicely

systemctl: show IPAddressAllow= and IPAddressDeny= in 'show' command

test: add tests for test_in_addr_prefix_to_string()

test: move tests for in_addr_prefix_from_string()

util: introduce in_addr_prefix_to_string()

util: use _cleanup_ attributes

core: fix received size of signal or status size

sd_bus_message_read_array() returns size of array in bytes, not number
of elements.

This also convert int to int32_t, as the dbus type 'i' is int32_t.

sd-hwdb: fix off_t vs. size_t confusion in cast

> on 32bit, size_t is 32bit, but .st_size is off_t hence 64bit

systemctl: show SuccessExitStatus= and friends

systemctl: use streq() if arguments must be non-NULL

systemctl: do not show negative values in {Success,Failure}ActionExitStatus=

bus-util: drop unnecessary re-formatting

systemctl: show SELinuxContext=, AppArmorProfile= and SmackProcessLabel=

Run: systemctl show -a dbus.service | grep -E "SELinuxContext|AppArmorProfile|SmackProcessLabel"

Before patch:
  SELinuxContext=[unprintable]
  AppArmorProfile=[unprintable]
  SmackProcessLabel=[unprintable]

After patch:
  SELinuxContext=[""|"value of context"]
  AppArmorProfile=[""|"value of context"]
  SmackProcessLabel=[""|"value of context"]

semaphoreci: caching and more robust creation of container image

lxc-create in semaphore sometimes fails with

    ERROR: Unable to fetch GPG key from keyserver

Which often happens behind proxies. As the default key server is a load
balancer, retry a few times.

Also, cache the container image between runs, and only recreate it when
it is older than a week.

test: do not assume test-chown-rec is running as root

Remove now-unused refcnt.h

sd-netlink: do not use atomic reference counters

Same as with the other users, any non-trivial use of the objects requires
use from a single thread only or external locking. Using atomic operations
just for reference counts is not useful.

sd-hwdb: some minor logging and style updates

sd-hwdb: use non-atomic reference counters

The sd-hwdb objects cannot be used concurrently from two threads in any
meaningful way, because query and iteration operations modify the object.
Thus atomic reference counts are pointless.

bus: make reference counting non-atomic

We had atomic counters, but all other operations were non-serialized. This
means that concurrent access to the bus object was only safe if _all_ threads
were doing read-only access. Even sending of messages from threads would not be
possible, because after sending of the message we usually want to remove it
from the send queue in the bus object, which would race. Let's just kill this.