bus: when releasing, taking, listing bus names check if they are valid

Given that the kernel does not validate bus names we have to when using
them.

tests: fix build

bus: sync with kdbus

Two new things here:

a) struct kdbus_notify_name_change now carries two struct
   kdbus_notify_id_change

b) a new KDBUS_CMD_BYEBYE ioctl that has no user yet in systemd

networkd: bridge - fix state machine

We were entering BRIDGE_STATE_CREATED rather than BRIDGE_STATE_READY.

networkd: link - remove useless states

Rework the state-machine a bit.

networkd: rename link_update_flags to link_update

We are likely to track more than the flags in the future.

bus-util: return errors when encountered

Introduced in:
9f6eb1cd58f2ddf2eb6ba0e4de056e13d938af75
313333b403439360c0396a50d77d0a1ee2bca4df

_noreturn_ --> noreturn for C11 compat

also define noreturn w/o <stdnoreturn.h>

util: no need for in_initrd() cache to be thread-local

the process only has one working directory, and a race is
harmless

__thread --> thread_local for C11 compat

Also make thread_local available w/o including <threads.h>.
(as the latter hasn't been implemented, but this part is trivial)

update TODO

libsystemd-bus: true/false instead of yes/no in msg dump

Due to this patch, message dump (for message which includes boolean
type) is more consistent with dbus-send (which display true/false
instead of yes/no for boolean). It's only simple 'cosmetics change'.

** For dbus-send **

dbus-send --system --dest=org.freedesktop.DBus --type=method_call --print-reply / org.freedesktop.DBus.NameHasOwner string:org.freedesktop.login1

method return sender=org.freedesktop.DBus -> dest=:1.97 reply_serial=2
   boolean true

** For libsystemd-bus (without this patch) **

‣ Type=method_call  Endian=l  Flags=0  Version=2 Serial=8
  Destination=org.freedesktop.DBus  Path=/org/freedesktop/DBus  Interface=org.freedesktop.DBus  Member=NameHasOwner
  MESSAGE "s" {
   STRING "org.freedesktop.login1";
  };

‣ Type=method_return  Endian=l  Flags=1  Version=2 Serial=51  ReplySerial=8
  Sender=:1.59  Destination=:1.67
  UniqueName=:1.59  WellKnownNames={org.freedesktop.DBus}
  MESSAGE "b" {
   BOOLEAN yes;
  };

For me true/false seems to be better readable than yes/no for BOOLEAN.

journal-file.c: remove redundant assignment of variable

we also do 'last_index = (uint64_t) -1;' at the end of the while
loop so there is no reason to also do it here.

bus: properly return an error code when release_name fails abnormally

bus-driverd: Fix return code in driver_request_name

RequestName return codes should be consistent with Dbus Specification.

VALUE - DESCRIPTION
1-The caller is now the primary owner of the name, replacing any previous owner,
2-The name already had an owner (QUEUE flag was not specified),
3-The name already has an owner (QUEUE flag was specified),
4-Application trying to request ownership of a name is already the owner of it.

journald: before closing /dev/kmsg let's unregister the event source

journald: close /dev/kmsg again if we cannot make use of it

journald: downgrade warning message when /dev/kmsg does not exist

dhcp: Properly handle unsuccessful cases for DHCP Ack

Ignore DHCP Ack packets if they are found to be erroneous or have
an unexpected xid by setting the result to zero instead of propagating
an error. Also remember to report a DHCP Nak and stop the DHCP client.

udev link-config: add asserts to silence scan-build

bus: removed unused variable in driverd

acked by Daniel Mack

bus: make driverd code more similar to other daemons, and make it exit on idle among other things

busctl: on kdbus the bus driver is a service like any other, hence don't skip it

bus: remarshal messages when necessary before sending

activate: fix error checking on epoll_ctl()

bus: install unit files for new driver service

bus: sync with kdbus

kdbus_cmd_{ep,ns,bus}_make are now consolidated, and bloom_size has
become an item of the dynamically sized list.

bus: sync up with kdbus

bus-driverd: Fix unique name return in driver_get_name_owner

man: networkd - clarify Address/Gateway keys in [Network] section

networkd: add support for Route sections

networkd: correct logging message

rtnl: fix for 32bits

Commit 0a0dc69b655cfb10cab39133f5d521e7b35ce3d5 broke tests for 32 bits

build-sys: fix distcheck

journal: fix against (theoretical) undefined behavior

While all the libc implementations I know return NULL when memchr's size
parameter is 0, without accessing any memory, passing NULL to memchr is
still invalid:

C11 7.24.1p2: Where an argument declared as "size_t n" specifies the length
of the array for a function, n can have the value zero on a call to that
function. Unless explicitly stated otherwise in the description of a
particular function in this subclause, pointer arguments on such a call
shall still have valid values, as described in 7.1.4. On such a call, a
function that locates a character finds no occurrence, a function that
compares two character sequences returns zero, and a function that copies
characters copies zero characters.

see http://llvm.org/bugs/show_bug.cgi?id=18247

man: describe journalctl --show-cursor

bus-driverd: actually add --version

systemctl: highlight filenames in cat

systemctl: fix return value from cat

man: reword awkward phrase

bus: use gperf to map error string to errno

bus: update kdbus.h

bus: when connecting to a container's system bus, double fork after joining the container's namespace

setns(CLONE_NEWPID) only applies to child processes, hence let's fork
once after joining the container's namespace.

build-sys: build bus-driverd if kdbus support is enabled

bus: when entering an existing namespace to connect to a container's system bus also switch over PID namespace

This is necessary to ensure that kdbus can collect creds of the
destination namespace when connecting.

core: always create /dev/kdbus/ns (and make it private 0700) after setting up the kdbus system bus

journal-file: protect against alloca(0)

bus: fix typo in systemd-bus-proxyd

Add bus-driverd

systemd-bus-driverd is a small daemon that connects to kdbus and
implements the org.freedesktop.DBus interface. IOW, it provides the bus
functions  traditionally taken care for by dbus-daemon.

Calls are proxied to kdbus, either via libsystemd-bus (were applicable)
or with the open-coded use of ioctl().

Note that the implementation is not yet finished as the functions to
add and remove matches and to start services by name are still missing.

bus: when a busname unit refuses to activate a service it should flush the queue

bus: also mask dbus.service in generator if kdbus is found

units: properly make bus proxy socket of type Accept=yes

systemctl: properly initialize and free sd_bus_error in "systemctl cat"

We need to properly initialize all error structs before use and free
them after use.

Also, there's no point in flushing stdout if we output a \n anyway...

bus: fix symlink to bus proxy socket

bus: use GREEDY_REALLOC() when allocating message queues

bus: let's use GREEDY_REALLOC() when allocating space for containers

execute: also set SO_SNDBUF when spawning a service with stdout/stderr connected to journald

loopback-setup: remove stray hunk

network: more asserts to shut up scan-build

kdbus.h: add alignment requirements

network: use SETLINK to bring up interfaces

rtnl: replace message_append by typesafe versions

rtnl: support interleaved reading and writing, and rewind

rtnl: simplify route_new()

Drop most of the arguments and instead introduce set_dst_prefixlen().

rtnl: simplify link_new()

Drop most of the arguments and instead introduce link_set_{flags,type}.

rtnl: match - only match on one type at a time

rtnl: clean up/add asserts

rtnl: add support for receiving route messages

bus: increase the bus socket buffer to 8 MB similar, to the log socket buffers

update TODO

util: try harder to increase the send/recv buffers of sockets

If we have the priviliges we will try SO_SNDBUFFORCE/SO_RCVBUFFORCE and
only fall back to SO_SNDBUF/SO_RCVBUF if that fails.

catalog: fix language detection

Detection would fail if language was not specified in the filename
but a dot appeared somewhere higher in the path.

core: refuse doing %h, %s, %U specifier resolving in PID 1

These specifiers require NSS lookups to work, and we really shouldn't do
them from PID 1 hence. With this change they are now only supported for
user systemd instance, or when the configured user for a unit is root.

build-sys: warn if builds are not byte-by-byte reproducible due to usage of __DATE__ and suchlike

Fix segv introduced by 2fd069b18e525860514a70d3ea08410ca122d3e2

n->path is pointing to the value now, we set s = NULL above.

build-sys: add cppcheck target

This uses --enable=all mode. Should be taken with a grain of salt
though. While many recommendations make sense we should probably keep
"int r" always on function scope, and many of the portability warnings
really don't matter to us because we only care for Linux/glibc.

Fix a few signed/unsigned format string issues

Since numbers involved are all small, behaviour was correct already.

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

Fix a few resource leaks in error paths

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

bus: _printf_ attributes should be on prototypes not function definitions for non-static functions

bus: update kdbus monitoring interface

udev-builtin-path: fix printf specifiers

bus: return negative errno on error

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

test-journal-syslog: use streq_ptr since we pass in NULL

systemctl: add the --plain option to the help message

man: remove advice to avoid setting the same var more than once

So far the compatibility with .desktop settings hasn't been imporant
at all, and we do not want people to write convoluted unit
files.

man: beef up ExecStart description

We have lots of questions from people who assume that shell syntax works
here, so let's be very explicit what is allowed and what is not. A few
examples should also help.

http://bugs.debian.org/732156

build: use -ftrapv for development

We want to find these bugs if they exist.

Add more _printf_'s for format-nonliterals

Clang is a bit more strict wrt format-nonliterals:
http://clang.llvm.org/docs/LanguageExtensions.html#format-string-checking

Adding these extra printf attributes also makes gcc able to find more
problems. E.g. this patch uncovers a format issue in udev-builtin-path_id.c

Some parts looked intetional about breaking the format-nonliteral check.
I added some supression for warnings there.

event: some snake-oil to speed up impossible error checks

event: instead of reset the revents field when we get new revents data from epoll, OR it in

event: dynamically adjust size of events array instead of pre-allocating it possibly too large or too small

util: port last code over to new namespace utility calls

util: when joining a namespace make sure to reset all uids to 0 after
the transition

bus: when connecting to a container's kdbus instance, enter namespace first

Previously we'd open the connection in the originating namespace, which
meant most peers of the bus would not be able to make sense of the
PID/UID/... identity of us since we didn't exist in the namespace they
run in. However they require this identity for privilege decisions,
hence disallowing access to anything from the host.

Instead, when connecting to a container, create a temporary subprocess,
make it join the container's namespace and then connect from there to
the kdbus instance. This is similar to how we do it for socket
conections already.

THis also unifies the namespacing code used by machinectl and the bus
APIs.

getty-generator: fix stripping /dev/

bus: install systemd-bus-proxyd unit files for compatibility with dbus1

getty-generator: fix segfault when $container_ttys is not set

event: make gcc shut up

journal: downgrade comments about sd_journal_print_with_location()

Effectviely these calls are ABI anyway, so downgrade the comments a bit,
since it might actually be useful for language bindings to make use of
them.

update TODO