Merge pull request #58676 from nextcloud/jail-wrapper-null-paths

sorbaugh · web-flow · commit ef87e2e5647b · 2026-06-09T17:25:34.000+02:00
fix: don't return cache entries with null paths
diff --git a/lib/private/Files/Cache/Wrapper/CacheJail.php b/lib/private/Files/Cache/Wrapper/CacheJail.php
@@ -102,7 +102,12 @@ protected function getJailedPath(string $path, ?string $root = null) {
 	#[\Override]
 	protected function formatCacheEntry($entry) {
 		if (isset($entry['path'])) {
-			$entry['path'] = $this->getJailedPath($entry['path']);
+			$jailedPath = $this->getJailedPath($entry['path']);
+			if ($jailedPath !== null) {
+				$entry['path'] = $jailedPath;
+			} else {
+				return false;
+			}
 		}
 		return $entry;
 	}

Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,12 @@ protected function getJailedPath(string $path, ?string $root = null) {`
`102`	`102`	`#[\Override]`
`103`	`103`	`protected function formatCacheEntry($entry) {`
`104`	`104`	`if (isset($entry['path'])) {`
`105`		`- $entry['path'] = $this->getJailedPath($entry['path']);`
	`105`	`+ $jailedPath = $this->getJailedPath($entry['path']);`
	`106`	`+ if ($jailedPath !== null) {`
	`107`	`+ $entry['path'] = $jailedPath;`
	`108`	`+ } else {`
	`109`	`+ return false;`
	`110`	`+ }`
`106`	`111`	`}`
`107`	`112`	`return $entry;`
`108`	`113`	`}`