Merge pull request #12290 from owncloud/ext-preventbreakageduetobugs2

Show warning when invalid user was passed
owncloud · Nov 19, 2014 · 284ba5b · 284ba5b
2 parents 1c8f956 + c941c3f
commit 284ba5b
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/apps/files_external/lib/config.php b/apps/files_external/lib/config.php
@@ -121,6 +121,14 @@ public static function initMountPointsHook($data) {
 
 		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());