loginctl: improve print_{session|user|seat}_status_info() functions

1) Instead of checking if we need to print a new line on each iteration,
pass the "new_line" as a pointer to those functions, so they can use
it to check if a new line is needed. This makes the code more consistent
as it is done in other places: machinectl, systemctl...

2) Move the error messages from show_{session|user|seat}() to their
appropriate print_{session|user|seat}_status_info() functions, this will
prevent from logging an error message twice in case show_properties()
fails and it will improve code readability.

3) Also do not ignore error codes on these functions.

loginctl: use show_properties() to get login1 properties

Commit f8f14b3654bcd introduced a regression that makes loginctl ignore
the "--property" option.

This patch fixes the bug, it uses a new show_properties() function to
query and filter properties.

loginctl: replace strv_append() by strv_extend()

journalctl,zsh-completion: fix several issues in --help message text

-- fix grammar and reword some descriptions for clarity
-- add a useful description of what --follow does
-- fix the description for --after-cursor
-- properly introduce the FSS acronym for "Forward Secure Sealing" in
both sections
-- clarify the --disk-usage command

[zj: perform similar changes to zsh completions]

squash! journalctl: fix several issues in --help message text

man: add DOI for refereed article on Forward Secure Sealing to journald.conf(5)

In journalctl(1), be more explicit about the reference to "Seal=" in
journald.conf(5) and what information can be found there.

man: be more explicit about option arguments that take 128-bit IDs in journalctl(1)

It may not be immediately obvious to the reader what "ID128" is, so replace the
example option argument "ID128" with "128-bit-ID".

man: fix grammar issues in journalctl(1)

And add a missing <option> tag around "--setup-keys" under "--force".

machinectl: show_properties() already logs the error

udevadm: modernization

udevadm,scsi_id: add short options to help strings and to the man page

Also clean things up a bit here and there.

scsi_id: cleanup

Remove -i option which would case exit(1) to happen.
Remove some unused code.
Convert to bool where appropriate.
Simplify things a bit.
Always free everything.

https://bugzilla.redhat.com/show_bug.cgi?id=1043304

driverd: implement AddMatch/RemoveMatch logic

do not run kmod-setup in a container

temporarily support "kdbus" keyword on the kernel commandline to load the module

libudev: monitor - only disallow "udev" type events

libudev: disable monitor inside a container

Uevents are events of the host, which should not leak into a container.
Containers do not support hotplug at the moment, and devices and uevents
are not namespace aware.

bus: catch up with kdbus changes

bus-driverd: fix flags translation in driver_request_name

Flags for a name entry (KDBUS_NAME_*) should be set for
cmd_name->flags (not conn_flags)

Revert "ptyfwd: make master terminal attributes raw, too"

This reverts commit e7d43b3cc30764138c90eaaf95d3d8f49e448890.

This broke the console terminal when booting up a container, so let's
not do this.

bus: reduce calls to KDBUS_CMD_MEMFD_SIZE_SET ioctl

Instead of calling it for each buffer append, increase allocation
exponentially and set the real value only at the end, when sealing off
the memfd.

This should drastically reduce the number of times we invoke the
ioctl().

ptyfwd: make master terminal attributes raw, too

units: when spawning a getty configure TERM explicitly

This way we can make use of our logic to automatically determine an
appropriate TERM for a specific tty.

execute: set TERM even if we don't open the tty on our own

This way, when a tty path is configured TERM is set, which is nice to
set a useful term for gettys.

loginctl: when showing device tree of seats with no devices show something useful

core,logind: libudev usage modernizations

Always use cleanup logic and don't eat up errors returned by libudev

core,logind,networkd: check for udev device initialization via enumeration matches

Instead of checking each device after we got it, check wuth an
enumeration filter instead, to make it more efficient.

core: priorize notification fd processing over notification fd process via sd-event's logic

log: when we log to /dev/console and got disconnected (maybe due to vhangup) reconnect

units: run systemd-networkd.service only if CAP_NET_ADMIN capability is around

This has the effect that systemd-networkd won't run in containers
without network namespacing wher CAP_NET_ADMIN is (usually) not
available. It will still run in containers with network namespacing on
(where CAP_NET_ADMIN is usually avilable).

We might remove this condition check again if networkd provides services
to apps that also are useful in containers lacking network namespacing,
however, as long as it doesn't it should be handled like udevd and be
excluded in such containers.

core: in a container log to /dev/console if "debug" is specified

log: don't reopen /dev/console each time we call log_open()

Instead, force reopen it only if we really really have to.

service: watch main pid even in final states

In some circumstances, for example when start-up times out we
immediately jump into the final state, at which point we still should
try to watch the main pid so that the SIGCHLD allows us to quickly
move into dead state.

libudev: devices received from udev are always initialized

login: Don't stop a running user manager from garbage-collecting the user.

With the current logic, a user will never be garbage-collected, since its
manager will always be around. Change the logic such that a user is
garbage-collected when it has no sessions and linger is disabled.

comment out udev's is_initialized call until the problem is sorted out

core,logind,networkd: don't pick up devices from udev before they finished udev initialization

Managers shouldn't pick up the devices the manage before udev finished
initialization, hence check explicitly for that.

busctl: output a single sorted list of names, including activatable and activated

bus: beef up driverd

Add validity checks for all parameters passed in.

Implement ReleaseName and StartServiceByName().

Fix access control in vtable.

bus: when releasing, taking, listing bus names check if they are valid

Given that the kernel does not validate bus names we have to when using
them.

tests: fix build

bus: sync with kdbus

Two new things here:

a) struct kdbus_notify_name_change now carries two struct
   kdbus_notify_id_change

b) a new KDBUS_CMD_BYEBYE ioctl that has no user yet in systemd

networkd: bridge - fix state machine

We were entering BRIDGE_STATE_CREATED rather than BRIDGE_STATE_READY.

networkd: link - remove useless states

Rework the state-machine a bit.

networkd: rename link_update_flags to link_update

We are likely to track more than the flags in the future.

bus-util: return errors when encountered

Introduced in:
9f6eb1cd58f2ddf2eb6ba0e4de056e13d938af75
313333b403439360c0396a50d77d0a1ee2bca4df

_noreturn_ --> noreturn for C11 compat

also define noreturn w/o <stdnoreturn.h>

util: no need for in_initrd() cache to be thread-local

the process only has one working directory, and a race is
harmless

__thread --> thread_local for C11 compat

Also make thread_local available w/o including <threads.h>.
(as the latter hasn't been implemented, but this part is trivial)

update TODO

libsystemd-bus: true/false instead of yes/no in msg dump

Due to this patch, message dump (for message which includes boolean
type) is more consistent with dbus-send (which display true/false
instead of yes/no for boolean). It's only simple 'cosmetics change'.

** For dbus-send **

dbus-send --system --dest=org.freedesktop.DBus --type=method_call --print-reply / org.freedesktop.DBus.NameHasOwner string:org.freedesktop.login1

method return sender=org.freedesktop.DBus -> dest=:1.97 reply_serial=2
   boolean true

** For libsystemd-bus (without this patch) **

‣ Type=method_call  Endian=l  Flags=0  Version=2 Serial=8
  Destination=org.freedesktop.DBus  Path=/org/freedesktop/DBus  Interface=org.freedesktop.DBus  Member=NameHasOwner
  MESSAGE "s" {
   STRING "org.freedesktop.login1";
  };

‣ Type=method_return  Endian=l  Flags=1  Version=2 Serial=51  ReplySerial=8
  Sender=:1.59  Destination=:1.67
  UniqueName=:1.59  WellKnownNames={org.freedesktop.DBus}
  MESSAGE "b" {
   BOOLEAN yes;
  };

For me true/false seems to be better readable than yes/no for BOOLEAN.

journal-file.c: remove redundant assignment of variable

we also do 'last_index = (uint64_t) -1;' at the end of the while
loop so there is no reason to also do it here.

bus: properly return an error code when release_name fails abnormally

bus-driverd: Fix return code in driver_request_name

RequestName return codes should be consistent with Dbus Specification.

VALUE - DESCRIPTION
1-The caller is now the primary owner of the name, replacing any previous owner,
2-The name already had an owner (QUEUE flag was not specified),
3-The name already has an owner (QUEUE flag was specified),
4-Application trying to request ownership of a name is already the owner of it.

journald: before closing /dev/kmsg let's unregister the event source

journald: close /dev/kmsg again if we cannot make use of it

journald: downgrade warning message when /dev/kmsg does not exist

dhcp: Properly handle unsuccessful cases for DHCP Ack

Ignore DHCP Ack packets if they are found to be erroneous or have
an unexpected xid by setting the result to zero instead of propagating
an error. Also remember to report a DHCP Nak and stop the DHCP client.

udev link-config: add asserts to silence scan-build

bus: removed unused variable in driverd

acked by Daniel Mack

bus: make driverd code more similar to other daemons, and make it exit on idle among other things

busctl: on kdbus the bus driver is a service like any other, hence don't skip it

bus: remarshal messages when necessary before sending

activate: fix error checking on epoll_ctl()

bus: install unit files for new driver service

bus: sync with kdbus

kdbus_cmd_{ep,ns,bus}_make are now consolidated, and bloom_size has
become an item of the dynamically sized list.

bus: sync up with kdbus

bus-driverd: Fix unique name return in driver_get_name_owner

man: networkd - clarify Address/Gateway keys in [Network] section

networkd: add support for Route sections

networkd: correct logging message

rtnl: fix for 32bits

Commit 0a0dc69b655cfb10cab39133f5d521e7b35ce3d5 broke tests for 32 bits

build-sys: fix distcheck

journal: fix against (theoretical) undefined behavior

While all the libc implementations I know return NULL when memchr's size
parameter is 0, without accessing any memory, passing NULL to memchr is
still invalid:

C11 7.24.1p2: Where an argument declared as "size_t n" specifies the length
of the array for a function, n can have the value zero on a call to that
function. Unless explicitly stated otherwise in the description of a
particular function in this subclause, pointer arguments on such a call
shall still have valid values, as described in 7.1.4. On such a call, a
function that locates a character finds no occurrence, a function that
compares two character sequences returns zero, and a function that copies
characters copies zero characters.

see http://llvm.org/bugs/show_bug.cgi?id=18247

man: describe journalctl --show-cursor

bus-driverd: actually add --version

systemctl: highlight filenames in cat

systemctl: fix return value from cat

man: reword awkward phrase

bus: use gperf to map error string to errno

bus: update kdbus.h

bus: when connecting to a container's system bus, double fork after joining the container's namespace

setns(CLONE_NEWPID) only applies to child processes, hence let's fork
once after joining the container's namespace.

build-sys: build bus-driverd if kdbus support is enabled

bus: when entering an existing namespace to connect to a container's system bus also switch over PID namespace

This is necessary to ensure that kdbus can collect creds of the
destination namespace when connecting.

core: always create /dev/kdbus/ns (and make it private 0700) after setting up the kdbus system bus

journal-file: protect against alloca(0)

bus: fix typo in systemd-bus-proxyd

Add bus-driverd

systemd-bus-driverd is a small daemon that connects to kdbus and
implements the org.freedesktop.DBus interface. IOW, it provides the bus
functions  traditionally taken care for by dbus-daemon.

Calls are proxied to kdbus, either via libsystemd-bus (were applicable)
or with the open-coded use of ioctl().

Note that the implementation is not yet finished as the functions to
add and remove matches and to start services by name are still missing.

bus: when a busname unit refuses to activate a service it should flush the queue

bus: also mask dbus.service in generator if kdbus is found

units: properly make bus proxy socket of type Accept=yes

systemctl: properly initialize and free sd_bus_error in "systemctl cat"

We need to properly initialize all error structs before use and free
them after use.

Also, there's no point in flushing stdout if we output a \n anyway...

bus: fix symlink to bus proxy socket

bus: use GREEDY_REALLOC() when allocating message queues

bus: let's use GREEDY_REALLOC() when allocating space for containers

execute: also set SO_SNDBUF when spawning a service with stdout/stderr connected to journald

loopback-setup: remove stray hunk

network: more asserts to shut up scan-build

kdbus.h: add alignment requirements

network: use SETLINK to bring up interfaces

rtnl: replace message_append by typesafe versions