MDL-55513 core_filestorage: Pass full path of temp dir to remove_dir()

davidknu · Aug 10, 2016 · e98483d · e98483d
1 parent 7752762
commit e98483d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/filestorage/file_storage.php b/lib/filestorage/file_storage.php
@@ -365,7 +365,7 @@ protected function create_converted_document(stored_file $file, $format) {
                 throw new file_exception('storedfileproblem', 'Could not copy file contents to temp file.');
             }
         } catch (file_exception $fe) {
-            remove_dir($uniqdir);
+            remove_dir($tmp);
             throw $fe;
         }
 
@@ -387,7 +387,7 @@ protected function create_converted_document(stored_file $file, $format) {
         $result = exec($cmd, $output);
         chdir($currentdir);
         if (!file_exists($newtmpfile)) {
-            remove_dir($uniqdir);
+            remove_dir($tmp);
             // Cleanup.
             return false;
         }
@@ -404,7 +404,7 @@ protected function create_converted_document(stored_file $file, $format) {
 
         $convertedfile = $this->create_file_from_pathname($record, $newtmpfile);
         // Cleanup.
-        remove_dir($uniqdir);
+        remove_dir($tmp);
         return $convertedfile;
     }