|
 |
d743bb5 |
From c5af9aa2ccab078cf6f5bf68b88a3defa2f768f8 Mon Sep 17 00:00:00 2001
|
|
|
d743bb5 |
From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
|
|
|
d743bb5 |
Date: Sat, 13 May 2017 16:40:09 -0400
|
|
|
d743bb5 |
Subject: [PATCH] shared/logs-show: avoid printing "(null)" when timestamp is
|
|
|
d743bb5 |
not specified
|
|
|
d743bb5 |
|
|
|
d743bb5 |
$ perl -e 'print("MESSAGE\n", pack("q<", 1), "A\n\nMESSAGE=test2\n")' > message.bin
|
|
|
d743bb5 |
$ systemd-journal-remote -o /tmp/out.journal message.bin
|
|
|
d743bb5 |
$ journalctl -o export --file /tmp/out.journal
|
|
|
d743bb5 |
__CURSOR=s=b16c464c2db44384b29e75a564d8388e;i=1;b=6b0be47627bd4932913dc126012c21c0;m=0;t=0;x=b04263a253e357a
|
|
|
d743bb5 |
__REALTIME_TIMESTAMP=0
|
|
|
d743bb5 |
__MONOTONIC_TIMESTAMP=0
|
|
|
d743bb5 |
_BOOT_ID=6b0be47627bd4932913dc126012c21c0
|
|
|
d743bb5 |
MESSAGE=A
|
|
|
d743bb5 |
|
|
|
d743bb5 |
$ journalctl -o verbose --file /tmp/out.journal
|
|
|
d743bb5 |
(null) [s=b16c464c2db44384b29e75a564d8388e;i=1;b=6b0be47627bd4932913dc126012c21c0;m=0;t=0;x=b04263a253e357a]
|
|
|
d743bb5 |
MESSAGE=A
|
|
|
d743bb5 |
|
|
|
d743bb5 |
This is changed to
|
|
|
d743bb5 |
$ build/journalctl -o verbose --file /tmp/out.journal
|
|
|
d743bb5 |
(no timestamp) [s=b16c464c2db44384b29e75a564d8388e;i=1;b=6b0be47627bd4932913dc126012c21c0;m=0;t=0;x=b04263a253e357a]
|
|
|
d743bb5 |
MESSAGE=A
|
|
|
d743bb5 |
|
|
|
d743bb5 |
We should deal gracefully with unexpected input.
|
|
|
d743bb5 |
|
|
|
d743bb5 |
(cherry picked from commit 8924973ae2e1f0a0c131dcec0578669dc26e5e26)
|
|
|
d743bb5 |
---
|
|
|
d743bb5 |
src/shared/logs-show.c | 7 ++++---
|
|
|
d743bb5 |
1 file changed, 4 insertions(+), 3 deletions(-)
|
|
|
d743bb5 |
|
|
|
d743bb5 |
diff --git a/src/shared/logs-show.c b/src/shared/logs-show.c
|
|
|
d743bb5 |
index 72c43e80cb..b4c72215c4 100644
|
|
|
d743bb5 |
--- a/src/shared/logs-show.c
|
|
|
d743bb5 |
+++ b/src/shared/logs-show.c
|
|
|
d743bb5 |
@@ -473,6 +473,7 @@ static int output_verbose(
|
|
|
d743bb5 |
_cleanup_free_ char *cursor = NULL;
|
|
|
d743bb5 |
uint64_t realtime = 0;
|
|
|
d743bb5 |
char ts[FORMAT_TIMESTAMP_MAX + 7];
|
|
|
d743bb5 |
+ const char *timestamp;
|
|
|
d743bb5 |
int r;
|
|
|
d743bb5 |
|
|
|
d743bb5 |
assert(f);
|
|
|
d743bb5 |
@@ -508,10 +509,10 @@ static int output_verbose(
|
|
|
d743bb5 |
if (r < 0)
|
|
|
d743bb5 |
return log_error_errno(r, "Failed to get cursor: %m");
|
|
|
d743bb5 |
|
|
|
d743bb5 |
+ timestamp = flags & OUTPUT_UTC ? format_timestamp_us_utc(ts, sizeof ts, realtime)
|
|
|
d743bb5 |
+ : format_timestamp_us(ts, sizeof ts, realtime);
|
|
|
d743bb5 |
fprintf(f, "%s [%s]\n",
|
|
|
d743bb5 |
- flags & OUTPUT_UTC ?
|
|
|
d743bb5 |
- format_timestamp_us_utc(ts, sizeof(ts), realtime) :
|
|
|
d743bb5 |
- format_timestamp_us(ts, sizeof(ts), realtime),
|
|
|
d743bb5 |
+ timestamp ?: "(no timestamp)",
|
|
|
d743bb5 |
cursor);
|
|
|
d743bb5 |
|
|
|
d743bb5 |
JOURNAL_FOREACH_DATA_RETVAL(j, data, length, r) {
|