zsh-completion: _loginctl - general bug fixes

1) the iterator `fun' has an local scope. after running the completer,
it will no longer be defined.

2) use _describe instead of calling compadd. Using compadd without
calling _description or something similar before, restricts the
user's ability to customize what is presented to them.
zstyle ':completion:*' format 'Completing %d'
  - now displays an header showing what is being completed.
zstyle ':completion::complete:loginctl-*::users' users user1 user2
  - allows the user to manually specify which users is offered
zstyle :completion::complete:loginctl-kill-user:\* \
ignored-patterns '(100<0-4>|user1)'
  - selectively ignore some users when completing loginctl kill-user
    <tab>
Sessions, UIDs now have descriptions when selecting them.

3) removed the call to _loginctl_all_seats in _loginctl_attach(), since
_loginctl_seats calls it a second time, right before adding matches.
There isn't a noticeable difference doing this.

Merge pull request #228 from teg/tmpfiles-btrfs-notdir

tmpfiles: silently ignore failed removal of btrfs submount from non-dir

tmpfiles: silently ignore failed removal of btrfs submount from non-dir

This fixes:
Jun 16 16:00:20 tomegun-x2402 systemd-tmpfiles[233]: rm_rf(/var/lib/machines/.#fedora.lck): Not a directory
Jun 16 16:00:20 tomegun-x2402 systemd-tmpfiles[233]: rm_rf(/var/lib/machines/.#Fedora-Cloud-Base-20141203-21.x86_64.raw.lck): Not a directory

Merge pull request #197 from dvdhrm/hashmap

hashmap: fix iterators to not skip entries

Merge pull request #223 from ronnychevalier/rc/warning_va_start

signal-util: fix incorrect argument of va_start

Merge pull request #222 from utezduyar/mem-leak-on-bus-error

sd-bus: use proper cleanup macro

signal-util: fix incorrect argument of va_start

The last argument of the function before the vargs is "old" not "how".

warning: second parameter of ‘va_start’ not last named argument

sd-bus: use proper cleanup macro

Merge pull request #218 from poettering/dual-timestamp-null

everywhere: actually make use of DUAL_TIMESTAMP_NULL macro

Merge pull request #219 from poettering/logind-docked

logind: expose "Docked" bool as property on the bus

logind: cast close() call to (void)

logind: expose "Docked" bool as property on the bus

We know the state anyway, let's expose it in the bus. It's useful for
debugging at least, but it might be useful for DEs too.

everywhere: actually make use of DUAL_TIMESTAMP_NULL macro

Let's use it as initializer where appropriate.

update TODO

Merge pull request #214 from poettering/signal-rework-2

everywhere: port everything to sigprocmask_many() and friends

Merge pull request #212 from poettering/gc-machine-snapshots

automatically remove old machine shapshots at boot

everywhere: port everything to sigprocmask_many() and friends

This ports a lot of manual code over to sigprocmask_many() and friends.

Also, we now consistly check for sigprocmask() failures with
assert_se(), since the call cannot realistically fail unless there's a
programming error.

Also encloses a few sd_event_add_signal() calls with (void) when we
ignore the return values for it knowingly.

Merge pull request #209 from crrodriguez/master

buildsys: missing SECCOMP_CFLAGS in various places

tmpfiles: automatically remove old machine snapshots at boot

Remove old temporary snapshots, but only at boot. Ideally we'd have
"self-destroying" btrfs snapshots that go away if the last last
reference to it does. To mimic a scheme like this at least remove the
old snapshots on fresh boots, where we know they cannot be referenced
anymore. Note that we actually remove all temporary files in
/var/lib/machines/ at boot, which should be safe since the directory has
defined semantics. In the root directory (where systemd-nspawn
--ephemeral places snapshots) we are more strict, to avoid removing
unrelated temporary files.

This also splits out nspawn/container related tmpfiles bits into a new
tmpfiles snippet to systemd-nspawn.conf

tmpfiles: make sure "R" lines also remove subvolumes

util: when creating temporary file names, allow including extra id string in it

This adds a "char *extra" parameter to tempfn_xxxxxx(), tempfn_random(),
tempfn_ranomd_child(). If non-NULL this string is included in the middle
of the newly created file name. This is useful for being able to
distuingish the kind of temporary file when we see one.

This also adds tests for the three call.

For now, we don't make use of this at all, but port all users over.

buildsys: missing SECCOMP_CFLAGS in various places

libcore, systemd and nspawn fail to build when seccomp headers
are not in the include path.

Merge pull request #208 from poettering/btrfs-rec-snapshot

btrfs-util: when snapshotting make sure we don't descent into subvolu…

btrfs-util: when snapshotting make sure we don't descent into subvolumes we just created

We already had a safety check in place that we don't end up descending
to the original subvolume again, but we also should avoid descending in
the newly created one.

This is particularly important if we make a snapshot below its source,
like we do in "systemd-nspawn --ephemeral -D /".

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

Merge pull request #154 from dmedri/master

Italian .po updates

Merge pull request #202 from victorenator/l10n-be

l10n: Add Belarusian translation

Merge pull request #206 from zonque/firewall-rename

firewall: rename fw-util.[ch] → firewall-util.[ch]

firewall: rename fw-util.[ch] → firewall-util.[ch]

The names fw-util.[ch] are too ambiguous, better rename the files to
firewall-util.[ch]. Also rename the test accordingly.

Merge pull request #180 from ronnychevalier/rc/coverity_cid_1304686

login: fix potential null pointer dereference

man: document that ExecStop= needs a synchronous tool

As requested in #199.

man: document that SIGCONT always follows SIGTERM

As requested in #199.

man: clarify overriding semantics of systemd-gpt-auto-generator

Specifically: /etc/fstab overrides the units itself, but not the deps.

See #168.

Merge pull request #205 from endocode/iaguis/seccomp-v2

nspawn: make seccomp loading errors non-fatal

hwdb: Update database of Bluetooth company identifiers

nspawn: make seccomp loading errors non-fatal

seccomp_load returns -EINVAL when seccomp support is not enabled in the
kernel [1]. This should be a debug log, not an error that interrupts nspawn.
If the seccomp filter can't be set and audit is enabled, the user will
get an error message anyway.

[1]: http://man7.org/linux/man-pages/man2/prctl.2.html

login: fix potential null pointer dereference

Fix CID 1304686: Dereference after null check (FORWARD_NULL)

However, this commit does not fix any bug in logind. It helps to keep
the elect_display_compare() function generic.

sysv-generator test: always log to console

Set $SYSTEMD_LOG_TARGET so that the output always goes to stdout/stderr. This
fixes running the test as root, as that logged to the journal previously.

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

update TODO

update TODO

l10n: Add Belarusian translation

Merge pull request #201 from mbiebl/drop-include_prefix

build-sys: Drop include_prefix

build-sys: Drop include_prefix

Appears to be unused and a leftover from the udev merge.

Merge pull request #144 from teg/udev-spawn-log-less-2

udevd: event - don't log about failures of spawn processes when this …

Merge pull request #200 from kaysievers/wip

build-sys: include libsystemd-journal and libudev in libshared

build-sys: include libsystemd-journal and libudev in libshared

Merge pull request #196 from dvdhrm/bus-map-props

tree-wide: fix memory leaks in users of bus_map_all_properties()

Merge pull request #198 from ivuk/fix_typo_timesyncd_conf

Fix typos in man/timesyncd.conf.xml

Merge pull request #192 from phomes/master

test-netlink-manual: typo fix

Fix typos in man/timesyncd.conf.xml

hashmap: fix iterators to not skip entries

Currently, the HASHMAP iterators stop at the first NULL entry in a
hashmap. This is non-obvious and breaks users like sd-device, which
legitimately store NULL values in a hashmap.

Fix all the iterators by taking a pointer to the value storage, instead of
returning it. The iterators now return a boolean that tells whether the
end of the list was reached.

Current users of HASHMAP_FOREACH() are *NOT* changed to explicitly check
for NULL. If it turns out, there were users that inserted NULL into
hashmaps, but didn't properly check for it during iteration, then we
really want to find those and fix them.

tree-wide: fix memory leaks in users of bus_map_all_properties()

If you use bus_map_all_properties(), you must be aware that it might
touch output variables even though it may fail. This is, because we parse
many different bus-properties and cannot tell how to clean them up, in
case we fail deep down in the parser.

Fix all callers of bus_map_all_properties() to correctly cleanup any
context structures at all times.

hwdb: add support for Alienware graphics amplifier

Unplugging and plugging in the cable will create various scancodes
on the keyboard controller.

Userspace within X should be able to interact with these to show
interesting messages. Assign them to generic prog1/prog2.

(David: add comment to hwdb explaining that these keycodes are reserved)

man: don't mention '/run' in hwdb.man

We do not support '/run' for hwdb files. Drop it from the man-pages so
people don't accidentally use it.

This was reported by: Peter Hutterer <peter.hutterer@who-t.net>

test-netlink-manual: typo fix

No functional change, but looked weird.

Merge pull request #178 from utezduyar/man-sd_bus_message_get_creds

Improve the documentation of bus credentials by mentioning send-time metadata. This needs more love, we should really clarify metadata details here. However, this is still better than nothing, so it's fine.

Merge pull request #183 from ssahani/net

Improve tun/tap logging by using the new log_*errno*() functions that set 'errno' explicitly. Also fix a bunch of incorrect errno/r confusions.

Merge pull request #191 from kaysievers/resolv

build-sys: merge convenience library libresolve

Merge pull request #189 from teg/rtnl-rename

Rename sd_rtnl to sd_netlink to prepare for further netlink-protocol support. Anything rtnl specific still uses the sd_rtnl prefix, but the generic parts (including the bus and message objects) are now called sd_netlink.

build-sys: merge convenience library libresolve

Merge pull request #187 from kaysievers/libcleanup

build-sys: simplify lib dependencies

build-sys: simplify lib dependencies

networkd: tuntap improve logging

Replaces  strerror() usage with log_netdev_error_errno()

sd-netlink: socket - move some functions from main source file

sd-netlink: message - split up source file

Split netlink-socket.c and rtnl-message.c from netlink-message.c.

sd-netlink: drop the write-queue

AF_NETLINK is not write-buffered, so this was actually never used.

sd-netlink: rename from sd-rtnl

Merge pull request #184 from l10n-tw/master

po: fix zh_TW mailing list again.

Merge pull request #167 from keszybz/line-oriented-ima-setup

ima-setup: write policy one line at a time

po: fix zh_TW mailing list again.

Merge pull request #179 from l10n-tw/master

po: fix zh_TW mailing list.

po: fix zh_TW mailing list.

Merge pull request #164 from l10n-tw/master

Add zh_TW translation.

po,catalog: add zh_TW translation.

man: mention sd_bus_message_get_creds as another way

of retrieving sd_bus_creds even though sd_bus_creds itself
contains very limited information.

selinux: whitespace fixes

Merge pull request #153 from crawford/cciss

rules: re-add cciss rules

Merge pull request #173 from mischief/ipforwarding-3

IPForwarding=kernel v3

Merge pull request #177 from ssahani/net

core: fix CID 996302

core: fix CID 996302

CID 996302:  Error handling issues  (CHECKED_RETURN)

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.

Merge pull request #171 from teg/rtnl-broadcast-2

sd-rtnl: make joining broadcast groups implicit

sd-rtnl: make joining broadcast groups implicit

Merge pull request #143 from teg/networkd-packets-per-slave-mode

networkd: bond - only set packets_per_slave on balance-rr mode

Merge pull request #156 from filbranden/journal_leading_whitespace

journald: do not strip leading whitespace from messages

Merge pull request #166 from zonque/kmod

kmod-setup: don't print warning on -ENOSYS

kmod-setup: don't print warning on -ENOSYS

-ENOSYS is returned from kmod_module_probe_insert_module() if a module isn't
available, not -ENOENT. Don't spit out a warning in that case unless the
warn_if_unavailable flag is set.

Also factor out the condition into an own variable for better readability.

Merge pull request #170 from teg/rtnl-recv

sd-rtnl: use netlink header rather than ucred in socket_recv_message()

sd-rtnl: use netlink header rather than ucred in socket_recv_message()

Lets us simplify the function and drop SO_PASSCRED.

Thanks to Alexander Larsson and David Herrmann.

ima-setup: write policy one line at a time

ima_write_policy() expects data to be written as one or more
rules, no more than PAGE_SIZE at a time. Easiest way to ensure
that we are not splitting rules is to read and write one line at
a time.

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

Merge pull request #84 from blueyed/zsh-optimize-filter_units_by_property

zsh-completion: optimize _filter_units_by_property

kmod-setup: when we fail to load a kmod, log the error cause

(Also, downgrade message from LOG_ERROR to LOG_WARNING, after all we
don't care much and just proceed)

core: Let two more booleans survive a daemon-reload

Without the boolean bus_name_good services as well as cgroup_realized
for units a unit of Type=dbus and ExecReload sending SIGHUP to $MAINPID
will be terminated if systemd will be daemon reloaded.

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=746151
https://bugs.freedesktop.org/show_bug.cgi?id=78311
https://bugzilla.opensuse.org/show_bug.cgi?id=934077

Merge pull request #158 from ssahani/net

Net

Merge pull request #157 from filbranden/revert_unused_result

build-sys: suppress warnings of unused results

Merge pull request #160 from kaysievers/libsplit

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

Merge pull request #150 from poettering/journal-no-recv-loop

journald: don't employ inner loop for reading from incoming sockets

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/

sd-network: fix CID 1306198

networkd: improve logging

Replace strerror() usage with log_netdev_error_errno()