Merge branch 'pr/761'

networkd-wait-online: only consider interfaces given on the commandline

If some interfaces are given on the commandline, ignore all others.

Merge pull request #777 from kaysievers/gtp-auto

gpt-auto-generator: merge efi-boot-generator

Merge pull request #774 from dvdhrm/gvariant3

gvariant: fix encoding of unary type and root-variant (v3)

gpt-auto-generator: merge efi-boot-generator

Merge pull request #776 from baracoder/master

man: unit: fix StartTimeoutSec

man: unit: fix StartTimeoutSec

should be TimeoutStartSec

sd-bus: fix encoding/decoding gvariant root container

The gvariant root container contains a 'variant' at the end, which embeds
the whole message body. This variant *must* contain a structure so we are
compatible to dbus1. Otherwise, it could encode at most 1 type, instead
of a full signature.

Our gvariant message parser already parses the variant-content as a
structure, so we're mostly good. However, it does *not* include the
opening and closing parantheses, nor does it parse them.

This patch fixes the decoder to verify a message contains the
parantheses, and also make the encoder add those parantheses into the
marshaled message.

sd-bus: fix marshaling of unary type

The unary type has a fixed size of 1 in gvariant. Make sure we properly
encode it as such. Right now, we encode/decode it as empty sequence.

NEWS: update 223 contributors

Update v223 contributors and release date.

build-sys: update git-tar

Merge pull request #769 from poettering/socket-util-no-log

socket-util: library calls shouldn't log on their own

sd-bus: drop redundant code

If c->item_size is 0, the next item to parse in a structure is empty.
However, this also implies that the signature must be empty. The latter
case is already handled just fine by enter_struct_or_dict_entry() so
there is no reason to handle the same case in the caller.

sd-bus: don't assert() on valid signatures

Right now sd_bus_message_skip() will abort execution if passed a
signature of the unary type "()". Regardless whether this should be
supported or not, we really must not abort. Drop the incorrect assertion
and add a test-case for this.

build: bump version info

Bump version to 223 and increase libsystemd libtool-info as we added a
structure-member (without increasing structure size) to sd-bus-vtable.

Merge pull request #768 from poettering/resolved-localhost

resolved: never attempt to resolve loopback addresses via DNS/LLMNR/mDNS

Merge pull request #767 from torstehu/fix-typo

treewide: fix typos

socket-util: library calls shouldn't log on their own

Instead, make sure that all callers log properly.

resolved: never attempt to resolve loopback addresses via DNS/LLMNR/mDNS

We already refuse to resolve "localhost", hence we should also refuse
resolving "127.0.0.1" and friends.

treewide: fix typos

NEWS: add 'macvtap' section

systemd-networkd now supports 'macvtap' devices, similar to the already
supported 'macvlan' devices.

Merge pull request #747 from dvdhrm/consoled

terminal: drop unfinished code

networkd-wait-online: fix race

We must consider 'pending' links as if they may be managed by networkd, as this
is the state we enter before deciding wether networkd should manage the link
or not, so we better wait for this decision being made.

Merge pull request #760 from teg/resolved-fix-nsec-types

resolved: fix NSEC type windows

resolved: packet - refuse empty type bitmaps

The NSEC type itself must at least be in the bitmap, so NSEC records with empty
bitmaps must be bogus.

resolved: packet - fix read_type_window()

We were tracking the bit offset inside each byte, rather than inside the whole bitmap.

resolve: packet - fix append_types()

We were counting the number of bits set rather than the number of bytes they occupied.

Merge pull request #759 from phomes/master

hwdb: add HP USB 1000dpi Laser Mouse

Merge pull request #758 from poettering/dns-is-equal

resolved: compare dns question arrays properly

hwdb: add HP USB 1000dpi Laser Mouse

Merge pull request #732 from ssahani/macvtap

networkd: add support for Macvtap

resolved: compare dns question arrays properly

Let's optimize things a bit and properly compare DNS question arrays,
instead of checking if they are mutual supersets. This also makes ANY
query handling more accurate.

udev: Fix udev_builtin_run_once()

Honour the boolean return value type.
This was an oversight in f89d10ae4.

udev: check more builtins[] pointers before dereferencing

Fix some more locations where pointers from builtins[] are dereferenced
before checking. Related to 8cacf69b1.

Merge pull request #756 from ldzhong/fix

udev: avoid coredump when initializing udev builtins

udev: avoid coredump when initializing udev builtins

If one of the macros(HAVE_BLKID/HAVE_KMOD/HAVE_ACL) is not
defined, there will be a coredump

Merge pull request #755 from karelzak/nomtab

mount: remove obsolete -n

mount: remove obsolete -n

It seems that systemd still uses legacy -n option. The option has been
originally designed to avoid write to /etc/mtab during boot when root
FS is not ready or read-only.

This is not necessary for long time, because /etc/mtab is not a real
file (it's symlink) and write to the file is impossible. All utils
should be able to detect the symlink and ignore mtab. This concept is
supported for very long time before systemd.

The userspase mount options are currently maintained by libmount
(mount(8) and mount.nfs) in /run/mount) which is tmpfs initialized
during early boot.

Merge pull request #375 from msekletar/test-install-crashers

install: make unit_file_get_list aware of UNIT_FILE_INDIRECT

Merge pull request #362 from d-hatayama/fix_selinux_unit_check_v2

selinux: fix missing SELinux unit access check

Merge pull request #748 from teg/dns-name-between-4

basic: dns-name - between()

Merge pull request #739 from intelfx/systemd-analyze-plot-user-instance

analyze: correctly draw the plot for user instances.

analyze: correctly draw the plot for user instances

Start-up timestamp of a user instance (userspace_time in struct boot_times)
actually may be arbitrarily big. This, because all timestamps are offset by
that value, leads to creation of arbitrarily wide SVGs which almost completely
consist of blank space.

Fix this by inverse-offsetting all timestamps by that value if user instance
operation is requested.

Fixes #740.

TODO

shared: dns-name - add dns_name_between()

Given three DNS names this function indicates if the second argument lies
strictly between the first and the third according to the canonical DNS
name order. Note that the order is circular, so the last name is
considered to be before the first.

shared: dns-name - use the canonical dns name ordering

The canonical DNS name ordering considers the rightmost label the most significant,
we were considering it the least significant. This is important when implementing
NSEC, which relies on the correct order.

shared: dns-name - introduce dns_label_unescape_suffix()

Intended to be called repeatedly, and returns then successive unescaped labels
from the most to the least significant (left to right).

This is slightly inefficient as it scans the string three times (two would be
sufficient): once to find the end of the string, once to find the beginning
of each label and lastly once to do the actual unescaping. The latter two
could be done in one go, but that seemed unnecessarily convoluted.

analyze: move acquire_boot_times() above acquire_time_data()

Next patch will make the latter call into the former.

NEWS: follow VNetHeader rename

VnetHeader was renamed to VNetHeader just now. Update NEWS accordingly.

Merge pull request #744 from poettering/vnethdr-fix

networkd: capitalize VNetHeader= as VnetHeader=

Merge pull request #746 from teg/resolved-connect-udp

resolved: UDP fixes

util: base32hex - explain distinction with base32

resolved: transaction - don't explicitly verify packet source

This is handled by the kernel now that the socket is connect()ed.

resolved: transaction - don't unref server when creating TCP socket

This was a bug.

resolved: scope - write() unicast DNS packets

As we have connect()ed to the desired DNS server, we no longer need to pass
control messages manually when sending packets. Simplify the logic accordingly.

resolved: transaction - introduce dns_transaction_emit()

This function emits the UDP packet via the scope, but first it will
determine the current server (and connect to it) and store the
server in the transaction.

This should not change the behavior, but simplifies the code.

networkd: capitalize VNetHeader= as VnetHeader=

Even when we use shortened, combined words, we still should uppercase
where a new word starts. I couldn't find a canonically capitalized
version of this term, hence I think we should follow our naming rules
here.

resolved: transaction - move a couple of functions

No functional change, but makes follow-up patch clearer.

terminal: drop unfinished code

This drops the libsystemd-terminal and systemd-consoled code for various
reasons:

 * It's been sitting there unfinished for over a year now and won't get
   finished any time soon.

 * Since its initial creation, several parts need significant rework: The
   input handling should be replaced with the now commonly used libinput,
   the drm accessors should coordinate the handling of mode-object
   hotplugging (including split connectors) with other DRM users, and the
   internal library users should be converted to sd-device and friends.

 * There is still significant kernel work required before sd-console is
   really useful. This includes, but is not limited to, simpledrm and
   drmlog.

 * The authority daemon is needed before all this code can be used for
   real. And this will definitely take a lot more time to get done as
   no-one else is currently working on this, but me.

 * kdbus maintenance has taken up way more time than I thought and it has
   much higher priority. I don't see me spending much time on the
   terminal code in the near future.

If anyone intends to hack on this, please feel free to contact me. I'll
gladly help you out with any issues. Once kdbus and authorityd are
finished (whenever that will be..) I'll definitely pick this up again. But
until then, lets reduce compile times and maintenance efforts on this code
and drop it for now.

resolved: transaction - move DNS UDP socket creation to the scope

With access to the server when creating the socket, we can connect()
to the server and hence simplify message sending and receiving in
follow-up patches.

resolved: transaction - close socket when changing server

Close the socket when changing the server in a transaction, in
order for it to be reopened with the right server when we send
the next packet.

This fixes a regression where we could get stuck with a failing
server.

resolved: transaction - don't request PKTINFO for unicast DNS

This was only ever used by LLMNR, so don't request this for unicast DNS packets.

resloved: transaction - unify IPv4 and IPv6 sockets

A transaction can only have one socket at a time, so no need to distinguish these.

sd-bus: fix path of object-manager signals

Each signal of the ObjectManager interface carries the path of the object
in question as an argument. Therefore, a caller will deduce the object
this signal is generated for, by parsing the _argument_. A caller will
*not* use the object-path of the message itself (i.e., message->path).
This is done on purpose, so the caller can rely on message->path to be
the path of the actual object-manager that generated this signal, instead
of the path of the object that triggered this signal.

This commit fixes all InterfacesAdded/Removed signals to use the path of
the closest object-manager as message->path. 'closest' in this case means
closest parent with at least one object-manager registered.

This fix raises the question what happens if we stack object-managers in
a hierarchy. Two implementations are possible: First, we report each
object only on the nearest object-manager. Second, we report it on each
parent object-manager. This patch chooses the former. This is compatible
with other existing ObjectManager implementations, which are required to
call GetManagedObjects() recursively on each object they find, which
implements the ObjectManager interface.

NEWS: group systemd-networkd items

… for better readability.

NEWS: fix typos

Merge pull request #738 from poettering/machined-caps

units: add more caps to machined

NEWS: add entries for v223

New features and API changes for v223. Please review carefully and amend!

units: add more caps to machined

Otherwise copying full directory trees between container and host won't
work, as we cannot access some fiels and cannot adjust the ownership
properly on the destination.

Of course, adding these many caps to the daemon kinda defeats the
purpose of the caps lock-down... but well...

Fixes #433

bus-proxy: augment debug message for dropped broadcasts a bit

Add the PID we are proxying for, as well as the message's sender and
destination string, to the debug message that is printed when the proxy
drops unmatched broadcasts.

man: netdev add man for macvtap

networkd: add support for macvtap

This patch add support for macvtap.

see http://virt.kernelnewbies.org/MacVTap

sd-netlink: Add macvtap support

networkd-wait-online: fix -i argument

-i (aka --interface) takes an argument. Tell getopt_long() that, so that optarg
isn't NULL.

Merge pull request #716 from michaelolbrich/automount-fixes

Automount fixes

Merge pull request #724 from dbuch/master

proxyd: downgrade to log_debug() for unmatched broadcasts

systemd-boot: fix whitespace

Merge pull request #634 from icarlosvenegas/sd-boot-show-efi-cmdline_v2

sd-boot: Show stub cmdline when edit (v2)

Merge pull request #727 from phomes/master

man: typo fixes

Merge pull request #725 from keszybz/network-file-masking

man: describe masking of .network files better

man: typo fixes

man: describe masking of .network files better

This should clear up some confusion in
https://github.com/systemd/systemd/issues/717.

This basically copies the description from systemd.unit to this
man page. Masking can happen also in /run, so strike the part
about /etc, and also add the magic work "mask".

proxyd: downgrade to log_debug() for unmatched broadcasts

Merge pull request #722 from keszybz/networkd-clarifications

Networkd clarifications

networkd: rename RootBlock to AllowPortToBeRoot

Justification is similar to BPDUGuard rename. "Positive" values
are easier. This is a rather uncommon option, so using a slightly
longer name should not be a problem, and may in fact may make it
easier to guess what the option does without reading the
documentation.

networkd: turn UnicastFlood on by default

Looking at the kernel commit, "on" seems to be the default value:
commit 867a59436fc35593ae0e0efcd56cc6d2f8506586
Author: Vlad Yasevich <vyasevic@redhat.com>
Date:   Wed Jun 5 10:08:01 2013 -0400

    bridge: Add a flag to control unicast packet flood.

    Add a flag to control flood of unicast traffic.  By default, flood is
    on and the bridge will flood unicast traffic if it doesn't know
    the destination.  When the flag is turned off, unicast traffic
    without an FDB will not be forwarded to the specified port.

... and it seems to be the reasonable thing to do by default.

networkd: rename BPDUGuard to UseBPDU

Rename to follow the follow the style of other options.

In general "positive" options are preferred to "negative" ones,
because they are easier to describe and easier for humans to
parse (c.f. the shortening on the man page entry).

man: reword new Bridge descriptions

network: rename DiffServiceCodePoint to CopyDSCP

Old name was slightly misleading, because this flag does not determine
whether DSCP is used overall, but only if it is copied to the
decapsulated packet. Rename to better reflect that.

"Copy" does not imply direction. This is on purpose, because we might
later on enhance the setting to allow/disallow copying in the other
direction, to the encapsulated packet. If that is implemented,
CopyDSCP could understand additional values. This is nicer than
having two separate settings and follows the example of DHCP=.

Also, we try to avoid abbreviations, but we allow acronyms
like MTU, in DiscoverPathMTU=.

This setting was recently added, so it's fine to rename it without
backwards compat.

man: try to better describe DiffServiceCodePoint= setting

http://marc.info/?l=linux-netdev&m=109507453227993&w=2

netlink-types: use consistent whitespace at EOL

Follow up for v222-124-g79e27dbcb1.

man: reword description of Hostname=

Also add dots at the end of sentences.

bootctl: add missing newline

Merge pull request #718 from phomes/master

ata_id: remove unused union member

ata_id: remove unused union member

The last use of octa was removed in 01f61d331bb5038f0c877ac03c54333328b6ea28

automount: handle state changes of the corresponding mount unit correctly

The expire timeout must be started/stopped if the corresponding mount unit
changes its state, e.g. it is started via local-fs.target or stopped by a
manual umount.

automount: don't try to umount if it already happened

Return the token immediately instead. Otherwise the token is never returned
to the kernel, because the umount job is a noop and will not trigger a
state change.

resolved: fix DNS_TYPE_ANY vs DNS_CLASS_ANY confusion

Assigning a TPYE enum value to a class variable is certainly wrong.
However, they both have the same value, so the result was correct
nevertheless.

Merge pull request #704 from richardmaw-codethink/empty-arg-unquote

unquote_first_word: parse ` '' ` as an empty argument instead of no arg

Merge pull request #714 from zonque/automount

automount: do not start expiration timer for TimeoutIdleSec=0