fix potential crash when trying to use the document session's user wh…

…en the session is not found Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
nextcloud · Jan 27, 2023 · 6640ad2 · 6640ad2
1 parent 1118c4a
commit 6640ad2
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/lib/Controller/SessionController.php b/lib/Controller/SessionController.php
@@ -113,9 +113,11 @@ public function mention(int $documentId, int $sessionId, string $sessionToken, s
 
 	private function loginSessionUser(int $documentId, int $sessionId, string $sessionToken) {
 		$currentSession = $this->sessionService->getSession($documentId, $sessionId, $sessionToken);
-		$user = $this->userManager->get($currentSession->getUserId());
-		if ($user !== null) {
-			$this->userSession->setUser($user);
+		if ($currentSession !== false) {
+			$user = $this->userManager->get($currentSession->getUserId());
+			if ($user !== null) {
+				$this->userSession->setUser($user);
+			}
 		}
 	}
 }