MAGETWO-36791: Fixed too broad permissions on files

-address CR comments

Author: Joan He

app/code/Magento/Catalog/Model/Product/Attribute/Backend/Media.php

@@ -397,7 +397,7 @@ public function addImage(
                 $this->_mediaDirectory->copyFile($file, $destinationFile);
 
                 $storageHelper->saveFile($this->_mediaConfig->getTmpMediaShortUrl($fileName));
-                $this->_mediaDirectory->changePermissions($destinationFile, DriverInterface::WRITEABLE_DIRECTORY_MODE);
+                $this->_mediaDirectory->changePermissions($destinationFile, DriverInterface::WRITEABLE_FILE_MODE);
             }
         } catch (\Exception $e) {
             throw new LocalizedException(__('We couldn\'t move this file: %1.', $e->getMessage()));

lib/internal/Magento/Framework/Filesystem/Io/Sftp.php

@@ -7,6 +7,7 @@
 // @codingStandardsIgnoreFile
 
 namespace Magento\Framework\Filesystem\Io;
+
 use Magento\Framework\Filesystem\DriverInterface;
 
 /**

pub/errors/processor.php

@@ -458,7 +458,7 @@ public function saveReport($reportData)
         }
 
         @file_put_contents($this->_reportFile, serialize($reportData));
-        @chmod($this->_reportFile, DriverInterface::WRITEABLE_DIRECTORY_MODE);
+        @chmod($this->_reportFile, DriverInterface::WRITEABLE_FILE_MODE);
 
         if (isset($reportData['skin']) && self::DEFAULT_SKIN != $reportData['skin']) {
             $this->_setSkin($reportData['skin']);