Don't return file handle if the mode supports writing and the file is not writable

Conflicts:
	apps/files_sharing/sharedstorage.php

diff --git a/apps/files_sharing/sharedstorage.php b/apps/files_sharing/sharedstorage.php
index 8b1933828386a37b48bad2b7520b9f4b7454fd2e..a2dfe34402f1c5542ef0a97c3b92f1fb40da09ec 100644 (file)
--- a/apps/files_sharing/sharedstorage.php
+++ b/apps/files_sharing/sharedstorage.php
@@ -416,6 +416,25 @@ class OC_Filestorage_Shared extends OC_Filestorage {
        public function fopen($path, $mode) {
                $source = $this->getSource($path);
                if ($source) {
+                       switch ($mode) {
+                               case 'r+':
+                               case 'rb+':
+                               case 'w+':
+                               case 'wb+':
+                               case 'x+':
+                               case 'xb+':
+                               case 'a+':
+                               case 'ab+':
+                               case 'w':
+                               case 'wb':
+                               case 'x':
+                               case 'xb':
+                               case 'a':
+                               case 'ab':
+                                       if (!$this->is_writable($path)) {
+                                               return false;
+                                       }
+                       }
                        $storage = OC_Filesystem::getStorage($source);
                        return $storage->fopen($this->getInternalPath($source), $mode);
                }