bus: open user, not system, bus in bus_open_user_systemd()

util.c: use read_one_line_file where possible

shared: procfs_file_alloca: handle pid==0

when pid is set to 0 use /proc/self

Update .mailmap file

This commit updates email addresses of people, who are already in the
.mailmap file, so I'd assume they have sorted out their viewpoint on
privacy within the .mailmap file.

The entries for this commit have been produced using:
        # Finding out duplicates by comparing email addresses:
        git shortlog -sne |awk '{ print $NF }' |sort |uniq -d

        # Finding out duplicates by comparing names:
        git shortlog -sne |awk '{ NF--; $1=""; print }' |sort |uniq -d

swap: remove if/else with the same data path

This was introduced in e1770af812 (2012-02-03, swap: replace failure
boolean by result enum).

This just removes unneeded lines of code, no functional change.

man: document some more bus functions

networkd: refuse to use .network files with missing Address/Gateway key

These keys are mandatory in [Address]/[Route] sections. Otherwise, we
hit an assert:

ens3: setting addresses
Assertion 'address->family == 2 || address->family == 10' failed at /build/amd64-generic/tmp/portage/sys-apps/systemd-9999-r1/work/systemd-9999/src/network/networkd-address.c:137, function address_configure(). Aborting.

Reported-by: Alex Polvi <alex.polvi@coreos.com>
At the same time make sure Route's Destination and Gateway uses the same address family.

man: typo fix

sd-event: add missing symbol, sd_event_source_set_io_fd

This appears to have been omitted in 30caf8f3afd29da8 when the
the function was added.

udev: net_id - handle virtio buses

This was already supported in path_id, so should be uncontroversial.

Revert "net-util: add support for Type=ethernet"

This reverts commit 4cd1214db6cf4b262e8ce6381bc710091b375c96.

This may still be fixed in the kernel, revert this for now until
we see how it all shakes out.

systemctl: improve readability on failed commands

Not long ago a failed command would print:
"Failed to start something.service: ..."
regardless of whether the command was to start/stop/restart/etc.

With e3e0314 this was improved to print the method used. E.g. for stopping:
"Failed to StopUnit something.service: ..."

This patch matches the method to a more human readable word. E.g:
"Failed to stop something.service: ..."

man: document a batch of bus functions

man: networkd - document DHCP option

net-util: add support for Type=ethernet

When DEVTYPE is not set for a nic, it means it is a wired/ethernet
device.

networkd: link hash uses 64, not 32 bit keys

networkd: print the ifindex of added links

This debug information may be useful when comapring to dropped rtnetlink messages.

networkd: improve logging

This gives a bit better messages when a link is added twice.

networkd: improve link state change logging

networkd: add some debug info about notifications we ignore

bus: always create --user bus when kdbus is active

We set the variable in the unit file for --user, so this check is
always true.

networkd: link - explicitly set the link to be up if the call to IFF_UP succeeds

No need to wait for the NEWLINK message to arrive.

networkd: fix memory leak in error path

networkd: do not deference null pointer in cleanup

core: fix gcc unused variable warning

bus: fix memleak in sd_bus_creds_new_from_pid

Introduce cleanup functions for cap_free

Unfortunately a different cleanup function is necessary per type,
because cap_t** and char** are incompatible with void**.

build-sys: add --disable-dbus autoconf option

build-sys: make valgrind-tests target output nicer

rtnl: fix memory corruptions after realloc

struct sd_rtnl_message would keep two additional pointers into the hdr
field. Every time hdr was realloced, those pointers should be adjusted,
but weren't. It seems less error-prone to keep offsets instead.

bus: break reference cycle between bus and messages

Because messages in the read and write queues hold a reference on the
bus, and the bus holds a reference on each of them, we would never
free the bus if the read or write queues were not empty. Explicitly
substract the number of messages in those queue from the bus reference
count when deciding whether to free or not.

A simple test which creates and unrefs simple objects is added.

Use format patterns for usec_t, pid_t, nsec_t, usec_t

It is nicer to predefine patterns using configure time check instead of
using casts everywhere.

Since we do not need to use any flags, include "%" in the format instead
of excluding it like PRI* macros.

Move bus path definitions to def.h

core/service: check if mainpid matches only if it is set

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

core/manager: print info about interesting signals

Information about signals which are not routinely received by systemd
are printed at info level. This should make it easier to see what is
happening in the system.

core/service: tweak messages on operation timeouts

libsystemd-dhcp: don't assert on client_stop()

We need to be able to stop the dhcp client at any time, and the function appears to be written to handle this.

networkd: dhcp - avoid null pointer dereference

networkd: add more asserts and ignore all events when link has failed

rules: drivers - do not reset RUN list

networkd: only track state of links we are managing

If a network is not (yet) set for a link, we do not care about its state (as we
anyway don't know what to do with it).

networkd: improve logging

Remove redundant messages, add some debugging ones and make wording more uniform.

networkd: fix NULL pointer deref

networkd: fix getting initial state

We were requesting the state and then ignoring it...

networkd: add DHCPv4 support

This adds basic DHCPv4 support. Link-sense is enabled unconditionally,
but the plan is to make that configurable.

I tested this in a VM with lots of NICs and over wifi in the various
coffee shops I found this Christmas, but more testing would definitely
be appreciated.

networkd: distinguish between static and dynamic addresses/routes

Static addresses/routes are associated with a network. Dynamic
addresses/routes are associtade with links (as the corresponding network
may be shared by several links).

network: add support for dropping address

build-sys: make test output a bit nicer

dhcp: fix creation of req_opts array

GREEDY_REALLOC needs to have two size variables: one for the
allocated size, and a second one for the used size. Using
the allocated size only lead to leaving some elements unitialized
and assigning some more than once.

build-sys: make test-dhcp-* statically linked

This makes them behave like everything else following
48d7417d3 'build-sys: link most internal libraries statically'.

sleep-config: fix double free

Before 34a3baa4d 'sleep-config: Dereference pointer before check for NULL'
oom conditions would not be detected properly. After that commit, a double
free was performed.

Rework the whole function to be easier to understand, and also replace
strv_split_nulstr with strv_new, since we know the strings anyway.

boot-efi: Remove superfluous assignment

2 lines after the changed line we assign err to efi_get_variable(...)
unconditionally, so it makes no sense to initialize it to some value.

sleep-config: Dereference pointer before check for NULL

This fixes a bug pointed out by http://css.csail.mit.edu/stack/
(Optimization-unstable code)
It is a similar fix as f146f5e159 (2013-12-30, core:
Forgot to dereference pointer when checking for NULL)

To explain this bug consider the following similar, but simpler code:
if (!p)
free(*p)

Assume the if condition evaluates to true, then we will access *p,
which means the compiler can assume p is a valid pointer, so it could
dereference p and use the value *p.
Assuming p as a valid pointer, !p will be false.
But initally we assumed the condition evaluates to true.

By this reasoning the optimizing compiler can deduce, we have dead code.
("The if will never be taken, as *p must be valid, because otherwise
accessing *p inside the if would segfault")

This led to an error message of the static code checker, so I checked the
code in question.

As we access *modes and *states before the check in the changed line of
this patch, I assume the line to be wrong and we actually wanted to check
for *modes and *states being both non null.

bootchart: Remove unneeded check for NULL

Directly before the changed line there is:

while ((parent->next_ps && parent->pid != ps->ppid))
parent = parent->next_ps;

which looks one element ahead of the list, hence we can rely on parent
being non null here.
If 'parent' were NULL at that while loop already, it would crash as we're
dereferencing 'parent' when checking for next_ps already.

Signed-off-by: Stefan Beller <stefanbeller@googlemail.com>

tmpfiles: rename --unsafe to --boot

As suggested by Kay, it is better to describe what is done,
not what might happen.

build-sys: fix systemd-stdio-bridge link to work with split /usr

shumski> there seems to be slight error in systemd git master Makefile.am

core: Forgot to dereference pointer when checking for NULL

Actually we already checked for !rt before, now we'd like to examine
the return value of the memory allocation.

man: expand on some more subtle points in systemd.socket(5)

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=727708#1694

systemd-python: fix listen_fds under Python 2

systemd-python: fix setting of exception codes

The return value of 0 would be treated as failure by mistake,
resulting in " SystemError: error return without exception set".
The way that set_error() is used is changed to be the same
everywhere.

acticate: rename --environment to --setenv

Nspawn has --setenv, and systemd itself accepts systemd.setenv.
It is nice to have the same parameter name everywhere.

Old name is accepted, but not advertised.

udev: LOG_PRIORITY -> LOG_LEVEL

autogen: add shortcut for running scan-build

scan-build is a static analyzer in llvm. As ususal static analyzers
tend to mostly find theoretical bugs in software that has been in
production for a while. For in-development code it can be useful to
check if new issues is added as there is a chance to spot real problems
before release. For systemd we are now down to 297 issues - the vast
majority are false positives because the tool does not understand the
cleanup attribute.

Running clang's static analyzer scan-build is a bit messy. You have to
run both configure and make "inside" the build-scan tool. To have an
easy shortcut from autogen.sh I thus call both directly from it. This
makes it different from the other options in autogen.sh. I chose 's'
for static analysis.

scan-build is in the package clang-analyzer on fedora.

autogen: add shortcut for building with clang

For a while I have been cleaning up warnings when building with clang.
There are currently only two sources of warnings left: Wcast-align and Wgnu.
I am not convinced that fixing up those up is feasible so I run with them
disabled to spot regressions. E.g. clang is a bit more strict wrt to unused
variables with the cleanup attribute and I have fixed a number of those since.

Like the other options in autogen.sh I have a shortcut for clang as well. I use
'l' for llvm.

gitignore: add back user@.service

bus: fix a couple of format string mistakes

bus: driverd; add missing format string parameter

This was causing a nasty coredump

bus: correct a number of gcc format string attribute usages

man: include autoconf snippet in daemon(7)

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

selinux: Check access vector for enable/disable perm for each unit file

SELinux check will be done using the context of the unit file as as a
target instead of the default init_t context, allowing selinux control
on the level of individual units.

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

shared: add simplistic XML parser for usage in the D-Bus policy language compat parser

bus: fix running tests in parallel

man: add a note about propagating signals

hwdb: Update database of Bluetooth company identifiers

build-sys: fix generation of user@.service

units: user@.service: fix user bus path

bus: PORTING-DBUS1: fix user bus path

systemctl: remove unused variable

bus: PORTING-DBUS1 typo fixes

man: fix Type= reference v2

grawity:

  It looks like the old version _was_ correct – the default value will
  be "Type=dbus" if the service has a BusName set.

  Suggested change: "if neither Type= nor BusName= is specified"

bus: PORTING-DBUS1 clarify pool size value

bus: PORTING-DBUS1 update

bus: update PORTING-DBUS1

bus: actually, the kernel does enforce validity of bus names...

bus: explain obsoletion of dbus1 .service files

bus: refuse invalid names from kbus meta data

bus: add some preliminary docs for porting existing dbus1 client libraries to kdbus

bus: update kdbus.h

man: fix Type= reference

Simple man page fix attached.

--
Marcos

From 268d10a2f8769fd1dcb9440670af15ac02c5df89 Mon Sep 17 00:00:00 2001
From: Marcos Mello <marcosfrm@gmail.com>
Date: Thu, 26 Dec 2013 17:19:04 -0200
Subject: [PATCH 1/1] man: fix Type= reference

Use enums to make it obvious what boolean params mean

Suggested-by: Russ Allbery <rra@debian.org>

systemctl: allow globbing in commands which take multiple unit names

systemctl: drop uninteresting units immediately

Also properly free memory if list-unit-files --root=... is used.

systemctl: simplify start_unit

man: fix description of is-enabled returned value

bus: add a temporary hard-coded policy to proxyd to make gdm work

gdm relies on the policy to deny its own requests to not deadlock. Given
that we currently do not enforce any policy in the dbus1 compat proxy
service this means that gdm will necessarily deadlock on our systems.

To work around this, enforce a fixed policy teomporarily, until we
interpret the legacy XML policy in full.

Booh, gdm, booh, for requring this and making me waste two days on
tracking this brokenness down.

bus: always show messages we send

man: document fail/nofail, auto/noauto

Also s/filesystem/file system/ in a few places.

Trim TODO

man: grammar and wording improvements

This is a recurring submission and includes corrections to:
- missing words, preposition choice.
- change of /lib to /usr/lib, because that is what most distros are
  using as the system-wide location for systemd/udev files.

man: improvements to comma placement

This is a recurring submission and includes corrections to:
comma placement.