build: remove AC_FUNC_MALLOC

What is the rationale to have AC_FUNC_MALLOC? It does not actually
abort the configure run if an "unsuitable" malloc was found, and
instead just replaces malloc by rpl_malloc, for which systemd however
has no definition, either.

Remove the call.

mising: add __NR_memfd_create syscall number for s390

Merge pull request #1341 from filbranden/werror2

Check behavior of -Werror=shadow before deciding to use it

update NEWS

Merge pull request #1339 from alkino/master

Hook more properties for transient units

Hook more properties for transient units

systemd-run can now launch units with WorkingDirectory, RootDirectory set.

Merge pull request #1346 from dvdhrm/user-shutdown

core: fix shutdown of --user

Merge pull request #1348 from larsu/master

busctl: also monitor messages to SERVICE arguments

busctl: also monitor messages to SERVICE arguments

Add a 'destination' match rule for every SERVICE argument in addition to
the 'sender' rule. This is consistent with busctl(1), which documents
monitor as dumping "messages to or from this peer".

core: fix shutdown of --user

Shutting down a user session currently fails with:

Sep 22 22:35:38 david-t2 systemd[640]: Reached target Shutdown.
Sep 22 22:35:38 david-t2 systemd[640]: Starting Exit the Session...
Sep 22 22:35:38 david-t2 systemd[640]: Received SIGRTMIN+24 from PID 659 (kill).
Sep 22 22:35:38 david-t2 systemd[640]: Shutting down.
Sep 22 22:35:38 david-t2 systemd[640]: Not executed by init (PID 1).
Sep 22 22:35:38 david-t2 systemd[640]: Critical error while doing system shutdown: Operation not permitted

This is a regression from:

    commit 287419c119ef961db487a281162ab037eba70c61
    Author: Alban Crequy <alban.crequy@gmail.com>
    Date:   Fri Sep 18 13:37:34 2015 +0200

        containers: systemd exits with non-zero code

Make sure we never ever execute systemd-shutdown from within a
user-manager. Restore the previous behavior by partially reverting given
commit.

Merge pull request #1342 from filbranden/gitignore1

build-sys: add new test binaries to .gitignore

build-sys: add new test binaries to .gitignore

build-sys: Check behavior of -Werror=shadow before deciding to use it

gcc versions 4.6 and earlier used to complain when a local variable
shadows a global function, 4.7 and above only complain if a local
variable shadows a global variable.

Fix this by checking whether gcc 4.7+ behavior is in place before
deciding to use -Werror=shadow in $(CFLAGS), by using a custom test
program source that shadows a global function with a local variable and
confirming that -Werror=shadow does not make the compile to break.

Tested:
- On gcc 4.7 and 4.8, confirmed nothing changed (other than the order of
  the -Werror=shadow argument, going to the end of CFLAGS.)
- On gcc 4.6, confirmed by looking at the config.log output that the
  check for -Werror=shadow failed and it was not included in CFLAGS.
- Ran `make V=1` to confirm -Werror=shadow was still in use, introduced
  a bogus shadowing issue and confirmed it was caught when building with
  a recent gcc.

build-sys: properly quote m4 macro arguments in CC_CHECK_FLAGS_APPEND

The variables should be quoted inside [...] to avoid double macro
expansion.  This is currently not an issue, since the values (-W...) are
not really macros, but we might as well just fix that issue now.

Tested by re-running autogen.sh and comparing the value of OUR_CFLAGS in
the generated Makefile.  Ran a full build from a clean tree to confirm
no other issues were introduced.

Merge pull request #1338 from kaysievers/pam

pam: systemd-user - call selinux module

udev: ata_id - ATA_ID_SATA_CAPABILITY == 76

pam: systemd-user - call selinux module

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

update TODO

Merge pull request #1334 from poettering/sd-bus-default-flush-close

sd-bus: introduce new sd_bus_default_flush_close() call

Merge pull request #1324 from pugs/master

Fixup WWN bytes for big-endian systems

Merge pull request #1335 from poettering/some-fixes

A variety of mostly unrelated fixes

Merge pull request #1336 from pszewczyk/functionfs_sockets_v3

core: add support for usb functionfs v3

importd: make sure we don't accidentally close fd 0

Fixes #1330

man: drop reference to yum from man pages

Apparently, yum is obsolete, and dnf is the new yum. Mention only dnf
hence, and don't mention yum anymore.

man: Add documentation for functionfs socket activation

core: Add FFSDescriptors and FFSStrings service parameters

By using these parameters functionfs service can specify ffs descriptors
and strings which should be written to ep0.

core: Add socket type for usb functionfs endpoints

For handling functionfs endpoints additional socket type is added.

notify: sort header files, follow CODING_STYLE

util: drop UID_IS_INVALID() in favour of uid_is_valid()

No need to keep both functions, settle on uid_is_valid() for everything.

util.h: order includes, as suggested by CODING_STYLE

Of course, because Linux is broken we cannot actually really order it,
and must keep linux/fs.h after sys/mount.h... Yay for Linux!

util: clean-ups to enum parsers

Never log when we fail due to OOM when translating enums, let the caller
do that. Translating basic types like enums should be something where
the caller logs, not the translatior functions.

Return -1 when NULL is passed to all enum parser functions.

The non-fallback versions of the enum translator calls already handle
NULL as failure, instead of hitting an assert, and we should do this
here, too.

cgtop: make sure help text doesn't cause main contents to move

Let's always keep space for the full help text. (We used to do that, but
recently another line of help was added which broke this.)

util: minor cleanups for loop_read() and friends

When 0 bytes are to be written, make sure to go into read() at least
once, in order to validate the parameters, such as the passed fd.

Return error on huge values, add a couple of asserts and casts where
appropriate.

cgtop: underline table header

Let's underline the header line of the table shown by cgtop, how it is
customary for tables. In order to do this, let's introduce new ANSI
underline macros, and clean up the existing ones as side effect.

sd-id128: make size constraints a bit more obvious

util: add safe_closedir() similar to safe_fclose()

copy: make copy_bytes() return whether we hit EOF or not

copy: be more careful when trying to reflink

sd-bus: introduce new sd_bus_default_flush_close() call

If code enqueues a message on one of the default busses, but doesn't
sync on it, and immediately drops the reference to the bus again, it
will stay queued and consume memory. Intrdouce a new call
sd_bus_default_flush_close() that can be invoked at the end of programs
(or threads) and flushes out all unsent messages on any of the default
busses.

login: support more than just power-gpio-key

Adding additional keys prevents this gpio-keys powerswitch from working,
e.g. this wouldn't poweroff:

button@23 {
    label = "power-switch";
    linux,code = <116>;
    gpios = <&gpio 23 1>;
};
button@25 {
    label = "KEY_A";
    linux,code = <30>;
    gpios = <&gpio 25 1>;
};

Changing ATTRS{keys}=="116" to ATTRS{keys}=="*116*" makes the
power-switch and the A key both work properly.

(David: rephrase and merge-commits)

Merge pull request #1333 from dvdhrm/sd-network-cleanup

sd-network: random API cleanups

sd-lldp: hide internal details

Currently, sd-lldp.h exports "UPDATE_INFO".. and defines it to a magic
constant '10'. This is completely bogus, so fix it to follow our coding
standards:

  - Prefix exported symbols by SD_LLDP_*

  - Define a separate event-enum for event types

  - Translate internal state to external event-types

sd-lldp: hide internal information

Don't export constant that are only used internally.

sd-pppoe: fix namespacing

Prefix all exported constants by SD_PPPOE_* to avoid namespacing
conflicts.

sd-ipv4ll: fix namespacing

Prefix all exported constants with SD_IPV4LL_* to avoid namespacing
conflicts.

sd-ipv4acd: fix namespacing

Prefix all exported constants with SD_IPV4ACD to prevent namespacing
conflicts.

sd-icmp6-nd: fix namespacing

Prefix all exported constants by SD_ICMP6_ND_* to avoid any namespacing
conflicts.

sd-dhcp6: fix namespacing

Prefix all exported constants with SD_DHCP6_CLIENT_* to avoid any
namespacing conflicts.

sd-dhcp: fix namespacing

Prefix all constants with SD_DHCP_CLIENT_* to avoid namespacing
conflicts.

Merge pull request #986 from karelzak/monitor

mount: use libmount to monitor mountinfo & utab

Merge pull request #1331 from dvdhrm/misc-cleanup

util, nspawn, machined: random cleanups

nspawn, machined: fix comments and error messages

A bunch of "Client -> Child" fixes and one barrier-enumerator fix.

(David: rebased on master)

nspawn: close unneeded sockets in outer child

(David: Note, this is just a cleanup and doesn't fix any bugs)

util: introduce {send,receive}_one_fd()

Introduce two new helpers that send/receive a single fd via a unix
transport. Also make nspawn use them instead of hard-coding it.

Based on a patch by Krzesimir Nowak.

NEWS: add some information about v227

core: Add list of additional file descriptors to socket port

Some additional files related to single socket may appear in the
filesystem and they should be opened and passed to related service.

This commit adds optional list of file descriptors, which are
dynamically discovered and opened.

Merge pull request #1323 from dvdhrm/mount-propagate

mount: propagate error codes correctly

Merge pull request #1325 from splantefeve/fr-translation

po: update french translation

po: update french translation

Fixup WWN bytes for big-endian systems

hwdb: fix typo in G9x DPI

Fix typo in DPI: 32000 -> 3200

hwdb: drop weird whitespace

Remove whitespace from empty line in hwdb.

Merge pull request #1317 from ronnychevalier/rc/ipv4ll_seed

sd-ipv4ll: do not assert_return when seed == 0

mount: propagate error codes correctly

Make sure to propagate error codes from mount-loops correctly. Right now,
we return the return-code of the first mount that did _something_. This is
not what we want. Make sure we return an error if _any_ mount fails (and
then make sure to return the first error to not hide proper errors due to
consequential errors like -ENOTDIR).

Reported by cee1 <fykcee1@gmail.com>.

Merge pull request #1322 from evverx/update-systemd-cgtop-bash-completion

Update systemd-cgtop bash completion

shell-completion: cgtop bash completion: add -r,--raw,-k,-P,--order

shell-completion: cgtop bash completion: -n,--iterations,-d,--delay require arg

Merge pull request #1249 from lnykryn/sysv-symlinks

sysv-generator: follow symlinks in /etc/rc.d/init.d

Merge pull request #1321 from evverx/cgtop-recurisive-option

shell-completion: update systemd-cgtop bash completion: --recursive

shell-completion: update systemd-cgtop bash completion: --recursive

Merge pull request #1320 from ronnychevalier/rc/core_fix_group_ownership

core: fix group ownership when Group is set

core: fix group ownership when Group is set

When Group is set in the unit, the runtime directories are owned by
this group and not the default group of the user (same for cgroup paths
and standard outputs)

Fix #1231

test-execute: add tests for RuntimeDirectory

Merge pull request #1315 from systemd-mailing-devs/1442692671-10134-1-git-send-email-dev@benjarobin.fr

systemd-notify: Always pass a valid pid to sd_pid_notify

Merge pull request #1318 from alban/alban/container-exit-code4

containers: systemd exits with non-zero code

containers: systemd exits with non-zero code

When a systemd service running in a container exits with a non-zero
code, it can be useful to terminate the container immediately and get
the exit code back to the host, when systemd-nspawn returns. This was
not possible to do. This patch adds the following to make it possible:

- Add a read-only "ExitCode" property on PID 1's "Manager" bus object.
  By default, it is 0 so the behaviour stays the same as previously.
- Add a method "SetExitCode" on the same object. The method fails when
  called on baremetal: it is only allowed in containers or in user
  session.
- Add support in systemctl to call "systemctl exit 42". It reuses the
  existing code for user session.
- Add exit.target and systemd-exit.service to the system instance.
- Change main() to actually call systemd-shutdown to exit() with the
  correct value.
- Add verb 'exit' in systemd-shutdown with parameter --exit-code
- Update systemctl manpage.

I used the following to test it:

| $ sudo rkt --debug --insecure-skip-verify run \
|            --mds-register=false --local docker://busybox \
|            --exec=/bin/chroot -- /proc/1/root \
|            systemctl --force exit 42
| ...
| Container rkt-895a0cba-5c66-4fa5-831c-e3f8ddc5810d failed with error code 42.
| $ echo $?
| 42

Fixes https://github.com/systemd/systemd/issues/1290

test-ipv4ll: use unsigned instead of an array for a seed

ipv4ll use an unsigned instead of an uint8_t array. Hence, use an
unsigned seed instead of declaring an array and then dereferencing it
later.

sd-ipv4ll: do not assert_return when seed == 0

Now that seed is an unsigned and not an array, we do not need to assert
on it.

Merge pull request #1288 from teg/ipv4acd-3

sd-ipv4acd: split out as separate library from sd-ipv4ll

Merge pull request #1311 from jsynacek/kill-errors-v2

core: extend KillUnit() to return error when no unit was killed

Merge pull request #1314 from evverx/cgtop-machine

cgtop: add -M/--machine

cgtop: add -M/--machine

core: extend KillUnit() to return error when no unit was killed

Merge pull request #1292 from gebi/resolve-specifier-in-runtime-directory

load-fragment: resolve specifiers in RuntimeDirectory

Merge pull request #1305 from mrkara/master

[l10n] Added Turkish translation

Merge pull request #1309 from nazgul77/master

[l10n] Update German translation

Update German translation

systemd-notify: Always pass a valid pid to sd_pid_notify

If the option --pid was used, take the pid from this option, unless take
the parend pid. Using 0 as pid (ucred of systemd-notify) will result 99% of the
time in a failure with this error: "Cannot find unit for notify message of PID"

Shouldn't we use always the ppid, since the MAINPID is something else ?

Signed-off-by: Benjamin Robin <dev@benjarobin.fr>

[l10n] Added Turkish translation

Merge pull request #1303 from evverx/show-controller

cgls: show controller without args too

cgls: show controller without args too

sd-ipv4{acd,ll}: add simple test programs

These programs should be run manually, typically two instances on a
veth pair to check conflict detection.

Both test programs take the ifname as input, the ACD also takes the
IP address to check, whereas LL (optionally) takes the seed, which
determines the sequence of IP addresses to try.

sd-ipv4acd: introduce new library split out from sd-ipv4ll

This splits the Address Conflict Detection out of the Link Local
library so that we can reuse it for DHCP and static addresses in
the future.

Implements RFC5227.

sd-ipv4ll: simplify conflict handling

Use stop() and start() to drop some pulicate code.

sd-ipv4ll: rework callbacks

Firstly, no longer distinguish between STOP and INIT states.

Secondly, do not trigger STOP events when calls to sd_ipv4ll_*() fail. The
caller is the one who would receive the event and will already know that the
call to sd_ipv4ll_*() has failed, so it is redundant.

STOP events will now only be triggered by calling sd_ipv4ll_stop() explicitly
or by some internal error in the library triggered by receiving a packet or
an expiring timeout (i.e., any error that would otherwise not be reported
back to the consumer of the library).

Lastly, follow CODING_STYLE and always return NULL on unref. Protect from
objects being destroyed in callbacks accordingly.

sd-ipv4ll: don't allow changing MAC address whilst running

This requires the caller to stop and restart the statemachine if they want to
change the MAC address.

sd-ipv4ll: code cleanups

Simplify timeout handling.

sd-ipv4ll: remove duplicate packet verification

Most packets are filtered out by the BPF, so only check for the parts that may
actually differ.

sd-ipv4ll: minor cleanups

sd-ipv4ll: split out on_conflict() from on_packet()