From c941c3fa5139ba4f122d4f40d9c9db5e50f8bcb7 Mon Sep 17 00:00:00 2001 From: Vincent Petry Date: Fri, 14 Nov 2014 12:45:36 +0100 Subject: [PATCH] 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. --- apps/files_external/lib/config.php | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/apps/files_external/lib/config.php b/apps/files_external/lib/config.php index fa44e446d9..9400bbdedc 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());