config: Give warn_about_relative_paths() const args

Also gives make_path_absolute() const args.

Obviously correct fixes to already reviewed code.

diff --git a/src/app/config/config.c b/src/app/config/config.c
index 14376e56ae74973d62d25dac5721451d2c22c948..9b75f96a83f73032cfea922d3479ee37e8fc2ed1 100644 (file)
--- a/src/app/config/config.c
+++ b/src/app/config/config.c
@@ -3303,7 +3303,7 @@ options_validate(const or_options_t *old_options, or_options_t *options,
  */
 static int
 warn_if_option_path_is_relative(const char *option,
-                                char *filepath)
+                                const char *filepath)
 {
   if (filepath && path_is_relative(filepath)) {
     char *abs_path = make_path_absolute(filepath);
@@ -3321,7 +3321,7 @@ warn_if_option_path_is_relative(const char *option,
  * Return 1 if there were relative paths; 0 otherwise.
  */
 static int
-warn_about_relative_paths(or_options_t *options)
+warn_about_relative_paths(const or_options_t *options)
 {
   tor_assert(options);
   int n = 0;

diff --git a/src/lib/fs/path.c b/src/lib/fs/path.c
index 1b4bc74e960dd76e0fd0c38b33afd360ec7ee0fd..9f297d98e8afa559fa746cf86464d5421ea074ab 100644 (file)
--- a/src/lib/fs/path.c
+++ b/src/lib/fs/path.c
@@ -258,7 +258,7 @@ alloc_getcwd(void)
  * Return a newly allocated string, which may be a duplicate of <b>fname</b>.
  */
 char *
-make_path_absolute(char *fname)
+make_path_absolute(const char *fname)
 {
 #ifdef _WIN32
   char *absfname_malloced = _fullpath(NULL, fname, 1);

diff --git a/src/lib/fs/path.h b/src/lib/fs/path.h
index 28a1838b8846a02a3d9c2e9de5870e6b837be1b0..0c2a57494171381018333b1aedcb655a074c091b 100644 (file)
--- a/src/lib/fs/path.h
+++ b/src/lib/fs/path.h
@@ -25,6 +25,6 @@ char *expand_filename(const char *filename);
 int path_is_relative(const char *filename);
 void clean_fname_for_stat(char *name);
 int get_parent_directory(char *fname);
-char *make_path_absolute(char *fname);
+char *make_path_absolute(const char *fname);
 
 #endif /* !defined(TOR_PATH_H) */