[or-cvs] r13944: options_init_from_torrc(): move code that loads torrc into i (tor/trunk/src/or)

[weasel at seul.org]

Author: weasel
Date: 2008-03-10 08:41:36 -0400 (Mon, 10 Mar 2008)
New Revision: 13944

Modified:
   tor/trunk/src/or/config.c
Log:
options_init_from_torrc(): move code that loads torrc into its own function

move code that loads torrc from disk and sets torrc_fname into its own function


Modified: tor/trunk/src/or/config.c
===================================================================
--- tor/trunk/src/or/config.c	2008-03-10 12:41:33 UTC (rev 13943)
+++ tor/trunk/src/or/config.c	2008-03-10 12:41:36 UTC (rev 13944)
@@ -3574,6 +3574,46 @@
   return fname;
 }
 
+/** Load torrc from disk, setting torrc_fname if successful */
+char *
+load_torrc_from_disk(int argc, char **argv)
+{
+  char *fname=NULL;
+  char *cf = NULL;
+  int using_default_torrc = 1;
+  int ignore_missing_torrc = 0;
+
+  fname = find_torrc_filename(argc, argv,
+                              &using_default_torrc, &ignore_missing_torrc);
+  tor_assert(fname);
+  log(LOG_DEBUG, LD_CONFIG, "Opening config file \"%s\"", fname);
+
+  tor_free(torrc_fname);
+  torrc_fname = fname;
+
+  /* Open config file */
+  if (file_status(fname) != FN_FILE ||
+      !(cf = read_file_to_str(fname,0,NULL))) {
+    if (using_default_torrc == 1 || ignore_missing_torrc ) {
+      log(LOG_NOTICE, LD_CONFIG, "Configuration file \"%s\" not present, "
+          "using reasonable defaults.", fname);
+      tor_free(fname); /* sets fname to NULL */
+      torrc_fname = NULL;
+      cf = tor_strdup("");
+    } else {
+      log(LOG_WARN, LD_CONFIG,
+          "Unable to open configuration file \"%s\".", fname);
+      goto err;
+    }
+  }
+
+  return cf;
+ err:
+  tor_free(fname);
+  torrc_fname = NULL;
+  return NULL;
+}
+
 /** Read a configuration file into <b>options</b>, finding the configuration
  * file location based on the command line.  After loading the options,
  * validate them for consistency, then take actions based on them.
@@ -3583,10 +3623,8 @@
 {
   or_options_t *oldoptions, *newoptions;
   config_line_t *cl;
-  char *cf=NULL, *fname=NULL, *errmsg=NULL;
+  char *cf=NULL, *errmsg=NULL;
   int i, retval;
-  int using_default_torrc = 1;
-  int ignore_missing_torrc = 0;
   static char **backup_argv;
   static int backup_argc;
 
@@ -3634,30 +3672,10 @@
     }
   }
 
-  fname = find_torrc_filename(argc, argv,
-                              &using_default_torrc, &ignore_missing_torrc);
-  tor_assert(fname);
-  log(LOG_DEBUG, LD_CONFIG, "Opening config file \"%s\"", fname);
+  cf = load_torrc_from_disk(argc, argv);
+  if (!cf)
+    goto err;
 
-  tor_free(torrc_fname);
-  torrc_fname = fname;
-
-  /* Open config file */
-  if (file_status(fname) != FN_FILE ||
-      !(cf = read_file_to_str(fname,0,NULL))) {
-    if (using_default_torrc == 1 || ignore_missing_torrc ) {
-      log(LOG_NOTICE, LD_CONFIG, "Configuration file \"%s\" not present, "
-          "using reasonable defaults.", fname);
-      tor_free(fname); /* sets fname to NULL */
-      torrc_fname = NULL;
-      cf = tor_strdup("");
-    } else {
-      log(LOG_WARN, LD_CONFIG,
-          "Unable to open configuration file \"%s\".", fname);
-      goto err;
-    }
-  }
-
   /* get config lines, assign them */
   retval = config_get_lines(cf, &cl);
   tor_free(cf);
@@ -3688,8 +3706,6 @@
 
   return 0;
  err:
-  tor_free(fname);
-  torrc_fname = NULL;
   config_free(&options_format, newoptions);
   if (errmsg) {
     log(LOG_WARN,LD_CONFIG,"Failed to parse/validate config: %s", errmsg);