tree-wide: drop redundant check for SD_VARLINK_METHOD_MORE flag
If the IDL declares the method requires 'more' yet the call doesn't have it set, varlink_idl_validate_method_call() should have rejected it and the callback shouldn't be reached.
This commit is contained in:
@@ -642,14 +642,12 @@ int vl_method_list_boot_entries(sd_varlink *link, sd_json_variant *parameters, s
|
|||||||
int r;
|
int r;
|
||||||
|
|
||||||
assert(link);
|
assert(link);
|
||||||
|
assert(FLAGS_SET(flags, SD_VARLINK_METHOD_MORE));
|
||||||
|
|
||||||
r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
|
r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
|
||||||
if (r != 0)
|
if (r != 0)
|
||||||
return r;
|
return r;
|
||||||
|
|
||||||
if (!FLAGS_SET(flags, SD_VARLINK_METHOD_MORE))
|
|
||||||
return sd_varlink_error(link, SD_VARLINK_ERROR_EXPECTED_MORE, NULL);
|
|
||||||
|
|
||||||
r = acquire_esp(/* unprivileged_mode= */ false,
|
r = acquire_esp(/* unprivileged_mode= */ false,
|
||||||
/* graceful= */ false,
|
/* graceful= */ false,
|
||||||
/* ret_part= */ NULL,
|
/* ret_part= */ NULL,
|
||||||
|
|||||||
@@ -1793,14 +1793,12 @@ static int vl_method_list_transfers(sd_varlink *link, sd_json_variant *parameter
|
|||||||
|
|
||||||
assert(link);
|
assert(link);
|
||||||
assert(parameters);
|
assert(parameters);
|
||||||
|
assert(FLAGS_SET(flags, SD_VARLINK_METHOD_MORE));
|
||||||
|
|
||||||
r = sd_varlink_dispatch(link, parameters, dispatch_table, &p);
|
r = sd_varlink_dispatch(link, parameters, dispatch_table, &p);
|
||||||
if (r != 0)
|
if (r != 0)
|
||||||
return r;
|
return r;
|
||||||
|
|
||||||
if (!FLAGS_SET(flags, SD_VARLINK_METHOD_MORE))
|
|
||||||
return sd_varlink_error(link, SD_VARLINK_ERROR_EXPECTED_MORE, NULL);
|
|
||||||
|
|
||||||
r = varlink_set_sentinel(link, "io.systemd.Import.NoTransfers");
|
r = varlink_set_sentinel(link, "io.systemd.Import.NoTransfers");
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
return r;
|
return r;
|
||||||
|
|||||||
@@ -5404,6 +5404,7 @@ static int vl_method_read_event_log(sd_varlink *link, sd_json_variant *parameter
|
|||||||
int r;
|
int r;
|
||||||
|
|
||||||
assert(link);
|
assert(link);
|
||||||
|
assert(FLAGS_SET(flags, SD_VARLINK_METHOD_MORE));
|
||||||
|
|
||||||
r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
|
r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
|
||||||
if (r != 0)
|
if (r != 0)
|
||||||
|
|||||||
@@ -1239,9 +1239,7 @@ static int vl_method_subscribe_query_results(sd_varlink *link, sd_json_variant *
|
|||||||
Manager *m = ASSERT_PTR(sd_varlink_get_userdata(ASSERT_PTR(link)));
|
Manager *m = ASSERT_PTR(sd_varlink_get_userdata(ASSERT_PTR(link)));
|
||||||
int r;
|
int r;
|
||||||
|
|
||||||
/* if the client didn't set the more flag, it is using us incorrectly */
|
assert(FLAGS_SET(flags, SD_VARLINK_METHOD_MORE));
|
||||||
if (!FLAGS_SET(flags, SD_VARLINK_METHOD_MORE))
|
|
||||||
return sd_varlink_error(link, SD_VARLINK_ERROR_EXPECTED_MORE, NULL);
|
|
||||||
|
|
||||||
r = verify_polkit(link, parameters, "org.freedesktop.resolve1.subscribe-query-results");
|
r = verify_polkit(link, parameters, "org.freedesktop.resolve1.subscribe-query-results");
|
||||||
if (r <= 0)
|
if (r <= 0)
|
||||||
@@ -1382,9 +1380,7 @@ static int vl_method_subscribe_dns_configuration(sd_varlink *link, sd_json_varia
|
|||||||
Manager *m = ASSERT_PTR(sd_varlink_get_userdata(ASSERT_PTR(link)));
|
Manager *m = ASSERT_PTR(sd_varlink_get_userdata(ASSERT_PTR(link)));
|
||||||
int r;
|
int r;
|
||||||
|
|
||||||
/* if the client didn't set the more flag, it is using us incorrectly */
|
assert(FLAGS_SET(flags, SD_VARLINK_METHOD_MORE));
|
||||||
if (!FLAGS_SET(flags, SD_VARLINK_METHOD_MORE))
|
|
||||||
return sd_varlink_error(link, SD_VARLINK_ERROR_EXPECTED_MORE, NULL);
|
|
||||||
|
|
||||||
r = verify_polkit(link, parameters, "org.freedesktop.resolve1.subscribe-dns-configuration");
|
r = verify_polkit(link, parameters, "org.freedesktop.resolve1.subscribe-dns-configuration");
|
||||||
if (r <= 0)
|
if (r <= 0)
|
||||||
|
|||||||
@@ -84,14 +84,12 @@ int metrics_method_describe(
|
|||||||
assert(metric_family_table);
|
assert(metric_family_table);
|
||||||
assert(link);
|
assert(link);
|
||||||
assert(parameters);
|
assert(parameters);
|
||||||
|
assert(FLAGS_SET(flags, SD_VARLINK_METHOD_MORE));
|
||||||
|
|
||||||
r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
|
r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
|
||||||
if (r != 0)
|
if (r != 0)
|
||||||
return r;
|
return r;
|
||||||
|
|
||||||
if (!FLAGS_SET(flags, SD_VARLINK_METHOD_MORE))
|
|
||||||
return sd_varlink_error(link, SD_VARLINK_ERROR_EXPECTED_MORE, NULL);
|
|
||||||
|
|
||||||
r = varlink_set_sentinel(link, "io.systemd.Metrics.NoSuchMetric");
|
r = varlink_set_sentinel(link, "io.systemd.Metrics.NoSuchMetric");
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
return r;
|
return r;
|
||||||
@@ -123,14 +121,12 @@ int metrics_method_list(
|
|||||||
assert(metric_family_table);
|
assert(metric_family_table);
|
||||||
assert(link);
|
assert(link);
|
||||||
assert(parameters);
|
assert(parameters);
|
||||||
|
assert(FLAGS_SET(flags, SD_VARLINK_METHOD_MORE));
|
||||||
|
|
||||||
r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
|
r = sd_varlink_dispatch(link, parameters, /* dispatch_table= */ NULL, /* userdata= */ NULL);
|
||||||
if (r != 0)
|
if (r != 0)
|
||||||
return r;
|
return r;
|
||||||
|
|
||||||
if (!FLAGS_SET(flags, SD_VARLINK_METHOD_MORE))
|
|
||||||
return sd_varlink_error(link, SD_VARLINK_ERROR_EXPECTED_MORE, NULL);
|
|
||||||
|
|
||||||
r = varlink_set_sentinel(link, "io.systemd.Metrics.NoSuchMetric");
|
r = varlink_set_sentinel(link, "io.systemd.Metrics.NoSuchMetric");
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
return r;
|
return r;
|
||||||
|
|||||||
@@ -2749,6 +2749,7 @@ static int vl_method_list(sd_varlink *link, sd_json_variant *parameters, sd_varl
|
|||||||
int r;
|
int r;
|
||||||
|
|
||||||
assert(link);
|
assert(link);
|
||||||
|
assert(FLAGS_SET(flags, SD_VARLINK_METHOD_MORE));
|
||||||
|
|
||||||
const char *class = NULL;
|
const char *class = NULL;
|
||||||
r = sd_varlink_dispatch(link, parameters, dispatch_table, &class);
|
r = sd_varlink_dispatch(link, parameters, dispatch_table, &class);
|
||||||
|
|||||||
Reference in New Issue
Block a user