Merge commit 'sebastian/bug1843'

2 files changed, 11 insertions, 0 deletions

diff --git a/changes/bug1843 b/changes/bug1843
new file mode 100644
index 000000000..f44054ee2
--- /dev/null
+++ b/changes/bug1843
@@ -0,0 +1,4 @@
+  o Minor features:
+    - Warn when CookieAuthFileGroupReadable is set but CookieAuthFile is
+      not. This would lead to a cookie that is still not group readable.
+      Closes bug 1843. Suggested by katmagic.
diff --git a/src/or/config.c b/src/or/config.c
index 9d98d1545..7ad272f74 100644
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -3501,6 +3501,13 @@ options_validate(or_options_t *old_options, or_options_t *options,
              "upgrade your Tor controller as soon as possible.");
   }
 
+  if (options->CookieAuthFileGroupReadable && !options->CookieAuthFile) {
+    log_warn(LD_CONFIG, "You set the CookieAuthFileGroupReadable but did "
+             "not configure a the path for the cookie file via "
+             "CookieAuthFile. This means your cookie will not be group "
+             "readable.");
+  }
+
   if (options->UseEntryGuards && ! options->NumEntryGuards)
     REJECT("Cannot enable UseEntryGuards with NumEntryGuards set to 0");