From 015232bd394367b8422b69222847e622a45b3714 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Tue, 24 Aug 2004 20:48:22 +0000 Subject: As far as I can tell, CONFIG_LEGAL_FILENAME_CHARACTERS is both pointless and broken. #if it out, pending agreement from arma. This fixes a bug on win32 that rejected paths with a : in them. svn:r2309 --- src/or/config.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'src/or') diff --git a/src/or/config.c b/src/or/config.c index 153385861..520725497 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -28,14 +28,17 @@ typedef enum config_type_t { /** Largest allowed config line */ #define CONFIG_LINE_T_MAXLEN 4096 +#if 0 static FILE *config_open(const unsigned char *filename); static int config_close(FILE *f); +#endif static struct config_line_t *config_get_commandlines(int argc, char **argv); static struct config_line_t *config_get_lines(FILE *f); static void config_free_lines(struct config_line_t *front); static int config_compare(struct config_line_t *c, const char *key, config_type_t type, void *arg); static int config_assign(or_options_t *options, struct config_line_t *list); +#if 0 /** Open a configuration file for reading */ static FILE *config_open(const unsigned char *filename) { tor_assert(filename); @@ -51,6 +54,7 @@ static int config_close(FILE *f) { tor_assert(f); return fclose(f); } +#endif /** Helper: Read a list of configuration options from the command line. */ static struct config_line_t *config_get_commandlines(int argc, char **argv) { @@ -670,9 +674,10 @@ int getconfig(int argc, char **argv, or_options_t *options) { } } } + tor_assert(fname); log(LOG_DEBUG,"Opening config file '%s'",fname); - cf = config_open(fname); + cf = fopen(fname, "r"); if(!cf) { if(using_default_torrc == 1) { log(LOG_NOTICE, "Configuration file '%s' not present, using reasonable defaults.",fname); @@ -692,7 +697,7 @@ int getconfig(int argc, char **argv, or_options_t *options) { if(config_assign(options,cl) < 0) return -1; config_free_lines(cl); - config_close(cf); + fclose(cf); } /* go through command-line variables too */ -- cgit v1.2.3