Make the bwauth.c module dirauth-only.

diff --git a/src/app/config/config.c b/src/app/config/config.c
index 3e0683eb56c2f259b2ed41cd6b0db4a43e6f44f5..26a3061a2673aa2527b2cbd2bf7ac1f5dd14fda5 100644 (file)
--- a/src/app/config/config.c
+++ b/src/app/config/config.c
@@ -3556,12 +3556,12 @@ options_validate(or_options_t *old_options, or_options_t *options,
           options->V3AuthoritativeDir))
       REJECT("AuthoritativeDir is set, but none of "
              "(Bridge/V3)AuthoritativeDir is set.");
+#ifdef HAVE_MODULE_DIRAUTH
     /* If we have a v3bandwidthsfile and it's broken, complain on startup */
     if (options->V3BandwidthsFile && !old_options) {
       dirserv_read_measured_bandwidths(options->V3BandwidthsFile, NULL, NULL,
                                        NULL);
     }
-#ifdef HAVE_MODULE_DIRAUTH
     /* same for guardfraction file */
     if (options->GuardfractionFile && !old_options) {
       dirserv_read_guardfraction_file(options->GuardfractionFile, NULL);

diff --git a/src/app/main/shutdown.c b/src/app/main/shutdown.c
index fd9512ce4b892a26b746b3ea6df1703013561ce4..92cd9c6f7b01193515dffff011fd1d436deb2e59 100644 (file)
--- a/src/app/main/shutdown.c
+++ b/src/app/main/shutdown.c
@@ -128,7 +128,6 @@ tor_free_all(int postfork)
   networkstatus_free_all();
   addressmap_free_all();
   dirserv_free_all();
-  dirserv_clear_measured_bw_cache();
   rend_cache_free_all();
   rend_service_authorization_free_all();
   rep_hist_free_all();

diff --git a/src/core/include.am b/src/core/include.am
index 63d42642104abff9dc8a2e62c3132a3601629f29..18b6046bfc38fe790627129038554c2055220a68 100644 (file)
--- a/src/core/include.am
+++ b/src/core/include.am
@@ -150,12 +150,6 @@ LIBTOR_APP_A_SOURCES =                             \
        src/feature/stats/rephist.c             \
        src/feature/stats/predict_ports.c
 
-# These should eventually move into module_dirauth_sources, but for now
-# the separation is only in the code location.
-LIBTOR_APP_A_SOURCES +=                        \
-       src/feature/dirauth/bwauth.c
-
-
 if BUILD_NT_SERVICES
 LIBTOR_APP_A_SOURCES += src/app/main/ntmain.c
 endif
@@ -172,6 +166,7 @@ LIBTOR_APP_TESTING_A_SOURCES = $(LIBTOR_APP_A_SOURCES)
 MODULE_DIRAUTH_SOURCES =                                       \
        src/feature/dirauth/authmode.c                          \
         src/feature/dirauth/bridgeauth.c                        \
+       src/feature/dirauth/bwauth.c                            \
        src/feature/dirauth/dirauth_periodic.c                  \
        src/feature/dirauth/dirauth_sys.c                       \
        src/feature/dirauth/dircollate.c                        \

diff --git a/src/feature/dirauth/dirauth_sys.c b/src/feature/dirauth/dirauth_sys.c
index b87fa5dc2973757e09095a7dcfd8e74de75bdd24..f691d5618aa696246b187e46df65756eaffefaa4 100644 (file)
--- a/src/feature/dirauth/dirauth_sys.c
+++ b/src/feature/dirauth/dirauth_sys.c
@@ -6,6 +6,7 @@
 
 #include "core/or/or.h"
 
+#include "feature/dirauth/bwauth.h"
 #include "feature/dirauth/dirauth_sys.h"
 #include "feature/dirauth/dirvote.h"
 #include "feature/dirauth/dirauth_periodic.h"
@@ -25,6 +26,7 @@ subsys_dirauth_shutdown(void)
 {
   dirserv_free_fingerprint_list();
   dirvote_free_all();
+  dirserv_clear_measured_bw_cache();
 }
 
 const struct subsys_fns_t sys_dirauth = {