]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
journal: use recognizable error code
authorYu Watanabe <watanabe.yu+github@gmail.com>
Thu, 25 Jun 2026 07:43:13 +0000 (16:43 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Sat, 27 Jun 2026 05:04:49 +0000 (14:04 +0900)
EINVAL is too generic and widely used. Let's use more specific error code.

src/journal/journalctl-misc.c
src/libsystemd/sd-journal/journal-authenticate.c

index 78d3cae91ac3380fe072d25d6f4e4e4519238a2a..38bc03def3762a463c08003b4eb436d648ca0ec3 100644 (file)
@@ -56,7 +56,7 @@ int action_verify(void) {
 #endif
 
                 k = journal_file_verify(f, arg_verify_key, &first, &validated, &last, /* show_progress= */ !arg_quiet);
-                if (k == -EINVAL)
+                if (k == -EKEYREJECTED)
                         /* If the key was invalid give up right-away. */
                         return k;
                 if (k < 0)
index 04fbe88b8cbac589b3ee8136ba9e86eba9fd0850..ff241fa0cb0d2e1cdb01447a08d20aef451af603 100644 (file)
@@ -515,24 +515,24 @@ int journal_file_parse_verification_key(JournalFile *f, const char *key) {
 
                 x = unhexchar(*k);
                 if (x < 0)
-                        return -EINVAL;
+                        return -EKEYREJECTED;
                 k++;
 
                 y = unhexchar(*k);
                 if (y < 0)
-                        return -EINVAL;
+                        return -EKEYREJECTED;
                 k++;
 
                 seed[c] = (uint8_t) (x * 16 + y);
         }
 
         if (*k != '/')
-                return -EINVAL;
+                return -EKEYREJECTED;
         k++;
 
         r = sscanf(k, "%llx-%llx", &start, &interval);
         if (r != 2)
-                return -EINVAL;
+                return -EKEYREJECTED;
 
         f->fsprg_seed = IOVEC_MAKE(TAKE_PTR(seed), seed_size);
         f->fss_start_usec = start * interval;