projects / radsecproxy.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Handle all libconfuse parse errors.
[radsecproxy.git] / lib / conf.c
diff --git a/lib/conf.c b/lib/conf.c
index 0f00fd8..9b8e77c 100644 (file)
--- a/lib/conf.c
+++ b/lib/conf.c
@@ -13,8 +13,11 @@
 #include "debug.h"
 
 #if 0
-  # client config options
-  config NAME {
+  # common config options
+  dictionary = STRING
+
+  # common realm config options
+  realm NAME {
       type = "UDP"|"TCP"|"TLS"|"DTLS"
       timeout = INT
       retries = INT
@@ -22,6 +25,10 @@
       #cacertpath = STRING
       certfile = STRING
       certkeyfile = STRING
+  }
+
+  # client specific realm config options
+  realm NAME {
       server {
           hostname = STRING
          service = STRING
@@ -30,14 +37,15 @@
   }
 #endif
 
+/* FIXME: Leaking memory in error cases?  */
 int
 rs_context_read_config(struct rs_context *ctx, const char *config_file)
 {
-  /* FIXME: Missing some error handling in rs_context_read_config().  */
-
-  cfg_t *cfg, *cfg_config, *cfg_server;
+  cfg_t *cfg, *cfg_realm, *cfg_server;
+  int err = 0;
   int i, j;
   const char *s;
+  struct rs_config *config = NULL;
 
   cfg_opt_t server_opts[] =
     {
@@ -46,7 +54,7 @@ rs_context_read_config(struct rs_context *ctx, const char *config_file)
       CFG_STR ("secret", "radsec", CFGF_NONE),
       CFG_END ()
     };
-  cfg_opt_t config_opts[] =
+  cfg_opt_t realm_opts[] =
     {
       CFG_STR ("type", "UDP", CFGF_NONE),
       CFG_INT ("timeout", 2, CFGF_NONE), /* FIXME: Remove?  */
@@ -60,39 +68,63 @@ rs_context_read_config(struct rs_context *ctx, const char *config_file)
     };
   cfg_opt_t opts[] =
     {
-      CFG_SEC ("config", config_opts, CFGF_TITLE | CFGF_MULTI),
+      CFG_STR ("dictionary", NULL, CFGF_NONE),
+      CFG_SEC ("realm", realm_opts, CFGF_TITLE | CFGF_MULTI),
       CFG_END ()
     };
 
   cfg = cfg_init (opts, CFGF_NONE);
-  if (cfg_parse (cfg, config_file) == CFG_PARSE_ERROR)
-    return rs_err_ctx_push (ctx, RSE_CONFIG, "%s: invalid configuration file",
-                           config_file);
-  for (i = 0; i < cfg_size (cfg, "config"); i++)
+  if (cfg == NULL)
+    return rs_err_ctx_push (ctx, RSE_CONFIG, "unable to initialize libconfuse");
+  err = cfg_parse (cfg, config_file);
+  switch (err)
+    {
+    case  CFG_SUCCESS:
+      break;
+    case CFG_FILE_ERROR:
+      return rs_err_ctx_push (ctx, RSE_CONFIG,
+                             "%s: unable to open configuration file",
+                             config_file);
+    case CFG_PARSE_ERROR:
+      return rs_err_ctx_push (ctx, RSE_CONFIG, "%s: invalid configuration file",
+                             config_file);
+    default:
+       return rs_err_ctx_push (ctx, RSE_CONFIG, "%s: unknown parse error",
+                               config_file);
+    }
+
+  config = rs_calloc (ctx, 1, sizeof (*config));
+  if (config == NULL)
+    return rs_err_ctx_push_fl (ctx, RSE_NOMEM, __FILE__, __LINE__, NULL);
+  ctx->config = config;
+  config->dictionary = cfg_getstr (cfg, "dictionary");
+
+  for (i = 0; i < cfg_size (cfg, "realm"); i++)
     {
-      struct rs_realm *r = rs_malloc (ctx, sizeof(*r));
+      struct rs_realm *r = rs_calloc (ctx, 1, sizeof(*r));
       const char *typestr;
 
       if (!r)
        return rs_err_ctx_push_fl (ctx, RSE_NOMEM, __FILE__, __LINE__, NULL);
-      memset (r, 0, sizeof(*r));
-      if (ctx->realms)
+      if (config->realms)
        {
-         r->next = ctx->realms->next;
-         ctx->realms->next = r;
+         r->next = config->realms->next;
+         config->realms->next = r;
        }
       else
-         ctx->realms = r;
-      cfg_config = cfg_getnsec (cfg, "config", i);
-      s = cfg_title (cfg_config);
+         config->realms = r;
+      cfg_realm = cfg_getnsec (cfg, "realm", i);
+      /* We use a copy of return value of cfg_title since it's a
+        const.  */
+      s = cfg_title (cfg_realm);
       if (s == NULL)
        return rs_err_ctx_push_fl (ctx, RSE_CONFIG, __FILE__, __LINE__,
-                                  "missing config name");
-      r->name = strdup (s);
+                                  "missing realm name");
+      r->name = strdup (s);    /* FIXME: Don't strdup.  */
       if (!r->name)
        return rs_err_ctx_push_fl (ctx, RSE_NOMEM, __FILE__, __LINE__, NULL);
 
-      typestr = cfg_getstr (cfg_config, "type");
+      typestr = cfg_getstr (cfg_realm, "type");
       if (!strcmp (typestr, "UDP"))
        r->type = RS_CONN_TYPE_UDP;
       else if (!strcmp (typestr, "TCP"))
@@ -104,16 +136,16 @@ rs_context_read_config(struct rs_context *ctx, const char *config_file)
       else
        return rs_err_ctx_push_fl (ctx, RSE_CONFIG, __FILE__, __LINE__,
                                   "invalid connection type: %s", typestr);
-      r->timeout = cfg_getint (cfg_config, "timeout");
-      r->retries = cfg_getint (cfg_config, "retries");
+      r->timeout = cfg_getint (cfg_realm, "timeout");
+      r->retries = cfg_getint (cfg_realm, "retries");
 
-      r->cacertfile = cfg_getstr (cfg_config, "cacertfile");
-      /*r->cacertpath = cfg_getstr (cfg_config, "cacertpath");*/
-      r->certfile = cfg_getstr (cfg_config, "certfile");
-      r->certkeyfile = cfg_getstr (cfg_config, "certkeyfile");
+      r->cacertfile = cfg_getstr (cfg_realm, "cacertfile");
+      /*r->cacertpath = cfg_getstr (cfg_realm, "cacertpath");*/
+      r->certfile = cfg_getstr (cfg_realm, "certfile");
+      r->certkeyfile = cfg_getstr (cfg_realm, "certkeyfile");
 
       /* Add peers, one per server stanza.  */
-      for (j = 0; j < cfg_size (cfg_config, "server"); j++)
+      for (j = 0; j < cfg_size (cfg_realm, "server"); j++)
        {
          struct rs_peer *p = peer_create (ctx, &r->peers);
          if (!p)
@@ -121,7 +153,9 @@ rs_context_read_config(struct rs_context *ctx, const char *config_file)
                                       NULL);
          p->realm = r;
 
-         cfg_server = cfg_getnsec (cfg_config, "server", j);
+         cfg_server = cfg_getnsec (cfg_realm, "server", j);
+         /* FIXME: Handle resolve errors, possibly by postponing name
+            resolution.  */
          rs_resolv (&p->addr, r->type, cfg_getstr (cfg_server, "hostname"),
                     cfg_getstr (cfg_server, "service"));
          p->secret = cfg_getstr (cfg_server, "secret");
@@ -130,7 +164,7 @@ rs_context_read_config(struct rs_context *ctx, const char *config_file)
 
   /* Save config object in context, for freeing in
      rs_context_destroy().  */
-  ctx->cfg =  cfg;
+  ctx->config->cfg =  cfg;
   return RSE_OK;
 }
 
@@ -139,7 +173,7 @@ rs_conf_find_realm(struct rs_context *ctx, const char *name)
 {
   struct rs_realm *r;
 
-  for (r = ctx->realms; r; r = r->next)
+  for (r = ctx->config->realms; r; r = r->next)
     if (!strcmp (r->name, name))
        return r;
   return NULL;
Unnamed repository; edit this file 'description' to name the repository.