resolve: ignore invalid service template name

Let's fiest test the template name, and then assign it.

resolve: make config_parse_dnssd_service_name() accepts an empty string

resolve: add a short comment about difference between dnssd_render_instance_name() and config_parse_dnssd_service_name()

resolve: wrap long line

Revert "resolve: check DNSSD service name template before assigning it"

This reverts commit 34136e1503cf60852051adbd8b9a002d6282b750.

Having the "%H" host name specifier in a DNSSD service name template
triggers a failed assertion during name template instantiation as
specifier_dnssd_host_name expects DnssdService in its userdata
pointer but finds NULL instead.

hwdb/60-keyboard: untabify and move comments to the same column

Follow-up for b0a3a3ff5d80d1605926fbba1d25a0f2448e383c.

Merge pull request #17640 from keszybz/meson-test-c++20

Also test headers against c++20

Merge pull request #17643 from keszybz/man-news-independent

NEWS and man page tweaks

NEWS,man: improve descriptions of Independent=

In NEWS, the new option was described twice, most likely because the first
description was tucked away in a paragraph about some other subject.

While at it, improve the descriptions in the man page to make it easier to grok
what that option really does.

man: drop misplaced phrase

I think it was added in 6c2b9c8da1eb1bac7e8f170b913f63dbd13a16d7 as c&p.

man: Add a paragraph to sd_bus_call explaning callback message lifetime

Merge pull request #17584 from yuwata/news-networkd

NEWS: several entries for networkd

network: use the common "cannot" form

Just for consistency.

man/systemd.netdev: remove bogus markup

There is no "Multicast" constant, and NULL doesn't make sense in the
context of addresses.

man/systemd.netdev: don't say "ranges"

NEWS: slightly fix explanation about oomd

man: DHCPv6PrefixDelegation= is disabled by dafault

NEWS: add several entries for networkd

Merge pull request #17599 from yuwata/meson-fuzz-tests

meson: do not build fuzzers when -Dfuzz-tests=false

test-oomd-util: remove memory_pressure == 0 checks

test_oomd_cgroup_context_acquire_and_insert reads the live cgroup data used
by the unit test. Under certain conditions, the memory pressure for the cgroup
can be non-zero (although most of the time it is 0 since these tests don't
generate much pressure).

Since these values are too dependent on the state of the system, remove the
checks. The type used is always >= 0 and test-psi-util already unit tests that
PSI values are parsed correctly from files so this test is redundant anyways.

man: mention that FirewallMark= optionally takes firewall mask

Closes #17587.

Merge pull request #17597 from yuwata/fix-typo

tree-wide: Fix typo

test: let's start getting ready for 2020

It'll be a wonderful year, I'm sure.

meson: use proper variable for libudev.h path

meson: fix build tests for c++14 and c++17

Merge pull request #17603 from yuwata/systemctl-fix-underline

systemctl: always show underline even if unit has no job ID

udev: use FOREACH_DEVICE_TAG() macro at one more place

Merge pull request #17612 from yuwata/fix-man-pages

man: fix two issues

Merge pull request #17614 from jwrdegoede/hwdb-logitech-kbd-fixes

hwdb Logitech keyboard fixes

man: Separated paragraph about making sd_bus object reply async

update TODO

man: add missing full stop

Merge pull request #17619 from yuwata/network-ndisc-dhcp6-client-always

network: honor M or O flag in RA even if IPv6AcceptRA.DHCPv6Cleint=always

meson: use "_" as separator in test names

Follow-up for ca121e20c42219e3bc4e5cb63dcc96cc5eae2879.

Fixes #17568.

sd-dhcp6-client: insert spaces around ternary operator

network: honor M or O flag in RA even if IPv6AcceptRA.DHCPv6Cleint=always

Follow-up for ac24e418d9bc988ecf114c464701b35934948178.

The original motivation of the commit and RFE #15339 is to start dhcpv6
client in managed mode when neither M nor O flag is set in the RA.
But, previously, if the setting is set to "always", then the DHCPv6
client is always started in managed mode even if O flag is set in the
RA. Such the behavior breaks RFC 7084.

missing: define several syscall numbers for MIPS arch

Fixes #17591.

Merge pull request #17600 from yuwata/drop-log-oom

util: drop log_oom() from several library functions

test: bump timeout for test-libcrypt-util

Apparently this test is prone to time out on slower architectures, so
bump it from 30s to 120s.

See
https://buildd.debian.org/status/fetch.php?pkg=systemd&arch=mipsel&ver=247%7Erc2-2&stamp=1605303506&raw=0
https://buildd.debian.org/status/fetch.php?pkg=systemd&arch=sparc64&ver=247%7Erc2-2&stamp=1605294845&raw=0

clang-format: set SpaceBeforeParens to ControlStatementsExceptForEachMacros

I contributed this option to LLVM a while ago to reduce the amount of
changes clang-format makes when formatting systemd source files. As
LLVM 11 is now starting to becomes widely available, let's configure
it in the clang-format config.

networkd: fix default value of DHCPv6Client

The configuration of networkd has a DHCPv6Client setting in its
[IPv6AcceptRA] section, which, according to the man page, can be
a boolean, or the special value "always". The man page states
that "true" is the default.

The default value is implemented in src/network/networkd-network.c
by setting field ipv6_accept_ra_start_dhcp6_client of network to
true. However, this field is not a boolean, but an enum type
IPv6AcceptRAStartDHCP6Client (src/network/networkd-ndisc.h).

Setting ipv6_accept_ra_start_dhcp6_client to true effectively
corresponds to the enum value IPV6_ACCEPT_RA_START_DHCP6_CLIENT_ALWAYS,
resulting in the DHCPv6Client setting having the default value
"always".

This patch changes the initialisation to the correct enum value
IPV6_ACCEPT_RA_START_DHCP6_CLIENT_YES.

hwdb: 60-keyboard: Update Logitech MX5000 key mappings

2 updates to the Logitech MX5000 key mappings:

1. Fix Logitech MX5000 Media key mapping:

The MX5000 has a key clearly marked "media" (in text) and when they gets
pressed the keyboards LCD briefly shows "MEDIA" in a big font.

Yet the keyboard sends a HID usage code which according to the HUTT
is config/control-panel. Map this to media so that the key behaves as
labeled.

2. Add mapping for the scan-code send by the Fn on/off toggle key

hwdb: 60-keyboard: Add Logitech Dinovo Edge special key key-mappings

The Logitech Dinovo Edge has 5 special hotkeys, one with a phone symbol
and the 4 usual Logitech A-D smartkeys. These send custom Logitech
0xc10xx keycodes, add mappings for these.

hwdb: 60-keyboard: Add Fn key mapping for Logitech wireless keyboards

Some Logitech wireless keyboards report when there Fn key is pressed,
add a mapping for this.

While it also fix identiation of 2 adjescent key-maps to use tabs like the
other Logitech entries

man: fix prototpe of sd_bus_message_peek_type()

Fixes #17609.

man: add missing <para> tag, and drop redundant ">"

Fixes #17609.

systemctl: fix minor coding style issue

systemctl: use unsigned for job_count

As, the number of units `c` is unsigned.

systemctl: always show underline even if unit has no job ID

io-util: do not call log_oom() in library functions

bpf: do not call log_oom() in library function

ci: also set -Dfuzz-tests=true if -Dslow-tests=true

meson: do not build fuzzers when fuzz-tests=false

Closes #17578.

test: fix typo

man: fix typo

tree-wide: fix "a the" or "the a"

ratelimit: fix integer overflow

If StartLimitIntervalSec is huge, or more specifically, set to "infinity", we need to take care about overflows.

Merge pull request #17583 from systemd/oomd-and-user-slices

Make oomd easier to enable and user slices to NEWS

NEWS: describe the new user slices

meson: allow oomd to be enabled even in release mode

A distro (Fedora in particular) may want to enable oomd in a unstable
branch for testing, even though the package as a whole is compiled in release
mode. Let's emit a warning but otherwise allow this.

hwdb: 60-keyboard:: add Dell Privacy Micmute Hotkey

Dell new Privacy feature provide new hardware level privacy
protect for users,it needs to map the scan code to F20 micmute

Signed-off-by: Perry Yuan <perry_yuan@dell.com>

Merge pull request #17561 from yuwata/man

meson: add option to skip installing to $sysconfdir

This is useful for development where overwriting files out side
the configured prefix will affect the host as well as stateless
systems such as NixOS that don't let packages install to /etc but handle
configuration on their own.

Alternative to https://github.com/systemd/systemd/pull/17501

tested with:

$ mkdir inst build && cd build
$ meson \
  -Dcreate-log-dirs=false \
  -Dsysvrcnd-path=$(realpath ../inst)/etc/rc.d \
  -Dsysvinit-path=$(realpath ../inst)/etc/init.d \
  -Drootprefix=$(realpath ../inst) \
  -Dinstall-sysconfdir=false \
  --prefix=$(realpath ../inst) ..
$ ninja install

license: LGPL-2.1+ -> LGPL-2.1-or-later

Follow-up for db9ecf050165fd1033c6f81485917e229c4be537.

man: set constant tag to NUL or NULL

man: append parentheses for function name

man: fix tag

man: mention that sd_bus_call() may return a negative errno mapped from D-Bus error reply

Closes #17556.

man: add missing period

Merge pull request #17567 from keszybz/various-small-cleanups

Various small cleanups

meson: use "_" as separator in test names

":" is prettier, but meson 0.56+ doesn't like it:
src/systemd/meson.build:73: DEPRECATION: ":" is not allowed in test name "cc-sd-bus.h:c", it has been replaced with "_"
src/systemd/meson.build:73: DEPRECATION: ":" is not allowed in test name "cc-sd-bus.h:c-ansi", it has been replaced with "_"
...

Fixes #17568.

Merge pull request #16603 from benzea/benzea/special-app-slice

Use app.slice by default in user manager (and define special user slices)

NEWS,man: tweak the description of FixedRandomDelay=

Explain why this is useful, but don't describe the implementation exactly,
since we're likely to want to change details in the future.

user: move "extrinsic" units to their root slice

With the grandparent change to move most units to app.slice,
those units would be ordered After=app.slice which doesn't make any sense.
Actually they appear earlier, before the manager is even started, and
conceputally it doesn't seem useful to put them under any slice.

Merge pull request #17566 from poettering/v247-rc2-prep

V247 rc2 prep

basic/fileio: constify struct timespec arguments

libsystemd-network: add comment explaining unusual memory access

Inspired by coverity CID#1435984. I'm confused by the union definion every time I look at it...
Let's at least add a comment to help future readers.

tree-wide: unsetenv cannot fail

... when called with a valid environment variable name. This means that
any time we call it with a fixed string, it is guaranteed to return 0.
(Also when the variable is not present in the environment block.)

basic/env-util: add little helper to call setenv or unsetenv

pid1: expose "extrinsic" status of swaps and mounts

The only visible change from this is that we show Extrinsic: yes/no
in dumps for swap units (this was already done for mount units).

man: properly list relative time event source API in man page

The content was already there, but it wasn't listed in the header
metadata. Fix that.

sd-event: minor modernization

With this change the pattern used for epoll_ctl() is the same in all calls in
this file. Consistency FTW!

sd-event: increase n_enabled_child_sources just once

Neither source_child_pidfd_register() nor event_make_signal_data() look at
n_enabled_child_sources.

sd-event: update state at the end in event_source_enable

Coverity in CID#1435966 was complaining that s->enabled is not "restored" in
all cases. But the code was actually correct, since it should only be
"restored" in the error paths. But let's still make this prettier by not setting
the state before all operations that may fail are done.

We need to set .enabled for the prioq reshuffling operations, so move those down.

No functional change intended.

hwdb: update

git-contrib: exclude -rc tags

update NEWS

Merge pull request #17497 from anitazha/randomizeonce

timer: add new feature FixedRandomDelay=

Add Logitech G403 Prodigy Gaming Mouse to hwdb. (#17537)

This PR adds logitech G403 to the hwdb.
Settings were obtained with 'sudo mouse-dpi-tool /dev/input/event3' and 'ratbagctl hollering-marmot info'.

Merge pull request #17555 from yuwata/ethtool-get-driver

ethtool: make ethtool_get_driver() return -ENODATA if driver is an empty string.

journal: refuse skip parameter for sd_journal_next_skip() larger than INT_MAX

Fixes #17502.

Merge pull request #17431 from msekletar/udev-link-update-race

udev: make algorithm that selects highest priority devlink less susceptible to race conditions

sd-bus: drop redundant abs()

strerror_safe() internally applies abs().

core: add missing oom check

Fixes #17552.

udev: log when we fail to query ID_NET_DRIVER

Inspired by #17532.

Merge pull request #17551 from yuwata/radv-router-lifetime

network: about router lifetime

ethtool: make ethtool_get_driver() return -ENODATA if ioctl succeeds but driver name is empty

Inspired by #17532.

ethtool: add several assertions

network: convert router lifetime from usec_t to uint16_t

sd-radv: refuse to set preference except medium when router lifetime is zero