remove some unneeded normalizePath calls

Signed-off-by: Robin Appelman <robin@icewind.nl>
nextcloud · Oct 31, 2023 · e4120e2 · e4120e2
1 parent 970ac3d
commit e4120e2
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 13 deletions.
diff --git a/apps/files_sharing/lib/SharedStorage.php b/apps/files_sharing/lib/SharedStorage.php
@@ -112,7 +112,7 @@ public function __construct($arguments) {
 
 		parent::__construct([
 			'storage' => null,
-			'root' => null,
+			'root' => '',
 		]);
 	}
 

diff --git a/lib/private/Files/Mount/MountPoint.php b/lib/private/Files/Mount/MountPoint.php
@@ -246,7 +246,7 @@ public function getInternalPath($path) {
 	 * @return string
 	 */
 	private function formatPath($path) {
-		$path = Filesystem::normalizePath($path);
+		$path = '/' . trim($path, '/');
 		if (strlen($path) > 1) {
 			$path .= '/';
 		}

diff --git a/lib/private/Files/Node/Node.php b/lib/private/Files/Node/Node.php
@@ -70,8 +70,8 @@ class Node implements INode {
 	 * @param FileInfo $fileInfo
 	 */
 	public function __construct(IRootFolder $root, $view, $path, $fileInfo = null, ?INode $parent = null, bool $infoHasSubMountsIncluded = true) {
-		if (Filesystem::normalizePath($view->getRoot()) !== '/') {
-			throw new PreConditionNotMetException('The view passed to the node should not have any fake root set');
+		if ($view->getRoot() !== '') {
+			throw new PreConditionNotMetException('The view passed to the node should not have any fake root set: ' . $view->getRoot());
 		}
 		$this->view = $view;
 		$this->root = $root;

diff --git a/lib/private/Files/Storage/Wrapper/Jail.php b/lib/private/Files/Storage/Wrapper/Jail.php
@@ -42,9 +42,9 @@
  */
 class Jail extends Wrapper {
 	/**
-	 * @var string
+	 * Root of the jail, without trailing slash
 	 */
-	protected $rootPath;
+	protected string $rootPath;
 
 	/**
 	 * @param array $arguments ['storage' => $storage, 'root' => $root]
@@ -54,11 +54,15 @@ class Jail extends Wrapper {
 	 */
 	public function __construct($arguments) {
 		parent::__construct($arguments);
-		$this->rootPath = $arguments['root'];
+		$this->rootPath = rtrim($arguments['root'], '/');
 	}
 
 	public function getUnjailedPath($path) {
-		return trim(Filesystem::normalizePath($this->rootPath . '/' . $path), '/');
+		if ($this->rootPath === '') {
+			return $path;
+		} else {
+			return $this->rootPath . '/' . trim($path, '/');
+		}
 	}
 
 	/**
@@ -70,9 +74,7 @@ public function getUnjailedStorage() {
 
 
 	public function getJailedPath($path) {
-		$root = rtrim($this->rootPath, '/') . '/';
-
-		if ($path !== $this->rootPath && !str_starts_with($path, $root)) {
+		if ($path !== $this->rootPath && !str_starts_with($path, $this->rootPath . '/')) {
 			return null;
 		} else {
 			$path = substr($path, strlen($this->rootPath));

diff --git a/lib/private/Files/View.php b/lib/private/Files/View.php
@@ -1522,7 +1522,7 @@ public function getDirectoryContent($directory, $mimetype_filter = '', \OCP\File
 							$rootEntry['permissions'] = $permissions & (\OCP\Constants::PERMISSION_ALL - (\OCP\Constants::PERMISSION_UPDATE | \OCP\Constants::PERMISSION_DELETE));
 						}
 
-						$rootEntry['path'] = substr(Filesystem::normalizePath($path . '/' . $rootEntry['name']), strlen($user) + 2); // full path without /$user/
+						$rootEntry['path'] = substr($path . '/' . $rootEntry['name'], strlen($user) + 2); // full path without /$user/
 
 						// if sharing was disabled for the user we remove the share permissions
 						if (\OCP\Util::isSharingDisabledForUser()) {

diff --git a/lib/private/legacy/OC_Helper.php b/lib/private/legacy/OC_Helper.php
@@ -491,7 +491,7 @@ public static function getStorageInfo($path, $rootInfo = null, $includeMountPoin
 		if (!$view) {
 			throw new \OCP\Files\NotFoundException();
 		}
-		$fullPath = Filesystem::normalizePath($view->getAbsolutePath($path));
+		$fullPath = rtrim($view->getAbsolutePath($path), '/');
 
 		$cacheKey = $fullPath. '::' . ($includeMountPoints ? 'include' : 'exclude');
 		if ($useCache) {