diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-08-27 19:28:19 +0200 |
|---|---|---|
| committer | Lukas Nykryn <lnykryn@redhat.com> | 2019-10-24 12:22:20 +0200 |
| commit | 7b00cae817e54ee3398ad3b42ec69a3b63676562 (patch) | |
| tree | b10f875fe7e14cce0d6a723dc4c9c33691674877 | |
| parent | ddd08e75b1e7fa1f6dfef3d30a0c1ef8c63e4d07 (diff) | |
| download | systemd-239-19.tar.gz | |
resolved: query polkit only after parsing the datav239-19
That's what we do everywhere else because it leads to nicer user experience.
Related: #1746857
| -rw-r--r-- | src/resolve/resolved-bus.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/resolve/resolved-bus.c b/src/resolve/resolved-bus.c index 4d6cc4fd48..3f6a6f9e12 100644 --- a/src/resolve/resolved-bus.c +++ b/src/resolve/resolved-bus.c @@ -1632,15 +1632,6 @@ static int bus_method_register_service(sd_bus_message *message, void *userdata, if (m->mdns_support != RESOLVE_SUPPORT_YES) return sd_bus_error_setf(error, SD_BUS_ERROR_NOT_SUPPORTED, "Support for MulticastDNS is disabled"); - r = bus_verify_polkit_async(message, CAP_SYS_ADMIN, - "org.freedesktop.resolve1.register-service", - NULL, false, UID_INVALID, - &m->polkit_registry, error); - if (r < 0) - return r; - if (r == 0) - return 1; /* Polkit will call us back */ - service = new0(DnssdService, 1); if (!service) return log_oom(); @@ -1765,6 +1756,15 @@ static int bus_method_register_service(sd_bus_message *message, void *userdata, if (r < 0) return r; + r = bus_verify_polkit_async(message, CAP_SYS_ADMIN, + "org.freedesktop.resolve1.register-service", + NULL, false, UID_INVALID, + &m->polkit_registry, error); + if (r < 0) + return r; + if (r == 0) + return 1; /* Polkit will call us back */ + r = hashmap_ensure_allocated(&m->dnssd_services, &string_hash_ops); if (r < 0) return r; |