import: use structured initializers

import: use _cleanup_ attribute for CURL object

Fix typo in hacking guide: "no possible" -> "not possible" (#10905)

analyze: Fix build without seccomp (#10899)

tests: add a fuzzer for server_process_native_file

dhcp6-client: handle IAID with value zero

config_parse_iaid(), dhcp_identifier_set_iaid() and sd_dhcp6_client_set_iaid() all
allow for the IAID to be zero. Also, RFC 3315 makes no mention that zero
would be invalid.

However, client_ensure_iaid() would take an IAID of zero as a sign that
the values was unset. Fix that by keeping track whether IAID is
initialized.

Merge pull request #10887 from poettering/run-love

systemd-run: modernizations, some new switches and let's set $INVOCATION_ID

Merge pull request #10893 from keszybz/udev-cleanup-followup

Udev cleanup follow-ups

pid1,sd-device: use PATH_STARTSWITH_SET more

basic/path-util: add missing header

PATH_STARTSWITH_SET uses STRV_FOREACH...

run: set $INVOCATION_ID for scope units

Services invoked by PID1 have $INVOCATION_ID initialized, hence let's do
that for scope units (where the payload process is invoked by us on the
client side) too, to minimize needless differences.

Fixes: #8082

run: port to static destructors

run: port to DEFINE_MAIN_FUNCTION()

run: add new --shell switch for spawning a shell as service

I keep running "systemd-run -t /bin/bash" to quickly get a shell running
in service context. I suspect I am not the only one, hence let's add a
shortcut for it. While we are at it, let's make it smarter, and
automatically inherit the $SHELL of the invoking user as well as the
working directory, and let's imply --pty. --shell (or -S) is hence
equivalent to "-t -d $SHELL".

run: add a switch for specifiying the working directory of a service

I find myself testing service management quite often with "systemd-run
-t /bin/bash". For that it is handy if the invoked shell would use the
working directory I am currently in. Hence introduce a shorthand for
that:

        $ systemd-run -dt /bin/bash

This will automatically insert a WorkingDirectory= property into the
transient service, pointing to the working directory of the caller.

libudev: drop unnecessary check

sd_device_get_subsystem returns -ENOENT if subsystem is not set.

Merge pull request #10861 from yuwata/udev-list-cleanups

libudev: several cleanups for udev-list

Merge pull request #10886 from yuwata/sd-device-monitor-fix-filter

sd-device: fix subsystem filter

unit: minor optimization, use stack over heap, when we can

70-mouse.hwdb: Set DPI for MS Classic IntelliMouse

It defaults to 3200 DPI and 1000 Hz. Note the ?? is required to
match the multibyte UTF-8 character '®'.

Merge pull request #10891 from yuwata/use-main-macro-2

tree-wide: use recently introduced macros

unit: drop an unused fields from Unit struct

test: add more tests for subsystem filter

core: add two minor comments (#10890)

locale: define main through macro

locale: store polkit_registry in Context

locale: rename context_free() to context_clear()

resolvectl: use static destructor and define main through macro

rfkill: define main through macro

rfkill: store write queue and rfkill fd in struct Context

socket-proxy: define main through macro

sysusers: use ordered_hashmap_steal_first_key_and_value()

time-wait: define main through macro

timedate: define main through macro

update-done: quit earlier on failure

update-utmp: use _cleanup_ attribute to finalize process

veritysetup-generator: use static destructors and define main through macro

veritysetup-generator: make arg_dest const

veritysetup: use static destructors and define main through macro

This also changes the return value when crypt_init_by_name() fails in 'detach'.

tests: add a fuzzer for catalog_import_file

man: update description of Description=

The way this is used drifted a bit from the original intent. Let's update
the description and add some examples to inspire people to texts that look
less bad during initial boot.

Merge pull request #10884 from yuwata/udev-debug

meson: add more debug options

sd-device-monitor: fix subsystem filter

This fixes a bug introduced by 759d9f3f8d07af2940bb3783acc3985ee47adfa5.

Fixes #10882.

udev-rules: update log messages

meson: also add option for debugging siphash

meson: add option for debugging udev

udev-rules: trivial coding style cleanups

udev: rename udev_rules_unref() to udev_rules_free()

As udev_rules do not have a reference counter.

libudev: do not ignore error in udev_list_entry_add()

libudev: introduce return_with_errno() and use it where applicable

test: add tests for libudev-list

libudev-list: drop unused udev argument from udev_list_init()

libudev-list: drop unused private functions

Merge pull request #10871 from keszybz/more-cleanup-2

Allow "synthetic" errno to be used in log calls

sysctl: when debug logging about sysctl changes, truncate trailing newline

basic/log: add note about operator precendence

CODING_STYLE: describe log & return operations

Also drop a few more unnecessary uses of synthethic errno

coccinelle: also mark previous synthetic errnos as such

coccinelle: make use of SYNTHETIC_ERRNO

Ideally, coccinelle would strip unnecessary braces too. But I do not see any
option in coccinelle for this, so instead, I edited the patch text using
search&replace to remove the braces. Unfortunately this is not fully automatic,
in particular it didn't deal well with if-else-if-else blocks and ifdefs, so
there is an increased likelikehood be some bugs in such spots.

I also removed part of the patch that coccinelle generated for udev, where we
returns -1 for failure. This should be fixed independently.

basic/log: add concept of "synthethic errnos"

Synthetic errnos are processed like normal, and may be used in %m and become
the return value from log_*(), but they are not logged in the journal.

vconsole: Don't skip udev call for dummy device

Kernel 4.19 supports deferred console takeover [0], i.e., fbcon will
take over the console only when the first text is displayed on the
console. Before that event, only the dummy console is active. Our
currently udev rules call systemd-vconsole on every vtcon except for
dummy consoles. Thus the exception for dummy consoles prevents a call
to systemd-vconsole when no text is displayed on the console, and as a
consequence, the keymap will not be set in that case. This is wrong and
leads to issues when keyboard input is expected without text on the
console, e.g., when a graphical password prompt is used in the boot
process.

This reverts commit 6b169c13ecf645a0a341f49b9dc2afb312a74206,
which introduced the exception for dummy devices to save unnecessary
udev calls.

Fixes #10826.

[0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=83d83bebf40132e2d55ec58af666713cc76f9764

basic/user-util: properly protect use of gshadow

Commit 100d5f6ee6 (user-util: add new wrappers for [...] database
files), ammended by commit 4f07ffa8f5 (Use #if instead of #ifdef for
ENABLE_GSHADOW) moved code from sysuser to basic/user-util.

In doing so, the combination of both commits properly propagated the
ENABLE_GSHADOW conditions around the function manipulating gshadow, but
they forgot to protect the inclusion of the gshadow.h header.

Fix that to be able to build on C libraries that do not provide gshadow
(e.g. uClibc-ng, where it does not exist.)

Merge pull request #9961 from fbuihuu/logind-fix-vt-reinit-race

Logind fix vt reinit race

Revert 5fdf2d51c244288ac41443d1bd81365fab7b7b81

This reverts 5fdf2d51c244288ac41443d1bd81365fab7b7b81, except for one improved
log message.

Fixes #10613.

Checking if resume= is configured is a good idea, but it turns out we cannot do
it reliably:
- the code only supported boot options with sd-boot, and it's not very widely
  used. This means that for most systemd we could only check the current
  commandline, not the next one.
- Various systems resume without, e.g. Debian has
  /etc/initramfs-tools/conf.d/resume in the initramfs.

Making those checks better would be possible with enough effort, but there'll
be always new systems that boot in a slightly different way and we would need
to keep adding new cases. Longer term, we want to rely on autodetecting the
resume partition, and then checks like this will not be necessary at all. It is
quite clear from the number of bug reports that the number of poeple impacted
by this is quite high now, so let's just drop this.

logind: stop managing VT switches if no sessions are registered on that VT

When no sessions are registered on a given VT (anymore), we should always let
the kernel processes VT switching (instead of simply emitting a warning)
otherwise the requests sent by the kernel are simply ignored making the VT
switch requested by users simply impossible.

Even if it shouldn't happen, this case was encountered in issue #9754, so
better to be safe than sorry.

terminal-util: introduce vt_release() helper

logind: become the controlling terminal process before restoring VT

Basically when a session ends, logind notices and restores VT_AUTO so the
kernel takes back VT-switching over.

logind achieves that by watching the process that took control of the session
(via the "TakeControl" D-Bus method), aka "the watched process", which can
be different from the one that initially opened the VT aka "the terminal
controlling process".

In this case the terminal controlling process can exit after the watched one
did and while logind is restoring the VT.

Even if logind took care to re-open the VT in case the VT was already in HUP
state, it wasn't enough because the terminal controlling process could have
exited right after, leaving the VT in HUP state and in VT_PROCESS mode making
further VT-switching impossible.

This patch fixes this situation by forcing logind to become the terminal
controlling process.

Fixes: #9754.

terminal-util: introduce vt_restore() helper

logind: make session_restore_vt() static

It's only used in logind-session.c.

generators: drop umask calls

systemd already sets the umask (see e3b8d0637dd755b3426f3363b2cdad63f738116c). When
running under systemd, we don't need to set it. And when *not* running under
systemd, for example during development, there is no reason to override the user
config. Let's just drop those calls.

$ git grep -e 'umask\(' -l 'src/*generator*' |xargs perl -i -0pe 's|^[^\n]*umask\([^\n]+\n\n||gms'

core: Detect initial timer state from serialized data

We keep a mark whether a single-shot timer was triggered in the caller's
variable initial. When such a timer elapses while we are
serializing/deserializing the inner state, we consider the timer
incorrectly as elapsed and don't trigger it later.

This patch exploits last_trigger timestamp that we already serialize,
hence we can eliminate the argument initial completely.

A reproducer for OnBootSec= timers:
        cat >repro.c <<EOD
        /*
         * Compile: gcc repro.c -o repro
         * Run: ./repro
         */
        #include <errno.h>
        #include <fcntl.h>
        #include <stdio.h>
        #include <stdlib.h>
        #include <sys/stat.h>
        #include <sys/types.h>
        #include <time.h>
        #include <unistd.h>

        int main(int argc, char *argv[]) {
         char command[1024];
         int pause;

         struct timespec now;

         while (1) {
         usleep(rand() % 200000); // prevent periodic repeats
                clock_gettime(CLOCK_MONOTONIC, &now);
         printf("%i\n", now.tv_sec);

         system("rm -f $PWD/mark");
         snprintf(command, 1024, "systemd-run --user --on-boot=%i --timer-property=AccuracySec=100ms "
         "touch $PWD/mark", now.tv_sec + 1);
         system(command);
         system("systemctl --user list-timers");
         pause = (1000000000 - now.tv_nsec)/1000 - 70000; // fiddle to hit the middle of reloading
         usleep(pause > 0 ? pause : 0);
         system("systemctl --user daemon-reload");
         sync();
         sleep(2);
         if (open("./mark", 0) < 0)
         if (errno == ENOENT) {
         printf("mark file does not exist\n");
         break;
         }
         }

         return 0;
        }
        EOD

Merge pull request #10866 from yuwata/libudev-util-cleanups

libudev-util: several cleanups and tests

Merge pull request #10860 from keszybz/more-cleanup-2

Do more stuff from main macros

test: add tests for util_resolve_subsys_kernel()

libudev: make util_resolve_subsys_kernel() return negative errno

This also replaces udev_device by sd_device in the function.

udev/scsi_id: fix buffer length

test: add tests for util_replace_whitespace()

libudev-util: make util_replace_whitespace() not count leading white spaces

udevd: define main through macro

This removes the call to log_close(), and refactors how fork() is done. Now
the parent also goes through normal cleanup. This isn't necessary to use the
macro, but it feels cleaner this way.

udevadm: define main through macro

This removes a call to log_close(). I don't think this should matter.
The call to mac_selinux_init() is moved after parse_argv(). We probably
don't need selinux when printing help().

Call mac_selinux_close() from main func macros, convert user-sessions and test-udev

systemctl: define main through macro and call ask_password_agent_close() from the macro

This doesn't save us anything, but I like consistency.

libudev-util: make util_replace_*() return size_t

libudev-util: add assertions

core: run env generators with non-zero umask

For PID 1 we adjust the umask to 0, but generators should not run that
way, given that they might be implemented as shell scripts and such.
Let's hence explicitly adjust the umask for them.

We already do this for unit generators. Let's do this for env
generators, too.

tests: skip test_exec_ambientcapabilities on Travis CI under ASan

Let's not bother contributors with spurious failures nobody can't
seem to reproduce. There is an issue about that where we're trying
to figure out what's going on: https://github.com/systemd/systemd/issues/10696.

Merge pull request #10853 from poettering/thaw-containers

main: don't freeze PID 1 in containers, exit with non-zero instead

Merge pull request #10858 from poettering/tmpfiles-fixlets

more tmpfiles fixlets

portablectl: make "arg_host" const

mount-tool: use static destructor and DEFINE_MAIN_FUNCTION() macro

modules-load: use static destructor and DEFINE_MAIN_FUNCTION() macro

machinectl: use static destructor and DEFINE_MAIN_FUNCTION() macro

shared/main-func: also close the polkit agent automatically

The agent is closed after the static destuctors but before the pager.
No users of DEFINE_MAIN_FUNCTION* were using a polkit agent, so this makes no
functional difference.

Split out pretty-print.c and move pager.c and main-func.h to shared/

This is high-level functionality, and fits better in shared/ (which is for
our executables), than in basic/ (which is also for libraries).

machine-id: use static destructor and DEFINE_MAIN_FUNCTION() macro

loginctl: use static destructor and DEFINE_MAIN_FUNCTION() macro

locale: make "arg_host" const

hostnamed: use DEFINE_MAIN_FUNCTION() macro

This also renames context_free() to context_clear().

hibernate-resume-generator: use static destructor and DEFINE_MAIN_FUNCTION() macro