mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #23295 from nextcloud/fix/versions-no-space-no-versions-undefined-index 

Fix undefined index and consequential damages in versions code
This commit is contained in:
Morris Jobke 2020-10-13 17:35:35 +02:00 committed by GitHub
parent 5111d6a3e2 6ce05bc7d7
commit f8ed08ca2a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
apps/files_versions/lib/Storage.php
View File

@ -496,7 +496,7 @@ class Storage {
$expiration = self::getExpiration();
$threshold = $expiration->getMaxAgeAsTimestamp();
$versions = self::getAllVersions($uid);
if (!$threshold || !array_key_exists('all', $versions)) {
if (!$threshold || empty($versions['all'])) {
return;
}
@ -578,7 +578,10 @@ class Storage {
// newest version first
krsort($versions);
$result = [];
$result = [
'all' => [],
'by_file' => [],
];
foreach ($versions as $key => $value) {
$size = $view->filesize(self::VERSIONS_ROOT.'/'.$value['path'].'.v'.$value['timestamp']);
@ -775,7 +778,7 @@ class Storage {
// if still not enough free space we rearrange the versions from all files
if ($availableSpace <= 0) {
$result = Storage::getAllVersions($uid);
$result = self::getAllVersions($uid);
$allVersions = $result['all'];
foreach ($result['by_file'] as $versions) {