networkd: manager - avoid unnecessary memory allocation

Don't allocate Address objects only to free them again when processing
rtnl events.

networkd: address - store active addresses in a Set rather than a List

We need to be able to look these things up quickly as we will be updating them
continuously and there can in principle be many of them.

networkd: address - process in manager.c rather than link.c

networkd: link - refactor (un)ref

networkd: link - fix error handling in link_add()

sd-netlink: add support for subscribing to ROUTE messages

Also, explicitly don't support subscribing to GET or SET messages, as these will
never be emitted by the kernel.

sd-netlink: rtnl - add route_get_{scope,tos,table,protocol}()

sd-ipv4ll: allow initial address to be set explicitly

This is useful in case the daemon is restarted and the state of the IPv4LL client should
be serialized/deserialized.

networkd: link/network - correctly align log messages

libsystemd-network: limit accuracy of timestamps in logging

We don't care about timestamps down to the last usec, round to the closest sec
as that will be plenty for debugging purposes.

networkd: address - add hash helpers for Address objects

Add compare_func and hash_func for the Address object. The notion of
address equality is the same as in the kernel, and hashing preserves
preserves equality.

Two addresses are considered equal if:
   - they have the same address family, and
     - they are neither IPv4 nor IPv6 addresses, or
     - the local addresses are identical, and
       - they are IPv6 addresses, or
         - they have the same prefixlength, and
         - their peer prefixes are identical

This fixes a bug in the old equality check, which got the local address
and the peer prefix mixed up.

networkd: manager - free bus after own objects

Freeing links/addresses may trigger sending signals over the bus, so let's make sure
the bus stays around until our own objects have been freed.

networkd: route/address - simplify and unify creators

Rename new_dynamic() to simply _new() and reuse that from new_static().

Merge pull request #1510 from mirco/fix-nspawn

nspawn: create /sys/fs/cgroup for unified hierarchy as well

Merge pull request #1516 from poettering/dontisolaterootslice

Don't isolate `-.slice`

core: ignore -.slice and init.scope when isolating

Otherwise, we might end up trying to isolate it away when starting user
instances.

While we are at it, also prohibit manual start/stop of these two units.

Fixes: #1507

sd-daemon: simply code simplification

No change in behaviour, just make the code more obvious.

core: remove taint flag about /etc/mtab not being a symlink.

We already stop boot if /etc/mtab is not a symlink right now, and most
likely we'll stop referecing it at all in the future, either way there's
no point in keeping it around as taint flag.

zsh: don't mention /etc/mtab

The file is obsolete, use /proc/self/mounts instead.

Also, yuck, the code is so awful...

Merge pull request #1512 from evverx/systemd-run-syslog-properties

systemd-run can launch units with SyslogIdentifier and SyslogLevelPrefix

Merge pull request #1501 from fbuihuu/fix-requires-mounts-for-directives

Make sure the mount units pulled by 'RequiresMountsFor=' are loaded, if they exist

shell-completion: systemd-run: add new properties

systemd-run can now launch units with SyslogIdentifier

systemd-run can now launch units with SyslogLevelPrefix

nspawn: create /sys/fs/cgroup for unified hierarchy as well

Merge pull request #1504 from evverx/systemd-run-new-properties

shell-completion: systemd-run: add new properties

shell-completion: systemd-run: add new properties

See:
* 602b83559a7cfcd3eee05
* b9c50073fcc1460ca1ff3

Make sure the mount units pulled by 'RequiresMountsFor=' are loaded (if they exist)

We should make sure that mount units involved by 'RequiresMountsFor='
directives are really loaded if not required by any others units so
that Requires= dependencies on the mount units are applied and thus
the mount unit dependencies are started.

Merge pull request #1499 from evverx/system-conf-update

core: system.conf: add DefaultTasksAccounting

core: system.conf: add DefaultTasksAccounting

Merge pull request #1496 from poettering/stdin-fd

allow passing in fds for stdin/stdout/stderr for transient services

Merge pull request #1497 from bengal/lldp-rename-struct

lldp: rename publicly visible structure

lldp: rename publicly visible structure

Rename struct 'tlv_packet' to 'sd_lldp_packet' and struct
'tlv_section' to 'sd_lldp_section' since the former is referenced in
public header sd-lldp.h.

Signed-off-by: Beniamino Galvani <bgalvani@redhat.com>

shell-completion: add pseudo machine ".host" to shell completion

Merge pull request #1492 from evverx/manager-dbus-properties

core: manager: add some missing properties

Merge pull request #1494 from reverendhomer/patch-1

networkd-link: remove unused variable assignment in link_save()

run: various modernizations and smaller fixes

Including a fix for properly freeing a calendarspec object after use.

shell-completion: add "machinectl shell" to bash completion logic

util: minor modernization of vt_disallocate()

util: do not reset terminal in acquire_terminal()

Before, we'd always reset acquired terminals, which is not really
desired, as we expose a setting TTYReset= which is supposed to control
whether the TTY is reset or not. Previously that setting would only
enable a second resetting of the TTY, which is of course pointless...

Hence, move the implicit resetting out of acquire_terminal() and make
the callers do it if they need it.

machined: when opening a shell via machined, pass tty fds in

With this change we'll open the shell's tty right from machined and then
pass it to the transient unit we create. This way we make sure the pty
is opened exactly as long as the transient service is around, and no
longer, and vice versa. This way pty forwarders do not have to deal with
EIO problems due to vhangup, as the pty is open all the time from the
point we set things up to the point where the service goes away.

networkd-link: remove unused variable assignment in link_save()

fixes Coverity #1317207

core: add support for setting stdin/stdout/stderr for transient services

When starting a transient service, allow setting stdin/stdout/stderr fds
for it, by passing them in via the bus.

This also simplifies some of the serialization code for units.

basic: move two more terminal-related calls into terminal-util.[ch]

update CODING_STYLE

Merge pull request #1491 from Danielmachon/danish-translation

po: added danish translation

po: added danish translation for systemd

po: added danish translation for systemd

Update da.po

fixed syntax errors

core: manager: add some missing dbus properties

keymap: Update Thinkpad Yoga 12 (2015)

Follow-up commit to e1fd19958: Add d8 scan code as well.

Fixes #1440

Merge pull request #1490 from poettering/machinectl-shell-race-fix

machinectl shell race fix

update TODO

machinectl: fix race when opening new shells with "machinectl shell"

Previously, we'd allocate the TTY, spawn a service on it, but
immediately start processing the TTY and forwarding it to whatever the
commnd was started on. This is however problematic, as the TTY might get
actually opened only much later by the service. We'll hence first get
EIOs on the master as the other side is still closed, and hence
considered it hung up and terminated the session.

With this change we add a flag to the pty forwarding logic:
PTY_FORWARD_IGNORE_INITIAL_VHANGUP. If set, we'll ignore all hangups
(i.e. EIOs) on the master PTY until the first byte is successfully read.
From that point on we consider a hangup/EIO a regular connection termination. This
way, we handle the race: when we get EIO initially we'll ignore it,
until the connection is properly set up, at which time we start
honouring it.

util: always enforce O_NOCTTY and O_CLOEXEC in openpt_in_namespace()

The child process is shortliving, hence always set O_NOCTTY so that the
tty doesn't quickly become controlling TTY and then gives it up again.
Also set O_CLOEXEC, because it's cleaner, and doesn't affect the parent
anyway.

build-sys: bump library version

build-sys: add sd_listen_fds_with_name() to .sym file

configure.ac: bump version number

update NEWS

NEWS: re-order entries slightly

Also, the release is actually done in Berlin ;)

Update NEWS

analyze: trivial code cleanups

update NEWS

Merge pull request #1484 from poettering/ask-pass-kernel-keyring

cache harddisk passwords in the kernel keyring

Merge pull request #1485 from jsynacek/machine-long-filename-v5

import: hash URL in paths if they are too long

Merge pull request #1487 from michich/fix-env-expansion

Fix env expansion

Merge pull request #1486 from michich/crash-handler

core: crash handler changes, CrashChangeVT parsing fix

man: fix typo ('and' -> 'are')

basic: fix env expansion for strings leading with two dollar signs

The way to escape a literal dollar sign is to write "$$". But this does
not work right if it's at the beginning of the argument. Fix it.

core: always let the kernel reap zombies when we're about to freeze

Regardless of whether we're going to spawn a crash shell or not, let the
kernel reap zombies. It's more consistent this way.

core: change how crash_shell and crash_reboot interact

Instead of freezing in PID1 and letting the forked child freeze or
reboot when exec("/bin/sh") fails, just wait for the child's
exit and then do the freeze_or_reboot in PID1 as usual.

This means that when both crash_shell and crash_reboot are enabled, the
system will reboot after the shell exits.

core: remove spurious assert in parsing CrashChangeVT=

"data" is always NULL (and unused) in config_parse_crash_chvt().

import: hash URL in paths if they are too long

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

core: adjust error message about /etc/mtab

Since having /etc/mtab as a regular file is now a fatal error, stop
mentioning irrelevant minor consequences.

ask-password: add support for caching passwords in the kernel keyring

This adds support for caching harddisk passwords in the kernel keyring
if it is available, thus supporting caching without Plymouth being
around.

This is also useful for hooking up "gdm-auto-login" with the collected
boot-time harddisk password, in order to support gnome keyring
passphrase unlocking via the HDD password, if it is the same.

Any passwords added to the kernel keyring this way have a timeout of
2.5min at which time they are purged from the kernel.

Merge pull request #1481 from again4you/devel/smack_sysuser_#4

smack: label /etc/passwd and friends as '_' smack label when --with-smack-run-label' is enabled (v3)

Merge pull request #1478 from steelman/analyze-without-hostnamed-fix

analyze: minor fixes according to review

Merge pull request #1483 from reverendhomer/patch-2

service: fix memory leak in service_add_fd_store()

Merge pull request #1482 from reverendhomer/patch-1

strv: fix infinite loop in strv_extend_n()

service: fix memory leak in service_add_fd_store()

fixes Coverity #1325767

strv: fix infinite loop in strv_extend_n()

Fixes Coverity #1325768

smack: label /etc/passwd and friends as '_' smack label when --with-smack-run-label' is enabled

systemd-sysusers.service unit creates system users and groups and it
could update /etc/passwd, /etc/group, /etc/shadow and /etc/gshadow.
Those files should have '_' smack label because of accessibility.

However, if systemd has its own smack label using '--with-smack-run-label'
configuration, systemd-sysusers process spawned by systemd(pid:1) has
its parent smack label and eventually updated files also is set as its
parent smack label.

This patch fixes that bug by labeling updated files as '_' smack label
when --with-smack-run-label' is enabled.

analyze: minor fixes according to review

+ use _cleanup_host_info in acquire_host_info()

+ revert the first strempty() to show "Linux" string in case
  os_pretty_name is missing.

Merge pull request #1475 from steelman/analyze-without-hostnamed

Analyze without hostnamed [V3]

Merge pull request #1474 from teg/siphash24

siphash: style fixes

analyze: replace isempty()+ternary with strempty()

analyze: do not require information from hostnamed

analyze: introduce _cleanup_host_info_

siphash24: coding-style fixes

Drop custom types. Drop unnecessary macros. Fix whitespace. Add asserts.

siphash24: unify API

Make the API of the new helpers more similar to the old wrapper.

In particular we now return the hash as a byte string to avoid
any endianness problems.

ask-password: various modernizations

Primarily clean-up error logging: log either all or no error messages in
the various functions. Mostly this means the actual password querying
calls no longer will log on their own, but the callers have to do so.

Contains various other fixes too, for example ports some code over to
use the clean-up macro.

Should contain no functional changes.

Merge pull request #1473 from michich/test-hashmap

test: hashmap - increase number of entries for crippled hash test

update CODING_STYLE

Merge pull request #1469 from poettering/even-more-fixed

Even more fixes

test: hashmap - increase number of entries for crippled hash test

The purpose of testing with the crippled hash function is to cover
the otherwise very unlikely codepath in bucket_calculate_dib() where
it has to fall back to recomputing the hash value.

This unlikely path was not covered by test-hashmap anymore after
57217c8f "test: hashmap - cripple the hash function by truncating the
input rather than the output".

Restore the test coverage by increasing the number of entries in the test.
The number was determined empirically by checking with lcov.

Merge pull request #1465 from teg/siphash24

hashmap/siphash24: refactor hash functions

update NEWS

util: some comment fixes in fdname_is_valid()

strv: properly return ENOMEM where we should in strv_extend_n()

activate: validate fdname when accepting it

man: use <function> where we refer to functions

NEWS