return NULL;
dns_resource_record_unref(txt_data->rr);
- dns_txt_item_free_all(txt_data->txt);
+ dns_txt_item_free_all(txt_data->txts);
return mfree(txt_data);
}
dropin_dirname = strjoina(service->name, ".dnssd.d");
r = config_parse_many(
- filename, DNSSD_SERVICE_DIRS, dropin_dirname,
+ STRV_MAKE_CONST(filename), DNSSD_SERVICE_DIRS, dropin_dirname,
"Service\0",
config_item_perf_lookup, resolved_dnssd_gperf_lookup,
CONFIG_PARSE_WARN,
"%s doesn't define service type",
service->name);
- if (LIST_IS_EMPTY(service->txt_data_items)) {
+ if (!service->txt_data_items) {
txt_data = new0(DnssdTxtData, 1);
if (!txt_data)
return log_oom();
- r = dns_txt_item_new_empty(&txt_data->txt);
+ r = dns_txt_item_new_empty(&txt_data->txts);
if (r < 0)
return r;
LIST_PREPEND(items, service->txt_data_items, txt_data);
- txt_data = NULL;
+ TAKE_PTR(txt_data);
}
r = hashmap_ensure_put(&manager->dnssd_services, &string_hash_ops, service->name, service);
if (r < 0)
return r;
- service = NULL;
+ TAKE_PTR(service);
return 0;
}
-static int specifier_dnssd_host_name(char specifier, const void *data, const void *userdata, char **ret) {
- DnssdService *s = (DnssdService *) userdata;
+static int specifier_dnssd_hostname(char specifier, const void *data, const char *root, const void *userdata, char **ret) {
+ const Manager *m = ASSERT_PTR(userdata);
char *n;
- assert(s);
- assert(s->manager);
- assert(s->manager->llmnr_hostname);
+ assert(m->llmnr_hostname);
- n = strdup(s->manager->llmnr_hostname);
+ n = strdup(m->llmnr_hostname);
if (!n)
return -ENOMEM;
return 0;
}
-int dnssd_render_instance_name(DnssdService *s, char **ret_name) {
+int dnssd_render_instance_name(Manager *m, DnssdService *s, char **ret) {
static const Specifier specifier_table[] = {
- { 'a', specifier_architecture, NULL },
- { 'b', specifier_boot_id, NULL },
- { 'B', specifier_os_build_id, NULL },
- { 'H', specifier_dnssd_host_name, NULL },
- { 'm', specifier_machine_id, NULL },
- { 'o', specifier_os_id, NULL },
- { 'v', specifier_kernel_release, NULL },
- { 'w', specifier_os_version_id, NULL },
- { 'W', specifier_os_variant_id, NULL },
+ { 'a', specifier_architecture, NULL },
+ { 'b', specifier_boot_id, NULL },
+ { 'B', specifier_os_build_id, NULL },
+ { 'H', specifier_dnssd_hostname, NULL },
+ { 'm', specifier_machine_id, NULL },
+ { 'o', specifier_os_id, NULL },
+ { 'v', specifier_kernel_release, NULL },
+ { 'w', specifier_os_version_id, NULL },
+ { 'W', specifier_os_variant_id, NULL },
{}
};
_cleanup_free_ char *name = NULL;
int r;
+ assert(m);
assert(s);
assert(s->name_template);
- r = specifier_printf(s->name_template, specifier_table, s, &name);
+ r = specifier_printf(s->name_template, DNS_LABEL_MAX, specifier_table, NULL, m, &name);
if (r < 0)
return log_debug_errno(r, "Failed to replace specifiers: %m");
"Service instance name '%s' is invalid.",
name);
- *ret_name = TAKE_PTR(name);
+ if (ret)
+ *ret = TAKE_PTR(name);
return 0;
}
int dnssd_load(Manager *manager) {
_cleanup_strv_free_ char **files = NULL;
- char **f;
int r;
assert(manager);
STRV_FOREACH_BACKWARDS(f, files) {
r = dnssd_service_load(manager, *f);
if (r < 0)
- log_warning_errno(r, "Failed to load '%s': %m", *f);;
+ log_warning_errno(r, "Failed to load '%s': %m", *f);
}
return 0;
}
int dnssd_update_rrs(DnssdService *s) {
- _cleanup_free_ char *n = NULL;
- _cleanup_free_ char *service_name = NULL;
- _cleanup_free_ char *full_name = NULL;
- DnssdTxtData *txt_data;
+ _cleanup_free_ char *n = NULL, *service_name = NULL, *full_name = NULL;
int r;
assert(s);
LIST_FOREACH(items, txt_data, s->txt_data_items)
txt_data->rr = dns_resource_record_unref(txt_data->rr);
- r = dnssd_render_instance_name(s, &n);
+ r = dnssd_render_instance_name(s->manager, s, &n);
if (r < 0)
return r;
goto oom;
txt_data->rr->ttl = MDNS_DEFAULT_TTL;
- txt_data->rr->txt.items = dns_txt_item_copy(txt_data->txt);
+ txt_data->rr->txt.items = dns_txt_item_copy(txt_data->txts);
if (!txt_data->rr->txt.items)
goto oom;
}
return 0;
}
-void dnssd_signal_conflict(Manager *manager, const char *name) {
+int dnssd_signal_conflict(Manager *manager, const char *name) {
DnssdService *s;
int r;
+ if (sd_bus_is_ready(manager->bus) <= 0)
+ return 0;
+
HASHMAP_FOREACH(s, manager->dnssd_services) {
if (s->withdrawn)
continue;
s->withdrawn = true;
r = sd_bus_path_encode("/org/freedesktop/resolve1/dnssd", s->name, &path);
- if (r < 0) {
- log_error_errno(r, "Can't get D-BUS object path: %m");
- return;
- }
+ if (r < 0)
+ return log_error_errno(r, "Can't get D-BUS object path: %m");
r = sd_bus_emit_signal(manager->bus,
path,
"org.freedesktop.resolve1.DnssdService",
"Conflicted",
NULL);
- if (r < 0) {
- log_error_errno(r, "Cannot emit signal: %m");
- return;
- }
+ if (r < 0)
+ return log_error_errno(r, "Cannot emit signal: %m");
break;
}
}
+
+ return 0;
}