NEWS: finalize

Merge pull request #26427 from bluca/hwdb

Update news and hwdb

hwdb: update

NEWS: update contributors list

cryptsetup: do not assert when unsealing token without salt

Salt was added in v253. We are not checking whether it was actually found
(non-zero size), so when an old tpm+pin enrollment is opened things go boom.
For good measure, check both the buffer and the size in both places.

Assertion 'saltlen > 0' failed at src/shared/tpm2-util.c:2490, function tpm2_util_pbkdf2_hmac_sha256(). Aborting.

ukify: fix padding length

Merge pull request #26417 from yuwata/sysusers-root-group

sysusers: also create root group

Merge pull request #26424 from ldv-alt/fix-typos

Fix a few typos in NEWS, docs and comments

man/tc: Fix hexadecimals being with an O

treewide: fix a few typos in NEWS, docs and comments

NEWS: mention that the default mDNS mode is now "yes"

C.f. issue #25252, PR #25255, and
RHBZ#2169786 (https://bugzilla.redhat.com/show_bug.cgi?id=2169786).

src: fix several typos in log messages

test-sysusers: add test for basic.conf

sysusers: also add root group

Follow-up for 49bb7fe5f88fc35b8529d7d8dfcd4c151a9aaf1a.

Fixes an issue reported at
https://github.com/systemd/systemd/pull/26270#issuecomment-1428945403.

hwdb: Add HP ENVY x360 2-in-1

Since #26144, RFKILL events are disabled for HP ENVY x360 Convertible.
This commit adds a variation of the name.

bootctl: avoid using __WORDSIZE macro

__WORDSIZE does not seem to be documented anywhere, and is probably
meant to be used internally by glibc headers.

In systemd, it was only being used in warning messages. We can avoid
using it by rewording the messages slightly.

Fixes a build error with musl libc.

Bug: https://bugs.gentoo.org/894430

core/execute: fix comment

tree-wide: fix typo and comment style update

test-boot-timestamp: Handle ERANGE error

Timestampfs from sysfs files can be zero in which case ERANGE will
be returned so let's make sure we catch that.

boot: Ensure raise() is not dropped by LTO

mkosi: Stop installing kernel headers to /usr

The selftests automatically pick up the headers from the kernel
build directory so we don't have to install them to /usr ourselves.

Merge pull request #26400 from ml-/fix-directory-and-typos

Fix directory for user home bind mounts in log error and man page

man: fix typos

man: fix directory for user home bind mounts

nspawn: fix directory in logged error

ci(labeler): fix missing emoji in `dont-merge` label

ci(labeler): fix missing emoji in `quick-review` label

NEWS: update date

Merge pull request #26392 from bluca/news

Update hwdb and news

dissect-image: unknown/unsupported diskseq is indicated by 0, not by UINT64_MAX

At almost all places if diskseq is not supported we encode this as
diskseq zero. But in two places we got the check for that wrong,
assuming it was UINT64_MAX.

Fix that.

hwdb: update database

NEWS: update contributors list

core: when isolating to a unit, also keep units running that are triggered by units we keep running

Inspired by: #26364

(this might even "fix" #26364, but without debug logs it's hard to make
such claims)

Fixes: #23055

userdb: Use json_dispatch_user_group_name() to parse GetMembership fields

It allows to relax the checks and allow characters like '\', used by
windows to split the domain name and user name.

For reference, discussion in the systemd-devel mailing list:
https://lists.freedesktop.org/archives/systemd-devel/2023-February/048804.html

Signed-off-by: Samuel Cabrero <scabrero@suse.de>

mkosi: Use globs instead of prepare script to install extra packages

This allows us to install everything in the same dnf command instead
of having to use a prepare script to run dnf from within the image.

This is a hack until mkosi supports release specific dropin files.

Merge pull request #26387 from bluca/swapon_util_linux

README/NEWS: note that we now explicitly require util-linux's swapon due to new option

NEWS: note that we require a swapon that supports --fixpgsz

README: explicitly note that util-linux's mount/swap are required

These are the most visible and hard requirements, as we use options that
busybox does not provide, so list them explicitly to avoid surprises

cryptenroll: drop deadcode

Follow-up for b0fc23fae51d244d2c33d70c10003aa5d5840223.

After the commit, 'signature_path' is now always non-NULL, hence the
condition can be dropped.

Fixes CID#1504492.

mkosi: Update to latest

Let's make sure we're testing unprivileged builds properly. Usage
of SourceFileTransfer= and SourceFileTransferFinal= are removed as
they were dropped by mkosi. SourceFileTransfer=mount is now the
default in mkosi so behavior for the build script is unchanged. We
stop copying sources in the final image until mkosi adds support
for virtiofs.

boot: Fix undefined reference to raise() on arm

This is just a workaround. Once we drop gnu-efi, the arm build system
for EFI binaries should be changed to use the arm-none-eabi toolchain,
which should not exhibit this behavior.

various: boldify version output

Follow-up for 4453ebe4db0511d25bed1040930ea6430c1bed91.
With the feature list all dandified, the most important part of the
output, i.e. the project name and version, are less visible.

boot: Make sure we take --root into account everywhere.

Merge pull request #26377 from keszybz/doc-fixups-2

Fix links in man pages

man: fix links to man pages

Done using linkchecker as usual.

man: fix section number

Fixes #26376.

units: change assert to condition to skip running in initrd/os

These units are also present in the initrd, so instead of an assert,
just use a condition so they are skipped where they need to be skipped.

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

bootctl: Add missing %m

Merge pull request #26366 from yuwata/nss-myhostname

nss-myhostname: two fixlets

nss-myhostname: do not return empty result with NSS_STATUS_SUCCESS

Fixes a bug introduced by db50d326a46beca3cc24b6354b6e1b3591902d45.

Fixes RHBZ#2167468 (https://bugzilla.redhat.com/show_bug.cgi?id=2167468).

nss-myhostname: fix inverted condition in

Fixes a bug introduced by db50d326a46beca3cc24b6354b6e1b3591902d45.

Merge pull request #26354 from bluca/news

NEWS: add future incompatible changes notice

process-util: add missing error check

hwdb: add override for IdeaPad5 insert key

Fixes #25968.

I wrote the rule to assume that all IdeaPad5 thingies are like that.

Merge pull request #26225 from qdeslandes/fix_delegate_cgroup_logs_filtering

Fix delegate cgroup logs filtering

ukify: add explanatory message when import fails

journal: modernize sd_journal_get_realtime_usec() a bit

This does what 404803e6caad2de2d8e74caab0b79ec3f030f801 did for the
monotonic timestamp getter, but for the realtime timestamp.

It also also makes the return value optional, exactly as for the
monotonic timestamp logic.

NEWS: note about future implicit PrivateUsers= in user units

Merge pull request #26350 from keszybz/reload-messages

Improve messages emitted when Reload or Reexec is requested

docs/NETWORK_ONLINE: fix example

Type=oneshot is necessary for systemd to actually wait for the service
to return. With RemainAfterExit=yes it won't be started again.

Fixes #26342.

journald: fix ignored filtering patterns for delegated cgroups

If a service defines Delegate=yes, its subcgroup won't inherit the
LogFilterPatterns= option, because the option is stored on the unit's
cgroup attributes, not on the subcgroup.

Fixed by using the unit's cgroup attributes instead.

core: add cg_path_get_unit_path()

From a given cgroup path, cg_path_get_unit() allows to retrieve the
unit's name. Although, this removes the path to the unit's cgroup,
preventing the result to be used to fetch xattrs.

Introduce cg_path_get_unit_path() which provides the path to the unit's
cgroup. This function behave similarly to cg_path_get_unit() (checking
the validity and escaping the unit's name).

sd-bus: adjust line breaks

manager: improve message about Reload/Reexec requests

If we fail to get the necessary information, let's just not print that
part of the message. 'n/a' looks pretty ugly.

I used a bunch of ternary operators instead of seperate log lines because
with two components that might or might not be there, we need four different
combinations.

Also, the unit name doesn't need to be quoted, it's always printable.

manager: "downgrade" message about command vanishing from the unit file

We would print "Current command vanished from the unit file, execution of
the command list won't be resumed." as a warning, but most of the time there
is nothing to resume, because a unit has just one command. So let's detect
the case where the command that was active is the last command in the sequence
and skip the warning.

I was considering how to store the information that the command is last. An
important consideration is not to use a format that would confuse older versions
of systemd. (It wouldn't be a big problem if older systemd just refused the
new serialization, since we require systemd to be newer, but we should avoid
the case where the deserialization is "successful", but actually incorrect.)
Similarly, the deserialization from the old systemd must not confuse new systemd.
For this command, we have a list of arguments at the end, so just adding a
new field either in the middle or at the end is problematic because it's hard
to ensure that we don't mix up the positional and variable arguments.

We actually need to store just one bit of information, so '+' is prefixed on
the index of the last command and used by new systemd to skip the warning.
When deserializing from older systemd, '+' is not present, so we detect all
commands as "not last", and still emit the warning, so we err on the side of
caution. If the user were to deserialize from newer to older systemd, nothing
untoward would happen, because the '+' is ignored. (Users shouldn't do this,
but we know that this occasionally happens with initrds or exitrds and package
downgrades.)

NEWS: copy future incompatible changes notice from 252

These are in the future, so it's good to re-emphasize on every release until they
actually happen

ci: Add names to steps in labeler workflow

This makes it easier to see what step failed/was skipped in the GitHub
Actions UI. It also makes future debugging easier.

hwdb: Add HP Envy x360 Convertible 15-cn0xxx to existing entry

test-parse-util: add tests with explicit plus character

I expected this to work, but our tests did not cover this
explicitly.

core/service: constify ExecCommand* in two functions

core: imply DeviceAllow=/dev/tpmrm0 with LoadCredentialEncrypted

If the device access policy is restricted, add implicitly access to the TPM
if at least one encrypted credential needs to be loaded.

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

cryptenroll: do not implicitly verify with default tpm policy signature

If it was not requested to use a tpm2 signature file when enrolling, do
not fallback to the default /run/systemd/tpm2-pcr-signature.json as it
likely will be unrelated if it exists.

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

unit: always return 1 in log_kill

This ensures that cg_kill_items returns the correct value to let the
manager know that a process was killed.

Merge pull request #26328 from yuwata/udev-worker-set-process-name

udev: set worker process name

journal: minor modernizations

Merge pull request #26337 from poettering/journal-display-ts

logs-show: rename "ts" paramater/variable to "display_ts"

Merge pull request #26336 from poettering/journal-etoomanyrefs

journal: print nicer log message when limit of open journal files during display is hit

Merge pull request #26335 from keszybz/dns-not-found

resolve: adjust message for NXDOMAIN lookup result

Fix Positivo-vaio VJPW12F11X key toggle touchpad

Merge pull request #26338 from jamacku/fix-labeling

ci: Fix automatic removing of labels when PR is closed

ci: remove `if: github.event.issue.pull_request` from `labeler.yml`

`github.event.issue.pull_request` is an object, not a boolean.
This is the root cause of why the step that is supposed to remove labels
is always skipped. Having this condition in place is not necessary since
the workflow is run on the `pull_request_target` event.

logs-show: rename "ts" paramater/variable to "display_ts"

When displaying log data we deal with two kind of timestamps: the one we
use for display (typically the source timestamp if available), and the
one we use internally (typically the reception timestamp of journald).

The user-facing output modes generally use the display timestamp, the
ones intended for further processing (i.e. json + export outputs) do
not, and directly query the timestamps of the entry, ignoring the
source. This gets a bit confusing, since it's not always clear why we
use which timestamp where. Let's address that by renaming the generic
"ts" parameter/variable to "display_ts" to emphasize that the stored
timestamp are "corrected" timestamps for display only.

No real code change, just some renaming.

ci: fix missing quotes in `labeler.yml`

update TODO

journal: print a useful error message if we hit the journal file open limit

See: #20921

resolve: adjust message for NXDOMAIN lookup result

Previously, we reported:
  nx.example.org: resolve call failed: 'nx.example.org' not found
But the call did succeed, and in fact all communication with the upstream
servers was successful, and we got an authoritative negative answer.
So instead of saying that the call fail, just say that the host doesn't exist:
  nx.example.org: Name 'nx.example.org' not found

I wanted to keep the prefix of "<name>: ", to keep the output uniform. But
it'd look a bit strange to say "<name>: <name> not found", so I added "Name "
to make the output more readable. (Another option would be to not display
the error string received from resolved, but that seems risky: even if right
now resolved uses just one message format, it could start doing something else
in the future, so it's better to display the error as received.)

Fixes #26233.

resolve: define normal macros for BUS_ERROR_DNS error codes

This result is identical after cpp is done, so we don't save anything
by not having the usual macros. And with the usual macros it's easier to
grep and code-crossreferencing works better.

tree-wide: set FORK_RLIMIT_NOFILE_SAFE flag

No functional changes, just refactoring.

udevd: configure a child process name for worker processes

This effectively reverts commit ff86c92e3043f71fc801cf687600a480ee8f6778,
and re-apply 49f3ee7e74c714f55aab395c080b1099fc17f7fd.

The change was dropped due to the process name was not correctly logged,
but the issue was fixed by dd15e4cb57129b915e01495e113696bfe0b70214.
Let's set the child process name again.

Merge pull request #26324 from yuwata/argv-util-update-short-name

argv-util: also update short invocation name

Merge pull request #26322 from keszybz/log-errno-fix

Log errno fix

Merge pull request #26321 from keszybz/flex-arrays

Enable new compiler diagnostics for invalid array accesses

Merge pull request #26320 from keszybz/operator-whitespace

Drop whitespace after shell redirection operators

NEWS: fix typo

update TODO

argv-util: also update program_invocation_short_name

Our logging uses program_invocation_short_name. Without this patch,
logs from forked client may become broken; spuriously truncated or
the short invocation name is not completely shown in the log.

test: use notice log level to make easily filter out logs from PID1

meson: enable -Wzero-length-bounds

This will warn if fake flexible arrays are re-introduced. I'm not using
-Werror=… because we may still get warnings when compiling against old kernel
headers. We can crank this up to error later.

meson: enable -Warray-bounds and -fstrict-flex-arrays

-fstrict-flex-arrays means that the compiler doesn't have to assume that any
trailing array is a flex array. I.e. unless the array is declared without a
specified size, only indices in the declared range are valid.

-Warray-bounds turns on the warnings about out-of-bounds array accesses.
-Warray-bounds=2 does some more warnings, with higher false positive rate. But
it doesn't seem to yield any false positives in our codebase, so enable it.

clang supports -Warray-bounds, but not -Warray-bounds=2.
gcc supports both.
gcc-13 supports -fstrict-flex-arrays.

See https://people.kernel.org/kees/bounded-flexible-arrays-in-c for a long
discussion of use in the kernel.

shared/json: avoid use of fake flex array