[or-cvs] make config_assign fail rather than warn for unrecognized c...

[arma@seul.org (Roger Dingledine)]

Update of /home/or/cvsroot/src/or
In directory moria.mit.edu:/home2/arma/work/onion/cvs/src/or

Modified Files:
	config.c 
Log Message:
make config_assign fail rather than warn for unrecognized config options


Index: config.c
===================================================================
RCS file: /home/or/cvsroot/src/or/config.c,v
retrieving revision 1.88
retrieving revision 1.89
diff -u -d -r1.88 -r1.89
--- config.c	28 Feb 2004 23:31:15 -0000	1.88
+++ config.c	2 Mar 2004 05:00:50 -0000	1.89
@@ -26,7 +26,7 @@
 static struct config_line *config_get_lines(FILE *f);
 static void config_free_lines(struct config_line *front);
 static int config_compare(struct config_line *c, char *key, int type, void *arg);
-static void config_assign(or_options_t *options, struct config_line *list);
+static int config_assign(or_options_t *options, struct config_line *list);
 
 /* open configuration file for reading */
 static FILE *config_open(const unsigned char *filename) {
@@ -141,9 +141,11 @@
   return 1;
 }
 
-static void config_assign(or_options_t *options, struct config_line *list) {
-
-  /* iterate through list. for each item convert as appropriate and assign to 'options'. */
+/* Iterate through list.
+ * For each item, convert as appropriate and assign to 'options'.
+ * If an item is unrecognized, return -1 immediately,
+ * else return 0 for success. */
+static int config_assign(or_options_t *options, struct config_line *list) {
 
   while(list) {
     if(
@@ -203,11 +205,13 @@
     ) {
       /* then we're ok. it matched something. */
     } else {
-      log_fn(LOG_WARN,"Ignoring unknown keyword '%s'.",list->key);
+      log_fn(LOG_WARN,"Unknown keyword '%s'. Failing.",list->key);
+      return -1;
     }
 
     list = list->next;
   }
+  return 0;
 }
 
 /* XXX are there any other specifiers we want to give so making
@@ -453,14 +457,16 @@
   } else { /* it opened successfully. use it. */
     cl = config_get_lines(cf);
     if(!cl) return -1;
-    config_assign(options,cl);
+    if(config_assign(options,cl) < 0)
+      return -1;
     config_free_lines(cl);
     config_close(cf);
   }
 
 /* go through command-line variables too */
   cl = config_get_commandlines(argc,argv);
-  config_assign(options,cl);
+  if(config_assign(options,cl) < 0)
+    return -1;
   config_free_lines(cl);
 
 /* Validate options */