src/shared/conf-parser.c

@@ -334,8 +334,6 @@ int config_parse(const char *unit,
                         return r;
                 }
 
-                line++;
-
                 l = skip_leading_chars(buf, WHITESPACE);
                 if (*l != '\0' && strchr(COMMENTS, *l))
                         continue;
@@ -392,7 +390,7 @@ int config_parse(const char *unit,
 
                 r = parse_line(unit,
                                filename,
-                               line,
+                               ++line,
                                sections,
                                lookup,
                                table,

src/shared/journal-util.c

@@ -80,10 +80,6 @@ static int access_check_var_log_journal(sd_journal *j, bool want_other_users) {
         return 1;
 }
 
-int journal_access_blocked(sd_journal *j) {
-        return hashmap_contains(j->errors, INT_TO_PTR(-EACCES));
-}
-
 int journal_access_check_and_warn(sd_journal *j, bool quiet, bool want_other_users) {
         Iterator it;
         void *code;
@@ -99,7 +95,7 @@ int journal_access_check_and_warn(sd_journal *j, bool quiet, bool want_other_use
                 return 0;
         }
 
-        if (journal_access_blocked(j)) {
+        if (hashmap_contains(j->errors, INT_TO_PTR(-EACCES))) {
                 if (!quiet)
                         (void) access_check_var_log_journal(j, want_other_users);
 

src/shared/journal-util.h

@@ -7,5 +7,5 @@
 #include "sd-journal.h"
 
 bool journal_field_valid(const char *p, size_t l, bool allow_protected);
-int journal_access_blocked(sd_journal *j);
+
 int journal_access_check_and_warn(sd_journal *j, bool quiet, bool want_other_users);

src/shared/logs-show.c

@@ -20,7 +20,6 @@
 #include "id128-util.h"
 #include "io-util.h"
 #include "journal-internal.h"
-#include "journal-util.h"
 #include "json.h"
 #include "log.h"
 #include "logs-show.h"
@@ -1233,21 +1232,7 @@ int show_journal(
 
                         if (r > 0 && not_before < cutoff) {
                                 maybe_print_begin_newline(f, &flags);
-
+                                fprintf(f, "Warning: Journal has been rotated since unit was started. Log output is incomplete or unavailable.\n");
-                                /* If we logged *something* and no permission error happened, than we can
-                                 * reliably emit the warning about rotation. If we didn't log anything and
-                                 * access errors happened, emit hint about permissions. Otherwise, give a
-                                 * generic message, since we can't diagnose the issue. */
-
-                                bool noaccess = journal_access_blocked(j);
-
-                                if (line == 0 && noaccess)
-                                        fprintf(f, "Warning: some journal files were not opened due to insufficient permissions.");
-                                else if (!noaccess)
-                                        fprintf(f, "Warning: journal has been rotated since unit was started, output may be incomplete.\n");
-                                else
-                                        fprintf(f, "Warning: journal has been rotated since unit was started and some journal "
-                                                "files were not opened due to insufficient permissions, output may be incomplete.\n");
                         }
 
                         warn_cutoff = false;

src/udev/udevadm-info.c

@@ -177,18 +177,18 @@ static int export_devices(void) {
 
         r = sd_device_enumerator_new(&e);
         if (r < 0)
-                return log_oom();
+                return r;
 
         r = sd_device_enumerator_allow_uninitialized(e);
         if (r < 0)
-                return log_error_errno(r, "Failed to set allowing uninitialized flag: %m");
+                return r;
 
         r = device_enumerator_scan_devices(e);
         if (r < 0)
-                return log_error_errno(r, "Failed to scan devices: %m");
+                return r;
 
         FOREACH_DEVICE_AND_SUBSYSTEM(e, d)
-                (void) print_record(d);
+                print_record(d);
 
         return 0;
 }