strbuf: fix leak on memory error

Not very likely, but let's fix it for the matter of
principle.

bootchart: fix a potential buffer overrun

If the configured number of samples was close to MAXSAMPLES,
the samples buffer could be overrun:

- by 1, because of off-by-one in the condition (samples > arg_samples_len),
and
- by many in case of an overrun, because the number of samples to
  capture was increased, instead of being decreased.

Simplify things by converting to a normal for-loop.

In store.c: change buffer size from 4095 to 4096. 4095 is a strange
number.

bootchart: split out option parsing from main()

journald: check return value of strappend

bus: check return value from bus_type_get_alignment/size

nspawn, machine-id-setup: warn if read-only mount call fails

They are not crucial, but they shouldn't fail.

nss-myhostname: use _cleanup_ and split function into two

The triply nested loop is just too much. Let's split out the
middle loop's body, so the whole thing is easier to read. Also
modernize the style a bit, using structure initialization to
avoid memset and such.

accelerometer: remove dead if and modernize

Based on a coverity warning.

execute: convert secure bits into mask properly

C.f. http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=5975c725dfd6f7d36f493ab1453fbdbd35c1f0e3

Remove some dead code

Based on coverity report.

journal-gatewayd: query PID 1 for virtalization

Since journal-gatewayd is now running unprivileged, and detecting
virtalization requires privileges, query PID1 via D-Bus for the used
virtualization.

This is also the first time we use libsystemd-bus for more than just
testing.

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

bus: rename sd_bus_get_peer() to sd_bus_get_server_id()

This function always returns the server side ID. The name suggested it
was actually always the peer's ID, but that's not correct if the call is
called on a server bus context. Hence, let's correct the name a bit.

bus: s/sd_message_handler_t/sd_bus_message_handler_t/g

bus: parse matches locally and allow registration of callbacks for them

This includes code to parse and split up match strings which will also
be useful to calculate bloom filter masks when the time comes.

bus: add missing test-bus-server.c

bus: implement server mode, and anonymous authentication

prioq: consider a NULL prioq empty

bus: if we recieve fds but fd passing is off, consider this an error

bus: enable SO_PASSCRED only if we are not connected to a bus

bus: properly set up bus connections with sd_bus_set_exec()

bus: consider it an error if the first message we get on the bus is not a reply to HELLO

bus: add C++ wrappers to public header files

dbus: let's make this const array static

udev: extern on functions is pointless

unit: no need to export variables if we can avoid it

TODO

gitignore: update

TODO: update

killall: print notice what we forcefully KILL

build-sys: force Python to write UTF-8

Here is a patch that fixes documentation with python 3.x in non utf-8
locales.  Specifically in my locale latin-1 is the default setting for
output going to stdout, which causes it to fail. By writing directly
to file we are able to set the locale to utf-8.

build-sys,man: use XML entities to substite strings

This makes it easier to add substitutions to man pages,
avoiding the separate transformation step.

mkdir -p's are removed from the rule, because xsltproc will
will create directories on it's own.

All in all, two or three forks per man page are avoided,
which should make things marginally faster.

Unfortunately python parsers must too be tweaked to handle
entities. This isn't particularly easy: with lxml a custom
Resolver can be used, but the stdlib etree doesn't support
external entities *at all*. So when running without lxml,
the entities are just removed. Right now it doesn't matter,
since the entities are not indexed anyway. But I intend to
add indexing of filenames in the near future, and then the
index generated without lxml might be missing a few lines.
Oh well.

build-sys: generate sed substitution from a list

I want to use the substitutions in different form for
xml entities.

Revert "build-sys: substitute strings in systemd.unit(5)"

This reverts commits c78ab91132aab9193f3c17a9a206f8825ff4be84
and 185c3be03cec26023acc11b49553753aa7330a1d.

It is simpler to just use includes...

test-path-util: fix another stupid test failure

units: fix network-online.target fields

update TODO

test-path-util: fix botched test

bootctl: fix help text

It currently says 'time settings', change that to 'boot settings'.

utmp: make gcc shut up, we need to pass an int here, not size_t

cgtop --help: default depth is 3 not 2

use default depth from variable for --help

Always use errno > 0 to help gcc

gcc thinks that errno might be negative, and functions could return
something positive on error (-errno). Should not matter in practice,
but makes an -O4 build much quieter.

journalctl: fix --update-catalog with not --root arg

grawity> `journalctl --update-catalog` from latest git prints:
         "Recursive mkdir .: Invalid argument" and
         "Failed to write : Invalid argument"

tests: add some silly tests for path-util.c

journalctl: support --root for message catalogs

catalog: open up catalog internals

In order to write tests for the catalog functions, they
are made non-static and start taking a 'database' parameter,
which is the name of a file with the preprocessed catalog
entries.

This makes it possible to make test-catalog part of the
normal test suite, since it now only operates on files
in /tmp.

Some more tests are added.

catalog: make sure strings are terminated

Coverity complains: systemd-199/src/journal/catalog.c:126:
buffer_size_warning: Calling strncpy with a maximum size argument of
32 bytes on destination array "i->language" of size 32 bytes might
leave the destination string unterminated.

...and unfortunately it was right. The string was defined as a
fixed-size string in some parts of the code, and used a
null-terminated string in others (e.g. in log statements). There's no
point in conserving one byte, so just define the max language tag
length to 31 bytes, and use null terminated strings everywhere.

Also, wrap some lines, zero-fill less bytes, use '\0' instead of just
0 to be more explicit that this is one byte.

udev: check return value of uname.

bootchart: properly terminate string

systemd-199/src/bootchart/store.c:289: buffer_size_warning: Calling
strncpy with a maximum size argument of 256 bytes on destination array
"ps->name" of size 256 bytes might leave the destination string
unterminated.

...and indeed, the string was used as NULL-terminated later on.

pid_cmdline_strncpy is renamed to pid_cmdline_strscpy to commemorate
the fact that it *does* properly terminate the string.

utmp-wtmp: don't try to read past end of string

systemd-199/src/shared/utmp-wtmp.c:228: buffer_size_warning: Calling
strncpy with a maximum size argument of 32 bytes on destination array
"store.ut_line" of size 32 bytes might leave the destination string
unterminated.

The destination string is unterminated on purpose, but we must
remember that.

build-sys: remove papersize option from sphinx

We don't use it currently for anything (no latex output),
but it was messing up stuff if /etc/papersize had comments.

coredump: do not free twice

build-sys: prepare release 200

unit: replace remote-fs-setup.target by network-online.target

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

hwdb: update

build-sys: fix HAVE/ENABLE_FIRMWARE

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

bootctl: ESP --> Partition

bootctl: Binary: --> File:

man/os-release: Add BUILD_ID field

BUILD_ID is a fairly generic field used to identify the system image
that was used to install the distribution.

catalog: fix error codes handling in catalog_list_items

Previously r was set to zero and so if(r<0) was never true.

log: fix error codes handling in catalog_list_items

It does not make sense to print error code from previous loop.

sd-bus: check c->path for null instead of path

systemd-sysctl: Handle missing /etc/sysctl.conf properly

Since fabe5c0e5fce730aa66e10a9c4f9fdd443d7aeda, systemd-sysctl returns
a non-zero exit code if /etc/sysctl.conf does not exist, due to a
broken ENOENT check.

shared: free dt (temporary dir name) on fail

[zj: modified to not to try to rmdir() dir we haven't created.]

core: check return value of rm_rf_dangerous and warn if it fails

bootctl: rename status output header

keymap: Fix Touchpad Toggle on MSI Wind U90/U100

This key is handled by the hardware already, so handling it again in software
nullifies the effect. Newer kernels read the real state and send out a separate
KEY_TOUCHPAD_ON or KEY_TOUCHPAD_OFF event, so in both cases we need to ignore
that key.

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

tests: skip bus test if bus cannot be opened

To make the result more visible, special return value
is used to tell automake that the test was skipped. While
at it, use the same return value in other skipped tests.

tests: fix size_t in format string

Simplify the meaning of %s

The rules governing %s where just too complicated. First of
all, looking at $SHELL is dangerous. For systemd --system,
it usually wouldn't be set. But it could be set if the admin
first started a debug shell, let's say /sbin/sash, and then
launched systemd from it. This shouldn't influence how daemons
are started later on, so is better ignored. Similar reasoning
holds for session mode. Some shells set $SHELL, while other
set it only when it wasn't set previously (e.g. zsh). This
results in fragility that is better avoided by ignoring $SHELL
totally.

With $SHELL out of the way, simplify things by saying that
%s==/bin/sh for root, and the configured shell otherwise.
get_shell() is the only caller, so it can be inlined.

Fixes one issue seen with 'make check'.

units: automatically order all mount units after network.target

Previously it was necessary to pull in remote-fs-pre.target to order the
mount units against network.target since the ordering was done
transitively via remote-fs-pre.target.

As network implementations shouldn't need to know about the specific
use-case of network mounts we instead now simply order network.target
against all mounts too. This should make it unnecessary for network
managing services to import remote-fs-pre.target explicitly, as
network.target will now suffice.

man: network.target is also a passive target

readahead: cleanups

- check for OOM
- no need to use floats and round()

readahead: chunk on spinning media

Readahead has all sorts of bad side effects depending on your
storage media. On rotating disks, it may be degrading startup
performance if enough requests are queued spanning linearly
over all blocks early at boot, and mount, blkid and friends
want to insert reads to the start of these block devices after.

The end result is that on spinning disks with ext3/4 that udev
and mounts take a very long time, and nothing really happens until
readahead is completely finished.

This has the net effect that the CPU is almost entirely idle
for the entire period that readahead is working. We could have
finished starting up quite a lot of services in this time if
we were smarter at how we do readahead.

This patch sorts all requests into 2 second "chunks" and sub-sorts
each chunk by block. This adds a single cross-drive seek per "chunk"
but has the benefit that we will have a lot of the blocks we need
early on in the boot sequence loaded into memory faster.

For a comparison of how before/after bootcharts look (ext4 on a
mobile 5400rpm 250GB drive) please look at:

    http://foo-projects.org/~sofar/blocked-tests/

There are bootcharts in the "before" and "after" folders where you
should be able to see that many low-level services finish 5-7
seconds earlier with the patch applied (after).

NEWS: tweak grammar

NEWS: typo fix

build-sys: bump version and .so revisions

cryptsetup: when prompting for password use GPT partition label

If there's a GPT partition label set for a LUKS partition, then it's
nicer to show that than the model number, when asking for a passphrase.

man: document that the passive units cannot be pulled in mnaully

fstab-generator: drop x-initrd.rootfs mount option

x-initrd.mount now has different meanings, if fstab-generator is called
in the initramfs.

initrd:/etc/fstab and x-initrd.mount defines mounts for the
initrd-root-fs.target

initrd:/sysroot/etc/fstab and x-initrd.mount defines mounts for the
initrd-fs.target

units: disallow manual starting of passive units

As passive units only are useful for ordering things within the initial
transaction there is no point in ever activating them manually, hence
refuse it.

fstab-generator: degrade the message about missing "root=" to log_debug

Some installation media (fedora at least) does not have and need a
"root=" argument on the kernel command line.

Drop src/login/uaccess.c, dead code

This moved to src/udev/udev-builtin-uaccess.c a while ago.

bus: fix missing variable initialization

modules-load: there's really no point in mentioning that a certain modules is already loaded

After all, this runs in parallel to udev, so there's quite a chance it
already is....

build-sys: make gcc shut up

build-sys: ship missing unit file

units: downgrade dependency on sockets.target/timers.target/paths.target by basic.target

There isn't really any need to require any targets but sysinit.target
from basic.target, so downgrade sockets.target, paths.target,
timers.target.

bus: when we are talking to a bus, SCM_CREDS/SCM_SECLABEL are not very useful

bus: automatically generate minimal introspection data to find installed objects

bus: rename send_hello flag to bus_cient

This way we can hide more than just the hello logic behind this flag,
for example, later on automatic match management.

build-sys: create kernel/install.d directories

NEWS: update

cryptsetup-generator: let's be a bit more efficient with strv_extend()

cryptsetup-generator: add a missing OOM check

update NEWS

prepare NEWS for 199

man: properly document the system targets that are also available for the user instance

man: rearrange systemd.special(7) to clarify which units are passive and which ones aren't

This also adds a short explanation paragraph for this.

units: there is no point in pulling in ordering 'provides'-style targets

Units such as nss-lookup.target, nss-user-lookup.target,
remote-fs-pre.target, local-fs-pre.target, time-sync.target,
rpcbind.target are to be pulled in by the implementing services, and
that's there only purpose. They should not have any 'active component'
otherwise, so let's drop all further deps from these units.

update TODO