From: moodler <moodler>
Date: Mon, 20 Jan 2003 14:04:33 +0000 (+0000)
Subject: Use $CFG->directorypermissions for new directories
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=17654e3845f1245287f9d8ac97a1504cd1282b74;p=moodle.git

Use $CFG->directorypermissions for new directories
---

diff --git a/user/lib.php b/user/lib.php
index 1a55dafd52..4f25cc7c26 100644
--- a/user/lib.php
+++ b/user/lib.php
@@ -63,13 +63,13 @@ function save_user_image($userid, $filename) {
     umask(0000);
 
     if (!file_exists("$CFG->dataroot/users")) {
-        if (! mkdir("$CFG->dataroot/users", 0777)) {
+        if (! mkdir("$CFG->dataroot/users", $CFG->directorypermissions)) {
             return false;
         }
     }
 
     if (!file_exists("$CFG->dataroot/users/$userid")) {
-        if (! mkdir("$CFG->dataroot/users/$userid", 0777)) {
+        if (! mkdir("$CFG->dataroot/users/$userid", $CFG->directorypermissions)) {
             return false;
         }
     }
@@ -80,9 +80,15 @@ function save_user_image($userid, $filename) {
         return false;
     }
 
-    chmod($originalfile, 0777);
+    chmod($originalfile, 0666);
 
     $imageinfo = GetImageSize($originalfile);
+    
+    if (empty($imageinfo)) {
+        unlink($originalfile);
+        return false;
+    }
+    
     $image->width  = $imageinfo[0];
     $image->height = $imageinfo[1];
     $image->type   = $imageinfo[2];