From 2e6d42735f3c343bec61a583802429e1ad352a29 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Mon, 20 Jan 2003 14:03:11 +0000
Subject: [PATCH] Added new configuration value $CFG->directorypermissions to
 set the permissions for new directories created in dataroot

---
 lib/moodlelib.php | 8 ++++----
 lib/setup.php     | 7 +++++++
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/lib/moodlelib.php b/lib/moodlelib.php
index e71c25468b..143d22343d 100644
--- a/lib/moodlelib.php
+++ b/lib/moodlelib.php
@@ -752,12 +752,13 @@ function make_upload_directory($directory) {
 
     $currdir = $CFG->dataroot;
 
+    umask(0000);
+
     if (!file_exists($currdir)) {
-        if (! mkdir($currdir, 0750)) {
+        if (! mkdir($currdir, $CFG->directorypermissions)) {
             notify("ERROR: You need to create the directory $currdir with web server write access");
             return false;
         }
-        chmod($currdir,0750);
     }
 
     $dirarray = explode("/", $directory);
@@ -765,11 +766,10 @@ function make_upload_directory($directory) {
     foreach ($dirarray as $dir) {
         $currdir = "$currdir/$dir";
         if (! file_exists($currdir)) {
-            if (! mkdir($currdir, 0750)) {
+            if (! mkdir($currdir, $CFG->directorypermissions)) {
                 notify("ERROR: Could not find or create a directory ($currdir)");
                 return false;
             }
-            chmod($currdir,0750);
         }
     }
 
diff --git a/lib/setup.php b/lib/setup.php
index 17cecc9b99..0c84d01bcb 100644
--- a/lib/setup.php
+++ b/lib/setup.php
@@ -60,6 +60,13 @@
     error_reporting($CFG->debug);   
 
 
+/// File permissions on created directories in the $CFG->dataroot
+
+    if (empty($CFG->directorypermissions)) {
+        $CFG->directorypermissions = 0777;      // Must be octal (that's why it's here)
+    }
+
+
 /// Location of standard files
 
     $CFG->wordlist    = "$CFG->libdir/wordlist.txt";
-- 
2.39.5