various: fix duplicated logging from parse_path_argument

As pointed out in review, parse_path_argument can fail for non-oom reasons.
But the function already logs, so the correct thing to do is to just
propagate the error.

diff --git a/src/bootctl/bootctl-install.c b/src/bootctl/bootctl-install.c
index 20958d0b0bc6fd9ca11da3bfa23917e746161f0c..2ad76264fa3900a69c73959f21971714ab456b8b 100644 (file)
--- a/src/bootctl/bootctl-install.c
+++ b/src/bootctl/bootctl-install.c
@@ -1509,7 +1509,7 @@ static int load_secure_boot_auto_enroll(
         if (arg_private_key_source_type == OPENSSL_KEY_SOURCE_FILE) {
                 r = parse_path_argument(arg_private_key, /* suppress_root= */ false, &arg_private_key);
                 if (r < 0)
-                        return log_error_errno(r, "Failed to parse private key path %s: %m", arg_private_key);
+                        return r;
         }
 
         r = openssl_load_private_key(

diff --git a/src/home/homectl.c b/src/home/homectl.c
index 80ee6e2155a94fc5ad3274250253467765bf614a..e9161dfb751e5c314361002df1577030d912551e 100644 (file)
--- a/src/home/homectl.c
+++ b/src/home/homectl.c
@@ -4648,7 +4648,7 @@ static int parse_argv(int argc, char *argv[]) {
                                 if (!eq) { /* --blob=/some/path replaces the blob dir */
                                         r = parse_path_argument(optarg, /* suppress_root= */ false, &arg_blob_dir);
                                         if (r < 0)
-                                                return log_error_errno(r, "Failed to parse path %s: %m", optarg);
+                                                return r;
                                         break;
                                 }
 
@@ -4664,7 +4664,7 @@ static int parse_argv(int argc, char *argv[]) {
 
                                 r = parse_path_argument(eq + 1, /* suppress_root= */ false, &path);
                                 if (r < 0)
-                                        return log_error_errno(r, "Failed to parse path %s: %m", eq + 1);
+                                        return r;
                         } else {
                                 const char *well_known_filename =
                                                   c == ARG_AVATAR ? "avatar" :
@@ -4678,7 +4678,7 @@ static int parse_argv(int argc, char *argv[]) {
 
                                 r = parse_path_argument(optarg, /* suppress_root= */ false, &path);
                                 if (r < 0)
-                                        return log_error_errno(r, "Failed to parse path %s: %m", optarg);
+                                        return r;
                         }
 
                         if (path) {

diff --git a/src/kernel-install/kernel-install.c b/src/kernel-install/kernel-install.c
index 0bd4e84da0ad0e9db87534109bebe46130bd2894..27e1a7426e4d1d89fdd2003a854913135936c6b9 100644 (file)
--- a/src/kernel-install/kernel-install.c
+++ b/src/kernel-install/kernel-install.c
@@ -1604,13 +1604,13 @@ static int parse_argv(int argc, char *argv[], char ***remaining_args) {
                 OPTION_LONG("esp-path", "PATH", "Path to the EFI System Partition (ESP)"):
                         r = parse_path_argument(opts.arg, /* suppress_root= */ false, &arg_esp_path);
                         if (r < 0)
-                                return log_oom();
+                                return r;
                         break;
 
                 OPTION_LONG("boot-path", "PATH", "Path to the $BOOT partition"):
                         r = parse_path_argument(opts.arg, /* suppress_root= */ false, &arg_xbootldr_path);
                         if (r < 0)
-                                return log_oom();
+                                return r;
                         break;
 
                 OPTION_COMMON_MAKE_ENTRY_DIRECTORY:

diff --git a/src/keyutil/keyutil.c b/src/keyutil/keyutil.c
index 2a66fabb19542374920d80ccfe5aed8dcf94b97a..abdec1591d01c54f2b54ff28fa755b29da84c78b 100644 (file)
--- a/src/keyutil/keyutil.c
+++ b/src/keyutil/keyutil.c
@@ -193,7 +193,7 @@ static int verb_validate(int argc, char *argv[], uintptr_t _data, void *userdata
         if (arg_private_key_source_type == OPENSSL_KEY_SOURCE_FILE) {
                 r = parse_path_argument(arg_private_key, /* suppress_root= */ false, &arg_private_key);
                 if (r < 0)
-                        return log_error_errno(r, "Failed to parse private key path %s: %m", arg_private_key);
+                        return r;
         }
 
         r = openssl_load_private_key(
@@ -259,7 +259,7 @@ static int verb_extract_public(int argc, char *argv[], uintptr_t _data, void *us
                 if (arg_private_key_source_type == OPENSSL_KEY_SOURCE_FILE) {
                         r = parse_path_argument(arg_private_key, /* suppress_root= */ false, &arg_private_key);
                         if (r < 0)
-                                return log_error_errno(r, "Failed to parse private key path %s: %m", arg_private_key);
+                                return r;
                 }
 
                 r = openssl_load_private_key(

diff --git a/src/measure/measure-tool.c b/src/measure/measure-tool.c
index eeb001f3fed4a23faa84ebe56666ca7eb1af4627..d5bb5bea2351cf2eb10e414ec7d16404f735b3aa 100644 (file)
--- a/src/measure/measure-tool.c
+++ b/src/measure/measure-tool.c
@@ -960,7 +960,7 @@ static int build_policy_digest(bool sign) {
                 if (arg_private_key_source_type == OPENSSL_KEY_SOURCE_FILE) {
                         r = parse_path_argument(arg_private_key, /* suppress_root= */ false, &arg_private_key);
                         if (r < 0)
-                                return log_error_errno(r, "Failed to parse private key path %s: %m", arg_private_key);
+                                return r;
                 }
 
                 r = openssl_load_private_key(

diff --git a/src/sbsign/sbsign.c b/src/sbsign/sbsign.c
index f5a88b2849fe24e758b0b7557d59a53f0642dbb7..9d163624a4997014d43600bc841edfa7f07293e4 100644 (file)
--- a/src/sbsign/sbsign.c
+++ b/src/sbsign/sbsign.c
@@ -461,7 +461,7 @@ static int verb_sign(int argc, char *argv[], uintptr_t _data, void *userdata) {
                 if (arg_private_key_source_type == OPENSSL_KEY_SOURCE_FILE) {
                         r = parse_path_argument(arg_private_key, /* suppress_root= */ false, &arg_private_key);
                         if (r < 0)
-                                return log_error_errno(r, "Failed to parse private key path %s: %m", arg_private_key);
+                                return r;
                 }
 
                 r = openssl_load_private_key(