diff --git a/src/udev/udevadm-info.c b/src/udev/udevadm-info.c index ccfc83cce4..45c39d4c73 100644 --- a/src/udev/udevadm-info.c +++ b/src/udev/udevadm-info.c @@ -1105,7 +1105,7 @@ static int parse_argv(int argc, char *argv[]) { if (streq(optarg, "env")) /* deprecated */ arg_query = QUERY_PROPERTY; else - return log_error_errno(SYNTHETIC_ERRNO(EINVAL), "unknown query type"); + return log_error_errno(SYNTHETIC_ERRNO(EINVAL), "Unknown query type '%s'", optarg); } break; @@ -1185,7 +1185,7 @@ static int parse_argv(int argc, char *argv[]) { case ARG_ATTR_MATCH: if (!strchr(optarg, '=')) return log_error_errno(SYNTHETIC_ERRNO(EINVAL), - "expect = instead of '%s'", optarg); + "Expected = instead of '%s'", optarg); r = strv_extend(&arg_attr_match, optarg); if (r < 0) @@ -1195,7 +1195,7 @@ static int parse_argv(int argc, char *argv[]) { case ARG_ATTR_NOMATCH: if (!strchr(optarg, '=')) return log_error_errno(SYNTHETIC_ERRNO(EINVAL), - "expect = instead of '%s'", optarg); + "Expected = instead of '%s'", optarg); r = strv_extend(&arg_attr_nomatch, optarg); if (r < 0) @@ -1205,7 +1205,7 @@ static int parse_argv(int argc, char *argv[]) { case ARG_PROPERTY_MATCH: if (!strchr(optarg, '=')) return log_error_errno(SYNTHETIC_ERRNO(EINVAL), - "expect = instead of '%s'", optarg); + "Expected = instead of '%s'", optarg); r = strv_extend(&arg_property_match, optarg); if (r < 0) diff --git a/src/udev/udevadm-util.c b/src/udev/udevadm-util.c index 09fb24f6e5..73c39f696c 100644 --- a/src/udev/udevadm-util.c +++ b/src/udev/udevadm-util.c @@ -152,8 +152,9 @@ int parse_resolve_name_timing(const char *str, ResolveNameTiming *ret) { return 1; } -int parse_key_value_argument(const char *s, bool require_value, char **key, char **value) { +int parse_key_value_argument(const char *str, bool require_value, char **key, char **value) { _cleanup_free_ char *k = NULL, *v = NULL; + const char *s = str; int r; assert(s); @@ -162,9 +163,9 @@ int parse_key_value_argument(const char *s, bool require_value, char **key, char r = extract_many_words(&s, "=", EXTRACT_DONT_COALESCE_SEPARATORS, &k, &v); if (r < 0) - return log_error_errno(r, "Failed to parse key/value pair %s: %m", s); + return log_error_errno(r, "Failed to parse key/value pair %s: %m", str); if (require_value && r < 2) - return log_error_errno(SYNTHETIC_ERRNO(EINVAL), "Missing '=' in key/value pair %s.", s); + return log_error_errno(SYNTHETIC_ERRNO(EINVAL), "Missing '=' in key/value pair %s.", str); if (!filename_is_valid(k)) return log_error_errno(SYNTHETIC_ERRNO(EINVAL), "%s is not a valid key name", k); diff --git a/src/udev/udevadm-util.h b/src/udev/udevadm-util.h index 9b41109df8..aca4618977 100644 --- a/src/udev/udevadm-util.h +++ b/src/udev/udevadm-util.h @@ -10,6 +10,6 @@ int find_device(const char *id, const char *prefix, sd_device **ret); int find_device_with_action(const char *id, sd_device_action_t action, sd_device **ret); int parse_device_action(const char *str, sd_device_action_t *ret); int parse_resolve_name_timing(const char *str, ResolveNameTiming *ret); -int parse_key_value_argument(const char *s, bool require_value, char **key, char **value); +int parse_key_value_argument(const char *str, bool require_value, char **key, char **value); int udev_ping(usec_t timeout, bool ignore_connection_failure); int search_rules_files(char * const *a, const char *root, char ***ret);