TODO: remove obsolete fedup related todos

Fedup is being replaced by a dnf plugin that relies on systemd offline
updates as per
https://fedoraproject.org/wiki/Changes/DNF_System_Upgrades

Merge pull request #878 from poettering/network-dhcp-array-fix

network: make enough space for string

network: make enough space for string

Follow-up for PR #877.

Merge pull request #877 from crawford/dhcp-private-options-v4

networkd: save private-zone DHCP options

udev: uaccess - do not log error when device node disappears

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

networkd: clean up duplicate code

networkd: serialize the private options

Save the private options along side the rest of the options in the lease
files.

networkd: add private options to lease struct

This stores private-zone DHCP options inside of their respective DHCP
lease. These options aren't used by networkd (what would it do with
them?), but saving them will allow other programs to query the values.
To improve performance, the options are stored in ascending order by
tag.

Merge pull request #876 from zonque/logind

logind: switch to sd_bus_track helper

logind: switch to sd_bus_track helper

Let logind use the sd_bus_track helper object to track the controllers of
sessions. This does not only remove quite some code but also kills the
unconditional matches for all NameOwnerChanged signals.

The latter is something we should never ever do, as it wakes up the daemon
every time a client connects, which doesn't scale.

Merge pull request #874 from richardmaw-codethink/doc-nspawn-machine-search-path-v2

man: Reference the canonical location of the search path in systemd-nspawn's documentation of the --machine option. (v2)

man: point nspawn --machine to machinectl search-path

The --machine option used to describe searching for machines in
/var/lib/machines, which is not the whole story, so let's link to where
it's described in more detail.

Merge pull request #871 from poettering/journald-meta-fields

journald: fix count of object meta fields

Merge pull request #864 from phomes/master

hwdb: add Logitech LX8 DPI and wheel click settings

journald: fix count of object meta fields

There are 12 object meta fields created in dispatch_message_real(), but
we only allocated space for 11. Fix this.

Fixes #866.

Merge pull request #869 from poettering/tmpfiles-enoent

sysctl: reword warning message a bit

sysctl: reword warning message a bit

Let's make this less dramatic, in order to not confuse people too much
making them think that this was something to actually fix.

Merge pull request #868 from jsynacek/jsynacek/sysctl

sysctl: bump loglevel when attempting to write invalid values [v2]

sysctl: bump loglevel when attempting to write invalid values

Merge pull request #865 from mattst88/alpha-getxpid

Use getxpid syscall on alpha for raw_getpid()

Use getxpid syscall on alpha for raw_getpid()

Alpha does not have a getpid syscall, but rather has getxpid to match
OSF/1.

Merge pull request #855 from richardmaw-codethink/machinectl-copy-to-from-relative-paths-v2

Allow relative paths in `machinectl copy-{from,to}` (v2)

gpt-auto-generator: remove unused variable

hwdb: add Logitech LX8 DPI and wheel click settings

Merge pull request #857 from dvdhrm/proxy-reload

bus-proxy: add ExecReload=

Merge pull request #860 from walyong/smack_v11

Smack v11: set only the default smack process label if the command path has no execute label

Merge pull request #858 from dvdhrm/proxy-env

bus-proxy: discard invalid environment updates

core: set default process label only exec label is none

When command path has access label and no SmackProcessLabel= is not
set, default process label will be set. But if the default process
label has no rule for the access label of the command path then smack
access error will be occurred.
So, if the command path has execute label then the child have to set
its label to the same of execute label of command path instead of
default process label.

smack-util: revise smack-util apis and add read smack attr apis

- Add smack xattr lookup table
- Unify all of mac_smack_apply_xxx{_fd}() to mac_smack_apply() and
  mac_smack_apply_fd().
- Add smack xattr read apis similar with apply apis as
  mac_smack_read{_fd}().

util: add getxattr helper apis

To get xattr of given path or fd on newly allocated buffer, add new
helper api getxattr_malloc() and fgetxattr_malloc().

bus-proxy: discard invalid environment updates

If the caller provides invalid assignments via
UpdateActivationEnvironment=, we now drop it and warn loudly about it. We
need this to be compatible to dbus1, but we also clearly want to tell the
caller that they better ought to follow POSIX recommendations and use
valid labels only.

Fixes #826.

bus-proxy: add ExecReload=

Make sure we support ExecReload= for bus-proxyd to reload configuration
during runtime. This is *really* handy when hacking on kdbus.

Package-managers are still recommended to run
 `busctl --address=unix:path=` directly.

Merge pull request #856 from richardmaw-codethink/doc-invalid-environmentfile-lines

man: Document invalid lines in EnvironmentFile

networkd: add DHCP options for private zone

basic: add LIST_INSERT_BEFORE

Similar in function to LIST_INSERT_AFTER, this will insert a new element
into the list before the specified position. If the specified position
is NULL, the element is added as the tail of the list.

man: Document invalid lines in EnvironmentFile

If a line doesn't contain an = separator, it is skipped, rather than
raising an error.

This is potentially useful, so let's document this behaviour.

machinectl: support relative host paths in copy-{from,to}

machined: Relax path checks for Copy{From,To}

Absolute paths should be sufficient to prevent funny business,
and while path_is_safe() checks this, it also checks whether the path
contains . or .. components, which while odd, aren't a security risk.

Merge pull request #850 from ssahani/dhcp

networkd: DHCP fix CID 1315105

networkd: DHCP fix CID 1315105

CID 1315105: Resource leaks (RESOURCE_LEAK)
/src/libsystemd-network/sd-dhcp-server.c: 800 in
dhcp_server_handle_message()
*** CID 1315105: Resource leaks (RESOURCE_LEAK)
/src/libsystemd-network/sd-dhcp-server.c: 800 in

Update bugtracker address

Merge pull request #845 from poettering/terminal-o-nonblock

add O_NONBLOCK to terminal handling

tree-wide: convert some more mfree() candidates

terminal-util: no real reason to assert on O_CREAT

That's just handle this as a normal error.

terminal-util: cast a couple of ioctl()s to void

terminal-util: when resetting terminals, don't wait for carrier

In case of non-CLOCAL lines (i.e. those with carrier detect configured)
we shouldnt wait for a carrier if all we try to do is reset the TTY.
Hence, whenever we open such a TTY pass O_NONBLOCK.

Note that we continue to open ttys we intend to write to without
O_ONBLOCK, we only add it in cases we invoke ioctl()s or other terminal
operations without reading or writing to the device.

Fixes #835.

Merge pull request #844 from poettering/clock-boottime2

tree-wide: convert bootchart and lldp code to use clock_boottime_or_m…

Merge pull request #843 from poettering/clock-boottime

sd-event: make sure sd_event_now() cannot fail

tree-wide: convert bootchart and lldp code to use clock_boottime_or_monotonic()

We should avoid using CLOCK_BOOTTIME directly unless we actually can
sensible distuingish it from CLOCK_MONOTONIC. CLOCK_BOOTTIME is only
fully feature on very recent Linux kernels, hence we should stick to a
fallback logic, which is already available in the
clock_boottime_or_monotonic() call.

Merge pull request #840 from systemd-mailing-devs/1438566907-9544-1-git-send-email-wangkefeng.wang@huawei.com

logind: add standard gpio power button support

sd-event: make sure sd_event_now() cannot fail

Previously, if the event loop never ran before sd_event_now() would
fail. With this change it will instead fall back to invoking now(). This
way, the function cannot fail anymore, except for programming error when
invoking it with wrong parameters.

This takes into account the fact that many callers did not handle the
error condition correctly, and if the callers did, then they kept simply
invoking now() as fall back on their own. Hence let's shorten the code
using this call, and make things more robust, and let's just fall back
to now() internally.

Whether now() is used or the cache timestamp may still be detected via
the return value of sd_event_now(). If > 0 is returned, then the fall
back to now() was used, if == 0 is returned, then the cached value was
returned.

This patch also simplifies many of the invocations of sd_event_now():
the manual fall back to now() can be removed. Also, in cases where the
call is invoked withing void functions we can now protect the invocation
via assert_se(), acknowledging the fact that the call cannot fail
anymore except for programming errors with the parameters.

This change is inspired by #841.

Merge pull request #841 from teg/resolved-exponential-resend

resolved: exponential backoff of packet resend

resolved: transaction - increase number of retry attempts

With the exponential backoff, we can perform more requests in the same amount of time,
so bump this a bit.

In case of large RTT this may be necessary in order not to regress, and in case
of large packet-loss it will make us more robust. The latter is particularly
relevant once we start probing for features (and hence may see packet-loss
until we settle on the right feature level).

resolved: transaction - exponentially increase retry timeouts

Rather than fixing this to 5s for unicast DNS and 1s for LLMNR, start
at a tenth of those values and increase exponentially until the old
values are reached. For LLMNR the recommended timeout for IEEE802
networks (which basically means all of the ones we care about) is 100ms,
so that should be uncontroversial. For unicast DNS I have found no
recommended value. However, it seems vastly more likely that hitting a
500ms timeout is casued by a packet loss, rather than the RTT genuinely
being greater than 500ms, so taking this as a startnig value seems
reasonable to me.

In the common case this greatly reduces the latency due to normal packet
loss. Moreover, once we get support for probing for features, this means
that we can send more packets before degrading the feature level whilst
still allowing us to settle on the correct feature level in a reasonable
timeframe.

The timeouts are tracked per server (or per scope for the multicast
protocols), and once a server (or scope) receives a successfull package
the timeout is reset. We also track the largest RTT for the given
server/scope, and always start our timouts at twice the largest
observed RTT.

resolved: cache - use clock_boottime_or_monotonic()

We cannot rely on CLOCK_BOOTTIME being supported by the kernel, so fallack
to CLOCK_MONOTONIC if the former is not supported.

Merge pull request #837 from ssahani/dhcp

networkd: fix neworkd crash

logind: add standard gpio power button support

Many boards like hisilicon D02 board use standard gpio key to power down system.
A description of gpio-key in dts shown below,

gpio_keys {
compatible = "gpio-keys";
#address-cells = <1>;
#size-cells = <0>;

pwrbutton {
label = "Power Button";
gpios = <&porta 8 1>;
linux,code = <116>;  // KEY_POWER, used by SC System Power Down
};
};

-bash-4.3# udevadm info -a /dev/input/event3

Udevadm info starts with the device specified by the devpath and then
walks up the chain of parent devices. It prints for every device
found, all possible attributes in the udev rules key format.
A rule to match, can be composed by the attributes of the device
and the attributes from one single parent device.

  looking at device '/devices/platform/gpio_keys/input/input3/event3':
    KERNEL=="event3"
    SUBSYSTEM=="input"
    DRIVER==""

  looking at parent device '/devices/platform/gpio_keys/input/input3':
    KERNELS=="input3"
    SUBSYSTEMS=="input"
    DRIVERS==""
    ATTRS{name}=="gpio_keys"
    ATTRS{phys}=="gpio-keys/input0"
    ATTRS{uniq}==""
    ATTRS{properties}=="0"

  looking at parent device '/devices/platform/gpio_keys':
    KERNELS=="gpio_keys"
    SUBSYSTEMS=="platform"
    DRIVERS=="gpio-keys"
    ATTRS{keys}=="116"
    ATTRS{switches}==""
    ATTRS{driver_override}=="(null)"
    ATTRS{disabled_keys}==""
    ATTRS{disabled_switches}==""

  looking at parent device '/devices/platform':
    KERNELS=="platform"
    SUBSYSTEMS==""
    DRIVERS==""

Merge pull request #836 from mbiebl/man-fix-typo-systemd-fsck

man: fix typo in systemd-fsck@

man: fix typo in systemd-fsck@

Merge pull request #834 from namhyung/sdbus-cleanup-v2

sd-bus: Small cleanups (v2)

sd-bus: cleanup bus_reset_parsed_address()

Both strv_free() and mfree() return NULL pointer after free.

sd-bus: use mfree() in bus_reset_queues()

Recently mfree() was introduced to reduce work of tedious free + reset
pointers.  Use it in bus_reset_queues() too.

sd-bus: cleanup bus_close_fds()

The safe_close() already checks the fd and returns -1.

networkd: fix neworkd crash

fix issue #827

hostname should be init to NULL.

Merge pull request #822 from hobbitalastair/master

hwdb: add resolution information for Dell Vostro 1510 touchpad

hwdb: add resolution information for Dell Vostro 1510 touchpad

Dell Vostro 1510, AlpsPS/2 ALPS GlidePoint touchpad.

Relevant issue: #763

Merge pull request #813 from dvdhrm/bus-client

busctl: make sure --address connects as bus-client

Merge pull request #812 from zonque/test

Add some more tests

Merge pull request #814 from dvdhrm/mfree

tree-wide: introduce mfree()

Merge pull request #815 from poettering/coding-style-for

CODING_STYLE: say that "for (;;)" is better than "while (1)"

busctl: make sure --address connects as bus-client

There is really no reason to use `busctl` to connect to legacy private
bus endpoints. Fix this and make sure `busctl --address=unix:path=/foo`
works!

CODING_STYLE: say that "for (;;)" is better than "while (1)"

test-socket-util: add test for in_addr_is_null()

test-util: add more tests

Add tests for safe_ato[iu]16() and some more unbase32hexmem() torture.

test: add test for static lists

Test af-list and arphdr-list.

tree-wide: introduce mfree()

Pretty trivial helper which wraps free() but returns NULL, so we can
simplify this:
        free(foobar);
        foobar = NULL;
to this:
        foobar = mfree(foobar);

Merge pull request #811 from namhyung/busctl-misc-v2

busctl: Misc cleanups and a fix (v2)

Merge pull request #810 from mamikk/bitmap-fixes-v2

Bitmap fixes v2

busctl: fix assertion failure on --size option

Using --size option triggers an assert failure below because
parse_size() requires the second argument, base, being either 1000 or
1024.  As it's for a packet size, it'd be better using IEC binary
suffix (base 1024) IMHO.

  $ busctl --size 2048
  Assertion 'base == 1000 || base == 1024' failed at src/basic/util.c:2222,
  function parse_size(). Aborting.
  Aborted (core dumped)

busctl: add missing description of --size option

The size option was to specify maximum captured patch length but was
missing its description in the command line help.  Add it.

busctl: add and use strcmp_ptr()

In member_compare_func(), it compares interface, type and name of
members.  But as it can contain NULL pointer, it needs to check them
before calling strcmp().  So make it as a separate strcmp_ptr
function (named after streq_ptr) so that it can be used by others.

Also let streq_ptr() to use it in order to make the code simpler.

hwdb: Update database of Bluetooth company identifiers

bitmap: make bitmap_clear free the bitmap array

Given two bitmaps and the following code:

  Bitmap *a = bitmap_new(), *b = bitmap_new();
  bitmap_set(a, 1);
  bitmap_clear(a);
  bitmap_set(a, 0);
  bitmap_set(b, 0);

These two bitmaps should now have the same bits set and they should be
equal but bitmap_equal() will return false in this case because while
bitmap_clear() resets the number of elements in the array it does not
clear the array and bitmap_set() expects the array to be cleared.
GREEDY_REALLOC0 looks at the allocated size and not the actual size so
it does not clear any memory.

Fix this by freeing the allocated memory and resetting the whole Bitmap
to an initial state in bitmap_clear().

This also adds test code for this issue.

bitmap: fix bitmap_equal on bitmaps with unset bits

Given two bitmaps and the following code:

Bitmap *a = bitmap_new(), *b = bitmap_new();
bitmap_set(a, 0);
bitmap_unset(a, 0);

These two bitmaps should now have the same bits set and they should be
equal but bitmap_equal() will return false in this case because the
bitmaps array in a is larger because of the bit which was previously
set.

Fix this by comparing only the bits which exists in both bitmaps and
then check that the rest of the bits (if any) is all zero.

This also adds test code for this issue.

bus-proxy: allow empty arguments to UpdateActivationEnvironment()

There is no reason to prevent empty argument lists on
UpdateActivationEnvironment(). Make sure we don't fail, but still skip
the call to pid1.

NEWS: update

Add new functionality and behavior changes to NEWS for the imminent bugfix
release 224.

build: bump version to 224

Prepare for bug-fix release and bump versions.

hwdb: update

Merge pull request #806 from dvdhrm/proxy-cookie

bus-proxy: fix cookie namespacing

logind: release VT-positions when closing sessions

Make sure we release VT-positions when a session is closed. Otherwise,
lingering sessions will occupy VTs and prevent next logins from
succeeding.

Note that we already release session-devices when closing a session, so
there cannot be anyone using the VT anymore.

bus-proxy: fix cookie namespacing

If we generate messages from within bus-proxyd, we really must make sure
that we do not clutter the cookie-namespace of our client. The client has
full control over it, so we cannot steal cookies from it. However, we can
re-use the cookie the client used for our request. As long as we only
send a single message, we're fine.

timesyncd: remove RLIMIT_NPROC

NSS plugins might create additional threads. Remove the limit, we cannot
really make any assumptions here.

Merge pull request #803 from mixi/master

build-sys: use 'test -f' instead of AC_CHECK_FILE

Merge pull request #804 from dvdhrm/proxy-directed2

bus-proxy: make NameAcquired/Lost directed (v2)

Merge pull request #805 from dvdhrm/proxy-activation

bus-proxy: make StartServiceByName synchronous

bus-proxy: make StartServiceByName synchronous

The StartServiceByName() call was provided by dbus-daemon to activate a
service without sending a message. On receiption, dbus-daemon schedules
an activation request (different modes are supported) and sends back the
reply once activation is done.

With kdbus, we marked StartServiceByName() as deprecated. There is no
real reason to start services explicitly. Instead, applications should
just *use* the service and rely on it being activated implicitly.
However, we provide compatibility with dbus-daemon and implement
StartServiceByName() on the proxy via a call to
org.freedesktop.DBus.Peer.Ping() on the destination. This will activate
the peer implicitly as part of the no-op Ping() method call (regardless
whether the peer actually implements that call).

Now, the problem is, StartServiceByName() was synchronous on dbus-daemon
but isn't on bus-proxy. Hence, on return, there is no guarantee that
ListNames includes the activated name. As this is required by some
applications, we need to make this synchronous.

This patch makes the proxy track the Ping() method call and send the
reply of StartServiceByName() only once Ping() returned. We do not look
at possible errors of Ping(), as there is no strict requirement for the
peer to implement org.freedesktop.DBus.Peer. Furthermore, any interesting
error should have already been caught by sd_bus_send() before.

Note:
        This race was triggered by gdbus. The gdbus-proxy implementation
        relies on a name to be available after StartServiceByName()
        returns. This is highly fragile and should be dropped by gdbus.
        Even if the call is synchronous, there is no reason whatsoever to
        assume the service did not exit-on-idle before ListNames()
        returns.
        However, this race is much less likely than the startup race, so
        we try to be compatible to dbus-daemon now.

build-sys: use 'test -f' instead of AC_CHECK_FILE

AC_CHECK_FILE fails when cross-compiling. It is intended to be used to
check for files that are used at runtime during build time (e.g.
/etc/passwd, /dev/*) [1]. For files which are only used at build time
'test -f' is sufficient.

[1]: https://lists.gnu.org/archive/html/autoconf/2000-10/msg00018.html

bus-proxy: don't filter directed signals

NameAcquired and NameLost are directed signals. Never ever filter them on
dbus1. On dbus1, filters *exclusively* apply to broadcasts! Hence, we
must always forward directed signals unconditionally!

bus-proxy: fix NameAcquired and NameLost to be directed

The NameAcquired and NameLost signals are _directed_ signals. Make sure
we properly set the destination correctly, and verify it in our
proxy-test.

basic: fix error/memleak in socket-util

Probably a typo, checking 'ret' instead of the return value 'p'. This
might cause the function to return failure, even though it succeeded.
Furthermore, it might leak resources.