summaryrefslogtreecommitdiff
path: root/systemd-solve-that-rsyslog-reads-journal-s-object-of.patch
blob: 2ff7fe37667c0bc8a3f132eb1291dfaa94387ebd (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
From 6aa35d7d911b6895043f222293703ef7cf60aca1 Mon Sep 17 00:00:00 2001
From: yefei25 <yefei25@huawei.com>
Date: Thu, 5 Mar 2020 21:45:36 +0800
Subject: [PATCH] systemd: solve that rsyslog reads journal's object of
size 0


Signed-off-by: yefei25 <yefei25@huawei.com>

---
 src/libsystemd/sd-journal/journal-file.c | 5 +++--
 src/libsystemd/sd-journal/sd-journal.c   | 7 ++++++-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/libsystemd/sd-journal/journal-file.c b/src/libsystemd/sd-journal/journal-file.c
index 2ead295..9962d3b 100644
--- a/src/libsystemd/sd-journal/journal-file.c
+++ b/src/libsystemd/sd-journal/journal-file.c
@@ -898,8 +898,9 @@ static int check_object(JournalFile *f, Object *o, uint64_t offset) {
                                                le64toh(o->tag.epoch), offset);
 
                 break;
+        default:
+                return -EBADMSG;
         }
-
         return 0;
 }
 
@@ -1804,7 +1805,7 @@ int journal_file_data_payload(
         }
 
         size = le64toh(READ_NOW(o->object.size));
-        if (size < journal_file_data_payload_offset(f))
+        if (size == 0 || o->object.type == 0 || size < journal_file_data_payload_offset(f))
                 return -EBADMSG;
 
         size -= journal_file_data_payload_offset(f);
diff --git a/src/libsystemd/sd-journal/sd-journal.c b/src/libsystemd/sd-journal/sd-journal.c
index 898218e..f6090dd 100644
--- a/src/libsystemd/sd-journal/sd-journal.c
+++ b/src/libsystemd/sd-journal/sd-journal.c
@@ -114,6 +114,10 @@ static void init_location(Location *l, LocationType type, JournalFile *f, Object
         assert(IN_SET(type, LOCATION_DISCRETE, LOCATION_SEEK));
         assert(f);
 
+        if(o->object.type != OBJECT_ENTRY || o->object.size == 0){
+            return;
+        }
+
         *l = (Location) {
                 .type = type,
                 .seqnum = le64toh(o->entry.seqnum),
@@ -851,7 +855,8 @@ static int real_journal_next(sd_journal *j, direction_t direction) {
                 return r;
 
         set_location(j, new_file, o);
-
+        if(o->object.size == 0)
+                return -EBADMSG;
         return 1;
 }
 
-- 
2.33.0