tests: move fileio related tests to test-fileio.c

tests: move stat-util related tests to test-stat-util.c

tests: move signal-util related tests to test-signal-util.c

tests: move cpu-set-util related tests to test-cpu-set-util.c

tests: move web-util related tests to test-web-util.c

tests: move alloc related tests to test-alloc-util.c

tests: move escape related tests to test-escape.c

tests: move strv related tests to test-strv.c

tests: move string related tests to test-string-util.c

tests: move hexdecoct tests to test-hexdecoct.c

Merge pull request #2790 from ronnychevalier/rc/systemctl_missing_oom_check

systemctl: add missing OOM check

systemctl: add missing OOM check

Merge pull request #2781 from keszybz/selinux-and-warnings

Selinux loading fix and warning cleanups

Merge pull request #2787 from ronnychevalier/rc/systemctl_edit_rm_empty_dropin_dirs

systemctl: rm empty dropin dirs after cancelled edit

systemctl: rm empty dropin dirs after cancelled edit

Fixes #2734

Redefine 32bit time_t format to signed

It seems that it is signed both on i386 and arm.

Avoids a stupid gcc warning on arm:

src/udev/udevadm-monitor.c: In function ‘print_device’:
src/udev/udevadm-monitor.c:44:16: warning: format ‘%u’ expects argument of type ‘unsigned int’, but argument 3 has type ‘__time_t {aka long int}’ [-Wformat=]
         printf("%-6s[%"PRI_TIME".%06ld] %-8s %s (%s)\n",
                ^

test-sizeof: add a helper which prints variable sizes and signedness

This helps to understand misleading gcc warnings about type mismatches.

test-selinux: add some simple tests which call functions and print the results and timings

selinux: use raw variants of security_compute_create and setfscreatecon

As suggested by Evgeny Vereshchagin as a follow up for
https://github.com/systemd/systemd/pull/2781#issuecomment-191043402.

Merge pull request #2779 from 0xAX/openat-opath-excess-flags

tree-wide: no need to pass excess flags to open()/openat() if O_PATH …

Merge pull request #2782 from AOSC-Dev/master

po/zh_CN: attempt to refresh zh_CN translation

po/zh_CN: attempt to refresh zh_CN translation

selinux: always try to load the full selinux db

https://github.com/systemd/systemd/pull/2508#issuecomment-190901170
Maybe fixes https://bugzilla.redhat.com/show_bug.cgi?id=1308771.

Merge pull request #2754 from eworm-de/ask-password

ask-password: do no print password when storing in keyring

Merge pull request #2777 from martinpitt/keymap

keymap fixes

tree-wide: no need to pass excess flags to open()/openat() if O_PATH is passed

As described in the documentation:

When O_PATH is specified in flags, flag bits other than O_CLOEXEC,
O_DIRECTORY, and O_NOFOLLOW are ignored.

So, we can remove unnecessary flags in a case when O_PATH is passed
to the open() or openat().

ask-password: add option --no-output to not print password to stdout

systemd-ask-password can store passwords in kernel keyring. However it
uses to print the passwords to standard output nevertheless. Depending
on where systemd-ask-password is called passwords may end on display
or in log, leaking sensitive information.
This allows to make systemd-ask-password quiet, effectively disabling
printing passwords to standard output.

keymap: Some HP refactoring, add Probook 440 G3 keys

Apparently the vendor string for newer models now uses "svnHP" instead of
"svnHewlett-Packard", so add this alternative match to the global HP map.

Generalize matches for micmute key on ProBook 4xx.

Silence two hardwired keys on the ProBook 440 G3 that should not produce evdev
events. Thanks to Hermann Kraus for those! (See PR #2679)

keymap: Add HP ZBook

https://launchpad.net/bugs/1535219

Merge pull request #2773 from eliasp/type-warnings

Use `PRIu64` to print `uint64_t` in log msgs

Merge pull request #2702 from poettering/resolved-iterate-fix

resolved iteration fix

Merge pull request #2767 from benjarobin/fix-warn

systemctl: Fix warn: action_to_runlevel() is not used

Use `PRIu64` to print `uint64_t` in log msgs

Merge pull request #2771 from 0xAX/core-kernel-timestamp-cleanup

core: use DUAL_TIMESTAMP_NULL to reset kernel_timestamp

Merge pull request #2769 from martinpitt/adjtime

Fixes for adjtime parsing

test-clock: fix FILE* leak

Close the FILE* instead of the underlying fd.

Merge pull request #2752 from eliasp/no-container-name-escaping

Don't escape the name of the container nspawn service

Merge pull request #2763 from tblume/shorten-hostname-before-removing-dots

shorten hostname before checking for trailing dot

Merge pull request #2756 from thom311/master

core: avoid compiler warning when compiling with -fexceptions

test-clock: fix assertions

Use assert_se() to ensure that they don't get optimized away.

timedated: trivial stylistic fix

Don't use {} for single-line "then" blocks.

Merge pull request #2766 from keszybz/coverity

test-clock: fix fd "leak"

systemctl: Fix warn: action_to_runlevel() is not used

If HAVE_SYSV_COMPAT is not defined: ‘action_to_runlevel’ defined but not used

test-clock: fix fd "leak"

CID #1352301.

core: use DUAL_TIMESTAMP_NULL to reset kernel_timestamp

instead of direct reset of kernel_timestamp fields.

Merge pull request #2764 from benjarobin/fix-stdio-bridge

stdio-bridge: Correctly propagate error

stdio-bridge: Correctly propagate error

Return EXIT_FAILURE and print the correct errno code if sd_bus_get_fd() or sd_bus_get_events() fail

shorten hostname before checking for trailing dot

Shortening can lead to a hostname that has a trailing dot.
Therefore it should be done before checking from trailing dots.

Merge pull request #2746 from daurnimator/patch-1

ISSUE_TEMPLATE: Use a list for submission type so that it appears as a checkbox

Merge pull request #2753 from phomes/trivial-fixes2

Trivial fixes

Merge pull request #2749 from martinpitt/adjtime

core: avoid compiler warning when compiling with -fexceptions

Initialize auto variables with cleanup attribute, otherwise we
get a compiler warning with -fexceptions.

    ./configure CFLAGS='-Wmaybe-uninitialized -fexceptions -O2'

stdio-bridge: remove dead code

tree-wide: indentation fixes

Don't escape the name of the container in instances of

When using `%I` for instances of `systemd-nspawn@.service`, the result
will be `systemd-nspawn` trying to launch a container named e.g.
`fedora/23` instead of `fedora-23`.
Using `%i` instead prevents escaping `-` in a container name and uses
the unmodified container name from the machine store.

timedated: be more tolerant in parsing /etc/adjtime

Similarly to the previous commit, make context_write_data_local_rtc()
understand /etc/adjtime files with just one or two lines, with or without a
final newline.

Normalize the file to the current definition in hwclock(8), in the spirit of
"be liberal what you accept and strict what you produce": Add line terminators,
and set the second line to "0" if missing.

Fixes: #2638

clock-util: be more tolerant in parsing /etc/adjtime

As we default to "hardware clock is in UTC" if /etc/adjtime is not present, it
also makes sense to have that default if /etc/adjtime contains only one or two
lines.

Drop the "gibberish" test case, as this was just EIO because of not containing
three lines, which is already contained in other tests. clock_is_localtime()
never actually validated the format of the first two lines, and there is little
point in doing that.

This addresses the reading half of issue #2638.

clock-util: make clock_is_localtime() testable and add initial tests

Add path argument to clock_is_localtime() and default to "/etc/adjtime" if it's
NULL. This makes the function testable.

Add test-clock: initial test cases for some scenarios, using a temporary file.
This also checks the behaviour with a NULL (i. e. the system's /etc/adjtime)
file.

.github/ISSUE_TEMPLATE: Use a list for submission type so that it appears as a checkbox

Merge pull request #2671 from 0xAX/move-pager-open-to-one-place

tree-wide: merge pager_open_if_enabled() to the pager_open()

tree-wide: merge pager_open_if_enabled() to the pager_open()

Many subsystems define own pager_open_if_enabled() function which
checks '--no-pager' command line argument and open pager depends
on its value. All implementations of pager_open_if_enabled() are
the same. Let's merger this function with pager_open() from the
shared/pager.c and remove pager_open_if_enabled() from all subsytems
to prevent code duplication.

Merge pull request #2743 from pfl/dhcp4_server

sd-dhcp-server: Send replies to BOOTP relay server port

sd-dhcp-server: Send replies to BOOTP relay server port

RFC 2131 Section 4.1 says that

 "If the ’giaddr’ field in a DHCP message from a client is non-zero,
  the server sends any return messages to the ’DHCP server’ port on the
  BOOTP relay agent whose address appears in ’giaddr’."

Fix this by adding a destination port when sending unicast UDP packets
and provide the server port when a BOOTP relay agent is being used.

Merge pull request #2722 from torstehu/fix-typo2

treewide: fix typos and then/that use

treewide: fix typos and then/that use

Merge pull request #2719 from evverx/add-test-to-makefile

tests: various fixes

Merge pull request #2726 from ssiloti/master

dnssec: only run tests that require gcrypt if we have it

Merge pull request #2727 from ian-kelling/man-pr-v3

man: clarify unit ordering language

man: clarify unit ordering language

dnssec: only run tests that require gcrypt if we have it

Merge pull request #2724 from vcaputo/minor-formatting-cleanups

tree-wide: minor formatting inconsistency cleanups

tree-wide: minor formatting inconsistency cleanups

Merge pull request #2639 from nafets227/master

Bugfix set virt=none when running on XEN Dom0

basic: Bugfix Detect XEN Dom0 as no virtualization

When running in XEN Dom0 the virtualization check:
1) detect_xen returns HYPERVISOR_NONE so next checks are executed
2) /proc/sys/hypervisor detects a XEN hypervisor
   it is lacking the special Dom0 detection as in detect_xen

With this patch, at the end of all virtualization checks we double-check if running in XEN Dom0 or DomU.

basic: Debug-logging of Virtualisation detection

print out every single detection executed and its result.

Merge pull request #2725 from phomes/misc-cleanup

Misc cleanup

networkctl: fix const

tree-wide: remove unused variables

basic: mark unused variable as such

Merge pull request #2664 from zonque/bootchart-removal

Remove systemd-bootchart

Merge pull request #2723 from poettering/importd-man

man: style fixes

man: change recommended order of NSS modules in /etc/nsswitch.conf

So far we recommended placing "nss-mymachines" after "nss-resolve" in the order
of preference in /etc/nsswitch.conf. This change reverse this order.

Rationale: single-label names are resolved via LLMNR by resolved, which has to
time out if no peer by that name exists. By placing "nss-mymachines" first
(which always responds immediately) we avoid running into this timeout for most
containers. Both modules should return the same data if LLMNR is used by the
container anyway.

While we are at it, improve the man pages of the three NSS modules in other
ways a bit.

tests: add exec-capabilityambientset* to Makefile.am

This is a follow-up to 70d7aea5c

tests: add exec-runtimedirectory* to Makefile.am

This is a follow-up to cc3ddc851

tests: fix x86 personality tests to only on x86

tests: add exec-spec-interpolation.service to Makefile.am

This is a follow-up to 2525499

Merge pull request #2708 from vcaputo/journal-restore-offline-state-on-error

journal: restore offline state on error

Merge pull request #2717 from keszybz/networkctl-prettification

Networkctl prettification

man: link some unit files to their online bus API documentation

man: add minimal man page for systemd-importd.service

networkctl: only print lldp legend for capabilities that were actually seen

basic: simplify ether_addr_is_null

Fix two typos

man: update link to Fedora Cloud images

Better to use the mirror manager and https.

architecture.h: remove PROC_CPUINFO_MODEL

This was only needed for bootchart, so it can go now.

Remove systemd-bootchart

This commit rips out systemd-bootchart. It will be given a new home, outside
of the systemd repository. The code itself isn't actually specific to
systemd and can be used without systemd even, so let's put it somewhere
else.

Merge pull request #2709 from vcaputo/normalize_inc_dec_usage

tree-wide: make ++/-- usage consistent WRT spacing

Merge pull request #2707 from teg/man-network

network: minor manpage clarifications

Merge pull request #2698 from poettering/personality

Personality fixes