libsystemd-network: add support for "Client FQDN" DHCP option

This adds support for the Client Fully Qualified Domain Name (FQDN)
option [RFC 4702] to libsystemd-network. The option can be used to
exchange information about a DHCPv4 client's fully qualified domain
name and about responsibility for updating the DNS RR related to the
client's address assignment.

Other popular DHCP clients (dhclient, dhcpcd) support this option and
it would be useful to have it in networkd too.

dns-domain: add dns_name_to_wire_format()

The function converts a domain name string to the wire format
described in RFC 1035 Section 3.1.

Merge pull request #1909 from keszybz/filetriggers-v2

Move daemon-reload from package %post scripts to file triggers

Merge pull request #1925 from evverx/fix-pam-systemd-user

tests: copy /etc/pam.d/systemd-user from the host on Debian, Ubuntu

Merge pull request #1926 from phomes/include-order-libudev

tree-wide: group include of libudev.h with sd-*

Merge pull request #1927 from evverx/no-entries-in-quiet-mode

journalctl: don't print -- No entries -- in quiet mode

Merge pull request #1924 from poettering/some-fixes-2

casting fixes, another siphash24 alignment fix, and more

journalctl: don't print -- No entries -- in quiet mode

tree-wide: group include of libudev.h with sd-*

Some additions to NEWS

tests: copy /etc/pam.d/systemd-user from the host on Debian, Ubuntu

Fixes:
systemd-testsuite systemd[34]: PAM _pam_load_conf_file: unable to open /etc/pam.d/system-auth
systemd-testsuite systemd[34]: PAM _pam_load_conf_file: unable to open /etc/pam.d/system-auth
systemd-testsuite systemd[34]: user@0.service: Failed at step PAM spawning /lib/systemd/systemd: Operation not permitted
...
on Debian, Ubuntu

siphash: minor coding style fixes and modernizations

Only cosmetics really, doesn't change any actual logic.

siphash: fix another alignment issue

tree-wide: make macros for converting fds to pointers and back generic and use them everywhere

remount-fs: modernize coding style a bit

a) Use _cleanup_ where it makes sense

b) Uniformly use negative errno-style errors internally, convert to
   EXIT_FAILURE/EXIT_SUCCESS only when actually exiting.

c) Use log_oom() where appropriate

d) Fix minor memory leak in hashmap addition error path.

e) Don't pretend we could continue sensibly on OOM or fork() failure

f) Use PR_SET_PDEATHSIG to make sure clients we don't kill on error are
   cleaned up.

g) Make use of STRV_MAKE() where it's pretty to do so.

h) Simplify error paths.

sd-ipv4ll: fix error path if sd-ipv4acd allocation fails

Let's make sure the destructor cannot hit the n_ref == 0 case.

journald: trivial simplification

coredump: modernize error logging a bit

core: don't generate warnings when write access to the cgroup fs fails in --user due to EACCES

After all, in the classic hierarchy that's pretty much the default case.

tree-wide: use right cast macros for UIDs, GIDs and PIDs

Merge pull request #1923 from zonque/siphash

siphash24: let siphash24_finalize() and siphash24() return the result…

Merge pull request #1922 from phomes/sort-includes

Sort includes

siphash24: let siphash24_finalize() and siphash24() return the result directly

Rather than passing a pointer to return the result, return it directly
from the function calls.

Also, return the result in native endianess, and let the callers care
about the conversion. For hash tables and bloom filters, we don't care,
but in order to keep MAC addresses and DHCP client IDs stable, we
explicitly convert to LE.

tree-wide: sort includes

Sort the includes accoding to the new coding style.

tree-wide: add missing includes

Add a few includes that we rely on to be include already.

Merge pull request #1919 from martinpitt/master

siphash42: add tests with unaligned input pointers

Merge pull request #1918 from dvdhrm/user3

login: allow re-using users (v3)

siphash42: add tests with unaligned input pointers

Add test case for calling siphash24 with unaligned input pointers, as we
commonly get with calling it on the result on basename() or similar.

This provides a test for PR #1916, rescued from the superseded PR #1911.

Thanks to Steve Langasek for the test!

login: ignore JobRemoved of old jobs

If we requeue jobs, we are no longer interested in old jobs. Hence, we
better ignore any JobRemoved signals for old jobs and concentrate on our
replacements.

Merge pull request #1916 from zonque/align

siphash: alignment

Merge pull request #1915 from poettering/btrfs-root-subvol

tmpfiles: create subvolumes for "v", "q", and "Q" only if / is a subv…

login: make sure to replace existing units

When queuing unit jobs, we should rather replace existing units than
fail. This is especially important when we queued a user-shutdown and a
new login is encountered. In this case, we better raplce the shutdown
jobs. systemd takes care of everything else.

login: fix re-use of users

If the last reference to a user is released, we queue stop-jobs for the
user-service and slice. Only once those are finished, we drop the
user-object. However, if a new session is opened before the user object is
fully dropped, we currently incorrectly re-use the object. This has the
effect, that we get stale sessions without a valid "systemd --user"
instance.

Fix this by properly allowing user_start() to be called, even if
user->stopping is true.

login: group static fields in "struct User"

Make sure to put static fields together in "struct User". This makes it
easier to figure out the lifetime of each field.

login: make user->service static

Just like user->slice, there is no reason to store the unit name in /run,
nor should we allocate it dynamically on job instantiation/removal. Just
keep it statically around at all times and rely on user->started ||
user->stopping to figure out whether the unit exists or not.

login: make user_new() and user_free() follow coding-style

Few changes to user_new() and user_free():
 - Use _cleanup_(user_freep) in constructor
 - return 'int' from user_new()
 - make user_free() deal with partially initialized objects
 - keep reverse-order in user_free() compared to user_new()
 - make user_free() return NULL
 - make user_free() accept NULL as no-op

login: keep user->slice constant

Currently, we allocate user->slice when starting a slice, but we never
release it. This is incompatible if we want to re-use a user object once
it was stopped. Hence, make sure user->slice is allocated statically on
the user object and use "u->started || u->stopping" as an indication
whether the slice is actually available on pid1 or not.

login: simply XDG_RUNTIME_DIR management

Lets not pretend we support changing XDG_RUNTIME_DIR via logind state
files. There is no reason to ever write the string into /run, as we
allocate it statically based on the UID, anyway. Lets stop that and just
allocate the runtime_path in "struct User" at all times.

We keep writing it into the /run state to make sure pam_systemd of
previous installs can still read it. However, pam_systemd is now fixed to
allocate it statically as well, so we can safely remove that some time in
the future.

Last but not least: If software depends on systemd, they're more than free
to assume /run/user/$uid is their runtime dir. Lets not require sane
applications to query the environment to get their runtime dir. As long as
applications know their login-UID, they should be safe to deduce the
runtime dir.

tmpfiles: create subvolumes for "v", "q", and "Q" only if / is a subvolume

It's not a good idea to create subvolumes for parts of the OS tree (such
as /home, or /var) if the root directory is not a subvolume too. We
shouldn't assume control of "heavier" objects such as subvolumes, if the
originating object (the root directory) is a "light-weight" object, i.e.
a plain directory.

Effectively this means that chroot() environments that are run on a
plain directory do not have to deal with problems around systemd
creating subvolumes that cannot be removed with a simple "rm" anymore.
However, if the chroot manager creates a proper subvolume for such an
environment it will also get further subvolumes placed in there, under
the assumption that the manager understands the concept of subvolumes in
that case.

siphash24: change result argument to uint64_t

Change the "out" parameter from uint8_t[8] to uint64_t. On architectures which
enforce pointer alignment this fixes crashes when we previously cast an
unaligned array to uint64_t*, and on others this should at least improve
performance as the compiler now aligns these properly.

This also simplifies the code in most cases by getting rid of typecasts. The
only place which we can't change is struct duid's en.id, as that is _packed_
and public API, so we can't enforce alignment of the "id" field and have to
use memcpy instead.

siphash24: fix memory alignment

Use unaligned_read_le64() to access input buffer when reading complete
64-bit words.

This should fix memory traps on platforms with strict aliasing.

basic: add unaligned macros for little endian

Also add test code for that.

Merge pull request #1886 from poettering/tasks-max

Enable TasksMax by default for all units

TODO: sort

Merge pull request #1907 from evverx/fix-pam-install-on-debian

tests: fix pam_modules installation on Debian (and Debian-like systems)

virt: sort #includes alphabetically

nitpick from PR #1910

Merge pull request #1908 from phomes/network-memleak

network: dhcp6 - fix mem leak

Merge pull request #1906 from poettering/pid-check

machined,logind: be more careful when accepting PIDs and UIDs from clients

nspawn: set TasksMax= for containers to 8192 by default

logind: change default to UserTasksMax= to 4096

core: enable TasksMax= for all services by default, and set it to 512

Also, enable TasksAccounting= for all services by default, too.

See:

http://lists.freedesktop.org/archives/systemd-devel/2015-November/035006.html

Merge pull request #1910 from martinpitt/master

virt: add missing #includes

virt: add missing #includes

src/basic/virt.c: In function 'detect_vm_device_tree':
src/basic/virt.c:117:17: error: unknown type name '_cleanup_closedir_'
                 _cleanup_closedir_ DIR *dir = NULL;

src/basic/virt.c:128:17: error: implicit declaration of function 'FOREACH_DIRENT' [-Werror=implicit-function-declaration]
                 FOREACH_DIRENT(dent, dir, return -errno)

tests: intsall /etc/login.defs

Fixes:
systemd-testsuite login[31]: cannot open login definitions /etc/login.defs [No such file or directory]
systemd-testsuite systemd[1]: Received SIGCHLD from PID 31 (login).
systemd-testsuite systemd[1]: Child 31 (login) died (code=exited, status=1/FAILURE)
systemd-testsuite systemd[1]: console-getty.service: Child 31 belongs to console-getty.service
systemd-testsuite systemd[1]: console-getty.service: Main process exited, code=exited, status=1/FAILURE
systemd-testsuite systemd[1]: console-getty.service: Changed running -> dead

on Debian/Ubuntu

Useful on other distros

Move daemon-reload from package %post scripts to file triggers

This uses new functionality added in rpm 4.13. Instead of doing
one daemon-reload per packages, we do just one or two
(When both installing and uninstalling packages, we do
two. Unfortunately this also includes the common case of upgrades.
When only installing or when only installing, we do just one.)

New file triggers.systemd can be built, but the contents have
to be copied into the rpm spec file by hand. Using %{load} does
not seem to work. It can serve as documentation.

network: dhcp6 - fix mem leak

from 7a695d8e
CID#1338679

tests: fix pam_modules installation on Debian (and Debian-like systems)

libpam_modules installs modules into /lib/$(dpkg-architecture -qDEB_HOST_MULTIARCH)/security
on Debian

Fixes:
systemd-testsuite login[36]: PAM unable to dlopen(pam_group.so): /lib/security/pam_group.so: cannot open shared object file: No such file or directory
systemd-testsuite login[36]: PAM adding faulty module: pam_group.so
systemd-testsuite login[36]: PAM unable to dlopen(pam_limits.so): /lib/security/pam_limits.so: cannot open shared object file: No such file or directory
systemd-testsuite login[36]: PAM adding faulty module: pam_limits.so
...
etc

machined,logind: be more careful when accepting PIDs and UIDs from clients

Always validate first before we start processing the data.

Merge pull request #1903 from phomes/gitignore

gitignore: add test-install-root

gitignore: add test-install-root

Merge pull request #1901 from evverx/various-fixes

Various fixes

Merge pull request #1899 from phomes/NEWS-typo

NEWS: typo fixes

tests: install execs from user units too

Fixes:
systemd[39]: systemd-exit.service: Executing: /bin/kill -s 58 29
systemd[39]: systemd-exit.service: Failed at step EXEC spawning /bin/kill: No such file or directory
systemd[29]: Received SIGCHLD from PID 39 ((kill)).
systemd[29]: Child 39 ((kill)) died (code=exited, status=203/EXEC)

firstboot: log on take_etc_passwd_lock error too

Merge pull request #1898 from marcosfrm/journalctl-shell-completion-2

shell-completion: journalctl: add --sync

NEWS: typo fixes

shell-completion: journalctl: add --sync

From 94b6551

Merge pull request #1896 from phomes/install-wrong-enum

install: don't mix state and type enums

install: don't mix state and type enums

from 0ec0deaa

hwdb: update

core: now that .snapshot unit are gone, we don't need the per-type .no_gc bool anymore

core: simplify scope unit GC checking code a bit

logind: add a new UserTasksMax= setting to logind.conf

This new setting configures the TasksMax= field for the slice objects we
create for each user.

This alters logind to create the slice unit as transient unit explicitly
instead of relying on implicit generation of slice units by simply
starting them. This also enables us to set a friendly description for
slice units that way.

logind: don't assert if the slice is missing

After all, we don't actually really need the slice to work, it's just
nice to have it.

core: enable transient unit support for slice units

man: document our definition of a year and a month

Let's be accurate here, as this might be surprising to people.

util-lib: when parsing time values, accept "M" as short for "month"

nginx defines an uppercase "M" that way (in contrast to the lowercase
"m" for "minute"), and it sounds like an OK logic to follow, so that we
understand a true superset of time values nginx understands.

http://nginx.org/en/docs/syntax.html

btrfs: properly handle the case when a subvol has no parent

Don't be confused by subvols without parent. This is after all how the
root subvol is set up.

core: add new DefaultTasksMax= setting for system.conf

This allows initializing the TasksMax= setting of all units by default
to some fixed value, instead of leaving it at infinity as before.

core: make sure DefaultLimitCPU= and DefaultLimitRTTIME= understand time units, too

We added this for the per-unit setting, hence let's enable this for the
global default settings too.

Merge pull request #1883 from evverx/man-systemd-service-sd-watchdog-enabled

man: add note about sd_watchdog_enabled

man: add note about sd_watchdog_enabled

Merge pull request #1880 from fsateler/sysctl-doc

man: document systemd-sysctl arguments

Merge pull request #1878 from hbrueckner/for-upstream

udev/path_id: improve and enhance bus detection for Linux on z Systems

man: document systemd-sysctl arguments

Merge pull request #1881 from michich/analyze-dot

analyze: dot graph missing Requisite, superfluous ConflictedBy

Merge pull request #1882 from teg/networkd-dhcp6-fix

networkd: link - fix ipv6ll_gained() handling

networkd: check explicit state rather than link->network

When deserializing we can now have an attached network without the various clients yet
having been configured. Hence, don't misused the link->network as a check to determine
if a link is ready to be used, but check the state explicitly.

analyze: do not draw ConflictedBy dependencies in dot graph

We already draw Conflicts. I see no reason for having every red line in
the graph duplicated in the opposite direction.

NEWS: document two more changes in behaviour

networkd: dhcp6 - don't log about losing a lease we never had

analyze: show Requisite dependencies in the dot graph again

We lost them a long time ago with commit 048ecf5b843.

Merge pull request #1879 from poettering/networkd-forward

stop managing per-interface IP forwarding settings

Merge pull request #1869 from poettering/kill-overridable

Remove support for RequiresOverridable= and RequisiteOverridable=

NEWS: add more stuff, and reorder things a bit

catalog: move danish catalog into the right place, and drop DOS line breaks

Merge pull request #1875 from marineam/fsck-after-resume

generator: order initrd fsck-root after local-fs-pre

networkd: stop managing per-interface IP forwarding settings

As it turns out the kernel does not support per-interface IPv6 packet
forwarding controls (unlike as it does for IPv4), but only supports a
global option (#1597). Also, the current per-interface management of the
setting isn't really useful, as you want it to propagate to at least one
more interface than the one you configure it on. This created much grief
(#1411, #1808).

Hence, let's roll this logic back and simplify this again, so that we
can expose the same behaviour on IPv4 and IPv6 and things start to work
automatically again for most folks: if a network with this setting set
is set up we propagate the setting into the global setting, but this is
strictly one-way: we never reset it again, and we do nothing for network
interfaces where this setting is not enabled.

Fixes: #1808, #1597.

networkd: rearrange checks when to write something into sysctl a bit

Move check whether ipv6 is available into link_ipv6_privacy_extensions()
to keep it as internal and early as possible.

Always check if there's a network attached to a link before we apply
sysctls. We do this for most of the sysctl functions already, with this
change we do it for all.

networkd: fix a couple of format string types

We really should use %i for ints, and %u for unsigneds, and be careful
what we pick depending on the type we want to print.