From: Vincent Petry <pvince81@owncloud.com>
Date: Fri, 14 Nov 2014 11:45:36 +0000 (+0100)
Subject: Show warning when invalid user was passed
X-Git-Tag: v7.0.4RC1~19
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=f64c6c9c9cb48ce2291c5c613e80794e6130a85d;p=nextcloud-server.git

Show warning when invalid user was passed

Sometimes there are bugs that cause setupFS() to be called for
non-existing users. Instead of failing hard and breaking the instance,
this fix simply logs a warning.

Backport c941c3fa5139ba4f122d4f40d9c9db5e50f8bcb7 from master
---

diff --git a/apps/files_external/lib/config.php b/apps/files_external/lib/config.php
index 9e58b1d54f7..717426c0e01 100644
--- a/apps/files_external/lib/config.php
+++ b/apps/files_external/lib/config.php
@@ -121,6 +121,14 @@ class OC_Mount_Config {
 
 		if ($data['user']) {
 			$user = \OC::$server->getUserManager()->get($data['user']);
+			if (!$user) {
+				\OC_Log::write(
+					'files_external',
+					'Cannot init external mount points for non-existant user "' . $data['user'] . '".',
+					\OC_Log::WARN
+				);
+				return;
+			}
 			$userView = new \OC\Files\View('/' . $user->getUID() . '/files');
 			$changePropagator = new \OC\Files\Cache\ChangePropagator($userView);
 			$etagPropagator = new \OCA\Files_External\EtagPropagator($user, $changePropagator, \OC::$server->getConfig());