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

automount: do not start expiration timer for TimeoutIdleSec=0

The timer value for automount unit specified with TimeoutIdleSec= is rounded
up to one second if that directive is set to 0.

Fix this by bailing early in automount_enter_runnning() in case no timeout is
requested.

Merge pull request #713 from daurnimator/577-sd_bus-vtable-methods-do-not-have-offset

sd-bus: remove _VTABLE from new method vtable initialiser

sd-bus: remove _VTABLE from new method vtable initialiser

Merge pull request #712 from daurnimator/577-sd_bus-vtable-methods-do-not-have-offset

sd-bus: Add offset member for vtable methods

sd-bus: add 'offset' member for vtable methods

Defaults to zero, which retains the current behaviour.
Fixes #577

Merge pull request #711 from zonque/const

tree-wide: do not use _cleanup_free_ on const pointers

tree-wide: do not use _cleanup_free_ on const pointers

free() cannot be used with const pointers. However, our _cleanup_free_
handler features cast logic that hides that qualifier, so we don't get a
warning.

sd-bus: don't treat KDBUS_ITEM_TIMESTAMP as unknown item

In bus_kernel_translate_message(), we print a DEBUG message on unknown
items. But right now, we also print this message for KDBUS_ITEM_TIMESTAMP
despite parsing it properly. Fix this!

Merge pull request #606 from dvdhrm/bus-proxy-pedantic-matches2

bus-proxy: never pass on unmatched broadcasts (v2)

Merge pull request #695 from poettering/journal-fixes

Journal fixes

Merge pull request #702 from ldzhong/fix

udev: fix parameter process

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

udev: fix parameter process

Merge pull request #699 from ysbnim/master

exit-status: add missing string for EXIT_SMACK_PROCESS_LABEL

Merge pull request #696 from poettering/automount-expiry-freq

automount: lower the idle polling frequency a bit

Merge pull request #697 from poettering/service-bus-name

core: print a nicer warning when two units have the same BusName= set…

exit-status: add a missing string for EXIT_SMACK_PROCESS_LABEL

core: print a nicer warning when two units have the same BusName= setting

This should make issues like #609 easier to debug.

automount: lower the idle polling frequency a bit

The autofs kernel idle logic requires us to poll the kernel for
idleness. This is of course suboptimal, but cannot be fixed without
kernel change.

Currently the polling frequency is set to 1/10 of the idle timeout. This
is quite high, as seen in #571. Let's lower this to 1/3.

journal: uppercase first character in verify error messages

In the english language the first character of a sentence is supposed to
be uppercase. Let's make sure this also applies to the journal
verification error messages.

journalctl: properly detect empty journal files

When we encounter a journal file with exactly zero entries, print a nice
message and exit, and don't print a weird error message.

journal: explain the error when we find a non-DATA object that is compressed

Only objects of type DATA may be compressed, generate a message about
that, like we do for all other errros.

journal: when verifying journal files, handle empty ones nicely

A journal file that carries no objects should be considered valid.

journal: avoid mapping empty data and field hash tables

When a new journal file is created we write the header first, then sync
and only then create the data and field hash tables in them. That means
to other processes it might appear that the files have a valid header
but not data and field hash tables. Our reader code should be able to
deal with this.

With this change we'll not map the two hash tables right-away after
opening a file for reading anymore (because that will of course fail if
the objects are missing), but delay this until the first time we access
them. On top of that, when we want to look something up in the hash
tables and we notice they aren't initialized yet, we consider them
empty.

This improves handling of some journal files reported in #487.

journal-verify: don't hit SIGFPE when determining progress

If we determine the progress based on a number of objects available,
don't blindly devide by the number of objects, given that it might be 0.

Merge pull request #539 from poettering/tmpfiles-journal-acl

tmpfiles: don't recursively descend into journal directories in /var

Merge pull request #694 from poettering/fileio-fixes

Fileio fixes

process: an empty environment block should be returned as such

An empty env block is completely valid, hence return it as such, and
don't turn it into an error.

process: return ESRCH when a PID is not valid anymore

so far, when we read something from /proc/$PID we would pass on the
ENOENT from the kernel as error, if the process was missing. With this
change we systematically convert this to ESRCH, which is the more
appropriate error code, and what all the other glibc/syscalls like
kill() use.

All code that calls these functions should be fine with this change. In
fact, one invocation of get_process_exe() in bus-creds.c already assumed
ESRCH would be returned if a process is missing, and this assumption is
now validated after the change.

fileio: get_status_field() don't clobber arg on OOM

According to our coding style guidelines we shouldn't clobber
pass-by-ref arguments on failure, hence don't do so here either.

Merge pull request #692 from poettering/fd-copy-directory-all

copy: when we recursively copy a directory tree, copy everything

copy: when we recursively copy a directory tree, copy everything

Don't ignore hidden files and directories.

Fixes #386

Merge pull request #683 from ssahani/tun1

networkd: ip6gre add support for flowlabel

Merge pull request #670 from floppym/ptsuid

nspawn: Don't pass uid mount option for devpts

Merge pull request #678 from eworm-de/oracle-kvm

Oracle kvm

Merge pull request #691 from teg/networkd-after-sysctl

units: order networkd after sysctl

virt: handle Virtualbox 5.0 with kvm hypervisor

Virtualbox 5.0 now supports kvm hypervisor. In this case cpuid
identidies as "kvm", which breaks units depending on
ConditionVirtualization=oracle.
So return "oracle" even with kvm hypervisor.

Merge pull request #682 from ssahani/bridge

networkd: add bridge link properties

units: order networkd after sysctl

This way networkd will correctly and race-freely inherit the default settings
applied by sysctl.

Suggested in issue #468.

man: add man for bridge params

networkd: add bridge link properties

new bridge properties

br.network

[Match]
Name=enp0s25

[Network]
Bridge=br-test

[Bridge]
Cost=332
BPDUGuard = true
HairPin = true
FastLeave = true
RootBlock = true
UnicastFlood = true

Merge pull request #604 from heftig/master

build-sys: Use slim LTO objects if possible

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

resolved: assorted fixes v2

resolve: transaction - stop processing packet when found to be invalid

We were stopping the transaction, but we need to stop processing the packet alltogether.

resolved: packet - fix segfault in truncate()

A size_t was being accessed as a char* due to the order of arguments being inverted.

resolved: rr - ignore pseudo types in NSEC(3) bitmaps

resolved: rr - fix parsing of NSEC3

We were appending rather than reading the bitmap.

resolved: rr - don't read past end of RR when parsing NSEC(3)

We can never read past the end of the packet, so this seems impossible
to exploit, but let's error out early as reading past the end of the
current RR is clearly an error.

Found by Lennart, based on patch by Daniel.

resolved: rr - SSHFP contains the fingerprint, not the key

Rename the field to make this clearer.

resolved: packet - fail on invalid zero-length data

Most blobs (keys, signatures, ...) should have a specific size given by
the relevant algorithm. However, as we don't use/verify the algorithms
yet, let's just ensure that we don't read out zero-length data in cases
where this does not make sense.

The only exceptions, where zero-length data is allowed are in the NSEC3
salt field, and the generic data (which we don't know anything about,
so  better not make any assumptions).

Merge pull request #687 from poettering/bitmap-fixes

bitmap: various clean-ups

bitmap: various clean-ups

a) use memcmp() to compare bitmaps efficiently

b) use UINT64_C() macro instead of ULL suffixes to get right suffix for
   uint64_t constants

c) add a few assert()s

d) when comparing integers with 0 we generally try to make this explicit
   with "!= 0".

e) remove redundant bitmap_isset() if check, as we don't have it in
   bitmap_isset() either.

f) It should be fine to invoke bitmap_unset() on a NULL bitmap

Merge pull request #669 from poettering/dns-rr-memdup

resolve: unify memdup() code when parsing RRs

networkd: ip6gre add support for flowlabel

sd-netlink: add bridge NL params

Add bridge NL params to missing.h

Merge pull request #677 from zonque/bitmap2

basic: bitmap: use uint64_t instead if long long unsigned

Merge pull request #674 from ssahani/tunnel

ip6 tunnel: add support for DSCP

man: add man for DSCP

Merge pull request #537 from poettering/nss-mymachines-userns

Hook up container userns with nss-mymachines

Merge pull request #663 from poettering/tmpfiles-chattr-enotty

tmpfiles: downgrade errors when a file system does not support file a…

Merge pull request #673 from poettering/dns-packet-append-type-window

resolved: make sure we alway initialize *start in dns_packet_append_t…

basic: bitmap: use uint64_t instead if long long unsigned

long long unsigned is always 64 bit wide, so use a more readable type.

Merge pull request #672 from poettering/bitmap-isclear

bitmap: bitmap_clear()

networkd: ip6 tunnel add DSCP

This patch adds support for setting the
DSCP field in the ip6 tunnel.

when set it inherits DSCP field between inner and outer header.

resolved: make sure we alway initialize *start in dns_packet_append_type_window()

bitmap: bitmap_clear()

No need to actually reset the bitmap, we can just truncate it back zero
size. That not only makes bitmap_clear() quicker, but also subsequent
bitmap_isclear().

resolve: unify memdup() code when parsing RRs

Let's make dns_packet_read_public_key() more generic by renaming it to
dns_packet_read_memdup() (which more accurately describes what it
does...). Then, patch all cases where we memdup() RR data to use this
new call.

This specifically checks for zero-length objects, and handles them
gracefully. It will set zero length payload fields as a result.

Special care should be taken to ensure that any code using this call
can handle the returned allocated field to be NULL if the size is
specified as 0!

nspawn: Don't pass uid mount option for devpts

Mounting devpts with a uid breaks pty allocation with recent glibc
versions, which expect that the kernel will set the correct owner for
user-allocated ptys.

The kernel seems to be smart enough to use the correct uid for root when
we switch to a user namespace.

This resolves #337.

Merge pull request #667 from poettering/dns-rr-memleak

resolve: fix two minor memory leaks

Merge pull request #666 from poettering/drop-good-dns-server

resolve: drop dns_scope_good_dns_server()

resolve: fix two minor memory leaks

strv_extend() already strdup()s internally, no need to to this twice.
(Also, was missing OOM check...).

Use strv_consume() when we already have a string allocated whose
ownership we want to pass to the strv.

This fixes 50f1e641a93cacfc693b0c3d300bee5df0c8c460.

resolve: drop dns_scope_good_dns_server()

It's not used anymore since 29815b6c608b836cada5e349d06a96b63eaa65f3,
hence let's remove it from the sources.