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

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

Partially revert "ma-setup: simplify"

copy_bytes() tries to do the write in chunks, but ima kernel code
needs every rule to be written in one write. Writing the whole file
at once avoids the issue.

http://lists.freedesktop.org/archives/systemd-devel/2015-June/032623.html
http://sourceforge.net/p/linux-ima/mailman/message/34145236/
https://bugzilla.redhat.com/show_bug.cgi?id=1226948

Merge pull request #68 from whot/hwdb-updates

hwdb: add Apple MagicMouse entry

hwdb: add Apple MagicMouse entry

Merge pull request #59 from eswierk/master

Add /dev/xvd* to 60-persistent-storage whitelist

Merge pull request #65 from teg/enumerate-accept-null

libudev: enumerate - accept NULL parameters in add_match()

Merge pull request #52 from mbiebl/systemctl-edit-default-editor

systemctl: Use /usr/bin/editor if available

kdbus: remove attach_flags_mask module parameter setting

libudev: enumerate - accept NULL parameters in add_match()

This was a regression introduced when moving to sd-device.

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

We unescape ExecStart line when parsing it, so escape device name
before adding it to unit file.

fixes #50

test-util: fix a memleak

Merge pull request #60 from Keruspe/master

build-sys: drop references to gobject-introspection

Merge pull request #55 from filbranden/rootprefix_empty1

Fix --with-rootprefix= (empty) with a workaround for now.

build-sys: disable gc-sections if optimization is disabled

This way, development builds will not rely on gc-sections to
paper over cyclic link dependencies. Newly introduced broken
link requirements will immediatley fail.

build-sys: drop references to gobject-introspection

It has been removed alongside gudev

Add /dev/xvd* to 60-persistent-storage whitelist
Without this, systemd-udevd does not create persistent storage symlinks
for xen block devices.

test-unit-file.c: fixup the test for commit 3b51f8ddd5

build-sys: Work around --with-rootprefix= (empty) not producing /

Since we introduced AX_NORMALIZE_PATH, using --with-rootprefix=/ does
produce an empty string, but using --with-rootprefix= (empty) now
produces "." instead which is wrong.

Work around it until we can find a better solution for AX_NORMALIZE_PATH
upstream at autoconf-archive.

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

build-sys: Recommend --with-rootprefix=/ for split-usr

Since we started using AX_NORMALIZE_PATH, that is a valid supported
setup and is more explicit than --with-rootprefix= (empty) which is
actually currently broken.

Let's advocate for it in the ./configure suggestion from autogen.sh.

Merge pull request #51 from zonque/unquote

util: fix another cunescape() regression

compiled and test-util exited with 0

configure.ac: add missing komma

point to github issues instead of freedesktop bugzilla

build-sys: split off libsystemd-machine convenience lib

Break the link order cycle by splitting off the machine parts which
use sd-bus but live in shared/.

build-sys: merge libsystemd-label convenience lib

Stop to pretend that we can split selinux related code from other.
We have too many cross-references and it breaks all the time and
I am no longer willing to maintain that mess for no real benefit.

We currently have cyclic dependencies which are only resolved on
machines with gc-sections toolchains. We need a simpler and at the
same time more strict model to manage our convenienc libraries and
linking.

The first thing to give up is the "optimization" of not linking
libselinux for a very few tools. If that is an issue, please fix
the mess that libselinux creates in selinux itself, and do not ask
consumers to work around it.

systemctl: Use /usr/bin/editor if available

If the EDITOR environment variable is not set, the Debian policy
recommends to use the /usr/bin/editor program as default editor.
This file is managed via the dpkg alternatives mechanism and typically
used in Debian/Ubuntu and derivatives to configure the default editor.

See section 11.4 of the Debian policy [1].

Therefor prefer /usr/bin/editor over specific editors if available.

[1] https://www.debian.org/doc/debian-policy/ch-customized-programs.html

util: fix another cunescape() regression

Fix a regression caused by 4034a06d ("util: rework word parsing and c
unescaping code") which broke octal escape sequences.

The reason for this breakage is that cunescape_one() expects 4 characters
in an octal encoding, which is a stray left-over from the old code which
operated on different variables to make the length check.

While at it, add a test case to prevent the same thing from happening
again.

build-sys: disable gc-sections for distcheck

We cannot rely on gc-sections to fix dependency cycles in our linking
setup. Disable it for distcheck to let it fail and find these bugs
earlier.

Merge pull request #43 from teg/udev-forking-rework

udev forking/socket-passing rework

build-sys: remove left-over gtk-doc

udevd: merge manager_new() and manager_listen() again

Now that listen_fds() have been split out, we can safely move the allocation
of the manager object after doing the forking (the fork is done to notify legcay
init-systems that the fds are ready).

Subsequently, we can merge manager_listen() back into managre_new().

This entails a minor behaviour change: the application of permissions to
static device nodes now happens after the fork (but still before notifying
systemd about being ready).

udevd: make sd_notify independent of forknig/notify mode

This will simply silently fail on non-systemd systems, so there is no reason
to make it conditional.

Also make it clear that we notify systemd about being ready as the last step
before starting the event loop, whereas the forking might need to happen
earlier.

udevd: manager - split listen_fds() out of manager_new()

This will allow us in a follow-up commit to listen to fds before forking and
still allocate the manager only after the fork.

udevd: unify fd handling in forking/notify modes

Hide the differenec in listen_fds, by simply opening the fds
here in case they are not passed in.

libudev: monitor - set nl_pid when reusing fd in udev_monitor_new_from_netlink_fd

This allows a fd to be created and configured as part of one monitor, to be passed in
to create a second monitor without having to redo any of the configuration.

udevd: make cgroup logic independent of socket passing

This should have no behavioural change, but it is odd to tie the cgroup cleaning to
whether or not we are passed sockets.

The point really is if we are guaranteed to be in a dedicated cgroup, so instead
check for our parent being PID1 (we already implicitly only do this on systemd
systems).

udevd: only bind ctrl and netlink sockets when we open them

If they are passed from PID1 this is not necessary.

udevd: rename systemd_fds to listen_fds

udevd: simplify signal mask handling

We used to block all signals, and restore the original signal mask before exec'ing
external processes.

Now we just block the signals we care about and unconditionally unblock all signals
before exec'ing.

Merge pull request #42 from jonboulle/master

README: update links to reference new home (GitHub)

Merge pull request #41 from mischief/ipforwarding

networkd: create "kernel" setting for IPForwarding

README: update links to reference new home (GitHub)

man: document IPForward=kernel option

networkd: create "kernel" setting for IPForwarding

In 5a8bcb674f71a20e95df55319b34c556638378ce, IPForwarding was introduced
to set forwarding flags on interfaces in .network files. networkd sets
forwarding options regardless of the previous setting, even if it was
set by e.g. sysctl. This commit creates a new option for IPForwarding,
"kernel", that preserves the sysctl settings rather than always setting
them.

See https://bugs.freedesktop.org/show_bug.cgi?id=89509 for the initial
bug report.

udev: input_id - use ABS_MT_SLOT{-1} to exclude non touch screen devices

Peek at the ABS_MT_SLOT-1 axis. Expect that touch screens only
have axes inside the MT range.

udev: input_id - use direct property and mt axis for touch screen detection

A lot of touch screens use INPUT_PROP_DIRECT to indicate that touch input
maps directly to the underlying screen, while the BTN_TOUCH bit might not be
set.

udev: input_id - refactor device detection

This change switches to bools and separates bit flag evaluation from
decision making and application of udev properties, while hopefully
keeping the same semantics. Apart from using BTN_LEFT instead of BTN_MOUSE
for mouse detection.

remove gudev and gtk-doc

The library moved to:
  https://git.gnome.org/browse/libgudev/

fix double semicolon typo

hwdb: update

hwdb: Update database of Bluetooth company identifiers

Merge pull request #12 from systemd-mailing-devs/1433236104-9967-1-git-send-email-m.olbrich@pengutronix.de

missing: add more btrfs defines

udev-builtin: path_id - don't pass NULL to udev_device_get_parent()

Being explicit about this makes the code easier to follow IMHO.

udev: add some asserts

Mostly for documentation purposes.