On Mon Mar 30, 2026 at 12:21 PM CEST, Suresh H A via lists.openembedded.org
wrote:
> From: sureshha <[email protected]>
>
> Backport patch to fix systemd journal-file assertion on removed or corrupted
> files
>
> Extracted from systemd MR:
> https://github.com/systemd/systemd/pull/40378
>
> Signed-off-by: sureshha <[email protected]>
> ---
> ...not-trigger-assertion-on-removed-or-.patch | 65 +++++++++++++++++++
> meta/recipes-core/systemd/systemd_257.8.bb | 1 +
> 2 files changed, 66 insertions(+)
> create mode 100644
> meta/recipes-core/systemd/systemd/0027-journal-file-do-not-trigger-assertion-on-removed-or-.patch
>
> diff --git
> a/meta/recipes-core/systemd/systemd/0027-journal-file-do-not-trigger-assertion-on-removed-or-.patch
>
> b/meta/recipes-core/systemd/systemd/0027-journal-file-do-not-trigger-assertion-on-removed-or-.patch
> new file mode 100644
> index 0000000000..e1bb556271
> --- /dev/null
> +++
> b/meta/recipes-core/systemd/systemd/0027-journal-file-do-not-trigger-assertion-on-removed-or-.patch
> @@ -0,0 +1,65 @@
> +From 1350f39db7e72116c3b2423db02da3ddc8e29082 Mon Sep 17 00:00:00 2001
> +From: Yu Watanabe <[email protected]>
> +Date: Sun, 18 Jan 2026 19:15:31 +0900
> +Subject: [PATCH] journal-file: do not trigger assertion on removed or
> + corrupted journal file
> +
> +When a journal file is removed or corrupted, then the value `p`, which is
> +read from Object.data.entry_offset, may be zero.
> +
> +Note, journal_file_move_to_object() checks the passed offset and return
> +-EBADMSG if it is invalid.
> +
> +Fixes the issue reported at
> +https://github.com/systemd/systemd/pull/40372#issuecomment-3762907261.
> +
> +Upstream-Status: Backport
> [https://github.com/systemd/systemd/commit/112cbc37906fb97afe0ad04164262cf62d0af5e9]
> +
> +Comment: Patch is refreshed as per codebase of v257
> +
> +Signed-off-by: sureshha <[email protected]>
> +---
> + src/libsystemd/sd-journal/journal-file.c | 7 +++----
> + 1 file changed, 3 insertions(+), 4 deletions(-)
> +
Hello,
This patch has a weird format. Every spaces at end of line were removed,
but in the empty or white space only context lines, those should be
kept. Examples below:
> +diff --git a/src/libsystemd/sd-journal/journal-file.c
> b/src/libsystemd/sd-journal/journal-file.c
> +index 08f3b82..633de5b 100644
> +--- a/src/libsystemd/sd-journal/journal-file.c
> ++++ b/src/libsystemd/sd-journal/journal-file.c
> +@@ -3305,7 +3305,9 @@ use_extra:
> +
^ Here should be a space.
> + static int test_object_offset(JournalFile *f, uint64_t p, uint64_t needle) {
> + assert(f);
> +- assert(p > 0);
> ++
> ++ if (p <= 0)
> ++ return -EBADMSG;
> +
^ And here
> + if (p == needle)
> + return TEST_FOUND;
> +@@ -3341,7 +3343,6 @@ static int test_object_seqnum(JournalFile *f, uint64_t
> p, uint64_t needle) {
> + int r;
> +
> + assert(f);
> +- assert(p > 0);
> +
> + r = journal_file_move_to_object(f, OBJECT_ENTRY, p, &o);
> + if (r < 0)
> +@@ -3382,7 +3383,6 @@ static int test_object_realtime(JournalFile *f,
> uint64_t p, uint64_t needle) {
> + int r;
> +
> + assert(f);
> +- assert(p > 0);
> +
> + r = journal_file_move_to_object(f, OBJECT_ENTRY, p, &o);
> + if (r < 0)
> +@@ -3423,7 +3423,6 @@ static int test_object_monotonic(JournalFile *f,
> uint64_t p, uint64_t needle) {
> + int r;
> +
> + assert(f);
> +- assert(p > 0);
> +
> + r = journal_file_move_to_object(f, OBJECT_ENTRY, p, &o);
> + if (r < 0)
> +--
^ here, the usual pattern to separate signature is "-- " (a space)
not "--" (no space)
Can you check and fix please? (also check your other patches in other
branches)
Patch otherwise looks good.
> +2.34.1
> diff --git a/meta/recipes-core/systemd/systemd_257.8.bb
> b/meta/recipes-core/systemd/systemd_257.8.bb
> index 549eb7fb29..a53959f816 100644
> --- a/meta/recipes-core/systemd/systemd_257.8.bb
> +++ b/meta/recipes-core/systemd/systemd_257.8.bb
> @@ -29,6 +29,7 @@ SRC_URI += " \
>
> file://0001-binfmt-Don-t-install-dependency-links-at-install-tim.patch \
> file://0002-implment-systemd-sysv-install-for-OE.patch \
> file://0001-Do-not-create-var-log-README.patch \
> +
> file://0027-journal-file-do-not-trigger-assertion-on-removed-or-.patch \
> "
>
> # patches needed by musl
--
Yoann Congal
Smile ECS
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#234463):
https://lists.openembedded.org/g/openembedded-core/message/234463
Mute This Topic: https://lists.openembedded.org/mt/118576543/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
