[systemd-commits] src/journal

[Michal Schmidt]

 src/journal/sd-journal.c |   12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit 668c965af4e803f460925dc616f328ed274d1f3a
Author: Michal Schmidt <mschmidt at redhat.com>
Date:   Fri Dec 19 17:11:22 2014 +0100

    journal: skipping of exhausted journal files is bad if direction changed
    
    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.

diff --git a/src/journal/sd-journal.c b/src/journal/sd-journal.c
index 89cab37..d46dc3c 100644
--- a/src/journal/sd-journal.c
+++ b/src/journal/sd-journal.c
@@ -723,13 +723,13 @@ static int next_beyond_location(sd_journal *j, JournalFile *f, direction_t direc
         assert(j);
         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 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
                  * iteration and the current location already points to a
                  * candidate entry. */