SymbX/systemd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

journal: skipping of exhausted journal files is bad if direction changed

Browse Source 
EOF is meaningless if the direction of iteration changes.
Move the EOF optimization under the direction check.

This fixes test-journal-interleaving for me.

Thanks to Filipe Brandenburger for telling me about the failure.
This commit is contained in:
Michal Schmidt
2014-12-19 17:14:11 +01:00
parent b29ddfcb38
commit 668c965af4
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/journal/sd-journal.c
+6 -6
View File
@@ -723,13 +723,13 @@ static int next_beyond_location(sd_journal *j, JournalFile *f, direction_t direc
assert(j); assert(j);
assert(f); assert(f);
/* If we hit EOF before, recheck if any new entries arrived. */
n_entries = le64toh(f->header->n_entries);
if (f->location_type == LOCATION_TAIL && n_entries == f->last_n_entries)
return 0;
f->last_n_entries = n_entries;
if (f->last_direction == direction && f->current_offset > 0) { if (f->last_direction == direction && f->current_offset > 0) {
/* If we hit EOF before, recheck if any new entries arrived. */
n_entries = le64toh(f->header->n_entries);
if (f->location_type == LOCATION_TAIL && n_entries == f->last_n_entries)
return 0;
f->last_n_entries = n_entries;
/* LOCATION_SEEK here means we did the work in a previous /* LOCATION_SEEK here means we did the work in a previous
* iteration and the current location already points to a * iteration and the current location already points to a
* candidate entry. */ * candidate entry. */