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

man: document that systemd-getty-generator can start additional container gettys on ptys

getty-generator: look add an environment variable $container_ttys set for PID 1 and start gettys on all ttys listed therein

nspawn: add new --setenv= switch to set an environment variable for the container to spawn

man: document more error codes for sd_bus_request_name()

libsystemd-bus: the same error codes for sd_bus_release_name() (for kdbus and dbus1)

Due to this patch, sd_bus_release_name() function
returns the same code errors for kdbus and dbus1
if we try release non-existing name or foreign
name.

build-sys: fix help text for --enable-kdbus

update TODO

bus: when a connection terminated use ECONNRESET as error

bus: rework how we attach fds to event loops

When trying a couple of addresses one after the other, make sure to
reattach the fds to the event loop.

event: add ability to change fd of an active event source

event: allow to query userdata and watchdog state

man: add another nspawn example

Taken from https://bugs.freedesktop.org/show_bug.cgi?id=68369.

event: when unreffing an event source from its own handler, detach fd from epoll

The pattern of unreffing an IO event source and then closing its fd is
frequently seen in even source callbacks. Previously this likely
resultet in us removing the fd from the epoll after it was closed which
is problematic, since while we were dispatching we always kept an extra
reference to event source objects because we might still need it later.

event: be more conservative when returning errors from event handler callbacks

We really should return errors from event handlers if we have a
continous problem and don't know any other solution.

git: update .gitignore

namespace: include boot id in private tmp directories

This way it is easy to only exclude directories from the current boot
from automatic clean up in /var/tmp.

Also, pick a longer name for the directories so that are globs in
tmp.conf can be simpler yet equally accurate.

event: rework sd-event exit logic

With this change a failing event source handler will not cause the
entire event loop to fail. Instead, we just disable the specific event
source, log a message at debug level and go on.

This also introduces a new concept of "exit code" which can be stored in
the event loop and is returned by sd_event_loop(). We also rename "quit"
to "exit" everywhere else.

Altogether this should make things more robus and keep errors local
while still providing a way to return event loop errors in a clear way.

bus: fix make check

bus: properly parse NameOwnerChanged messages when caller explicitly wants to match against names coming/going

man: update sd_bus_request_name() man page

bus: make sure exit-on-idle logic works on kdbus systems that do not generate NameLost

bus: fix parsing of matches against empty strings

scan-build: silence some warnings

test-fileio/test-strv:
Use the streq_ptr to make build-scan not worry about passing in a null
to a nonnull function.

test-dhcp-option:
Prevent a theoretical null pointer dereference

bus: fix rewind logic

service: process watchdog timeouts with lowest priority

bus: remove explicit activator-specific flags, the kdbus supports it now

bus: update kdbus.h

bus: always pass valid timeout to kdbus

bus: enforce endianess and marshalling for messages we send

timedatectl: work with old timedated

Which does have TimeUSec. Should we specifically check for this method
instead of assuming time=0 means it doesn't exist?

Before:
shawn@debian-T61:~/git/systemd$ ./timedatectl
      Local time: Wed 1969-12-31 16:00:00 PST
  Universal time: Thu 1970-01-01 00:00:00 UTC
        RTC time: n/a
        Timezone: America/Los_Angeles (PST, -0800)
     NTP enabled: n/a
NTP synchronized: no
 RTC in local TZ: no
      DST active: no
 Last DST change: DST ended at
                  Sun 1969-10-26 01:59:59 PDT
                  Sun 1969-10-26 01:00:00 PST
 Next DST change: DST begins (the clock jumps one hour forward) at
                  Sun 1970-04-26 01:59:59 PST
                  Sun 1970-04-26 03:00:00 PDT

After:
shawn@debian-T61:~/git/systemd$ ./timedatectl
      Local time: Wed 2013-12-11 14:03:21 PST
  Universal time: Wed 2013-12-11 22:03:21 UTC
        RTC time: n/a
        Timezone: America/Los_Angeles (PST, -0800)
     NTP enabled: n/a
NTP synchronized: no
 RTC in local TZ: no
      DST active: no
 Last DST change: DST ended at
                  Sun 2013-11-03 01:59:59 PDT
                  Sun 2013-11-03 01:00:00 PST
 Next DST change: DST begins (the clock jumps one hour forward) at
                  Sun 2014-03-09 01:59:59 PST
                  Sun 2014-03-09 03:00:00 PDT

build-sys: use internal library for dhcp client tests

Also clean up AM_CFLAGS in a few places.

dhcp: Add function to free DHCP client data

dhcp: Add notification callback

Define a notification callback and events for stopping and client
lease expiry. Add functions to fetch IP parameters from a lease.

dhcp: Compute expire, T1 and T2 timers

Compute the default T1 and T2 timer values if they were not set by
the DHCP server. Verify that the values are reasonable.

dhcp: Process DHCP Ack/Nak message

Process a DHCP Ack/Nak in much the same way as an DHCP Offer. Factor
out header verification and process options sent. Add notification
functionality with discrete values for the outcome of the DHCP Ack/
Nak processing.

dhcp: Add maximum message size option

Add maximum message size option to keep some DHCP server implementations
from sending too big messages. See ConnMan commit
0c5c862749c05193cf4c513628328c6db02b5222.

dhcp: Send DHCP Request to acquire an IP address

Create and send a DHCP Request message reusing already existing parts
of the code. This causes factoring out IP and UDP header creation and
moving next timeout calculation to be done every time in the timer
callback function independent of DHCP state. Also add an exponential
part to the timer calculation, bail out if there are errors while
resending the DHCP message for the sixth or more times.

dhcp: Handle received DHCP Offer message

Create a function for handling the full IP, UDP and DHCP packet
and tie it to the main loop. Verify IP and UDP headers and checksum.
Creat a new lease structure with using the values supplied in the
DHCP message. Free the lease structure when client is stopped.

Split out socket handling into a creation and a sending part. As a
result modify the test code.

dhcp: Add timeout and main loop support

Require a main loop to be set when creating a DHCP client. Set up
a timer to resend DHCP Discover messages and add a 0-2 second
delay to the timeout value. Move to state Selecting after successful
sending of a Discover message.

build: Add dependency on libsystemd-bus needed for main loop

dhcp: Add function to stop the DHCP client

The client is stopped and brought back to its initial state.

dhcp: Support seconds elapsed since start of DHCP negotiation

It was noticed by Grant Erickson in ConnMan commit
95e15c09350acf58d4707056ae2614570883ef66 that:

   "Certain DHCP servers, such as that implemented in Mac OS X
    (< 10.7) for its "Internet Sharing" feature, refuse to issue
    a DHCP lease to clients that have not set a non-zero value
    in their DISCOVER or REQUEST packets."