.gitignore: add new tests and sort tests alphabetically

Add new function to filter fstab options

This fixes parsing of options in shared/generator.c. Existing code
had some issues:

- it would treate whitespace and semicolons as seperators. fstab(5)
  is pretty clear that only commas matter. And the syntax does
  not allow for spaces to be inserted in the field in fstab.
  Whitespace might be escaped, but then it should not seperate
  options. Treat whitespace and semicolons as any other character.
- it assumed that x-systemd.device-timeout would always be followed
  by "=". But this is not guaranteed, hasmntopt will return this
  option even if there's no value. Uninitialized memory could be read.
- some error paths would log, and inconsistently, some would just
  return an error code.

Filtering is split out to a separate function and tests are added.

Similar code paths in other places are adjusted to use the new function.

network: apply static addresses in specified order

https://bugs.freedesktop.org/show_bug.cgi?id=83270

shared/list: add LIST_APPEND

path-lookup: allow /run to override /etc in generator search

Generators are different than unit files: they are never automatically
generated, so there's no point in allowing /etc to override /run. On
the other hand, overriding /etc might be useful in some cases.

test-path-lookup: add simple test for path lookup functions

test-util: make sure that masking and overriding works

Implement masking and overriding of generators

Sometimes it is necessary to stop a generator from running. Either
because of a bug, or for testing, or some other reason. The only way
to do that would be to rename or chmod the generator binary, which is
inconvenient and does not survive upgrades. Allow masking and
overriding generators similarly to units and other configuration
files.

For the systemd instance, masking would be more common, rather than
overriding generators. For the user instances, it may also be useful
for users to have generators in $XDG_CONFIG_HOME to augment or
override system-wide generators.

Directories are searched according to the usual scheme (/usr/lib,
/usr/local/lib, /run, /etc), and files with the same name in higher
priority directories override files with the same name in lower
priority directories. Empty files and links to /dev/null mask a given
name.

https://bugs.freedesktop.org/show_bug.cgi?id=87230

Simplify execute_directory()

Remove the optional sepearate opening of the directory,
it would be just too complicated with the change to
multiple directories.

Move the middle of execute_directory() to a seperate
function to make it easier to grok.

bus-proxy: implement 'at_console'

The 'at_console' policy-category allows to apply policy-items to clients
depending on whether they're run from within a valid user-session or not.
We use sd_uid_get_seats() to check whether a user has a valid seat (which
excludes remote-sessions like ssh).

bus-proxy: print message direction in policy logs

Make sure to print "dbus-1 to kernel" or "kernel to dbus-1" in policy logs
to better diagnose the situation.

bus-proxy: fix receiver policy on dbus-1 to kdbus signals

If a dbus-1 client sends a broadcasted signal via the bus-proxy to kdbus,
the bus-proxy has no idea who the receiver is. Classic dbus-daemon has
bus-access and can perform policy checks for each receiver, but we cant.
Instead, we know the kernel will perform receiver policy checks for
broadcasts, so we can skip the policy check and just push it into the
kernel.

This fixes wpa_supplicant which has DENY rules on receive_type=signal for
non-root. As we never know the target, we always DENY all broadcasts from
wpa_supplicant.

Note that will still perform receiver-policy checks for signals that we
get from the kernel back to us. In those cases, we know the receiver
(which is us).

bus-proxy: fix swapped path/interface debug messages

The policy debug messages swapped "path=" and "interface=", fix this.

bus-proxy: fix policy for expected/non-expected reply tags

dbus-1 distinguishes expected and non-expected replies. An expected reply
is a reply that is sent as answer to a previously forwarded method-call
before the timeout fires. Those replies are, by default, forwarded and
DENY policy tags are ignored on them (unless explicitly stated otherwise).

We don't track reply-windows in the bus-proxy as the kernel already does
this. Furthermore, the kernel prohibits any non-expected replies (which
breaks dbus-1, but it was an odd feature, anyway).

Therefore, skip policy checks on replies and always let the kernel deal
with it!

To be correct, we should still process DENY tags marked as
send_expected_reply=true (which is *NOT* the default!). However, so far we
don't parse those attributes, and no-one really uses it, so lets not
implement it for now. It's marked as TODO if anyone feels like fixing it.

log: fix log_full_errno() with custom facilities

Make sure to extract the log-priority when comparing against
max-log-level, otherwise, we will always drop those messages.

This fixes bus-proxyd to properly send warnings on policy blocks.

build-sys: fix link-order and avoid "label" functions in libsystemd-shared

kdbus.h: update

push xorg information to the systemd --user instance

update TODO

loginctl: make session/user arguments optional for a number commands, and imply calling session/user instead

This turns "lock-session", "activate", "unlock-session",
"enable-linger", "disable-linger" into commands that take no argument,
optionally in which case the callers session/user is implied.

sd-bus: even if we need a cookie when sending a message there's no need to needlessly send it if we don't actually need it

bus-proxy: make sure we have creds when two legacy clients talk to each other

hwdb: add Kay's Laser Mouse to the dpi database

bus-proxy-test: show parsed system/session policy

bus-proxy: eat up "*" matches, they are pointless

bus-proxy: dbus-daemon implies that connections from UIDs that are identical to the bus owner should be allowed

Hence, copy this behaviour for bus-proxy too.

bus-proxyd: xml - consider empty tags as recv

bus-proxy: make sure sure eavesdrop= XML attributes are properly handled

logind: unify how we cast between uid_t and pointers for hashmap keys

logind: when a bus call is done on a session, user or seat, optionally determine them from the caller credentials

More specifically, if an operation is requested on a session with an
empty name, the caller's session is used. If an operation is requested
on a seat with an empty name, the seat of the caller's session is used.
Finally, if an operation on the user with UID -1 is requested, the user
of the client's session is used (and not the UID of the client!).

man: document that 'loginctl activate' only takes a single argument

logind: include "self" object links in dbus introspection

Makes "busctl introspect" a lot more fun.

loginctl: make "loginctl session-status" without session ID show the caller's session status

Similar for user-status and seat-status.

core: modernize execution code a bit

Among other things, avoid log_struct() unless we really need it.

Also, use "r" as variable to store function errors in, instead of "err".
"r" is pretty much what we use everywhere else, hence using the same
here make sense.

FInally, in the child, when we want to log, make sure to open the
logging framework first, since it is explicitly closed in preparation
for the exec().

journal: do not check for number of files

Now that we bump rlimit, we do not really know how many files
we can open. Remove the check.

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

systemctl: add missing output modes to help message

shell-completion: add missing output modes

https://bugs.freedesktop.org/show_bug.cgi?id=88216

bus: append bloom-filter to all signals

Make sure to append bloom-filters to all signal-messages, not only
broadcasts.

hwdb: add missing line between the G400 entries

Error, DATA expected but got 'mouse:usb:v046dpc24c:name:Logitech G400s Optical
Gaming Mouse:' in '/etc/udev/hwdb.d/70-mouse.hwdb':
Error, MATCH expected but got ' MOUSE_DPI=400@1000 *800@1000 2000@1000
4000@1000' in '/etc/udev/hwdb.d/70-mouse.hwdb':

Introduced in 6366e349

core: check both EPERM and EACCES for OOM adjustments

update TODO

core: make EPERM errors when applying OOM adjustment for forked processes non-fatal

This should be useful for user namespaces.

loginctl: port to generic verbs.h API

man: remove references to systemadm from systemctl man page

The tool is badly maintained and we shouldn't refence such old cruft.

machined: when cloning a raw disk image, also set the NOCOW flag

man: bring machinectl man page up-to-date

man: bring systemctl man page and --help text into the same order

man: add the same command sections to the man page as the --help text shows

systemctl,loginctl: start polkit agent for all polkit enabled operations

machinectl: given that machinectl invokes a number of polkit enabled methods, start the polkit agent on terminals

machinectl: show most recent log output in "machinectl status", too

loginctl: show the 10 most recent log user/session log lines in "loginctl user-status" and "loginctl session-status"

hwdb: note that we care about the maximum frequency in MOUSE_DPI

Devices with dynamic frequency scaling adjust the frequency as needed. For
those we only care about the maximum frequency, not the various in betweens.

https://bugs.freedesktop.org/show_bug.cgi?id=87435#c8

hwdb: adjust max frequency of Logitech RX 250

https://bugs.freedesktop.org/show_bug.cgi?id=87435

bash-completion: systemd-nspawn update

*Autocompletion for dirs, doesn't leave until you press space.
*Added tmpfs, volatile and network-macvlan options.

I tried with the SELinux options with seinfo(setools-console), but too
messy to get it right. Even Daniel Walsh haven't done it yet. :)

cryptsetup: support header= option

https://bugs.freedesktop.org/show_bug.cgi?id=66396

test-verbs: add unit tests for verbs minilib

doc: fix stale references to systemd-verify

This was subsumed into systemd-analyze back in 142c4ecaa98.

man: add hostnamed chassis type "embedded" to machine-info(5)

man machine-info lacks hostnamed chassis type "embedded" as introduced in 218. The following lines should fix this.

man: mention where timesyncd keeps the timestamp

bus-proxyd: fix EPERM on replies

Imagine a kdbus peer sending a method-call without EXPECT_REPLY set
through the proxy to a dbus1 peer. The proxy turns the missing
EXPECT_REPLY flag into a dbus1 NO_REPLY_EXPECTED flag. However, if the
receipient ignores that flag (valid dbus1 behavior) and sends a reply, the
proxy will try to forward it to the original peer. This will fail with
EPERM as the kernel didn't track the reply.

We have two options now: Either we ignore EPERM for reply messages, or we
track reply-windows in the proxy so we can properly ignore replies if
EXPECT_REPLY wasn't set.

This commit chose the first option: ignore EPERM for replies. The only
down-side is that replies without matching method call will no longer be
forwarded by the proxy. This works on dbus1, though.
Nobody sane does this, so lets ignore it.

bus-proxyd: optimize replies if they're not requested

If a caller does not request a reply, dont send it. This skips message
creation and speeds up NO_REPLY_EXPECTED cases. Note that sd-bus still
handles this case internally, but if we handle it in bus-proxyd, we can
skip the whole message creation step.

bus-proxy: augment credentials from /proc for cmdline update

dbus1 does not provide cmdline, so we have to augment our credentials from
/proc to beautify the bus-proxyd cmdline. We dont use this for anything
but beautification, so there shouldn't be any problems due to /proc
pid-recycling races.

This fixes bus-proxyd to no longer display 'xxxxxxxxxxxxxxxxxxxxxxxxxxx'
in its cmdline.

nspawn: fix error message when mknod fails

path-util: plug leak

dbus1-generator: fix array sentinal

Add a {} sentinal entry so the config-iterator can properly iterate all
array elements. Fixes a segfault in the dbus1-generator.

build: print conditions of ldconfig, hibernate configs

sd-bus: fix copy-paste error

update TODO

journal: bump RLIMIT_NOFILE when journal files to 16K (if possible)

When there are a lot of split out journal files, we might run out of fds
quicker then we want. Hence: bump RLIMIT_NOFILE to 16K if possible.

Do these even for journalctl. On Fedora the soft RLIMIT_NOFILE is at 1K,
the hard at 4K by default for normal user processes, this code hence
bumps this up for users to 4K.

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

udevadm: don't hit an assert when obsolete parameters are passed

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

update TODO

util: make it easy to initialize the crtime from the current time in fd_setcrtime()

import: also set NOCOW for gpt disk images

Given the write patterns on disk images, we better should turn COW off
for them. In particular as the file systems used inside the disk images
should do their own data integrity checks anyway and we don't need
multiple layers of it.

journald: turn off COW for journal files on btrfs

btrfs' COW logic results in heavily fragment journal files, which is
detrimental for perfomance. Hence, turn off COW for journal files as we
create them.

Turning off COW comes at the cost of data integrity guarantees, but this
should be acceptable, given that we do our own checksumming, and
generally have a pretty conservative write pattern.

Also see discussion on linux-btrfs:

http://www.spinics.net/lists/linux-btrfs/msg41001.html

util: upgrade default $TERM from vt102 to vt220 if we have no idea about the connected terminal

So far, if we had no knowledge about the correct $TERM we defaulted to
v102, as a safe, conservative choice. However, the terminfo data for
vt102 is not aware of pageup/pagedown, which makes "less" much harder
work with than necessary. Setting vt220 allows them to work correctly.

"vt220" should be a sufficiently safe choice too, given that xterm,
gnome-terminal and the linux console all strive to implement vt220 as
baseline, already to pass pageup/pagedown correctly to apps.

Effectively, with this change "journalctl -e" run inside a
"systemd-nspawn" terminal will now run a pager where pageup/pagedown
works, which is quite an improvement of usability for containers.

bus-util: fix error number output when waiting for job

update TODO

units: make resolved pull in its own .busname unit, but only on kdbus systems

The daemon requires the busname unit to operate, since it contains the
policy that allows it to acquire its service name.

service: automatically create After= dependency from services to their .busname units, if BusName= is set

conf-parse: don't accept invalid bus names as BusName= arguments in service units

conf-parse: make syntax logging functions behave more like other log functons

In particular, don't patch the error number to EINVAL if 0, and don't
negate it.

(Also, add do {} while (false) around multi-line macro)

units: improve Description= for systemd's own busname unit

hwdb: mouse DPI data, Logitech Gaming Mouse G400

busctl: exit cleanly when the bus connection is severed

sd-bus: when we synthesize messages, initialize timestamps ourselves

sd-bus: unify how we set the sender of synthetic messages

sd-bus: make use of the newly added timestamps on kdbus kernel messages

sd-bus: always catch name requests for the special names "org.freedesktop.DBus" and "org.freedesktop.DBus.Local" and refuse them

update TODO

machinectl: prettify "machinectl list" output

sysv-generator: initialize units before use to ensure correct ordering

The original loop called fix_order() on each service immediately after
loading it, but fix_order() would reference other units which were not
loaded yet.
This resulted in bogus and unnecessary orderings based on the static
start priorities.

Therefore call load_sysv() for every init script when traversing them in
enumerate_sysv(). This ensures that all units are loaded when
fix_order() is called.

Bug-Debian: https://bugs.debian.org/771118

update TODO

man: document nss-mymachines

man: add an alias to libnss_myhostname.so2 for nss-myhostname

The module appears under the name "libnss_myhostname.so.2" in the file
system, hence let's link it up under that name.

man: rework nss-myhostname man page

Remove a number of incorrect links, and explain that "localhost" and
"gateway" are now resolved by it, too.

Update TODO

ptyfwd: simplify how we handle vhangups a bit

btrfs-util: rework how we iterate through the results of the TREE_SEARCH results

Let's introduce some syntactic sugar with iteration macros, and add
correct key increment calls.

sysv-generator: handle Provides: for non-virtual facility names

The list of provided facility names as specified via Provides: in the
LSB header was originally implemented by adding those facilities to the
Names= property via unit_add_name().

In commit 95ed3294c632f5606327149f10cef1eb34422862 the internal SysV
support was replaced by a generator and support for parsing the Names=
option had been removed from the unit file parsing in v186.
As a result, Provides: for non-virtual facility was dropped when
introducing the sysv-generator.

Since quite a few SysV init scripts still use that functionality (at
least in distros like Debian which have a large body of SysV init
scripts), add back support by making those facility names available via
symlinks to the unit filename to ensure correct orderings between
SysV init scripts which use those facility names.

Bug-Debian: https://bugs.debian.org/774335