From: Roger Dingledine <arma@torproject.org>
Date: Tue, 9 Nov 2004 06:18:17 +0000 (+0000)
Subject: During options_dup, don't copy LINELIST_S or OBSOLETE options.
X-Git-Tag: debian-version-0.0.8+0.0.9pre5-1~52
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=677124fd0c34e94ba88b658bdc239100e873cfd2;p=thirdparty%2Ftor.git

During options_dup, don't copy LINELIST_S or OBSOLETE options.
And recognize obsolete options now in config_get_assigned_option().


svn:r2729
---

diff --git a/src/or/config.c b/src/or/config.c
index 3e3a3f7d5c..0843e855a2 100644
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -604,6 +604,11 @@ config_get_assigned_option(or_options_t *options, const char *key)
       else
         result->value = tor_strdup("");
       break;
+    case CONFIG_TYPE_OBSOLETE:
+      log_fn(LOG_WARN,"You asked me for the value of an obsolete config option %s.", key);
+      tor_free(result->key);
+      tor_free(result);
+      return NULL;
     default:
       tor_free(result->key);
       tor_free(result);
@@ -915,6 +920,10 @@ options_dup(or_options_t *old)
 
   newopts = tor_malloc_zero(sizeof(or_options_t));
   for (i=0; config_vars[i].name; ++i) {
+    if(config_vars[i].type == CONFIG_TYPE_LINELIST_S)
+      continue;
+    if(config_vars[i].type == CONFIG_TYPE_OBSOLETE)
+      continue;
     line = config_get_assigned_option(old, config_vars[i].name);
     if (line) {
       if (config_assign(newopts, line, 0) < 0) {