Check whether an app archive can be extracted

If extraction fails we should not continue the installation/update
process as the info.xml cannot be loaded and an unrelated error
occurs.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

1 files changed, 8 insertions, 1 deletions

diff --git a/lib/private/Installer.php b/lib/private/Installer.php
index 4d7e5016653..d5082a7fad2 100644
--- a/lib/private/Installer.php
+++ b/lib/private/Installer.php
@@ -288,7 +288,14 @@ class Installer {
 					$archive = new TAR($tempFile);
 
 					if($archive) {
-						$archive->extract($extractDir);
+						if (!$archive->extract($extractDir)) {
+							throw new \Exception(
+								sprintf(
+									'Could not extract app %s',
+									$appId
+								)
+							);
+						}
 						$allFiles = scandir($extractDir);
 						$folders = array_diff($allFiles, ['.', '..']);
 						$folders = array_values($folders);