core: always handle method call params via sd_varlink_dispatch(), even if we accept...

This results in much more precise errors for the clients, and is what we
do everywhere else these days.

diff --git a/src/core/manager-varlink.c b/src/core/manager-varlink.c
index c124b2fc9fc857bce99ec08701715a1851e13b91..3c7e746640bdde469b9190b8a7f5ddcb05cfc747 100644 (file)
--- a/src/core/manager-varlink.c
+++ b/src/core/manager-varlink.c
@@ -158,8 +158,9 @@ int vl_method_describe_manager(sd_varlink *link, sd_json_variant *parameters, sd
 
         assert(parameters);
 
-        if (sd_json_variant_elements(parameters) > 0)
-                return sd_varlink_error_invalid_parameter(link, parameters);
+        r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
+        if (r != 0)
+                return r;
 
         r = sd_json_buildo(&v,
                         SD_JSON_BUILD_PAIR_CALLBACK("Context", manager_context_build_json, manager),