11291: Fix warnings, add changes file, rename 'mask'.

diff --git a/changes/ticket-11291 b/changes/ticket-11291
new file mode 100644 (file)
index 0000000..4c19f3c
--- /dev/null
+++ b/changes/ticket-11291
@@ -0,0 +1,4 @@
+  o Minor features (hidden services):
+    - New HiddenServiceDirGroupReadable option to cause hidden service
+      directories and hostname files to be created group-readable.
+      Patch from "anon", David Stainton, and "meejah".

diff --git a/src/common/util.c b/src/common/util.c
index b616d1f389173b25c80d5f8f74849db92cb1bb05..50097dac93a9a4d8d08caaa57fbed397e0642f5a 100644 (file)
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -2012,8 +2012,7 @@ check_private_dir(const char *dirname, cpd_check_t check,
   struct stat st;
   char *f;
 #ifndef _WIN32
-  int mask = 0;
-  int perm = 0;
+  unsigned unwanted_bits = 0;
   const struct passwd *pw = NULL;
   uid_t running_uid;
   gid_t running_gid;
@@ -2112,11 +2111,11 @@ check_private_dir(const char *dirname, cpd_check_t check,
     return -1;
   }
   if (check & (CPD_GROUP_OK|CPD_GROUP_READ)) {
-    mask = 0027;
+    unwanted_bits = 0027;
   } else {
-    mask = 0077;
+    unwanted_bits = 0077;
   }
-  if (st.st_mode & mask) {
+  if ((st.st_mode & unwanted_bits) != 0) {
     unsigned new_mode;
     if (check & CPD_CHECK_MODE_ONLY) {
       log_warn(LD_FS, "Permissions on directory %s are too permissive.",
@@ -2129,7 +2128,7 @@ check_private_dir(const char *dirname, cpd_check_t check,
     if (check & CPD_GROUP_READ) {
       new_mode |= 0050; /* Group should have rx */
     }
-    new_mode &= ~mask; /* Clear the other bits that we didn't want set...*/
+    new_mode &= ~unwanted_bits; /* Clear the bits that we didn't want set...*/
     if (chmod(dirname, new_mode)) {
       log_warn(LD_FS, "Could not chmod directory %s: %s", dirname,
                strerror(errno));

diff --git a/src/test/test_checkdir.c b/src/test/test_checkdir.c
index 7bf735e06138101306ab06c204f30ab2e7030b85..409ba8000485699e6f4ffdfd5ba9dbd0cbeb1d89 100644 (file)
--- a/src/test/test_checkdir.c
+++ b/src/test/test_checkdir.c
@@ -12,6 +12,7 @@
 static void
 test_checkdir_perms(void *testdata)
 {
+  (void)testdata;
   or_options_t *options = get_options_mutable();
   const char *subdir = "test_checkdir";
   char *testdir;