aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMorris Jobke <hey@morrisjobke.de>2016-09-30 14:57:54 +0200
committerLukas Reschke <lukas@statuscode.ch>2016-10-03 14:00:57 +0200
commit5dacd51c144f6262b9db583b096f5ec82c793081 (patch)
treefb917d525ade636ef9ffad11068b49278ed2cb1c
parenta62f6fac516de2c06a2df6eadd2f086f545c9c4e (diff)
downloadnextcloud-server-5dacd51c144f6262b9db583b096f5ec82c793081.tar.gz
nextcloud-server-5dacd51c144f6262b9db583b096f5ec82c793081.zip
Use proper casing of username in avatar filesystem setup
* before you could request an avatar for User instead of user which sets up the filesystem for that user twice causing the sharing codes collision detection to detect a lot of collisions Signed-off-by: Morris Jobke <hey@morrisjobke.de>
-rw-r--r--lib/private/AvatarManager.php2
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/private/AvatarManager.php b/lib/private/AvatarManager.php
index 0eabc3a1754..6f1f380a903 100644
--- a/lib/private/AvatarManager.php
+++ b/lib/private/AvatarManager.php
@@ -91,6 +91,8 @@ class AvatarManager implements IAvatarManager {
if (is_null($user)) {
throw new \Exception('user does not exist');
}
+ // sanitize userID - fixes casing issue (needed for the filesystem stuff that is done below)
+ $userId = $user->getUID();
/*
* Fix for #22119