resolve-host: support parsing numeric interface names

If the user specifies an interface by its ifindex we should handle this
nicely. Hence let's try to parse the ifindex as a number before we try
to resolve it as an interface name.

resolved: remove duplicate handling of "no servers" query result

So far we handled immediate "no server" query results differently from
"no server" results we ran into during operation: the former would cause
the dns_query_go() call to fail with ESRCH, the later would result in
the query completion callback to be called.

Remove the duplicate codepaths, by always going through the completion
callback. This allows us to remove quite a number of lines for handling
the ESRCH.

This commit should not alter behaviour at all.

resolved: replace transaction list by hashmap

Right now we keep track of ongoing transactions in a linked listed for
each scope. Replace this by a hashmap that is indexed by the RR key.
Given that all ongoing transactions will be placed in pretty much the
same scopes usually this should optimize behaviour.

We used to require a list here, since we wanted to do "superset" query
checks, but this became obsolete since transactions are now single-key
instead of multi-key.

Merge pull request #1012 from gentoo-root/master

sd-device: fix enumeration of devices without subsystem

sd-bus: don't list activators as proper peers

If a connection passed KDBUS_HELLO_ACTIVATOR, it cannot do I/O on the
bus. Hence, we should not treat it as proper peer. To actually query it,
you have to explicitly ask for activators.

This makes kdbus in-line with what dbus-daemon does.

Revert "sd-bus: include queried path in GetManagedObjects"

This reverts commit 92d16a53e385781a55d9231d9f8f89c1747ab0e4. As it turns
out, this is not how ObjectManager is supposed to work. It is just a
special behavior of BlueZ, but no-one else implements it this way.

Revert the patch as discussed on github, and as such revert to the
previous behavior (as described in the spec).

Merge pull request #1014 from whot/hwdb-updates

hwdb: add more DPI entries

hwdb: add more DPI entries

Provided by Francois Marier

sd-device: fix enumeration of devices without subsystem

Prior to commit c32eb440bab953a0169cd207dfef5cad16dfb340, libudev's
function udev_enumerate_scan_devices() had behaved differently. If
parent match was added with udev_enumerate_add_match_parent(),
udev_enumerate_scan_devices() did not return error if some child devices
had no subsystem symlink in sysfs. An example of such devices is USB
endpoints /sys/bus/usb/devices/*/ep_*. If there was a parent match
against USB device, old implementation of udev_enumerate_scan_devices()
did not treat ep_* device directories without subsystem symlink as error
and just ignored them, but new implementation returns -ENOENT (also
ignoring these devices) though correctly enumerates all other matching
devices.

To compare, you could look at 96df036fe3d25525a44f5efdb2fc8560e82e6cfd,
in src/libudev/libudev-enumerate.c, function parent_add_child():

    if (!match_subsystem(enumerate, udev_device_get_subsystem(dev)))
            goto nomatch;

udev_device_get_subsystem() was returning NULL, match_subsystem() was
returning false, and USB endpoint device was ignored.

New parent_add_child() from src/libsystemd/sd-device/device-enumerator.c
checks return value of sd_device_get_subsystem() and fails if subsystem
was not found. Absence of subsystem symlink should not be really treated
as error because all enumerations of children of USB devices will fail
with -ENOENT. This new behavior also breaks system-config-printer.

So restore old behavior and treat absence of subsystem symlink as no
match.

Merge pull request #1010 from poettering/resolved-question-key

only maintain one question RR key per transaction and other fixes

Merge pull request #1009 from phomes/master

remove unused variables

resolved: always split up questions into per-RR transactions

We do so for Unicast DNS and LLMNR anyway, let's also do this for mDNS,
and simplify things.

resolved: only maintain one question RR key per transaction

Let's simplify things and only maintain a single RR key per transaction
object, instead of a full DnsQuestion. Unicast DNS and LLMNR don't
support multiple questions per packet anway, and Multicast DNS suggests
coalescing questions beyond a single dns query, across the whole system.

resolved: add extra check for family when doing LLMNR TCP connections

It shouldn't happen that we try to resolve IPv4 addresses via LLMNR on
IPv6 and vice versa, but let's explicitly verify that we don't turn an
IPv4 LLMNR lookup into an IPv6 TCP connection.

resolved: add reference to negative caching RFC

remove unused variables

Merge pull request #1005 from poettering/resolved-refuse-compression

Don't do name compression when passing RRs across the bus

Merge pull request #1004 from poettering/systemd-run-man

man: rework systemd-run man page a bit

resolve-host: Minor wording improvement

resolved: when passing RRs across the bus, make sure not to use name compression

We explicitly need to turn off name compression when marshalling or
demarshalling RRs for bus transfer, since they otherwise refer to packet
offsets that reference packets that are not transmitted themselves.

man: rework systemd-run man page a bit

Merge pull request #1003 from poettering/kmod-log-debug

core: downgrade "Module inserted" message for kmod to DEBUG

core: downgrade "Module inserted" message for kmod to DEBUG

Closes #919.

Merge pull request #1002 from poettering/resolved-various

resolved: synthesize more RRs locally and other fixes

resolved: rework synthesizing logic

With this change we'll now also generate synthesized RRs for the local
LLMNR hostname (first label of system hostname), the local mDNS hostname
(first label of system hostname suffixed with .local), the "gateway"
hostname and all the reverse PTRs. This hence takes over part of what
nss-myhostname already implemented.

Local hostnames resolve to the set of local IP addresses. Since the
addresses are possibly on different interfaces it is necessary to change
the internal DnsAnswer object to track per-RR interface indexes, and to
change the bus API to always return the interface per-address rather than
per-reply. This change also patches  the existing clients for resolved
accordingly (nss-resolve + systemd-resolve-host).

This also changes the routing logic for queries slightly: we now ensure
that the local hostname is never resolved via LLMNR, thus making it
trustable on the local system.

resolved: make DnsQuestion logic handle NULL arrays as empty arrays

Following our usual logic of treating NULL arrays as empty arrays (for
example, see strv.c) do the same for questions too.

resolved: minor typo comment fix

dns-domain: add call for concatenating two domain names

This is specifically useful for appending the mDNS ".local" suffix to a
single-label hostname in the most correct way. (used in later commit)

Merge pull request #924 from pfl/systemd-dhcp6

sd-dhcpv6: support DNS and NTP information

Merge pull request #1001 from major/add-bonding-docs

man: networkd - adding bonding examples for systemd-networkd

hostname-util: introduce new is_gateway_hostname() call

This moves is_gateway() from nss-myhostname into the basic APIs, and
makes it more like is_localhost(). Also, we rename it to
is_gateway_hostname() to make it more expressive.

Sharing this function in src/basic/ allows us to reuse the function for
routing name requests in resolved (in a later commit).

network: Save DNS and NTP data for a DHCPv6 link

Append DNS and NTP data obtained via DHCPv6 when the Link is saved.

network: Add function to serialize an IPv6 address

test-dhcp6-client: Add tests for DNS and NTP options

Test option setting and getting in test_advertise_option(). Verify
that the information provided in DHCPv6 Reply messages is also
available in the Information and Solicit callbacks.

sd-dhcp6: Support deprecated SNTP Configuration Option

Although the SNTP option specified in RFC 4075 has been deprecated, some
servers are still sending NTP information with this option. Use the SNTP
information provided only if the NTP option is not present.

Update the test case as SNTP information is also requested.

sd-dhcp6: Add support for DHCPv6 NTP Server Option

Support NTP server and multicast addresses and NTP server domain names
as specified in RFC 5908.

sd-dhcp6: Add support for DHCPv6 DNS Domain Search List option

Support DHCPv6 DNS search list option as specified in RFC 3646. This
option contains a list of DNS search domains encoded without compression
as specified in Section 8. of RFC 3315.

sd-dhcp6: Add support for DHCPv6 DNS Recursive Name Server option

Support DHCPv6 DNS server option as specified in RFC 3646. This option
contains a list of IPv6 DNS server addresses.

dhcp6-option: Add helper function for uncompressed domain names

Add a helper function containing a modified version of dns_packet_read_name()
that does not use DnsPacket to extract a string array of domain names from
the provided option data. The domain names are stored uncompressed as defined
in Section 8. of RFC 3315.

dhcp6-option: Add helper function for fetching IPv6 addresses

Add a helper function that extracts a block of IPv6 addresses from
the provided option data.

sd-dhcp6-client: Save a DHCPv6 lease also with Information Reply

As the lease structure contains interesting information, save it also
for the Information Reply.

sd-dhcp6-client: Fix unreferencing DHCPv6 lease on client reset

When the DHCPv6 client is started by the library user or stopped for
any reason, unref the DHCPv6 lease when resetting the DHCPv6 client
data structure. This makes the DHCPv6 client always start from a clean
state and not keep unnecessary an lease structure around when stopped.

If this is not done, a previously existing lease information can be
interpreted to be from another server when restarting DHCPv6.

Merge pull request #996 from kostrowski/man-systemd-special

man: fix typo in description of system.slice

Merge pull request #995 from poettering/timedate-utc

timedatectl: when no timezone is set consider UTC the default

Adding bonding examples for systemd-networkd

This commit provides some basic bonding configuration examples for .netdev and .network files.

man: fix typo in description of system.slice

update TODO

timedatectl: when no timezone is set consider UTC the default

This fixes #993, and ensures that the libc does not consider any
old timezone information into account, that was set earlier.

Merge pull request #990 from owtaylor/issue-989

Issue 989 - logind: VT is not properly reset on session close

Merge pull request #993 from xnox/timedate

timedatectl: assert timezone is not null in setenv() call.

timedatectl: assert timezone is not null in setenv() call.

setenv is declared as:

extern int setenv (const char *__name, const char *__value, int __replace)
     __THROW __nonnull ((2));

And i->timezone can be NULL, if for example /etc/localtime is
missing. Previously that worked, but now result in a libc dumping
core, as seen with gcc 2.22, due to:

https://sourceware.org/ml/glibc-cvs/2015-q2/msg00075.html

Merge pull request #988 from spartacus06/update-completion-machinectl-v2

Update completion machinectl (part 2)

Merge pull request #992 from zonque/journal-json

journal: fix json output of unicode characters

logind: get a fresh file descriptor to clean up a VT

When the controlling process exits, any existing file descriptors
for that FD will be marked as hung-up and ioctls on them will
file with EIO. To work around this, open a new file descriptor
for the VT we want to clean up.

Thanks to Ray Strode for help in sorting out the problem and
coming up with a fix!

https://github.com/systemd/systemd/issues/989

logind: use open_terminal() instead of open()

The open_terminal() function adds retries in case a terminal
is in the process of being closed when we open it, and should
generally be used to open a terminal. We especially need it
for code that a subsequent commit adds that reopens the terminal
at session shut-down time; such races would be more likely in
that case.

Found by Ray Strode.

shell-completion: bash: make list-transfers and cancel-transfer STANDALONE

list-transfers and cancel-transfers don't take machine arguments.

shell-completion: bash: add image transfer subcommands

journal: fix json output of unicode characters

Merge pull request #984 from whot/hwdb-updates

hwdb: fix capitalization of product version for the Macbooks

Merge pull request #983 from phomes/journal_dead_check

journal: remove error check that never happens

hwdb: fix capitalization of product version for the Macbooks

We need uppercase hex here.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

journal: remove error check that never happens

remove_directory will always return 0 so this can never happen.
Besides that, d->path and d are freed so we would end up with
a null pointer dereference anyway.

Merge pull request #982 from evverx/add-systemd-cgls

man: add systemd-cgls as tool that shows the cgroup hierarchy

man: add systemd-cgls as tool that shows the cgroup hierarchy

Merge pull request #977 from richardmaw-codethink/machinectl-userns-login-v2

Fix machinectl login with containers in user namespaces (v2)

Merge pull request #976 from elfring/Remove_unnecessary_checks2

Delete unnecessary checks before some function calls

Merge pull request #958 from stefwalter/fix-journalctl-f-regression

Regression: 'journalctl -f -t unmatched' doesn't block properly

namespace helpers: Allow entering a UID namespace

To be able to use `systemd-run` or `machinectl login` on a container
that is in a private user namespace, the sub-process must have entered
the user namespace before connecting to the container's D-Bus, otherwise
the UID and GID in the peer credentials are garbage.

So we extend namespace_open and namespace_enter to support UID namespaces,
and we enter the UID namespace in bus_container_connect_{socket,kernel}.

namespace_open will degrade to a no-op if user namespaces are not enabled
in the kernel.

Special handling is required for the setns call in namespace_enter with
a user namespace, since transitioning to your own namespace is forbidden,
as it would result in re-entering your user namespace as root.

Arguably it may be valid to check this at the call site, rather than
inside namespace_enter, but it is less code to do it inside, and if the
intention of calling namespace_enter is to *be* in the target namespace,
rather than to transition to the target namespace, it is a reasonable
approach.

The check for whether the user namespace is the same must happen before
entering namespaces, as we may not be able to access /proc during the
intermediate transition stage.

We can't instead attempt to enter the user namespace and then ignore
the failure from it being the same namespace, since the error code is
not distinct, and we can't compare namespaces while mid-transition.

Bug #944: Deletion of unnecessary checks before a few calls of systemd functions

The following functions return immediately if a null pointer was passed.
* calendar_spec_free
* link_address_free
* manager_free
* sd_bus_unref
* sd_journal_close
* udev_monitor_unref
* udev_unref

It is therefore not needed that a function caller repeats a corresponding check.

This issue was fixed by using the software Coccinelle 1.0.1.

Merge pull request #953 from poettering/ebadf

tree-wide: generate EBADF when we get invalid fds

Merge pull request #970 from poettering/timedatectl-set-netp

man: extend documentation for timedatectl's set-ntp command

Merge pull request #973 from poettering/sd-bus-error-ret

sd-bus: always fill in sd_bus_error paramters, on error

Merge pull request #974 from teg/resolved-fixes-2

resolved: debugging improvements

journalctl: make sure 'journalctl -f -t unmatched' blocks

Previously the following command:

$ journalctl -f -t unmatchedtag12345

... would block when called with criteria that did not match any
journal lines. Once log lines appeared that matched the criteria
they were displayed.

Commit 02ab86c732576a71179ce12e97d44c289833236d broke this
behavior and the journal was not followed, but the command
exits with '-- No entries --' displayed.

This commit fixes the issue.

More information downstream:

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

Bug #944: Deletion of unnecessary checks before calls of the function "free"

The function "free" is documented in the way that no action shall occur for
a passed null pointer. It is therefore not needed that a function caller
repeats a corresponding check.
http://stackoverflow.com/questions/18775608/free-a-null-pointer-anyway-or-check-first

This issue was fixed by using the software Coccinelle 1.0.1.

Bug #944: Replacement of a free() call by mfree()

The function "mfree" should be called instead of "free" at a specific
source code place.

resolved: cache - add more detailed cache debug logging

sd-bus: always fill in sd_bus_error paramters, on error

Whenever one of our calls is invoked with a non-NULL, writable
sd_bus_error parameter, let's fill in some valid error on failure. We
previously only filled in remote errors, but never local errors, which is
hard to handle by users. Hence, let's clean this up to always fill in
the error.

This introduces a new bus_assert_return() macro that works like
assert_return() but optionally also initializes a bus_error struct.

Fixes #224.

Based on a patch by Umut Tezduyar.

update TODO

Merge pull request #908 from richardmaw-codethink/nspawn-path-escapes-v3

Allow arbitrary file paths to be passed to nspawn (v3)

README: drop reference to Endocode

Chris Kühl does not work at Endocode anymore, the contact data is no
longer valid.

TODO

resolve-host: print RTT

resolved: packet - fix typo in read_rr()

Merge pull request #955 from poettering/resolved-localhost-xyz

resolved: enable synthesizing of A, AAAA and PTR records for "localhost"

Merge pull request #954 from poettering/nss-myhostname-loopback

nss-myhostname: use LOOPBACK_IFINDEX instead of if_nametoindex("lo")

Merge pull request #972 from poettering/resolve-host-auto-start

resolve-host: enable auto start of resolved

resolve-host: enable auto start of resolved

There's no reason to explicitly turn off bus activation for resolved
here. The reason this was done before was that the code was copied from
nss-resolve, which has a fallback to glibc's nss-dns if resolved is not
reachable. However, such a logic makes no sense for resolve-host since
such a fallback doesn't make sense here, which means we can actually
turn on activation. Let's do it hence.

Merge pull request #971 from evverx/news-fix

NEWS: fix the "indirect" state description

NEWS: fix the "indirect" state description

Merge pull request #931 from spartacus06/update-completion-machinectl

Update shell completion for machinectl

man: extend documentation for timedatectl's set-ntp command

This extends on the relationship between timedatectl's set-ntp command
and its effect on the systemd-timesyncd.service unit. This also links
that unit back to the timedatectl man page.

Closes #798.

update TODO

shell-completion: use list-images rather than list

"machinectl list" only lists running machines while many of the MACHINES
commands use names of images; both running and non-running.

List machines from both "list" and "list-images" and use sort -u to
avoid duplicates.

shell-completion: update machinectl bash completion

Many new subcommands have been added since the bash completion was last
updated.  Bring it up to speed.

Merge pull request #930 from richardmaw-codethink/man-when-exec-start-post

man: Describe when ExecStartFoo= commands are run in more detail

Merge pull request #918 from thomasmey/uml-fix

UML: Fix block device setup

Merge pull request #947 from evverx/bash-completion

Update bash-completion: generate all unit types

bash-completion: generate unit type list from systemctl -t help option

Merge pull request #932 from kaysievers/bus

 sd-bus: do not connect to dbus-1 socket when kdbus is available