fix: unbloc ffmpeg on some broken videos

Co-authored-by: Joas Schilling <[email protected]>

Signed-off-by: Arthur Schiwon <[email protected]>

Author: blizzz

.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
 

lib/private/Preview/Movie.php

@@ -135,8 +135,8 @@ private function generateThumbNail(int $maxX, int $maxY, string $absPath, int $s
 		$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;
 		}

tests/data/broken-video.webm

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+/**
+ * SPDX-FileCopyrightText: 2025 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+namespace Test\Preview;
+
+/**
+ * Class MovieTest
+ *
+ * @group DB
+ *
+ * @package Test\Preview
+ */
+class MovieBrokenStuckFfmpegTest extends MovieTest {
+	protected string $fileName = 'broken-video.webm';
+}

tests/lib/Preview/MovieBrokenStuckFfmpegTest.php

@@ -15,6 +15,10 @@
  * @package Test\Preview
  */
 class MovieTest extends Provider {
+	protected string $fileName = 'testimage.mp4';
+	protected int $width = 560;
+	protected int $height = 320;
+
 	protected function setUp(): void {
 		$avconvBinary = \OC_Helper::findBinaryPath('avconv');
 		$ffmpegBinary = ($avconvBinary) ? null : \OC_Helper::findBinaryPath('ffmpeg');
@@ -25,10 +29,7 @@ protected function setUp(): void {
 			\OC\Preview\Movie::$avconvBinary = $avconvBinary;
 			\OC\Preview\Movie::$ffmpegBinary = $ffmpegBinary;
 
-			$fileName = 'testimage.mp4';
-			$this->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');