bus-proxy: ECONNRESET/ENOTCONN can hit us on every step, hence handle it on every step

bus-proxy: a few simplifications

sd-bus: initialize a few structs at time or declaration

bus-proxy: close each connection fd only once

After passing the fds over to the sd_bus object, we should forget them,
so that we don't close them a second time when the object goes away.

bus-proxy: also consider ENOTCONN a clean termination condition

Sometimes, when we try to reply to messages we don't check return
values. This means we might miss a ECONNRESET, and will get a ENOTCONN
on next command. Treat both the same hence.

exit-on-idle: only exit if actually idle

sd_event_wait() returning 0 usually means that it timed out, which means it must
have been idle. However, sd_event_wait() may return 0 in case an event was triggered
but it turned out there was nothing to do. Make the check for idle explicit to avoid
this edge-case.

bus-proxy: whenever we cannot forward a message, report this back to caller, but don't exit

Errors like EPERM from the kernel should certainly not be reason to
exit. Let's try to be defensive here, and try to continue on most send
errors, but possibly tell the sender about it.

bus-proxy: rename synthetic_reply_return_strv() to synthetic_reply_method_return_strv()

That way it matches more closely the nomenclature of our other
success reply calls.

bus-proxy: minor simplifications

sd-bus: export sd_bus_error_set_errnofv()

bus-proxy: no need to negate error codes, log_error_errno() already does it

bus-proxy: tell Coverity we don't care about these return values

bus-proxy: we don't pointlessly abbreviate function names

It's fine to abbreviate local variables, but it's not OK to abbreviate
function names needlessly. This is not an excercise in writing
unreadable code.

sd-bus: sync kdbus.h (ABI break)

Another slice logic rework in kdbus made KDBUS_ITEM_PAYLOAD_OFF items
relative to the message header again. Catch up with that in sd-bus.

units: turn on --network-veth by default for systemd-nspawn@.service

Given the recent improvements in networkd, it's probably the better
default now.

units: fix systemd-networkd.service in containers lacking CAP_NET_ADMIN

hwdb: update name databases

build-sys: already bump package version in preparation for next release

(don't misunderstand this, the release is still out quite a bit...)

build-sys: bump sonames in preparation for next release

NEWS: add contributors list

rules: simplify mmc RPMB handling

We don't actually want a by-path/ symlink for MMC RPMB devices, so just add
them to the blacklist. This will prevent creating wrong by-path links and
blkid'ing those.

rules: Fix by-path of mmc RPMB partitions and don't blkid them

Linux 3.10+ exposes RPMB (Replay Protected Memory Block) partitions of MMC
devices [1] ; trying to read them with blkid or other unspecific means will
cause kernel buffer I/O errors and timeouts. So don't run blkid on these.

Also ensure that /dev/disk/by-path creates proper symlinks and exposes the
-rpmb partition separately, instead of letting the "normal" partition symlink
point to the rpbm device (this is a race condition).

[1] http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=090d25fe224c0

https://launchpad.net/bugs/1333140

include <poll.h> instead of <sys/poll.h>

include-what-you-use automatically does this and it makes finding
unnecessary harder to spot. The only content of poll.h is a include
of sys/poll.h so should be harmless.

Add missing includes in header files

This fixes various issues found by globally reordering the include
sections of all .c files.

test-util: remove superfluous const

remove unused variable

exec: also evaluate working_directory_missing_ok when not applying chroots

NEWS: begin collecting news items for 219

networkd: explicitly enable/disable ipv6ll addresses

The kernel does not like getting an empty container, so just always pass the parameter, even if
it is the default.

hwdb: adjust --help text to be more like other --help texts

Fix capitalization, indentation, make sure we support --version, and
similar.

core: disarm shutdown watchdog if we fail to set timeout

Better safe than sorry, if drivers are stupid, and reset immediately on
device closing if the timeout could not be initialized.

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=777735

core: don't fail to run services in --user instances if $HOME is missing

Otherwise we cannot even invoke systemd-exit.service anymore, thus not
even exit.

https://bugs.freedesktop.org/show_bug.cgi?id=83100
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=759320

Revert "tree-wide: Always use recvmsg with MSG_CMSG_CLOEXEC"

This reverts commit d6d810fbf8071f8510450dbacd1d083f37603656.

It's apparently not OK to pass MSG_CMSG_CLOEXEC to recvmsg() of raw
sockets.

missing: add __NR_getrandom for powerpc architecture

core: don't watch for socket events when service is in SERVICE_STOP

If a service has an ExecStop= program that uses the socket the service was
activated by to tell it to terminate, then that should not be used as
trigger to start the service again.

http://lists.freedesktop.org/archives/systemd-devel/2015-February/028058.html

systemctl: don't update the reboot parameter if none is given

Otherwise systemd-reboot.service will remove the parameter that was set
before. This was broken in commit "b986229efe2cc96157aa14c37bab7843311bbef1
systemctl: bugfix for systemctl reboot command with argument"

core: correct a log message

The log message talks specifically about services, though it actually
applies to any kind of unit.

tree-wide: whenever we include libgen.h, immediately undefine basename()

Also, document in adjacent comments and in CODING_STYLE why we do that.

core: remove unneeded <libgen.h> include

execute.c only uses basename (the GNU version in <string.h>)

Revert "units: add SecureBits"

This reverts commit 6a716208b346b742053cfd01e76f76fb27c4ea47.

Apparently this doesn't work.

http://lists.freedesktop.org/archives/systemd-devel/2015-February/028212.html

ata_id: remove unused header files

Signed-off-by: Robert Milasan <rmilasan@suse.com>

units: add SecureBits

No setuid programs are expected to be executed, so add
SecureBits=noroot noroot-locked
to unit files.

man: fix typo

network-internal: fix inverted matching

This fixes the test-suite again.

networkd: fdb - fix const warning

networkd: don't do exit-on-idle if there is something wrong with the dbus connection

Just fall back to plain old event loop in this (highly unlikely) case.

In the glorious future when kdbus is upstream we can of course drop all this.

sysv-generator: Skip init scripts for existing native services

This avoids taking the SysV init script enablement state into account if we
have native units. Otherwise systemctl disable on native unit would not
be respected in the presence of an enabled SysV script.

Also, there's no need to do all the parsing and creation of service files if we
already have a native systemd unit for the processed SysV init script.

Update TODO

networkd DHCPv4 logging endian fix

On Tue, Feb 10, 2015 at 08:10:43PM +0100, Lennart Poettering wrote:

> Hmm, I think it would be nicer to use be32toh() here instead, since it
> ensures the macro is (to a limited degree) typesafe.
>
> Any chance you could rework that?

From: Paul Martin <paul.martin@codethink.co.uk>
Date: Wed, 11 Feb 2015 11:47:16 +0000
Subject: [PATCH] networkd dhcpv4 logging endian fix

On a big-endian host, systemd-networkd prints out IPv4 network
addresses byte reversed:

Feb 10 16:43:32 hostname systemd-networkd[151]: eth0 : DHCPv4 address 158.1.24.10/16 via 1.1.24.10

The address obtained is 10.24.1.158/16 and the route is

  10.24.0.0/16 dev eth0  src 10.24.1.187

The macro ADDRESS_FMT_VAL() unpacks a "struct in_addr" in a
little-endian specific manner.

This patch forces the passed address into host order, then unpacks it.

On an x86 later than i486, compiled with -O2, the only extra overhead
is a single bswap instruction.

man: boilerplate unification

man: fix reference to glob manpage

man: add systemd.generator(7)

This is largely based on http://www.freedesktop.org/wiki/Software/systemd/Generators/,
and obsoletes that page. It seems that we do a much better
job of keeping man pages up-to-date compared to wiki pages.
Man pages are also easier to find for users.

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

TODO

net: support globbing and disjunction in Match logic

Match{Name,OrginalName,Type,Driver,Path} can now take a space-separated glob of matches.

networkd: fdb - refactor a bit

Pass around Link objcets rather than FdbEntry objects. The link objects have an up-to-date
ifname we can use for logging. match_name sholud _never_ be used for anything except
matching. Firstly, it may be unset (usually is), and secondly it may not be up-to-date.

networkd: bus - expose MatchMAC

man: sd_bus_path_encode - fix order of arguments

networkd: network-bus - use sd_bus_path_{en,de}code

CODING_STYLE: elaborate on O_CLOEXEC a bit

tree-wide: Always use recvmsg with MSG_CMSG_CLOEXEC

test-lldp: fix minor OOM

test-lldp: initialize structs when we define them

test-lldp: use strndup() where possible

libsystemd-network: Avoid potential NULL dereference in test-lldp

As a malloc0 could fail, doing a strncpy without checking could
cause issues. Adding an assert should be good enough and in line
with other similar routines in the code.

Found with Coverity Fixes: CID#1261402

update TODO

bus-proxyd: initialize ioctl structure only once

.gitignore: add systemd-pull

journald: don't specify inline in local functions

Leave it to the compiler to figure out whether it shall inline stuff or
not.

Only place where using static inline is OK to use is in in header
files, really.

shared: untabify

logind: tell Coverity that we knowingly ignore mkdir()'s return value

update TODO

keymap: Add microphone mute keymap for several HP laptops

On these several HP laptops, the microphone mute hotkey is "Fn+F8" and
the scancode for this hotkey is 0x81, but this scancode was mapped to
fn_esc in the HP generic keymap section. To fix this problem, we add
a machine specific keymap section to add the correct keymap rule.

BugLink: https://bugs.launchpad.net/bugs/1409721
BugLink: https://bugs.launchpad.net/bugs/1334968
Signed-off-by: Hui Wang <hui.wang@canonical.com>

networkd: link_object_find - don't accept invalid input

Reported by Zbigniew.

networkd: support route scopes

For now we only support the hardcoded values RT_SCOPE_{UNIVERSE,LOCAL,HOST},
and not numerical values or values from /etc/iproute2/rt_scopes.

This addresses https://bugs.freedesktop.org/show_bug.cgi?id=88508.

treewide: correct typos and use consistent "MAC" spelling

networkd: bus - switch to properly escaped object paths

networkd: add support for IPv6 tokens

This allows the admin to set the host-specific part of IPv6 addresses, but still
receive the prefix via SLAAC.

.network file snippet:

[Network]
IPv6Token=::12

gives:

$ ip token
token ::12 dev eth0

This closes https://bugs.freedesktop.org/show_bug.cgi?id=81177.

networkd: generalize IPv4LL to LinkLocal

This allows both IPv4 and IPv6 link-local addresses to be enabled or disabled. By default
we still enable IPv6LL and disable IPv4LL. The old config option is kept for backwards
compatibility, but removed from the documentation.

sd-rtnl: fix typesystem for IFLA_AF_SPEC

Got this one wrong, it is not a union, just a nested container.

language fallback: it_CH (Italian, Swiss) -> it_IT (Italian, Italy)

networkd: add basic org.freedesktop.network1.Network interface

networkd: add network_get_by_name

build-sys: properly enable the networkd dbus activation

sd-rtnl: add support for IFLA_INET6_*

sd-rtnl: extend type system to allow address-family to decide the union members

So far we only supported selecting them by sibling attributes.

(This stuff is all a bit crazy, but there seems to be no other way...)

networkd: use valid bus paths

Object path components must start with [A-Za-z_] (AFAIK).
Also the value of 'p' is undefined if asprintf fails.
Compare to user_bus_path() in src/login/logind-user-dbus.c:281.

sd-rtnl: don't treat 0-length messages specially

Still parse the CMSG data, and most importantly make sure we drop the message when peeking.

sysctl: consider --prefix while parsing the files

not while applying the parsed sysctl values. Otherwise
info "Overwriting earlier assignment of %s in file %s" is
visible many times even though the given --prefix doesn't
try to set the overridden value.

This also optimizes the startup tiny bit since we have udev
rules running on network devices and setting sysctl through
the rules.

Add Silesian and Kashubian to language fallback map

Silesian and Kashubian speakers would prefer to fall back to Polish (pl)
translations instead of English (C).

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

bus-proxyd: fix 'ListQueuedOwners' call

Set proper kdbus_cmd_list object size, otherwise:

dbus-send --system --dest=org.freedesktop.DBus --type=method_call \
print-reply / org.freedesktop.DBus.ListQueuedOwners string:org.freedesktop.systemd1

Error org.freedesktop.DBus.Error.InvalidArgs: Invalid argument

localed: add LANGUAGE= fallback when LANG= is specified

For the entries listed in the first column of language-fallback-map,
the entry from the second column will be used for LANGUAGE=, if
LANGUAGE= is not explicitly specified.

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

Add language fallback map

This map will be used to provide a fallback for translations.
For example, a Niederdeutsch (nds) speaker prefers to fall back to
German (de) translations rather then the English (C) ones.

https://bugzilla.redhat.com/show_bug.cgi?id=624158#c9

build-sys: remove reference to long gone file

Interesting that automake doesn't complain about this at all.
That file was removed in 3fb97a58fa3f233.

units: networkd - fix busname to work on kdbus

sd-rtnl: don't fall over when receiving 0 bytes from the socket

causes EOF. Seems like a kernel bug. Ignoring it seems to work be the best we
can do for now...

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

sd-rtnl: fix creation of synthetic error replies

udev: net_setup - clarify reason for failure of persistent mac address policy

shared/log: read /proc/cmdline only in daemons

http://lists.freedesktop.org/archives/systemd-devel/2015-February/027960.html

update TODO

networkd: move the connection to the bus out of manager_new (again)

This would otherwise make the tests fail as we cannot grab the bus name.