condition: Fix file descriptor leak in test_capability()

Detected by cppcheck.

random-seed: convert poolsize from bits to bytes

The problem was first noted in a bug report against Arch's initscripts.

Reported-by: Taylan Ulrich Bayırlı <taylanbayirli@gmail.com>
Reported-by: Gerardo Exequiel Pozzi <vmlinuz386@yahoo.com.ar>

plymouth: fix ply proto endianess issues

Plymouth enforces LE even for the local Ply proto, hence we should do
the conversion properly for BE arch compat.

Tracked down by Harald Hoyer.

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

README: explain dependency on cgroups

update TODO

systemctl: make list-unit-files output more economical

The first column is given the width of the widest entry,
if possible, otherwise all entries are ellipsized to fit
in ($COLUMNS - (width of second column)).

[ Added a few fixes, calculate state_cols too, respect '--no-legend',
  better handling of '--full' -- michich ]

systemctl-completion: always invoke with --no-legend

In the case of completion for the 'restart' verb, passing the invalid
unit name (the colums header) causes completion to cease functioning
entirely, with the error:

  Failed to issue method call: Unit name UNIT is not valid.

This adds a small wrapper function for systemctl which can have common
options added to it.

audit: do not complain if kernel lacks audit

When running on a kernel without audit support, systemd currently
writes a mysterious-sounding error to its log:

systemd[1]: Failed to connect to audit log: Protocol not supported

Better to suppress the audit_open() failure message when (and only
when) it is due to running on a kernel without audit support, since in
this case the admin probably does not mind systemd not writing to the
audit log.  This way, more serious errors like ENOMEM and EACCES will
stand out more.

service: Drop rcN.d runlevels from SysV services that also exist in rcS.d

Services which claim to start in both rcN.d and rcS.d generate
loops which for some reason seems to usually end up with dbus not
starting and the whole machine being quite unhappy. We now rather
assume that if a service can be started in rcS, it should not also
start in rcN.d.

Fixes Debian bug #637037

manager: fix a crash in isolating

HASHMAP_FOREACH is safe against the removal of the current entry, but
not against the removal of other entries. job_finish_and_invalidate()
can recursively remove other entries.

It triggered an assertion failure:
  Assertion 'j->installed' failed at src/manager.c:1218, function
  transaction_apply(). Aborting.

Fix the crash by iterating from the beginning when there is a
possibility that the iterator could be invalid.

It is O(n^2) in the worst case, but that's better than a crash.

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

util: properly detect what the last capability is

build-sys: bump release for v37

service: don't try to guess PID for SysV services anymore

As it turns out there are quite a number of SysV services too broken to
make the guessing work: instead of returning in the parent only after
the child is fully initialized they return immediately. The effect is
that the guessing in systemd might happen too early, at a time where the
final main process doesn't exist yet.

By turning this off we won't try to detect the main pid anymore, with
the effect that all processes of the service in question are considered
equally likely to be the main process.

localed: make sure s-s-k doesn't create any X11 config files anymore

localed: shorten generate-kbd-model-map

Output is identical.

pager: add _noreturn_ to pager_fallback()

src/pager.c: In function ‘pager_fallback’:
src/pager.c:35:13: warning: function might be possible candidate for attribute ‘noreturn’ [-Wmissing-noreturn]

unit: introduce ConditionCapability

units: forgot target units

units: increase LimitNOFILE a bit

since we need one fd per session (for logind) and one fd per service
(for stdout-syslog-bridge) increase the default rlimit a bit.

logind: fail gracefully if too many sessions are created

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

timedate: fall back to /etc/sysconfig/clock on Fedora, for compatibility with legacy

units: remount root and API FS before all mount units are applied

In order to ensure that bind mounts copy the final mount settings to the
new bind mount make the root and API FS mount options are applied before
the other file systems are mounted.

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

units: introduce local-fs-pre.target and remote-fs-pre.target

This hook target enables services to order themselves between
network.target and remote mounts, which is needed for GFS2 and similar
systems.

l10n: Reword the polish translation a bit

s/siedzenie/stanowisko/: they both translate to seat in English, but 'stanowisko'
is a place where you work, while 'siedzenie' is like a chair.

s/podłączanie/podłączenie/: we are doing it now, now sometime in the future.

man: document list-unit-files

It's documented in the --help, but not in the manpage.

sd-login.h: correct spelling mistakes in comments

hostname-setup: Frugalware switched to /etc/hostname

readahead: lower max file size for readahead

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

man: .include directive does not include as textual include it includes by parsing the include file.

This means that section headers must be used inside of
a .include file otherwise all the lines are ignored.

man: for ExecStart= provide more details on env var substitution and how that turns into arguments.

For EnvironmentFile= explain that double quotes can be used
to protect whitespace.

update TODO

util: fix close() call on wrong variable

Detected by "cppcheck" (actually it detected a file descriptor leak)

tmpfiles: fix file descriptor leak

Detected by "cppcheck"

update TODO

test_virtualization: do not try to compare id in !virt context

update TODO

update TODO

hashmap: use different version of DJB's hash algorithm that uses shifting instead of multiplication

update TODO

logind: properly generate session creation response on dbus

This fixes the various reported issues with dbus message parsing when
running su.

systemctl: fix corrupted output of units

On some systems the list of units in systemctl output came out wrong,
all on one line and with missing descriptions.

It turns out printf() really attempts to mmap 2G memory when INT_MAX is
passed as the field width. On machines with small virtual memory
without overcommit the mmap inside printf() failed.

Do not use INT_MAX for unlimited width.

localed: handle multiple X11 kbd layouts

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

systemctl: fix error message when no connection to systemd is available

systemctl: fix detection whether the dbus connection is available

build-sys: fix build

vconsole: don't parse Fedora's KEYMAP= kernel parameters anymore

KEYMAP are use to pass keymap configuration to initrd, but not to the
system itself. Since the initrd might get out of date we need to make
sure that changes made in userspace override the settings from the
cmdline, hence drpo any use of it all for these variables.

localed: add SetX11Keyboard() and SetVConsoleKeyboard() bus calls

analyze: always draw top kernel, initramfs, userspace bars

Based on Koen Kooi's patch. Renamed initrd to initramfs.

systemctl: better width and alignment of fields

Provide nicer output by taking advantage of the available columns.

Expand UNIT and DESCRIPTION if possible. Economize the space:
 - never give UNIT more width than it can use.
 - if there's a shortage of space, give 25 columns to UNIT, even
   if it would result in overflowing the total columns. The field would not
   be useful below width 25.
 - then if possible, assign up to 25 columns to DESCRIPTION.
 - then split any remaining space evenly between UNIT and DESCRIPTION.

Keep the columns aligned even with '--full'.

systemctl: print the legend regardless of stdout type, add --no-legend

Print the legend (the column headers and the footer with hints) by
default even to non-tty output. People seem to get confused by the
difference when they redirect the output.

Add a parameter to suppress the printing of the legend.

Related-to: https://bugzilla.redhat.com/show_bug.cgi?id=713567

pager: add a trivial internal pager

In the very unlikely scenario where none of the external pagers is
available, use an internal implementation to pass stdin to stdout.

Don't bother with trying 'cat', because it's no more useful than the
internal pager.

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

build-sys: bump release

readahead: opt out of bridge logging for readahead in order not to delay things unnecessarily

service: fix up std output/error before we add dependencies to the bridge socket

virt: add missing header inclusion

condition: optionally test against type of virtualization (vm vs. container)

util: move virtualization detection into its own files, and extend return codes

analyze: report startup time in plot mode as well

It now prints something like "Startup finished in 1507ms (kernel) + 850ms (userspace) = 2357ms" below the legend.

analyze: always draw 1s marker for scale

In situations like this:

root@omap4430-panda:~# systemd-analyze
Startup finished in 1499ms (kernel) + 916ms (userspace) = 2416ms

The svg plot will only have the 0s marker and no subsequent markers for scale. This patch forces the 1s marker to always be drawn.

unit: fix complementing of requirement deps with After deps for targets

'man systemd.target' says:
  Unless DefaultDependencies= is set to false, target units will
  implicitly complement all configured dependencies of type
  Wants=, Requires=, RequiresOverridable= with dependencies of type
  After=  if the units in question also have DefaultDependencies=true.

It did not work because of a forgotten negation.

util: don't fail if no id was passed to detect_container()

util: detect systemd-nspawn without relying on ns cgroup tree

nspawn: set env var container=systemd-nspawn, following the scheme lxc introduced

loginctl: fix typo

llvm-analyze: change a few things to make llvm-analyze show fewer false positives

llvm-analyze: fix some bugs found by llvm-analyze

util: improve signal_to_string, signal_from_string

signal_to_string:
 Produce names for SIGRTMIN+n.
 Never give an "n/a" result. In the worst case give the number itself as
 a string.

signal_from_string:
 Parse "RTMIN+n".
 Parse any valid signal number.

manager: fix job mode for SIGRTMIN+1, +2

The test did not work as intended and always resulted in JOB_REPLACE.

condition: fix reversed tests if path does not exist at all

CONDITION_PATH_IS_DIRECTORY, CONDITION_PATH_IS_SYMBOLIC_LINK and
CONDITION_FILE_IS_EXECUTABLE gave reversed results when the path
did not exist at all.

coverity: change a few things so that coverity doesn't show so many false positives

coverity: fix a couple of bugs found by coverity

units: fix rescue.service race with plymouth

It was possible for the "ExecStartPre=-/bin/plymouth quit" to race
with plymouth-start.service which is pulled in indirectly by
basic.target -> sysinit.target.

The race left plymouth running on the terminal, making it unusable for
rescue purposes.

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

man: document that PathChanged= does not care for simple writes but only for close()-after-write()

unit: don't recheck conditions when a unit is already starting but unit_start() is invoked

man: document pkg-config line for sd-daemon

man: document that sd-daemon.[ch] is now available as shared library

service: warn if a service fails to write its PID file

Warn if a service promises to write a PID file (using 'PIDFile=' in the
unit file or '# pidfile:' in SysV header), but fails to keep the
promise.

This warning will likely trigger also for the forking services with a
racy daemonization, which exit the original process before the PID file
is written.

update TODO

selinux: reopen log when loading the SELinux policy fails

service: if StandardInput=socket and StandardOutput=inherit imply socket for output, don't imply default output

This is useful for inetd-style per-connection services, so that they
again can simply specify StandardOutput=socket to connect all three fds
to the socket.

udev: asynchronously mount fuse/confifgs as soon as the kmods are loaded

units: introduce unit file for FUSE fs

FUSE and configfs is very very similar, so handle both the same way.

units: In MeeGo there was more than one ExecStart lines present that caused systemd to refuse the rescue.service

mount: pull in quota tools from fstab lines with 'quota' option, too

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

job: after converting a job from restart to start readd it to run queue

When we converted a restart job it's time to look again whether any of
its dependencies are now runnable.

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

update TODO

units: we don't need to check whether /sys/kernel/security is already mounted since we'll enumerate that fact anyway

update TODO

units: drop automount unit for /dev/hugepages

Since hugetlbfs cannot be compiled as kernel module there's little point
in doing on-demand mounting via autofs for it.

units: drop automount unit for /dev/mqueue

Since the mqueue support cannot be built as a module there's little
benefit in having an autofs mount point set up for this.

units: drop automount unit for /sys/kernel/security

Since securityfs cannot be build as module there's little value in
having an on-demand autofs mount point for it.

units: drop automount unit for /sys/kernel/debug

Since debugfs cannot be compiled as module there's little benefit in
having it as autofs mount point.

units: drop /sys/kernel/config automount unit

The mount point directory /sys/kernel/config is only created after the
module is loaded, hence there's little value in having this an automount
unit: the runtime penalty for mounting an autofs here should be the same
as for a real mount.

modules-load: filter out double modules

service: minor change in service_load_pid_file return value

Return 0 only if the PID was really loaded. If no PIDFile= is defined,
return -ENOENT.

Only one caller cares about the return value of this function and this
change makes the usage nicer.

condition: add ConditionPathIsSymbolicLink

condition: always follow symlinks for condition checks, to mimic test

service: handle forking services that move to a new PID

When some forking daemons receive a SIGHUP, they re-execute themselves
and consequently change to a new main PID. As long as they update the
PID file in the right order (before exiting the old PID), we can detect
that and avoid killing them.

units: don't initialize automount point for securityfs if the initrd already mounted the directory

unit: introduce ConditionPathIsMountPoint=