build-sys: split internal basic/ library from shared/

basic/      can be used by everything
            cannot use anything outside of basic/

libsystemd/ can use basic/
            cannot use shared/

shared/     can use libsystemd/

Merge pull request #3 from threatgrid/more_cgtop_enhancements

cgtop enhancements for easier machine-readable output

cgtop: IO readings are valid if any data is available, even if unchanged since last tick

Emit "0" rather than "-" if no change in IO values are seen for a process since
last tick, so long as accounting has registered content at all.

Merge pull request #6 from xnox/drop-name

shared: Drop 'name=' prefix from SYSTEMD_CGROUP_CONTROLLER define.

Merge pull request #152 from zonque/buildsys

build-sys: make man/systemd.directives.xml depend on man/custom-entit…

Merge pull request #148 from teg/sd-network-race

sd-network: allow the state dir to be created after the monitor

Merge pull request #151 from mischief/dns-shared-3

move dns code from resolve to shared v3

build-sys: make man/systemd.directives.xml depend on man/custom-entities.ent

Currently, the following command sequence fails:

  make distclean
  ./autogen.sh c
  make distcheck

That's because the command invoked to build man/systemd.directives.xml needs
man/custom-entities.ent to function, which itself isn't a dependency.

The $(filter-out $<,$^) logic used to filter out everything from the
prerequisites except for the first word, which doesn't work anymore
now. Use $(SOURCE_XML_FILES) instead.

Include tools/compile-unifont.py in the source tarball

This prevents a build failure when /usr/share/fonts/unifont/unifont.hex
is newer than unifont-glyph-array.bin.

bus: we now support path_namespace=/

Our bloom-filters support root-path matching. Make sure we properly add
the path_namespace= tag.

bus: fix pattern matching

DBus-spec defines two different pattern matchings:

  1) Path and namespace prefix matching. In this case, A matches B either
     if both are equal, or if B is fully included in the namespace of A.
     In other words, A has to be a prefix of B, but end with a separator
     character (or the following character in B must be one).

     This is used for path_namespace= and arg0namespace=

  2) The other pattern matching is used for arg0path= which does a two-way
     matching. That is, A must be a prefix of B, or B a prefix of A.
     Furthermore, the prefix must end with a separator.

Fix the sd-bus helpers to reflect that. The 'simple_' and 'complex_'
prefixes don't make any sense now, but.. eh..

bus: fix test-bus-kerne-bloom.c to match properly

Make sure we actually verify our match-rules are executed properly. Right
now all we test is the bloom-matches, which are non-reliable as they leave
through false-positives.

bus: fix arg0path= two-way matching

DBus spec clearly defines arg0path= to be a two-way matching. That is,
either the matcher or the matchee can be a prefix of the other to match.
This is not possible to implement with bloom-filters. Instead, we'd have
to add a separate filter for each prefix. This is non-trivial, though.
Hence, just skip the match for now and match locally.

bus: fix bloom_add_prefixes() to add all required data

Lets look at an example where we add arg0="/foo/bar/waldo" to a
bloom-filter. The following strings are added:
  "arg0:/foo/bar/waldo"
  "arg0-slash-prefix:/foo/bar"
  "arg0-slash-prefix:/foo"

Two problems arise:

1) If we match on "arg0path=/foo/bar/waldo", the dbus-spec explicitly
states that equal strings are also considered prefixes. However, in the
bloom-match, we can only provide a single match-filter. Therefore, we have
to add "arg0-slash-prefix:/foo/bar/waldo" there, but this never occured in
the bloom-mask of the message.
Hence, this patch makes sure bloom_add_prefixes() adds the full path as
prefix, too.

2) If we match on "arg0path=/foo/", the dbus-spec states that arg0path
does prefix-matching with the trailing slash _included_, unlike
path_namespace= matches, which does *not* include them. This is
inconsistent, but we have to support the specs. Therefore, we must add
prefixes with _and_ without trailing separators.
Hence, this patch makes sure bloom_add_prefixes() adds all prefixes with
the trailing slash included.

The final set of strings added therefore is:
  "arg0:/foo/bar/waldo"
  "arg0-slash-prefix:/foo/bar/waldo"
  "arg0-slash-prefix:/foo/bar/"
  "arg0-slash-prefix:/foo/bar"
  "arg0-slash-prefix:/foo/"
  "arg0-slash-prefix:/foo"
  "arg0-slash-prefix:/"

libsystemd-network: use domain validation instead of hostname validation for dhcp domain option

previously hostname_is_valid was used to validate domain names, which
would silently drop perfectly valid dns names that were longer than a
single dns label.

shared: add convenience function for validating dns names

resolve: move dns routines into shared

sd-network: allow the state dir to be created after the monitor

We now listen for new subdirs of /run/systemd, and /run/systemd/netif in case
/run/systemd/netif/links does not exist.

Merge pull request #147 from poettering/cmsg

util: introduce CMSG_FOREACH() macro and make use of it everywhere

util: introduce CMSG_FOREACH() macro and make use of it everywhere

It's only marginally shorter then the usual for() loop, but certainly
more readable.

Merge pull request #146 from arvidjaar/pr/udev-import-program-exit-code-man

man: clarify that IMPORT{program} is done only for zero exit code

Fix typo

Follow up for 7c918141ed.

man: clarify that IMPORT{program} is done only for zero exit code

Merge pull request #142 from teg/sd-network-unref-NULL

sd-network: allow NULL in sd_network_monitor_unref

Merge pull request #140 from teg/man-udev-timeout

man: udevd - correct default event timeout

sd-network: allow NULL in sd_network_monitor_unref

Match rest of codebase, we always allow unref'ing NULL.

Merge pull request #85 from keszybz/selinux-context

test-copy: test copy_bytes()

man: udevd - correct default event timeout

This was changed from 30 to 180 seconds quite some time ago.

sd-bus: remove ucred parameter from bus_message_from_header() since we don't use it anymore

sd-bus: fix early exit when we lack all data in bus_get_owner_creds_dbus1()

build-sys: upgrade shadow variable warnings to errors

bus-message: remove shadow warning with log_debug_bus_message()

journald: simplify context handling

By using our homegrown function we can dispense with all the iffdefery.

sd-bus: store selinux context at connection time

This appears to be the right time to do it for SOCK_STREAM
unix sockets.

Also: condition bus_get_owner_creds_dbus1 was reversed. Split
it out to a separate variable for clarity and fix.

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

Merge pull request #132 from ssahani/bond

networkd: bond  improve logging

Merge pull request #138 from utezduyar/use-async-convenience-function-on-setting-hostname

networkd: use async convenience call to set hostname

Merge pull request #133 from ssahani/net

networkd: vxlan improve logging

sd-bus: do not use per-datagram auxiliary information

SELinux information cannot be retrieved this way, since we are
using stream unix sockets and SCM_SECURITY does not work for
them.

SCM_CREDENTIALS use dropped to be consistent. We also should
get this information at connection time.

https://bugzilla.redhat.com/show_bug.cgi?id=1224211
"SCM_SECURITY was only added for datagram sockets."

Merge pull request #137 from torstehu/correct-typo

sd-bus: Correct typo

networkd: use async convenience call to set hostname

sd-bus: Correct typo

Merge pull request #136 from dslul/master

keymap: Add Samsung NP350V and NP670Z

keymap: Add Samsung NP350V and NP670Z

typo

keymap: Add Samsung NP350V and NP670Z

Merge pull request #117 from utezduyar/feat/dump-sync-dbus-message-with-logging-on

sd-bus: dump sync messages in debug mode

Merge pull request #135 from zonque/buildsys

build-sys: turn some warnings into errors

build-sys: turn some warnings into errors

Make the build sys error out on missing function prototypes, missing
variable declarations, implicit function declarations or forgotten return
statements.

None of these conditions are acceptable, and by making them hard errors, the
build bots can detect them earlier.

Merge pull request #130 from whot/hwdb-updates

Hwdb updates

sd-bus: dump sync messages in debug mode

network: veth imprve logging

Replaces a lof of strerror() usage with log_netdev_error_errno()

networkd: vxlan improve logging

Replaces a lof of strerror() usage with log_netdev_error_errno()

networkd: bond  improve logging

Replaces a lof of strerror() usage with log_netdev_error_errno()

hwdb: update Logitech's unifying receiver devices

Since 3.19, the devices have the proper vid/pid and the model number in the
name.

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

cgtop: more sensible flushing behavior w/ non-TTY output

- Explicitly flush stdout before sleep between iterations
- Only clear user keystrokes when output is to TTY
- Add a newline between output batches when output is not to TTY

cgtop: allow user to force looping behavior even in non-TTY mode

cgtop: raw output option (disable conversion to human-readable units)

logind,sd-event: drop spurious new-lines

tree-wide: whenever we fork off a foreign child process reset signal mask/handlers

Also, when the child is potentially long-running make sure to set a
death signal.

Also, ignore the result of the reset operations explicitly by casting
them to (void).

signal-util: modernize and share more code

Merge pull request #126 from phomes/master

core: log oom during killing spree

core: log oom during killing spree

but don't do anything else. We still want to kill as much as
possible.

Coverity CID#996306

Merge pull request #122 from martinpitt/master

path-util: Fix path_is_mount_point for parent mount points in symlink mode

path-util: Fix path_is_mount_point for parent mount points in symlink mode

When we have a structure like this:

  /bin -> /usr/bin
  /usr is a mount point

Then path_is_mount_point("/bin", AT_SYMLINK_FOLLOW) needs to look at the pair
/usr/bin and /usr, not at the pair / and /usr/bin, as the latter have different
mount IDs. But we only want to consider the base name, not any parent.

Thus we have to resolve the given path first to get the real parent when
allowing symlinks.

Bug: https://github.com/systemd/systemd/issues/61

bus-creds: always set SD_BUS_CREDS_PID when we set pid in the mask

Also reorder the code a bit to be easier to parse.

Merge pull request #121 from martinpitt/master

build-sys: always dist *.policy.in files

build-sys: always dist *.policy.in files

Unconditionally dist org.freedesktop.{import1,machine1}.policy.in, like all the
other *.policy.in files. This avoids missing policy files in the tarball.

Spotted by "make distcheck" failure with --disable-importd.

Merge pull request #111 from phomes/cleanup_warning_suppression

build-sys: don't suppress irrelevant warnings

Merge pull request #120 from systemd/revert-115-master

Revert "hwdb: actually search /run/udev/hwdb.d"

Revert "hwdb: actually search /run/udev/hwdb.d"

Merge pull request #118 from haraldh/set_consume2

util:bind_remount_recursive() fix "use after free" - 2

Merge pull request #77 from haraldh/cryptsetup

cryptsetup: craft a unique ID with the source device

Merge pull request #116 from utezduyar/feat/async-api-for-method-call

sd-bus: add async convenience method call API

Merge pull request #113 from mezcalero/address-family

networkd: actually always use AddressFamilyBoolean as the bit mask it is

util:bind_remount_recursive(): handle return 0 of set_consume()

set_consume() does not return -EEXIST, but 0, in case the key is already
in the Set.

Revert "util:bind_remount_recursive() fix "use after free""

This reverts commit 46be6129d3e52556eb0f2ae4d07818f9f3f7af7a.

cryptsetup: craft a unique ID with the source device

If cryptsetup is called with a source device as argv[3], then craft the
ID for the password agent with a unique device path.

If possible "/dev/block/<maj>:<min>" is used, otherwise the original
argv[3] is used.

This enables password agents like petera [1] to provide a password
according to the source device. The original ID did not carry enough
information and was more targeted for a human readable string, which
is specified in the "Message" field anyway.

With this patch the ID of the ask.XXX ini file looks like this:
ID=cryptsetup:/dev/block/<maj>:<min>

[1] https://github.com/npmccallum/petera

sd-bus: add async convenience method call API

Merge pull request #115 from whot/master

hwdb: actually search /run/udev/hwdb.d

Merge pull request #114 from crawford/xen

rules: whitelist xvd* devices

hwdb: add Logitech TrackMan Marble Wheel USB

DPI is guesswork, no specs found on the web and calculating DPIs on a
trackball is tedious.

hwdb: actually search /run/udev/hwdb.d

The documentation claims hwdb entries may be placed in the volatile
/run/udev/hwdb.d directory but nothing actually looked at it.

rules: whitelist xvd* devices

Xen disks need to be whitelisted as well.

networkd: actually always use AddressFamilyBoolean as the bit mask it is

build-sys: don't suppress irrelevant warnings

We do not trigger these warnings so no need to suppress them.

Merge pull request #108 from phomes/master

tree-wide: remove spurious space

Merge pull request #110 from systemd/revert-41-ipforwarding

Revert "networkd: create "kernel" setting for IPForwarding"

Revert "networkd: create "kernel" setting for IPForwarding"

tree-wide: remove spurious space

Merge pull request #106 from teg/man-mac-random

man: systemd.link - explain random MAC addresses

man: systemd.link - explain random MAC addresses

Two of the bits in the MAC address are set unconditioanlly, and the rest is randomized,
make this clear in the documentation (as it currently read as if it was all random).

Merge pull request #101 from zonque/kmod

kmod-setup: split warn flags

kmod-setup: split warn flags

Traditionally, we used to warn about ipv6 being a module or being
unavailable. This was changed in b4aa82f16 ("kmod-setup: don't warn
when ipv6 can't be loaded") in a way that neither of the two conditions
will cause a log message.

Now, while running a setup without any IPv6 is completely fine and
shouldn't cause any warning, we should still warn about ipv6 being a
module instead of built-in.

To achieve this, split the boolean warn flag into two: one for a
feature not being built-in but shipped as a module, and one to
print an error when a module is entirely unavailable.

We will, however, still warn if kmod returns anything else than
-ENOENT in the attempt of loading the module, and at the very least,
turn the message into a debug log.

Merge pull request #96 from haraldh/set_consume

util:bind_remount_recursive() fix "use after free"

util:bind_remount_recursive() fix "use after free"

set_consume(done, x) consumes x with free(x)
but
mount(…, x, …) uses it afterwards.

coverity CID 1299006

Merge pull request #92 from zonque/README

README.md: add Coverity scan status badge

README.md: add Coverity scan status badge

Merge pull request #79 from zonque/fdo-87475

kmod-setup: don't warn when ipv6 can't be loaded (FDO bug #87475)

Merge pull request #82 from gmacario/bootchart-fix-typo

systemd-bootchart: Trivial typo fix in warning

systemd-bootchart: Trivial typo fix in warning

Signed-off-by: Gianpaolo Macario <gmacario@gmail.com>

Merge pull request #80 from zonque/README.md

Add README.md