diff --git a/.reuse/dep5 b/.reuse/dep5 index 4320fe5bd01..e47bb6c42cb 100644 --- a/.reuse/dep5 +++ b/.reuse/dep5 @@ -107,7 +107,7 @@ Files: tests/data/integritycheck/htaccessWithValidModifiedContent/.htaccess Copyright: 2016 ownCloud, Inc., 2019 Nextcloud GmbH and Nextcloud contributors License: AGPL-3.0-only -Files: core/img/favicon*.* core/img/logo/logo*.* tests/data/testimage.webp apps/testing/img/logo.png core/img/apps/spreed.svg +Files: core/img/favicon*.* core/img/logo/logo*.* tests/data/testimage.webp tests/data/broken-video.webm apps/testing/img/logo.png core/img/apps/spreed.svg Copyright: 2016-2024 Nextcloud GmbH License: LicenseRef-NextcloudTrademarks diff --git a/lib/private/Preview/Movie.php b/lib/private/Preview/Movie.php index 7b7d6fe40e8..e52aa392c20 100644 --- a/lib/private/Preview/Movie.php +++ b/lib/private/Preview/Movie.php @@ -122,8 +122,8 @@ class Movie extends ProviderV2 { $returnCode = -1; $output = ''; if (is_resource($proc)) { - $stdout = trim(stream_get_contents($pipes[1])); $stderr = trim(stream_get_contents($pipes[2])); + $stdout = trim(stream_get_contents($pipes[1])); $returnCode = proc_close($proc); $output = $stdout . $stderr; } diff --git a/tests/data/broken-video.webm b/tests/data/broken-video.webm new file mode 100644 index 00000000000..0cafb9314f8 Binary files /dev/null and b/tests/data/broken-video.webm differ diff --git a/tests/lib/Preview/MovieBrokenStuckFfmpegTest.php b/tests/lib/Preview/MovieBrokenStuckFfmpegTest.php new file mode 100644 index 00000000000..e66d5e64649 --- /dev/null +++ b/tests/lib/Preview/MovieBrokenStuckFfmpegTest.php @@ -0,0 +1,20 @@ +imgPath = $this->prepareTestFile($fileName, \OC::$SERVERROOT . '/tests/data/' . $fileName); - $this->width = 560; - $this->height = 320; + $this->imgPath = $this->prepareTestFile($this->fileName, \OC::$SERVERROOT . '/tests/data/' . $this->fileName); $this->provider = new \OC\Preview\Movie; } else { $this->markTestSkipped('No Movie provider present');