core: simplify fd collection code, return number of fds as return value

Let's simplify the fd collection code a bit, and return the number of
collected fds as positive integer, the way it's customary in our usual
code.

socket: don't double close the same fd for FIFOs

gpt-auto-generator: fix duplicate backing partition check

We shouldn't exit the loop early, otherwise our duplicate backing
partition check won't work.

Merge pull request #1462 from lnykryn/analyze-manpage

man: LEVEL in systemd-analyze set-log level is not optional

Merge pull request #1406 from blaskovic/journal-remote-typo

journal-remote: typo in log_error when no sources are specified

[tomegun: this was a pun, but let's not do that]

Merge pull request #1448 from bengal/lldp-export-v5

LLDP: add API to export neighbors list (v5)

Merge pull request #1459 from ssahani/bridge1

networkd: add bridge properties

Merge pull request #1372 from jemk/prefsrc

networkd: add support to configure preferred source of static routes

man: LEVEL in systemd-analyze set-log level is not optional

rhbz#1268336

Merge pull request #1461 from Danielmachon/patch-1

Update NEWS

Update NEWS

Fixed spelling error: seriban -> serbian

NEWS update

man: add man for bridge params

networkd: add bridge properties

ForwardDelaySec:   forward delay
HelloTimeSec:      hello time
MaxAgeSec:         maximum message age

for more information see
http://www.tldp.org/HOWTO/BRIDGE-STP-HOWTO/set-up-the-bridge.html

In kernel
br_dev_newlink: does not have the this functionality to set while
creation.
br_changelink: after creation we can change the parameters.

we need to first create then set it the parameters.

Introduce new callback post_create .This should
set the properties after the creation.

sd-rtnl: introduce new API to set the NL header flags

By default we set as NLM_F_CREATE | NLM_F_EXCL in
sd_rtnl_message_new_link
But incase of bridge we need to set NLM_F_REQUEST | NLM_F_ACK.

If NLM_F_EXCL is set then we are unable to set the parameters. As bridge
supports setting properties after creation not during creation.

sd-rtnl: sd-netlink: add support for bridge NL parametrs

Rename rtnl_link_info_data_bridge_types to
rtnl_link_bridge_management_types
as they are of nested types of IFLA_AF_SPEC.

missing.h : add bridge params

Merge pull request #1457 from martinpitt/master

keymap: Add Thinkpad Yoga 12 (2015)

Merge pull request #1454 from kostich/master

po, catalog: added Serbian translation.

keymap: Add Thinkpad Yoga 12 (2015)

Fixes #1440

po, catalog: added Serbian translation.

Merge pull request #1450 from evverx/man-journal-fields

man: systemd.journal-fields: add info about _TRANSPORT=audit

man: systemd.journal-fields: add info about _TRANSPORT=audit

lldp: add reception tests

Add some tests to simulate the reception of LLDP frames and to verify
the correctness of the data in the MIB.

lldp: fix parsing of TLV length

tlv_packet_read_bytes() and tlv_packet_read_string() returned the
wrong length when called after other functions which modify the offset
in the container.

In other words, if the TLV data length is X and we do a
tlv_packet_read_u8(), a subsequent tlv_packet_read_bytes() should
return a length of (X - 1).

lldp: move lldp_receive_packet() to lldp-internal.c

In order to implement tests for the LLDP state machine, we need to
mock lldp_network_bind_raw_socket(). Move the other function
lldp_receive_packet() to another file so that we can replace the first
function with a custom one and keep the second one.

lldp: check return value of lldp_tlv_packet_exit_container()

lldp: add support for organizationally specific TLVs

LLDP TLVs of type 127 are used to carry organizationally specific
information and include additional fields to specify the OUI and
subtype.

Add support for parsing such fields and functions to access the most
common IEEE 802.1 specific TLVs.

lldp: factor out common code in lldp-tlv.c

lldp: add sd_lldp_tlv_packet_get_destination_type()

It can be useful to know the destination address of a LLDP frame
because it determines the scope of propagation of the frame and thus
this information be used to know whether the neighbor is connected to
the same physical link.

See clause 7.1 of IEEE Std 802.1AB-2009.

lldp: add public function to export LLDP TLV packets

Add a public function to get a list of current LLDP neighbours' TLV
packets. The function populates an array of pointers to the opaque
type sd_lldp_packet and returns the number of elements found. Callers
must take care of freeing the array and decreasing the refcount of
elements when done.

lldp: move TLV related functions to lldp-tlv.c

Move some public functions from lldp-internal.c to lldp-tlv.c, as now
they are not internal functions anymore.

lldp: export opaque TLV type and accessor functions

Export struct tlv_packet as a public opaque sd_lldp_packet type and
make its accessor functions public.

lldp: change order of arguments of lldp_read_*() functions

These functions are going to be exported, swap the 'data' and 'length'
arguments so that their signature is consistent with the rest of the
code.

lldp: add reference counter to struct tlv_packet

Add a reference counter to the tlv_packet structure so that it can be
shared between multiple users and properly free'd when no longer in
use.

Merge pull request #1439 from poettering/CODING_STYLE

Coding style

logind: remove warn_melody() logic

The internal speaker is usually not available on modern latops that
support suspend, and even if it is available in the hardware, most
distributions turned support for it off in the kernel. And even if it is
enabled, it's probably still a bad idea to make use of it for the
suspend-failures. If anything a proper sound should be played.

Long story short, let's remove support of this anachronism.

update CODING_STYLE with various additions

Merge pull request #1428 from franciozzy/tagenhance

Enhance matching of TAG keys

Merge pull request #1438 from poettering/rfkill-rework

An rfkill rework among other things

Merge pull request #1430 from evverx/driver-syslog

journald: add syslog fields for driver messages

update NEWS

rfkill: rework and make it listen on /dev/rfkill

With this rework we introduce systemd-rfkill.service as singleton that
is activated via systemd-rfkill.socket that listens on /dev/rfkill. That
way, we get notified each time a new rfkill device shows up or changes
state, in which case we restore and save its current setting to disk.

This is nicer than the previous logic, as this means we save/restore
state even of rfkill devices that are around only intermittently, and
save/restore the state even if the system is shutdown abruptly instead
of cleanly.

This implements what I suggested in #1019 and obsoletes it.

core: add new setting Writable= to ListenSpecial= socket units

Writable= is a new boolean setting. If ture, then ListenSpecial= will
open the specified path in O_RDWR mode, rather than just O_RDONLY.

This is useful for implementing services like rfkill, where /dev/rfkill
is more useful when opened in write mode, if we want to not only save
but also restore its state.

systemctl: fix how we decode types and states

There was a bad memory access among other smaller issues.

man: minor wording improvements to USB FunctionFS documentation

fix typo in NEWS

Merge pull request #1434 from reverendhomer/patch-1

systemctl: fix memory leak in systemctl_parse_argv

Merge pull request #1433 from michich/log-errno

tree-wide: add missing errno arguments to log_*_errno()

udev: fix format string after conversion to log_*_errno()

Use %m where previously %s was used together with strerrno().

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

sd-device: fix format strings after conversion to log_*_errno()

Use %m where previously %s was used together with strerrno().

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

systemctl: fix memory leak in systemctl_parse_argv

This commit fixes Coverity #1325228

tree-wide: add missing errno arguments to log_*_errno()

A few of the recent conversions to log_*_errno() were missing the errno
value arguments.

Fixes: e53fc357a9b "tree-wide: remove a number of invocations of
       strerror() and replace by %m"

Merge pull request #1429 from poettering/dhcp-hostname

DHCP and journal remoting fix, as well as NEWS update

Merge pull request #1427 from dbuch/master

sd-pppoe: use strna() instead of strempty()

Merge pull request #1426 from poettering/log-syntax

logging fixes and more

journald: add syslog fields for driver messages

journal: make sure to set MHD_USE_PIPE_FOR_SHUTDOWN for libmicrohttpd servers

This makes sure libmicrohttpd won't call shutdown() on our listening
sockets, which make sure socket activation and re-activation will work
cleanly.

See:

https://github.com/systemd/systemd/pull/1286
https://lists.gnu.org/archive/html/libmicrohttpd/2015-09/msg00014.html

Fixes #1286

dhcp: make sure we can deal with a single trailing NUL byte in the hostname

Fixes #1337

update NEWS

sd-pppoe: use strna() instead of strempty()

In cases where we use for instance. log_debug() with a statement + string
it is better to output (Service-name: N/A) instead of (Service-name: )

Support OP_NOMATCH for TAG key

The TAG key can be used in rules for event matching. At the moment, it
does not support inequality tests. This patch enhances the key test to
validate the rule if it does not contain a given TAG (by TAG!="value").

Signed-off-by: Felipe Franciosi <felipe@paradoxo.org>

Merge pull request #1425 from kaysievers/wip

gpt-auto-generator: check fstab for /boot entries

Merge pull request #1422 from keszybz/sd-daemon-mq-badf

sd-daemon: fix return value for sd_is_mq

Merge pull request #1419 from keszybz/shell-completion

Shell completion tweaks

core: make sure we expose the empty string when we detect no virtualization

On the bus, stick to exposing the empty string as "no virtualization",
instead of none.

Fixes #1423

tree-wide: remove a number of invocations of strerror() and replace by %m

Let's clean up our tree a bit, and reduce invocations of the
thread-unsafe strerror() by replacing it with printf()'s %m specifier.

basic: split out cpu set specific APIs into cpu-set-util.[ch]

conf-parser: make use of DEFINE_PARSER macro to parse mode_t

Let's unify more code.

Also, rework the macro to accept a trailing semicolon, to make the code
prettier.

util: there cannot be trailing garbage when parsing cpu sets

extract_first() already skips trailing whitespace, hence no reason to
explicitly check for it.

util: rename parse_cpu_set() to parse_cpu_set_and_warn()

It's pretty untypical for our parsing functions to log on their own.
Clarify in the name that this one does.

log: move log_invalid_utf8() to log.h

Also, make sure it follows the same scheme as log_syntax() does in its
behaviour.

tree-wide: clean up log_syntax() usage

- Rely everywhere that we use abs() on the error code passed in anyway,
  thus don't need to explicitly negate what we pass in

- Never attach synthetic error number information to log messages. Only
  log about errors we *receive* with the error number we got there,
  don't log any synthetic error, that don#t even propagate, but just eat
  up.

- Be more careful with attaching exactly the error we get, instead of
  errno or unrelated errors randomly.

- Fix one occasion where the error number and line number got swapped.

- Make sure we never tape over OOM issues, or inability to resolve
  specifiers

Merge pull request #1424 from evverx/journalctl-rotate

journalctl: add --rotate option

gpt-auto-generator: check fstab for /boot entries

We need to prevent the creation of the gpt automount unit, which will not
get overridden by the fstab mount unit.

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

man: journalctl: add info about --rotate

shell-completion: journalctl: add --rotate

journalctl: add --rotate option

shortcut for `systemctl kill --kill-who main --signal SIGUSR2 systemd-journald`

sd-daemon: fix sd_is_mq for non-mq fds

mq_getattr returns -1/EBADF for file descriptors which are not mq.
But we should return 0 in this case.

We first check that fd is a valid fd, so we can assume that if
mq_getattr returns EBADF, it is simply a non-mq fd. There is a slight
race, but there doesn't seem to be a nice way to fix it.

Merge pull request #1421 from keszybz/formatting-fixes

logind: avoid a dot before parenthesis

Merge pull request #1420 from keszybz/doc-updates

Doc updates

systemctl: do not set variable to be immediately overwritten

NEWS: typos

man: update description of exit.target

exit.target is now used for both system and user sessions,
so remove "on user service manager exit". Also reword that
paragraph: services will be killed before the manager exits,
even if they do not conflict with shutdown target, but we
recommend that they conflict with shutdown target so that
systemd schedules them to be stopped immediately when starting
to exit.

In the first paragraph, containers should be mentioned last,
and the more general systems first.

logind: avoid a dot before parenthesis

systemd-logind[27]: System is rebooting. (Applied kernel updates.)

is changed to

systemd-logind[27]: System is rebooting (Applied kernel updates).

Users should not add a dot in the sentence in --message, i.e. the correct usage is now:
$ systemctl reboot --message "Applied kernel updates"

man: add link to kernel docs for net_cls

man: use "=" when referring to configuration file settings

This convention is almost universal in systemd man pages, and makes
it easier to visually parse the docs.

Also fix some markup along the way.

man: add link to functionfs docs

Also reword some ungainly grammar constructs.

Merge pull request #1415 from aakoskin/bootchart-v2

[v2] bootchart: don't fail if "model name" is missing from cpuinfo

Merge pull request #1418 from poettering/base-fs

base-fs: consider inability to create /root non-fatal

Merge pull request #1417 from poettering/nspawn-and-more

Various fixes, primarily for nspawn, but other stuff too

base-fs: consider inability to create /root non-fatal

Fixes #1414

Merge pull request #1394 from steelman/fsck-force-reboot

fsck: start reboot.target irreversibly

analyze: add new set-log-target subcommand

We already have the property writable, hence let's add a command to set
it.

log: properly return -EINVAL from log_set_max_level_from_string()

If we just return the value we got from log_level_from_string() on
failure we'll return -1, which is not a proper error code.

log_set_target_from_string() did get this right already, hence let's fix
this here too.

nspawn: mount /sys as tmpfs, and then mount only select subdirs of the real sysfs below it

This way we can hide things like /sys/firmware or /sys/hypervisor from
the container, while keeping the device tree around.

While this is a security benefit in itself it also allows us to fix
issue #1277.

Previously we'd mount /sys before creating the user namespace, in order
to be able to mount /sys/fs/cgroup/* beneath it (which resides in it),
which we can only mount outside of the user namespace. To ensure that
the user namespace owns the network namespace we'd set up the network
namespace at the same time as the user namespace. Thus, we'd still see
the /sys/class/net/ from the originating network namespace, even though
we are in our own network namespace now. With this patch, /sys is
mounted before transitioning into the user namespace as tmpfs, so that
we can also mount /sys/fs/cgroup/* into it this early. The directories
such as /sys/class/ are then later added in from the real sysfs from
inside the network and user namespace so that they actually show whatis
available in it.

Fixes #1277

bootchart: use get_proc_field and PROC_CPUINFO_MODEL

Use get_proc_field and PROC_CPUINFO_MODEL to get the CPU type. Also don't
fail if it missing, instead restore the old behaviour and print "Unknown".

Fixes the following error seen with v225 on powerpc:

Unable to read module name from cpuinfo.
Error generating svg file: No such file or directory

architecture.h: define PROC_CPUINFO_MODEL

Define PROC_CPUINFO_MODEL to indicate the CPU model field in /proc/cpuinfo.

fileio: make get_status_field() more generic

All users of get_status_field() expect the field pattern to occur in
the beginning of a line, and the delimiter is ':'.

Hardcode this into the function, and also skip any whitespace before ':'
to support fields in files like /proc/cpuinfo. Add support for returning
the full field value (currently stops on first whitespace).

Rename the function so it's easier to ensure all callers switch to new
semantics.

fix typos in NEWS