journald: Fix EOF reporting when not advancing next on LMI_JournalMessageLog.GetRecord()

Testing wrong return code when sd_journal_next() was not called as it
was not needed. When step_next == FALSE the EOF flag must be never set
since the journal EOF case is tested at the beginning of the function
and we're not advancing next thus staying on a valid cursor.

1 files changed, 3 insertions, 1 deletions

diff --git a/src/journald/instutil.c b/src/journald/instutil.c
index b2ed0e7..12f63bf 100644
--- a/src/journald/instutil.c
+++ b/src/journald/instutil.c
@@ -656,6 +656,7 @@ journal_iter_get_data(gchar **iter_id, sd_journal *journal, gboolean step_next)
         return NULL;
     }
 
+    eof_set = FALSE;
     if (step_next) {
         r = sd_journal_next(journal);
         if (r < 0) {
@@ -663,9 +664,10 @@ journal_iter_get_data(gchar **iter_id, sd_journal *journal, gboolean step_next)
             g_free(d);
             return NULL;
         }
+        eof_set = (r == 0);
     }
 
-    if (! update_iter(iter_id, r == 0, journal)) {
+    if (! update_iter(iter_id, eof_set, journal)) {
         lmi_error("Error getting record message\n");
         return NULL;
     }