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.

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.

Merge pull request #799 from andreaskem/hwdb-pull

hwdb: DPI info for Logitech G402 mouse

hwdb: Add mouse DPI info for Logitech G402.

DPI information was taken from the device's setup guide.

Additional (shortened) information from udevadm info:

P: .../usb2/2-1/2-1:1.0/0003:046D:C07E.0002/input/input6/event2
N: input/event2
S: input/by-id/usb-Logitech_Gaming_Mouse_G402_6D93117D5049-event-mouse
S: input/by-path/pci-0000:00:12.0-usb-0:1:1.0-event-mouse
...
E: ID_BUS=usb
E: ID_INPUT=1
E: ID_INPUT_MOUSE=1
E: ID_MODEL=Gaming_Mouse_G402
E: ID_MODEL_ENC=Gaming\x20Mouse\x20G402
E: ID_MODEL_ID=c07e
E: ID_PATH=pci-0000:00:12.0-usb-0:1:1.0
E: ID_PATH_TAG=pci-0000_00_12_0-usb-0_1_1_0
E: ID_REVISION=9002
E: ID_SERIAL=Logitech_Gaming_Mouse_G402_6D93117D5049
E: ID_SERIAL_SHORT=6D93117D5049
E: ID_TYPE=hid
E: ID_USB_DRIVER=usbhid
E: ID_USB_INTERFACES=:030102:030000:
E: ID_USB_INTERFACE_NUM=00
E: ID_VENDOR=Logitech
E: ID_VENDOR_ENC=Logitech
E: ID_VENDOR_ID=046d
E: LIBINPUT_DEVICE_GROUP=3/46d/c07e/111:usb-0000:00:12.0-1
E: MAJOR=13
E: MINOR=66
E: SUBSYSTEM=input
...

hwdb: Update info for potential contributors.

Update the location of the bug tracker and mention that pull requests
are preferred.

systemd-boot: sort /EFI/Linux/*.efi files

unit-name: fix memory leak

Fix a memory leak introduced by 2fc09a9c. 's' is used twice in this
function, and if free_and_strdup() fails, the pointer is left untouched.

Merge pull request #787 from dvdhrm/bus-list

sd-bus: fix parsing of KDBUS_CMD_LIST

sd-bus: fix parsing of KDBUS_CMD_LIST

We *must not* assume that an entry returned by KDBUS_CMD_LIST only
carries a single KDBUS_ITEM_OWNED_NAME. Similarly, we already parse
multiple such items for message-metadata, so make sure we support the
same on KDBUS_CMD_LIST.

By relying on the kernel to return all names separately, we limit the
kernel API significantly. Stop this and let the kernel decide how to
return its data.

Merge pull request #784 from eworm-de/typo

man: fix typo in man systemd-machine-id-commit

Merge pull request #785 from zonque/free-and-strdup

tree-wide: use free_and_strdup()

tree-wide: use free_and_strdup()

Use free_and_strdup() where appropriate and replace equivalent,
open-coded versions.

man: fix typo in man systemd-machine-id-commit

Merge pull request #783 from whot/hwdb-updates

Hwdb updates

hwdb: add min/max/resolution override for Appletouch one-button touchpads

Macbook2,1, late 2006 model.

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

hwdb: add resolution values for Asus K52JT touchpad

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

Merge pull request #779 from poettering/fflush-and-check

tree-wide: port everything over to fflush_and_check()

Merge pull request #764 from ssahani/vxlan1

networkd: Add VXLAN Netlink attributes

Merge branch 'master' of github.com:systemd/systemd

tree-wide: port everything over to fflush_and_check()

Some places invoked fflush() directly with their own manual error
checking, let's unify all that by using fflush_and_check().

This also unifies the general error paths of fflush()+rename() file
writers.

Merge branch 'pr/761'

networkd-wait-online: only consider interfaces given on the commandline

If some interfaces are given on the commandline, ignore all others.

sd-boot: stub - export LoaderDevicePartUUID

Merge pull request #778 from ssahani/flow

networkd: fix size of networkd binary

build-sys: fix "make doc-sync"

networkd: fix size of networkd binary

This patch fixes the size of networkd
Bug #775

before:

size systemd-networkd
   text    data     bss     dec     hex filename
1493755 8424392    2337 9920484  975fe4 systemd-networkd

After

$ size systemd-networkd
   text    data     bss     dec     hex filename
1493555   35752    2337 1531644  175efc systemd-networkd

Merge pull request #777 from kaysievers/gtp-auto

gpt-auto-generator: merge efi-boot-generator

Merge pull request #774 from dvdhrm/gvariant3

gvariant: fix encoding of unary type and root-variant (v3)

gpt-auto-generator: merge efi-boot-generator

Merge pull request #776 from baracoder/master

man: unit: fix StartTimeoutSec

man: unit: fix StartTimeoutSec

should be TimeoutStartSec

sd-bus: fix encoding/decoding gvariant root container

The gvariant root container contains a 'variant' at the end, which embeds
the whole message body. This variant *must* contain a structure so we are
compatible to dbus1. Otherwise, it could encode at most 1 type, instead
of a full signature.

Our gvariant message parser already parses the variant-content as a
structure, so we're mostly good. However, it does *not* include the
opening and closing parantheses, nor does it parse them.

This patch fixes the decoder to verify a message contains the
parantheses, and also make the encoder add those parantheses into the
marshaled message.

sd-bus: fix marshaling of unary type

The unary type has a fixed size of 1 in gvariant. Make sure we properly
encode it as such. Right now, we encode/decode it as empty sequence.

NEWS: update 223 contributors

Update v223 contributors and release date.

build-sys: update git-tar

Merge pull request #769 from poettering/socket-util-no-log

socket-util: library calls shouldn't log on their own

sd-bus: drop redundant code

If c->item_size is 0, the next item to parse in a structure is empty.
However, this also implies that the signature must be empty. The latter
case is already handled just fine by enter_struct_or_dict_entry() so
there is no reason to handle the same case in the caller.

sd-bus: don't assert() on valid signatures

Right now sd_bus_message_skip() will abort execution if passed a
signature of the unary type "()". Regardless whether this should be
supported or not, we really must not abort. Drop the incorrect assertion
and add a test-case for this.

build: bump version info

Bump version to 223 and increase libsystemd libtool-info as we added a
structure-member (without increasing structure size) to sd-bus-vtable.

Merge pull request #768 from poettering/resolved-localhost

resolved: never attempt to resolve loopback addresses via DNS/LLMNR/mDNS

Merge pull request #767 from torstehu/fix-typo

treewide: fix typos

socket-util: library calls shouldn't log on their own

Instead, make sure that all callers log properly.

resolved: never attempt to resolve loopback addresses via DNS/LLMNR/mDNS

We already refuse to resolve "localhost", hence we should also refuse
resolving "127.0.0.1" and friends.

treewide: fix typos

NEWS: add 'macvtap' section

systemd-networkd now supports 'macvtap' devices, similar to the already
supported 'macvlan' devices.

Merge pull request #747 from dvdhrm/consoled

terminal: drop unfinished code

man: add man for VXLAN Group Policy option

networkd: vxlan add support for GBP

This patch add support for vxlan  VXLAN Group Policy Option.

https://tools.ietf.org/html/draft-smith-vxlan-group-policy-00
http://lwn.net/Articles/628683/

sd-netlink: introduce api for new NL type NLA_FLAG

sd-netlink: add support for new type NETLINK_TYPE_FLAG

NETLINK_TYPE_FLAG is NLA_FLAG.

This new type will be used in NLA_FLAG for example

IFLA_VXLAN_GBP  and IFLA_VXLAN_REMCSUM_NOPARTIAL

sd-netlink: add support for vxlan attributes

networkd-wait-online: fix race

We must consider 'pending' links as if they may be managed by networkd, as this
is the state we enter before deciding wether networkd should manage the link
or not, so we better wait for this decision being made.

Merge pull request #760 from teg/resolved-fix-nsec-types

resolved: fix NSEC type windows

resolved: packet - refuse empty type bitmaps

The NSEC type itself must at least be in the bitmap, so NSEC records with empty
bitmaps must be bogus.

resolved: packet - fix read_type_window()

We were tracking the bit offset inside each byte, rather than inside the whole bitmap.

resolve: packet - fix append_types()

We were counting the number of bits set rather than the number of bytes they occupied.

Merge pull request #759 from phomes/master

hwdb: add HP USB 1000dpi Laser Mouse

Merge pull request #758 from poettering/dns-is-equal

resolved: compare dns question arrays properly

hwdb: add HP USB 1000dpi Laser Mouse

Merge pull request #732 from ssahani/macvtap

networkd: add support for Macvtap