Postpone resolving of DNS names of server.

[libradsec.git] / lib / radsec.c

diff --git a/lib/radsec.c b/lib/radsec.c
index a05a22b..e176b6d 100644 (file)
--- a/lib/radsec.c
+++ b/lib/radsec.c
@@ -28,41 +28,14 @@
 
 /* Public functions.  */
 int
-rs_context_create (struct rs_context **ctx, const char *dict)
+rs_context_create (struct rs_context **ctx)
 {
-  int err = RSE_OK;
   struct rs_context *h;
-  char *buf1 = NULL, *buf2 = NULL;
-  char *dir, *fn;
 
-  assert (dict);
-
-  if (ctx)
-    *ctx = NULL;
-  h = (struct rs_context *) malloc (sizeof(struct rs_context));
-  if (!h)
+  h = calloc (1, sizeof(*h));
+  if (h == NULL)
     return RSE_NOMEM;
 
-  /* Initialize freeradius dictionary.  */
-  buf1 = malloc (strlen (dict) + 1);
-  buf2 = malloc (strlen (dict) + 1);
-  if (!buf1 || !buf2)
-    {
-      err = RSE_NOMEM;
-      goto err_out;
-    }
-  strcpy (buf1, dict);
-  dir = dirname (buf1);
-  strcpy (buf2, dict);
-  fn = basename (buf2);
-  if (dict_init (dir, fn) < 0)
-    {
-      err = RSE_FR;
-      goto err_out;
-    }
-  free (buf1);
-  free (buf2);
-
 #if defined (RS_ENABLE_TLS)
   ssl_init ();
 #endif
@@ -72,36 +45,68 @@ rs_context_create (struct rs_context **ctx, const char *dict)
 #endif
   debug_init ("libradsec");    /* radsecproxy compat, FIXME: remove */
 
-  memset (h, 0, sizeof(struct rs_context));
   fr_randinit (&h->fr_randctx, 0);
   fr_rand_seed (NULL, 0);
 
-  if (ctx)
+  if (ctx != NULL)
     *ctx = h;
 
   return RSE_OK;
+}
 
- err_out:
-  if (buf1)
-    free (buf1);
-  if (buf2)
-    free (buf2);
-  if (h)
-    free (h);
-  return err;
+/** Initialize freeradius dictionary.  */
+int
+rs_context_init_freeradius_dict (struct rs_context *ctx, const char *dict)
+{
+  int r = RSE_OK;
+  size_t dictlen;
+  char *dir = NULL;
+  char *fn = NULL;
+
+  if (dict == NULL)
+    if (ctx->config != NULL && ctx->config->dictionary)
+      dict = ctx->config->dictionary;
+
+  if (dict == NULL)
+    dict = RS_FREERADIUS_DICT;
+
+  dictlen = strlen (dict);
+  dir = rs_calloc (ctx, 1, dictlen + 1);
+  fn = rs_calloc (ctx, 1, dictlen + 1);
+  if (dir == NULL || fn == NULL)
+    {
+      r = rs_err_ctx_push_fl (ctx, RSE_NOMEM, __FILE__, __LINE__, NULL);
+      goto out;
+    }
+  strncpy (dir, dict, dictlen);
+  strncpy (fn, dict, dictlen);
+
+  if (dict_init (dirname (dir), basename (fn)) < 0)
+    {
+      r = rs_err_ctx_push_fl (ctx, RSE_FR, __FILE__, __LINE__,
+                             "failing dict_init(\"%s\")", dict);
+      goto out;
+    }
+
+ out:
+  if (dir)
+    rs_free (ctx, dir);
+  if (fn)
+    rs_free (ctx, fn);
+  return r;
 }
 
-struct rs_error *         /* FIXME: Return int as all the others?  */
-rs_resolv (struct evutil_addrinfo **addr,
-          rs_conn_type_t type,
-          const char *hostname,
-          const char *service)
+struct rs_error *
+rs_resolve (struct evutil_addrinfo **addr,
+            rs_conn_type_t type,
+            const char *hostname,
+            const char *service)
 {
   int err;
   struct evutil_addrinfo hints, *res = NULL;
 
   memset (&hints, 0, sizeof(struct evutil_addrinfo));
-  hints.ai_family = AF_INET;   /* IPv4 only.  TODO: Set AF_UNSPEC.  */
+  hints.ai_family = AF_UNSPEC;
   hints.ai_flags = AI_ADDRCONFIG;
   switch (type)
     {
@@ -137,27 +142,40 @@ rs_context_destroy (struct rs_context *ctx)
   struct rs_realm *r = NULL;
   struct rs_peer *p = NULL;
 
-  for (r = ctx->realms; r; )
+  if (ctx->config)
     {
-      struct rs_realm *tmp = r;
-      for (p = r->peers; p; )
+      for (r = ctx->config->realms; r; )
        {
-         struct rs_peer *tmp = p;
-         if (p->addr)
-           evutil_freeaddrinfo (p->addr);
-         p = p->next;
+         struct rs_realm *tmp = r;
+         for (p = r->peers; p; )
+           {
+             struct rs_peer *tmp = p;
+             if (p->addr_cache)
+                {
+                  evutil_freeaddrinfo (p->addr_cache);
+                  p->addr_cache = NULL;
+                }
+             p = p->next;
+             rs_free (ctx, tmp);
+           }
+         free (r->name);
+          rs_free (ctx, r->transport_cred);
+         r = r->next;
          rs_free (ctx, tmp);
        }
-      rs_free (ctx, r->name);
-      r = r->next;
-      rs_free (ctx, tmp);
     }
 
-  if (ctx->cfg)
-    cfg_free (ctx->cfg);
-  ctx->cfg = NULL;
+  if (ctx->config)
+    {
+      if (ctx->config->cfg)
+       {
+         cfg_free (ctx->config->cfg);
+         ctx->config->cfg = NULL;
+       }
+      rs_free (ctx, ctx->config);
+    }
 
-  rs_free (ctx, ctx);
+  free (ctx);
 }
 
 int