|
 |
163b89a |
From 6f0aa79c3e8dd93e723f29bf46e1b8b14403254f Mon Sep 17 00:00:00 2001
|
|
|
163b89a |
From: Sumit Bose <sbose@redhat.com>
|
|
|
163b89a |
Date: Mon, 5 Dec 2016 18:25:44 +0100
|
|
|
163b89a |
Subject: [PATCH] Kerberos: fall back to tcp SRV lookup
|
|
|
163b89a |
|
|
|
163b89a |
---
|
|
|
163b89a |
service/realm-kerberos-provider.c | 48 +++++++++++++++++++++++++++++++--------
|
|
|
163b89a |
1 file changed, 39 insertions(+), 9 deletions(-)
|
|
|
163b89a |
|
|
|
163b89a |
diff --git a/service/realm-kerberos-provider.c b/service/realm-kerberos-provider.c
|
|
|
163b89a |
index 2b3a0f8..1477ae8 100644
|
|
|
163b89a |
--- a/service/realm-kerberos-provider.c
|
|
|
163b89a |
+++ b/service/realm-kerberos-provider.c
|
|
|
163b89a |
@@ -19,6 +19,7 @@
|
|
|
163b89a |
#include "realm-kerberos-provider.h"
|
|
|
163b89a |
|
|
|
163b89a |
#include <errno.h>
|
|
|
163b89a |
+#include <string.h>
|
|
|
163b89a |
|
|
|
163b89a |
struct _RealmKerberosProvider {
|
|
|
163b89a |
RealmProvider parent;
|
|
|
163b89a |
@@ -38,28 +39,54 @@ realm_kerberos_provider_init (RealmKerberosProvider *self)
|
|
|
163b89a |
|
|
|
163b89a |
}
|
|
|
163b89a |
|
|
|
163b89a |
+typedef struct {
|
|
|
163b89a |
+ gchar *name;
|
|
|
163b89a |
+ const char *prot;
|
|
|
163b89a |
+} NameProtPair;
|
|
|
163b89a |
+
|
|
|
163b89a |
+static void
|
|
|
163b89a |
+name_prot_pair_free (gpointer data)
|
|
|
163b89a |
+{
|
|
|
163b89a |
+ NameProtPair *name_prot_pair = data;
|
|
|
163b89a |
+ g_free (name_prot_pair->name);
|
|
|
163b89a |
+ g_free (name_prot_pair);
|
|
|
163b89a |
+}
|
|
|
163b89a |
+
|
|
|
163b89a |
static void
|
|
|
163b89a |
on_kerberos_discover (GObject *source,
|
|
|
163b89a |
GAsyncResult *result,
|
|
|
163b89a |
gpointer user_data)
|
|
|
163b89a |
{
|
|
|
163b89a |
GTask *task = G_TASK (user_data);
|
|
|
163b89a |
- const gchar *domain = g_task_get_task_data (task);
|
|
|
163b89a |
+ NameProtPair *name_prot_pair = g_task_get_task_data (task);
|
|
|
163b89a |
GError *error = NULL;
|
|
|
163b89a |
RealmDisco *disco;
|
|
|
163b89a |
GList *targets;
|
|
|
163b89a |
+ GResolver *resolver;
|
|
|
163b89a |
|
|
|
163b89a |
targets = g_resolver_lookup_service_finish (G_RESOLVER (source), result, &error);
|
|
|
163b89a |
if (targets) {
|
|
|
163b89a |
g_list_free_full (targets, (GDestroyNotify)g_srv_target_free);
|
|
|
163b89a |
- disco = realm_disco_new (domain);
|
|
|
163b89a |
- disco->kerberos_realm = g_ascii_strup (domain, -1);
|
|
|
163b89a |
+ disco = realm_disco_new (name_prot_pair->name);
|
|
|
163b89a |
+ disco->kerberos_realm = g_ascii_strup (name_prot_pair->name, -1);
|
|
|
163b89a |
g_task_return_pointer (task, disco, realm_disco_unref);
|
|
|
163b89a |
|
|
|
163b89a |
} else if (error) {
|
|
|
163b89a |
- g_debug ("Resolving %s failed: %s", domain, error->message);
|
|
|
163b89a |
+ g_debug ("Resolving %s failed: %s", name_prot_pair->name, error->message);
|
|
|
163b89a |
g_error_free (error);
|
|
|
163b89a |
- g_task_return_pointer (task, NULL, NULL);
|
|
|
163b89a |
+
|
|
|
163b89a |
+ if (strcmp (name_prot_pair->prot, "tcp") == 0) {
|
|
|
163b89a |
+ g_task_return_pointer (task, NULL, NULL);
|
|
|
163b89a |
+ } else {
|
|
|
163b89a |
+ /* Try tcp */
|
|
|
163b89a |
+ name_prot_pair->prot = "tcp";
|
|
|
163b89a |
+ resolver = g_resolver_get_default ();
|
|
|
163b89a |
+ g_resolver_lookup_service_async (resolver, "kerberos", name_prot_pair->prot,
|
|
|
163b89a |
+ name_prot_pair->name,
|
|
|
163b89a |
+ g_task_get_cancellable (task),
|
|
|
163b89a |
+ on_kerberos_discover, g_object_ref (task));
|
|
|
163b89a |
+ g_object_unref (resolver);
|
|
|
163b89a |
+ }
|
|
|
163b89a |
}
|
|
|
163b89a |
|
|
|
163b89a |
g_object_unref (task);
|
|
|
163b89a |
@@ -76,7 +103,7 @@ realm_kerberos_provider_discover_async (RealmProvider *provider,
|
|
|
163b89a |
GTask *task;
|
|
|
163b89a |
const gchar *software;
|
|
|
163b89a |
GResolver *resolver;
|
|
|
163b89a |
- gchar *name;
|
|
|
163b89a |
+ NameProtPair *name_prot_pair;
|
|
|
163b89a |
|
|
|
163b89a |
task = g_task_new (provider, NULL, callback, user_data);
|
|
|
163b89a |
|
|
|
163b89a |
@@ -86,12 +113,15 @@ realm_kerberos_provider_discover_async (RealmProvider *provider,
|
|
|
163b89a |
g_task_return_pointer (task, NULL, NULL);
|
|
|
163b89a |
|
|
|
163b89a |
} else {
|
|
|
163b89a |
- name = g_hostname_to_ascii (string);
|
|
|
163b89a |
+ name_prot_pair = g_new0 (NameProtPair, 1);
|
|
|
163b89a |
+ name_prot_pair->name = g_hostname_to_ascii (string);
|
|
|
163b89a |
+ name_prot_pair->prot = "udp";
|
|
|
163b89a |
resolver = g_resolver_get_default ();
|
|
|
163b89a |
- g_resolver_lookup_service_async (resolver, "kerberos", "udp", name,
|
|
|
163b89a |
+ g_resolver_lookup_service_async (resolver, "kerberos", name_prot_pair->prot,
|
|
|
163b89a |
+ name_prot_pair->name,
|
|
|
163b89a |
realm_invocation_get_cancellable (invocation),
|
|
|
163b89a |
on_kerberos_discover, g_object_ref (task));
|
|
|
163b89a |
- g_task_set_task_data (task, name, g_free);
|
|
|
163b89a |
+ g_task_set_task_data (task, name_prot_pair, name_prot_pair_free);
|
|
|
163b89a |
g_object_unref (resolver);
|
|
|
163b89a |
}
|
|
|
163b89a |
|
|
|
163b89a |
--
|
|
|
163b89a |
2.9.3
|
|
|
163b89a |
|