job: add waiting jobs to run queue in unit_coldplug

When we have job installed and added to run queue for service which is
still in dead state and systemd initiates reload then after reload we
never add deserialized job to the run queue again. This is caused by
check in service_coldplug() where we check if deserialized state is
something else than dead state, which is not the case thus we never call
service_set_state() and finally unit_notify() where we would have added
job to the run queue.

Thanks to Michal Sekletar <msekleta@redhat.com> for the original patch.

build-sys: add configure switch for -fsanitize=undefined

--enable-undefined-sanitizer mirrors --enable-memory-sanitizer.

test-journal-flush: avoid predictable names in /var/tmp

man: networkd typo fixes

networkd-wait-online: refactor a bit

Clarify that we are waiting for any link to reach 'carrier' state, regardless
of who manages it. This will be useful when we add support for waiting for more
operational states.

networkd-wait-online: drop config file and add commandline options instead

core: reindent {selinux, ima, smack}-setup.c

7-space indentation is just too weird to leave alone.
Make it 8 spaces, as per CODING_STYLE. No other changes.

update TODO

core: expose CFS CPU time quota as high-level unit properties

core: make sure we always write changed cgroup attributes to the cgroupfs

update TODO

bootchart: print to stdout and display default values in help

bootchart: rewrite usage message more generally

bootchart: add control group option

udev: warn when name_to_handle_at is not implemented

We have a bunch of reports from people who have a custom kernel and
are confused why udev is not running. Issue a warning on
error. Barring an error in the code, the only error that is possible
is ENOSYS.

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

service: rename StartLimitAction enum to FailureAction

It's used for the FailureAction property as well.

service: add FailureAction= option

It has the same possible values as StartLimitAction= and is executed
immediately if a service fails.

udev: increase the size of RESULT buffer

Under some conditions, in udev_rules_apply_to_event the fact that
result is 1024 bytes, creates problems if the output of the running
command/app is bigger then 1024 bytes.

bus: update kdbus.h

test-resolve.c: Cleanup and make use of sockaddr_pretty()

Fix keysize handling in cryptsetup (bits vs. bytes)

The command line key-size is in bits but the libcryptsetup API expects bytes.

Note that the modulo 8 check is in the original cryptsetup binary as well, so
it's no new limitation.

(v2: changed the point at which the /= 8 is performed, rebased, removed tabs)

Add more password agent information

Add an (optional) "Id" key in the password agent .ask files. The Id is
supposed to be a simple string in "<subsystem>:<target>" form which
is used to provide more information on what the requested passphrase
is to be used for (which e.g. allows an agent to only react to cryptsetup
requests).

(v2: rebased, fixed indentation, escape name, use strappenda)

errno: make sure to handle the 3 errnos that are aliases for others properly

util: make sure all our name_to_handle_at() code makes use of file_handle_union

networkd-wait-online: improve interoptability and enable by default

To make sure we don't delay boot on systems where (some) network links are managed by someone else
we don't block if something else has successfully brought up a link.

We will still block until all links we are aware of that are  managed by networkd have been
configured, but if no such links exist, and someone else have configured a link sufficiently
that it has a carrier, it may be that the link is ready so we should no longer block.

Note that in all likelyhood the link is not ready (no addresses/routes configured),
so whatever network managment daemon configured it should provide a similar wait-online
service to block network-online.target until it is ready.

The aim is to block as long as we know networking is not fully configured, but no longer. This
will allow systemd-networkd-wait-online.service to be enabled on any system, even if we don't
know whether networkd is the main/only network manager.

Even in the case networking is fully configured by networkd, the default behavior may not be
sufficient: if two links need to be configured, but the first is fully configured before the
second one appears we will assume the network is up. To work around that, we allow specifying
specific devices to wait for before considering the network up.

This unit is enabled by default, just like systemd-networkd, but will only be pulled in if
anyone pulls in network-online.target.

hostnamectl: read kernel name and release from remote

hostnamed: expose KernelName and KernelRelease on the bus

update TODO

analyze: fix plot with bad y size

systemd-analyze plot > test.svg produces output with all y and height
element attributes equal to zero. This of course causes the resulting
svg to appear blank (zero height). Bug does not affect x86. Looks like
a compiler optimization may be the culprit.

https://github.com/archlinuxarm/PKGBUILDs/issues/815

journal-gatewayd: bugfix for invalid level case

introduced in cafc7f91306ea17ace4a6c3d76d81c8780c87452

man: recommend that XDG_SESSION_DESKTOP and XDG_CURRENT_DESKTOP use the same identifiers

use new name DRAW_TREE_VERTICAL

Was renamed in 6b01f1d3911bd7c7eadbb8a3b4375bd3ac05c98f

delta: draw arrows with draw_special_char()

Let's unify generation of unicode chars at one place.

Also, don't add an extra space into chars we print, except for the tree
chars where this is really necessary.

install: simplification

label: there is no label_retest_selinux() call

bus: simplification

backlight: warn if kernel exposes backlight device with bogus max_brightness

We shouldn't silently tape over broken kernel drivers.

update TODO

login: rename session_mute_vt() to session_prepare_vt()

This function is no longer just about muteing the VT. We do all kinds of
VT setup for sessions using the controller-API. Rename the function to
something more appropriate.

login: give session ownership of VT

The tty associated with a VT should be owned by the owner of the session
running on the VT. This is important for supporting a socket activated X
server, since the X server will open the tty itself.

This commit makes sure to chown the tty any time a session is
created and and chown it back to root any time the session
is removed. This behavior is copied from /bin/login.

bus: remove "on behalf" kdbus logic (ABI break)

networkd: link - add missing curly brackets

introduced in 1e9be60bbabe179f5233217384f1daec757c17c7

networkd: link - reduce default verbosity a bit

networkd: link - support IFLA_OPERSTATE

This properly detects the state of the link based on both the link flags and the
operstate.

Moreover, always log state-changes even if we are not yet managing the link.

networkd: link - don't react to successfull link up

Avoid having two code-paths racing with eacother to do the same thing. The change
of flags will be detected in the normal way, so only use the link_up_handler
to detect if the 'up' failed and in that case fail the link.

networkd: link - log when starting clients fail

networkd: link - avoid cycle when failing

networkd: link - stop clients when failing

remove bus-driverd, the interface is now handled natively by bus-proxyd

bus-proxyd: handle org.freedesktob.DBus calls natively

Revert "networkd: netdev - set predictable mac address when creating netdev"

This reverts commit cdc85c875b842b9309f72caefc51c262f521cf92.

There appears to be a kernel bug that (among other things) cause bridges
not to get a bridge id set when supplying IFLA_ADDRESS when creating the
netdev.

Simply revert the whole thing until we sort this out in the kernel.

See: http://www.spinics.net/lists/netdev/msg279807.html

Reported-by: C. R. Oldham <cr@saltstack.com>

service: add support for reboot argument when triggered by StartLimitAction=

When rebooting with systemctl, an optional argument can be passed to the
reboot system call. This makes it possible the specify the argument in a
service file and use it when the service triggers a restart.
This is useful to distinguish between manual reboots and reboots caused by
failing services.

systemctl: delete REBOOT_PARAM_FILE if no parameter is specified

And move it to sperate function.

implement a union to pad out file_handle

Cases where name_to_handle_at is used allocated the full struct to be
MAX_HANDLE_SZ, and assigned this size to handle_bytes. This is wrong
since handle_bytes should describe the length of the flexible array
member and not the whole struct.

Define a union type which includes sufficient padding to allow
assignment of MAX_HANDLE_SZ to be correct.

condense assignment and path_kill_slashes calls

doc: fix items' names in PORTING-DBUS1

Remove duplicate includes

silence warnings

networkd: link - add explicit unmanaged state

networkd: link - set mac addresses when starting clients

MAC addreses are also set when NEWLINK messages are receieved, but only if the clients have
already been initialized.

sd-dhcp-client: log ifindex and mac address when starting

networkd: link - log changes in some common link flags

networkd: link - use IFF_DORMANT instead of IFF_RUNNING

Should be no functional change, but avoids spurious log
messages when in operstate UNKNOWN and !IFF_LOWER_UP.

networkd: link - make debug logging a bit more readable

Drop leading zeros which will never be used when printing the link flags.

udev: link-config - treat any non-random mac address as persistent

The address may be already changed by other processes, or set
when creating the netdev from userspace.

sd-rtnl: log when queues are exhausted

cryptsetup: copy value, not key for (rd.)luks.key

Handle Unix domain socket connections from outside our namespace v2

This is a second attempt at 9754d56, reverted in 2f20a8e, because
I lost a 'break;' when moving chunks around.

networkd: order after udev kernel socket

Otherwise we will not be able to queuery whether devices are initialized on kdbus enabled systems.

networkd: manager - add a bit of debug output to udev ADD events

networkd: link - track interface renames

sd-rtnl: socet_read_message - make sure we only shrink the partial rqueue when processing multi-part message

network: implement masking of .link, .network and .netdev files

networkd: netdev - give up early when setting already set ifindex

We may receive RTM_NEWLINK messages with missing LINKINFO after the initial NEWLINK message,
don't bother verifying these, just drop out early after checking that the ifindex is not in conflict.

networkd: netdev - verify name of newlink messages

networkd: netdev - set predictable mac address when creating netdev

networkd: tie links to rtnl rather than udev

This essentially swaps the roles of rtnl and udev in networkd. After this
change libudev is only used for waiting for udev to initialize devices and
to get udev-specific information needed for some [Match] attributes.

This in particular simplifies the code in containers where udev is not really
useful, but also simplifies things and reduces round-trips in the non-container
case.

nss-myhostname: port to sd-rtnl

sd-rtnl: message - expose DUMP flag in the api

sd-rtnl: add multi-part message test

sd-rtnl: message - concatenate multi-part messages from different packets

sd-rtnl: message - don't put NULL message on rqueue

If nothing interesting was receieved we should not put anything on
the queue.

sd-rtnl: message - make room for all pending messages, not just the first

Also, don't actually read any of the message when peeking, just get its length.

Revert "Handle Unix domain socket connections from outside our namespace"

This reverts commit 9754d56e9b21bfe89fc18f47987d6bef491b8521.

It causes a crash in PID1:
  Apr 19 13:49:32 lon systemd[1]: Code should not be reached 'Unhandled socket type.'
                                  at src/core/socket.c:684, function instance_from_socket(). Aborting.
  Apr 19 13:49:32 lon systemd[1]: Caught <ABRT>, dumped core as pid 336.
  Apr 19 13:49:32 lon systemd[1]: Freezing execution.

tmpfiles: fix permissions on new journal files

When starting up journald on a new system, set the proper permissions on
the system.journal files, not only on the journal directory.

build-sys: do not try to build test-rtnl-manual w/o kmod

build-sys: add libkmod flags for test_rtnl_manual

core: minor typo fix

delta: do not use unicode chars in C locale

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

install: create_symlink() check unlink() return value

create_symlink() do not check the return value of unlink(), this may
confuse the user.

Before the unlink() call we check the 'force' argument. If it is not set
we fail with -EEXIST, otherwise we unlink() the file, therefore the next
symlink() should not fail with -EEXIST (do not count races...).

However since callers may not have appropriate privileges to unlink()
the file we lose the -EPERM or any other errno code of unlink(), and
return the -EEXIST of the next symlink(). Fix this by checking unlink()
results.

Before:
$ systemctl --force --root=~/container-03 set-default multi-user.target
Failed to set default target: File exists

After:
$ systemctl --force --root=~/container-03 set-default multi-user.target
Failed to set default target: Permission denied

systemctl: allow to change the default target without the --force switch

Currently "systemctl set-default" will fail to change the default target
due to the 'default.target' being a symlink which is always the case.

To work around this, the user must specify the "--force" switch to be
able to overwrite the existing symlink.

This is clearly a regression that was introduced by commit  718db96199e
since it worked before without the "--force" switch and the man pages do
not mention that you need to specify it. It is expected that this is a
symlink.

So just explicity set the force flag to make it work again.

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

Reported-by: <code@progandy.de>

Handle Unix domain socket connections from outside our namespace

NixOS uses Unix domain sockets for certain host <-> container
interaction; i.e. the host connects to a socket visible in the
container's directory tree, where the container uses a .socket unit to
spawn the handler program on demand. This worked in systemd 203, but
in 212 fails with "foo.socket failed to queue service startup job
(Maybe the service file is missing or not a template unit?): No data
available".

The reason is that getpeercred() now returns ENODATA if it can't get
the PID of the client, which happens in this case because the client
is not in the same PID namespace. Since getpeercred() is only used to
generate the instance name, this patch simply handles ENODATA by
creating an instance name "<nr>-unknown".

[zj: reorder clauses and remove (unsigned long) casts.]

man: document relationship between RequiresMountsFor and noauto

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

test-path-util: fix running with separate build dir

test-path-utils attempts to find itself, but if the binary is not
in current directory, the test would fail.

sysctl: replaces some slashes with dots

It turns out that plain sysctl understands a.b/c syntax to write to
/proc/sys/a/b.c. Support this for compatibility.

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

systemctl: add --recursive for list-timers

systemctl: add --recursive for list-sockets

bus-proxy: remove unused variable

cgls: avoid writing an unused value

silences a build-scan warning for the unused value in r

timedatectl: avoid using uninitialized var

sec is not set if have_time is false so avoid using it. have_time
was introduced in 9ff09bcb86fb125768667aca9bc0b10b1745370a but only
the first uses for sec were covered

Found with scan-build