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:/"

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>

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

Add README.md

A README.md allows us to include a badge for the current build status of
Semaphore CI. Keep other information in this file minimal, and avoid
information duplication but point readers to the official README.

bus: don't force send-masks on kdbus buses

Right now we always pass KDBUS_ITEM_ATTACH_FLAGS_RECV to
KDBUS_CMD_BUS_MAKE, effectively forcing every bus connection to do the
same during KDBUS_CMD_HELLO. This used to be a workaround to make sure all
metadata is always present. However, we refrained from that approach and
intend to make all metadata collection solely rely on /proc access
restrictions. Therefore, there is no need to force the send-flags mask on
newly created buses.

kmod-setup: don't warn when ipv6 can't be loaded

Not having IPv6 is a valid setup. Let's not print a warning in that
case.

Addresses:

  https://bugs.freedesktop.org/show_bug.cgi?id=87475

logind: Fix user_elect_display() to be more stable

The previous implementation of user_elect_display() could easily end up
overwriting the user’s valid graphical session with a new TTY session.
For example, consider the situation where there is one session:
   c1, type = SESSION_X11, !stopping, class = SESSION_USER
it is initially elected as the user’s display (i.e. u->display = c1).

If another session is started, on a different VT, the sessions_by_user
list becomes:
   c1, type = SESSION_X11, !stopping, class = SESSION_USER
   c2, type = SESSION_TTY, !stopping, class = SESSION_USER

In the previous code, graphical = c1 and text = c2, as expected.
However, neither graphical nor text fulfil the conditions for setting
u->display = graphical (because neither is better than u->display), so
the code falls through to check the text variable. The conditions for
this match, as u->display->type != SESSION_TTY (it’s actually
SESSION_X11). Hence u->display is set to c2, which is incorrect, because
session c1 is still valid.

Refactor user_elect_display() to use a more explicit filter and
pre-order comparison over the sessions. This can be demonstrated to be
stable and only ever ‘upgrade’ the session to a more graphical one.

https://bugs.freedesktop.org/show_bug.cgi?id=90769

Merge pull request #63 from arvidjaar/issue/50

fstab-generator: cescape device name in root-fsck service

Merge pull request #76 from crrodriguez/master

core: Remove "old kernel" warning if PR_SET_CHILD_SUBREAPER fails

core: Remove "old kernel" warning if PR_SET_CHILD_SUBREAPER fails

This made sense when systemd ran on older kernels, nowdays not so much.

Merge pull request #74 from systemd-mailing-devs/1432753344-31461-1-git-send-email-jonathan.boulle@coreos.com

fix extraneous space in equality check

bus: update kdbus.h

Sync with upstream.

Merge pull request #72 from teg/event-fork-unref

sd-event: don't touch fd's accross forks

Merge pull request #73 from zonque/mountinfo

core/mount: skip incomplete mountinfo entries

sd-event: don't touch fd's accross forks

We protect most of the API from use accross forks, but we still allow both
sd_event and sd_event_source objects to be unref'ed. This would cause
problems as it would unregister sources from the underlying eventfd, hence
also affecting the original instance in the parent process.

This fixes the issue by not touching the fds on unref when done accross a fork,
but still free the memory.

This fixes a regression introduced by
        "udevd: move main-loop to sd-event": 693d371d30fee

where the worker processes were disabling the inotify event source in the
main daemon.

core/mount: skip incomplete mountinfo entries

Skip /proc/mountinfo entries for which libmount returns a NULL pointer
for 'source' or 'target'. This happened on Semaphore CI's build servers
when the test suite is run.

Merge pull request #71 from kloun/patch-1

small fix ru translation

Merge pull request #58 from pwithnall/wip/pwithnall/user-active-on-vt-switch

logind: Save the user’s state when a session enters SESSION_ACTIVE

small fix ru translation

Merge pull request #57 from pwithnall/wip/pwithnall/udev-virtualbox-rules

logind: Add a udev rule to tag all DRM cards with master-of-seat