man: sd_unref does not necessarily free the bus

Document that sd_unref() does not necessarily free the bus,
even if no part of the program explicitly refs it, due to
the queued messages.

Merge pull request #531 from dvdhrm/boot-buildid

boot: use BUILD_ID if VERSION_ID is not available

Merge pull request #532 from dvdhrm/bus-managed-root

sd-bus: include queried path in GetManagedObjects

basic/util.c fopen_temporary(): close fd if failed

Merge pull request #529 from dvdhrm/bus-slot-description

sd-bus: sd_bus_slot_get_description() should return const strings

Merge pull request #426 from tblume/reload-system-conf-at-daemon-reload

Reload manager defaults at daemon-reload

sd-bus: include queried path in GetManagedObjects

If GetManagedObjects is called on /foo/bar, then it should also include
the object /foo/bar, if it exists. Right now, we only include objects
underneath /foo/bar/.

This follows the behavior of existing dbus implementations.

Obsoletes #527 and fixes #525. Reported by: Nathaniel McCallum

boot: use BUILD_ID if VERSION_ID is not present

According to os-release(5), VERSION_ID is not mandatory and BUILD_ID only
needs to be unique underneath VERSION_ID. Therefore, assuming a missing
VERSION_ID field means 'empty', we can rely on BUILD_ID to be unique.

Use BUILD_ID if VERSION_ID is not present. This way, rolling-release
distros can still provide a proper os-release entry without crafting
random VERSION_ID strings.

This fixes #186.

boot: fix memleaks in os-release parser

There is no guarantee that the os-release section contains each key only
once, nor any guarantee that all keys are present. Make sure we properly
free memory in both cases.

Not that it matters much, as we're short-living, anyway. But correct code
is always nicer to read..

sd-bus: sd_bus_slot_get_description() should return const strings

All other *_get_description() functions use 'const char**', so make sure
sd_bus_slot_get_description() does the same.

This changes API, but ABI stays stable. I think this is fine, but I
wouldn't mind bumping SONAME.

Reported in #528.

Reload manager defaults at daemon-reload

"systemctl daemon-reload" should also update the manager defaults from
/etc/systemd/system.conf.
For details, see:
http://lists.freedesktop.org/archives/systemd-devel/2015-June/033062.html

Amended to use  manager_set_defaults() as common function.

Merge pull request #526 from phomes/master

machine: remove unused variables

machine: remove unused variables

Merge pull request #522 from mbiebl/no-install-readme-md

build-sys: Do not install README.md as end-user documentation

build-sys: Do not install README.md as end-user documentation

This file is mostly GitHub glue and not particularly useful to
end-users. So only add it to the dist tarball but do not install it as
documentation.

Merge pull request #516 from utezduyar/consistent-get-callback-return

property callback returns are consistent

Merge pull request #500 from zonque/fileio

fileio: consolidate write_string_file*()

Merge pull request #520 from zonque/pr-384-rework

man-pages: PR 384 rework

man: minor updates to the sd_bus_request_name() documentation

man: document user slice sd-login calls we added a while back

man: fix sd_bus_negotiate_timestamps documentation link-up

man: fully document sd_bus_creds subsystem

[@zonque: typo fixed, reported by @ronnychevalier]

man: fully document sd-bus' error APIs

[@zonque: Some minor nits fixed as pointed out by @ronnychevalier,
 dropped class='sd-bus-errors' to fix python logic]

Merge pull request #519 from poettering/man-bus-message-append

man: update and extend the various sd_bus_message_append_*() man pages

Merge pull request #514 from teg/resolved-rrsig-marshal

resolved: fix marshalling of RRSIG records

Merge pull request #515 from xnox/no-dot-files

automake: Do not install developer files on end-user systems.

Merge pull request #510 from zonque/journal-gatewayd

journal-gatewayd: fix tmpfile logic

Merge pull request #501 from keszybz/remove-python-systemd

Remove python-systemd

property callback returns are consistent

It is no different to return 0 over 1 in the property
callback. It is confusing to return 1 which made me think
1 has a special purpose. This way code is consistent with
the rest of the tree.

automake: Do not install developer files on end-user systems.

autogen.sh, .dir-locals.el, .vimrc, .ycm_extra_conf.py, .travis.yml,
.mailmap files are only useful with the source tree, for the
developers. Do not install these files as documentation on the
end-user systems, but keep them distributed with the tarball.

resolved: fix marshalling of RRSIG records

The key tag is 16, not 8 bits.

journal-gatewayd: fix tmpfile logic

"rw" is not a valid mode string for f*open(). This got broken in
cc02a7b33049 ("journal-gatewayd: factor out opening of temp
file").

man: update and extend the various sd_bus_message_append_*() man pages

Some calls changed their signature since the man pages were written.
Also extend on a number of details.

Merge pull request #509 from dvdhrm/logind

logind: fix CreateSession for lightdm

logind: fail on CreateSession if already in session

Right now, if you're already in a session and call CreateSession, we
return information about the current session of yours. This is highy
confusing and a nasty hack. Avoid that, and instead return a commonly
known error, so the caller can detect that.

This has the side-effect, that we no longer override XDG_VTNR and XDG_SEAT
in pam_systemd, if you're already in a session. But this sounds like the
right thing to do, anyway.

hwdb: Update database of Bluetooth company identifiers

logind: allow sessions to share a VT if it's a greeter

Old gdm and lightdm start the user-session during login before they
destroy the greeter-session. Therefore, the user-session will take over
the VT from the greeter. We recently prevented this by never allowing
multiple sessions on the same VT. Fix this now, by explicitly allowing
this if the owning session is a GREETER.

Note that gdm no longer behaves like this. Instead, due to wayland, they
always use a different VT for each session. All other login-managers are
highly encouraged to destroy the greeter-session _before_ starting the
user-session. We now work around this, but this will probably not last
forever (and will already have nasty side-effects on the greeter-session).

Bump version to 222

Remove repeated 'the's

NEWS: give packagers a heads-up on the upcoming python bindings removal

Merge pull request #502 from keszybz/login-small-cleanup

Login small cleanup

Remove python-systemd

python-system has moved to it's own repository:
https://github.com/systemd/python-systemd

treewide: fix typos of let's

login: use normal comparison to zero for integers

! is supposed to be used for booleans and pointers.

tree-wide: fix write_string_file() user that should not create files

The latest consolidation cleanup of write_string_file() revealed some users
of that helper which should have used write_string_file_no_create() in the
past but didn't. Basically, all existing users that write to files in /sys
and /proc should not expect to write to a file which is not yet existant.

fileio: consolidate write_string_file*()

Merge write_string_file(), write_string_file_no_create() and
write_string_file_atomic() into write_string_file() and provide a flags mask
that allows combinations of atomic writing, newline appending and automatic
file creation. Change all users accordingly.

fileio: add 'enforce_newline' argument to write_string_stream()

Add a flag to control whether write_string_stream() should always enforce a
trailing newline character in the file.

get_process_environ: exit early when there is nothing to read

Merge pull request #492 from richardmaw-codethink/nspawn-automatic-uid-shift-fix-v2

nspawn: Communicate determined UID shift to parent version 2

Merge pull request #497 from phomes/master

man: sd-bus: typo fix

Merge pull request #496 from poettering/ipv6-privacy

networkd: various fixes for the IPv6 privacy extensions support

man: sd-bus: typo fix

networkd: various fixes for the IPv6 privacy extensions support

- Make sure that the IPv6PrivacyExtensions=yes results in
  prefer-temporary, not prefer-public.

- Introduce special enum value "kernel" to leave setting unset, similar
  how we have it for the IP forwarding settings.

- Bring the enum values in sync with the the strings we parse for them,
  to the level this makes sense (specifically, rename "disabled" to
  "no", and "prefer-temporary" to "yes").

- Make sure we really set the value to to "no" by default, the way it is
  already documented in the man page.

- Fix whitespace error.

- Make sure link_ipv6_privacy_extensions() actually returns the correct
  enum type, rather than implicitly casting it to "bool".

- properly size formatting buffer for ipv6 sysctl value

- Don't complain if /proc/sys isn't writable

- Document that the enum follows the kernel's own values (0 = off, 1 =
  prefer-public, 2 = prefer-temporary)

- Drop redundant negating of error code passed to log_syntax()

- Manpage fixes

This fixes a number of issues from PR #417

Merge pull request #495 from poettering/forwarding-fix

networkd: be more defensive when writing to ipv4/ipv6 forwarding sett…

NEWS: mention ipv6 privacy extensions

bus-proxyd: fix log message and explain dbus-1 compat matches

docs: add .gitignore

bus-proxyd: subscribe to unicast signals directed to the proxy connection

sd-bus: support matching on destination names

Right now, we never install destination matches on kdbus as the kernel did
not support MATCH rules on those. With the introduction of
KDBUS_ITEM_DST_ID we can now match on destination IDs, so add explicit
support for those.

This requires a recent kdbus module to work. However, there seems to be no
user-space that uses "Destination=''" matches, yet, so old kdbus modules
still work fine (we couldn't find any real user).

This is needed to match on unicast signals in bus-proxy. A followup will
add support for this.

Merge pull request #491 from torstehu/fix-typo

treewide: fix typos

nspawn: Communicate determined UID shift to parent

There is logic to determine the UID shift from the file-system, rather
than having it be explicitly passed in.

However, this needs to happen in the child process that sets up the
mounts, as what's important is the UID of the mounted root, rather than
the mount-point.

Setting up the UID map needs to happen in the parent becuase the inner
child needs to have been started, and the outer child is no longer able
to access the uid_map file, since it lost access to it when setting up
the mounts for the inner child.

So we need to communicate the uid shift back out, along with the PID of
the inner child process.

Failing to communicate this means that the invalid UID shift, which is
the value used to specify "this needs to be determined from the file
system" is left invalid, so setting up the user namespace's UID shift
fails.

networkd: be more defensive when writing to ipv4/ipv6 forwarding settings

1) never bother with setting the flag for loopback devices

2) if we fail to write the flag due to EROFS (which is likely to happen
   in containers where /proc/sys is read-only) or any other error, check
   if the flag already has the right value. If so, don't complain.

Closes #469

treewide: fix typos

update TODO

nspawn: fix indenting

Merge pull request #443 from ssahani/lldp

lldp: set correct state for processing

Merge pull request #417 from ssahani/ipv6-private

Ipv6 private extensions

login: simplify assignment

rules: block - add dasd to whitelist

build-sys: bump sonames

hwdb: update

NEWS: update

core: fix missing bus-util.h include

Whoopsy, forgot to 'git add' this, sorry.

core: don't mount kdbusfs if not wanted

Just like we conditionalize loading kdbus.ko, we should conditionalize
mounting kdbusfs. Otherwise, we might run with kdbus if it is builtin,
even though the user didn't want this.

NEWS: mention accelerometer removal for 222

Add NEWS entry for 222 and mention the removal of the udev accelerometer.
iio-sensor-proxy-1.0 is required if you want orientation data to still
work.

TODO: remove ipv6 Ipv6 privacy extensions

man: add manual for ipv6 privacy extension

networkd: Add support for ipv6 privacy extension

This patch add support for ipv6 privacy extensions.

The variable  /proc/sys/net/ipv6/conf/<if>/use_tempaddr

can be changed via the boolean

IPv6PrivacyExtensions=[yes/no/prefer-temporary]

When true enables privacy extensions, but prefer public addresses over
temporary addresses.
prefer-temporary  prefers temporary adresses over public addresses.
Defaults to false.

[Match]
Name=enp0s25

[Network]
IPv6PrivacyExtensions=prefer-temporary

Merge pull request #488 from miguelinux/boot

bootctl: fix sdboot to systemd-boot

bootctl: fix sdboot to systemd-boot

sdboot was renamed to systemd-boot

Fixes: e7dd673d1e0a ("gummiboot/sd-boot/systemd-boot: rename galore")

Signed-off-by: Miguel Bernal Marin <miguel.bernal.marin@linux.intel.com>

man: fix sysctl references in networkd-manpage

We refer to the same sysctl-setting twice, which is misleading. Correctly
list all global forwarding options. As we _always_ change the forwarding
setting on links, they will get disabled by default. The global sysctl
defaults thus will not have any effect.

Merge pull request #485 from poettering/sd-bus-flush-close-unref

sd-bus: introduce new sd_bus_flush_close_unref() call

core: fix coding style in agent-handling

Avoid late bail-out based on a condition. This makes code hard to read.
Instead, reverse the forwarding-condition.

core: harden cgroups-agent forwarding

On dbus1, we receive systemd1.Agent signals via the private socket, hence
it's trusted. However, on kdbus we receive it on the system bus. We must
make sure it's sent by UID=0, otherwise unprivileged users can fake it.

Furthermore, never forward broadcasts we sent ourself. This might happen
on kdbus, as we forward the message on the same bus we received it on,
thus ending up in an endless loop.

busctl: flush stdout after dumping data

Running `busctl monitor` currently buffers data for several seconds /
kilobytes before writing stdout. This is highly confusing if you dump in a
file, ^C busctl and then end up with a file with data of the last few
_seconds_ missing.

Fix this by explicitly flushing after each signal.

Merge pull request #484 from xnox/persistent-journal

journal: in persistent mode create /var/log/journal, with all parents.

sd-bus: introduce new sd_bus_flush_close_unref() call

sd_bus_flush_close_unref() is a call that simply combines sd_bus_flush()
(which writes all unwritten messages out) + sd_bus_close() (which
terminates the connection, releasing all unread messages) +
sd_bus_unref() (which frees the connection).

The combination of this call is used pretty frequently in systemd tools
right before exiting, and should also be relevant for most external
clients, and is hence useful to cover in a call of its own.

Previously the combination of the three calls was already done in the
_cleanup_bus_close_unref_ macro, but this was only available internally.

Also see #327

Merge pull request #478 from systemd/revert-429-nspawn-userns-uid-shift-autodetection-fix

Revert "nspawn: determine_uid_shift before forking"

journal: in persistent mode create /var/log/journal, with all parents.

systemd-journald races with systemd-tmpfiles-setup, and hence both are
started at about the same time. On a bare-bones system (e.g. with
empty /var, or even non-existent /var), systemd-tmpfiles will create
/var/log. But it can happen too late, that is systemd-journald already
attempted to mkdir /var/log/journal, ignoring the error. Thus failing
to create /var/log/journal. One option, without modifiying the
dependency graph is to create /var/log/journal directory with parents,
when persistent storage has been requested.

Merge pull request #480 from rinrinne/fix-message

Fix error message for enumerate addresses

Revert "nspawn: determine_uid_shift before forking"

Fix error message for enumerate addresses

Error message for enumerating addresses was not 'addresses' but 'links'.
This patch fixes it.

Merge pull request #473 from richardmaw-codethink/machinectl-import-earlier-than-3-15

util: fall back in rename_noreplace when renameat2 isn't implemented

login: add rule for qemu's pci-bridge-seat

Qemu provides a separate pci-bridge exclusively for multi-seat setups.
The normal pci-pci bridge ("-device pci-bridge") has 1b36:0001. The new
pci-bridge-seat was specifically added to simplify guest-side
multiseat configuration.  It is identical to the normal pci-pci bridge,
except that it has a different id (1b36:000a) so we can match it and
configure multiseating automatically.

Make sure we always treat this as separate seat if we detect this, just
like other "Pluggable" devices.

(David: write commit-message)

util: fall back in rename_noreplace when renameat2 isn't implemented

According to README we only need 3.7, and while it may also make sense
to bump that requirement when appropriate, it's trivial to fall back
when renameat2 is not available.

Merge pull request #472 from keszybz/documentation-updates2

Documentation updates

Merge pull request #470 from marineam/escape

escape: fix exit code

Merge pull request #427 from keszybz/man-list-fixes

Update Makefile-man.am after recent changes and fix regeneration of Makefile-man.am
Fixes fallout from https://github.com/systemd/systemd/pull/282.

man: update sysctl example about netfilter

It turns out that since kernel 3.18 netfilter on bridged packets
is off anyway, so the example should be reworded (and the module
name updated).

man: information about available properties

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