1
0
mirror of https://github.com/systemd/systemd.git synced 2024-12-22 17:35:35 +03:00

journalctl: fix output when --until is used with --lines

Before this commit, when --lines is specified, we jump to the tail and
search afterwards from there, thus breaking --until if used together.

After this commit:
If both --until and any of --reverse and --lines is specified, things get
a little tricky. We seek to the place of --until first. If only --reverse or
--reverse and --lines is specified, we search backwards and let the output
counter handle --lines for us. If only --lines is used, we just jump backwards
arg_lines and search afterwards from there.
This commit is contained in:
Mike Yuan 2023-03-04 19:06:00 +08:00
parent 561f17a1e4
commit 81fb5375b3
No known key found for this signature in database
GPG Key ID: 417471C0A40F58B3

View File

@ -2482,12 +2482,21 @@ static int run(int argc, char *argv[]) {
r = sd_journal_next(j);
} else if (arg_until_set && arg_reverse) {
} else if (arg_until_set && (arg_reverse || arg_lines >= 0)) {
/* If both --until and any of --reverse and --lines is specified, things get
* a little tricky. We seek to the place of --until first. If only --reverse or
* --reverse and --lines is specified, we search backwards and let the output
* counter handle --lines for us. If only --lines is used, we just jump backwards
* arg_lines and search afterwards from there. */
r = sd_journal_seek_realtime_usec(j, arg_until);
if (r < 0)
return log_error_errno(r, "Failed to seek to date: %m");
r = sd_journal_previous(j);
if (arg_reverse)
r = sd_journal_previous(j);
else /* arg_lines >= 0 */
r = sd_journal_previous_skip(j, arg_lines);
} else if (arg_reverse) {
r = sd_journal_seek_tail(j);
@ -2552,7 +2561,7 @@ static int run(int argc, char *argv[]) {
break;
}
if (arg_until_set && !arg_reverse) {
if (arg_until_set && !arg_reverse && arg_lines < 0) {
usec_t usec;
r = sd_journal_get_realtime_usec(j, &usec);