diff options
| author | Lennart Poettering <lennart@poettering.net> | 2024-06-27 10:12:04 +0200 |
|---|---|---|
| committer | Lennart Poettering <lennart@poettering.net> | 2024-06-27 10:43:21 +0200 |
| commit | 6678b9acc6435be56d0193dd0b40325fa615bd1f (patch) | |
| tree | 0d770d75be469ab254d768e5027828c95de42f21 /src/hostname | |
| parent | hostnamed: if polkit authentication fails for Varlink Describe() call, don't ... (diff) | |
| download | systemd-6678b9acc6435be56d0193dd0b40325fa615bd1f.tar.xz systemd-6678b9acc6435be56d0193dd0b40325fa615bd1f.zip | |
hostnamed: make sure we can actually properly parse 'allowInteractiveAuthentication' varlink parameter
If people want they should be able to turn on this flag, to allow
interactive auth. Let's make sure this actually works. i.e. add it to
the introspection data and don't refuse the parameter in Describe().
(note the varlink handling already does parameter validation through
varlink_dispatch(), hence we can just drop any further validation)
Diffstat (limited to 'src/hostname')
| -rw-r--r-- | src/hostname/hostnamed.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/hostname/hostnamed.c b/src/hostname/hostnamed.c index a8ebf97cd2..a0bf1bfdc8 100644 --- a/src/hostname/hostnamed.c +++ b/src/hostname/hostnamed.c @@ -1636,9 +1636,6 @@ static int vl_method_describe(Varlink *link, sd_json_variant *parameters, Varlin * the product ID which we'll check explicitly. */ privileged = r > 0; - if (sd_json_variant_elements(parameters) > 0) - return varlink_error_invalid_parameter(link, parameters); - _cleanup_(sd_json_variant_unrefp) sd_json_variant *v = NULL; r = build_describe_response(c, privileged, &v); if (r < 0) |