From: Lukas Reschke <lukas@owncloud.com>
Date: Wed, 12 Nov 2014 14:56:02 +0000 (+0100)
Subject: Try to read the file only instead of trying to touch
X-Git-Tag: v7.0.4RC1~14
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d815638f7b086fad627dbbd22c10a068acfcae6c;p=nextcloud-server.git

Try to read the file only instead of trying to touch

The permissions are already catched properly on the installation so we just have to check whether the file is readable to prevent fatal errors from happening.

Fixes https://github.com/owncloud/core/issues/12135

Conflicts:
	lib/private/config.php
---

diff --git a/lib/private/config.php b/lib/private/config.php
index 7bf3863e2a6..cc07d6a1ed1 100644
--- a/lib/private/config.php
+++ b/lib/private/config.php
@@ -138,12 +138,12 @@ class Config {
 
 		// Include file and merge config
 		foreach ($configFiles as $file) {
-			if(!@touch($file) && $file === $this->configFilePath) {
-				// Writing to the main config might not be possible, e.g. if the wrong
+			$filePointer = @fopen($file, 'r');
+			if($file === $this->configFilePath && $filePointer === false) {
+				// Opening the main config might not be possible, e.g. if the wrong
 				// permissions are set (likely on a new installation)
 				continue;
 			}
-			$filePointer = fopen($file, 'r');
 
 			// Try to acquire a file lock
 			if(!flock($filePointer, LOCK_SH)) {