fix: properly catch invalid exit FFmpeg statuses

Bionus · Feb 3, 2024 · bad1f28 · bad1f28
1 parent c1cf658
commit bad1f28
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/lib/src/ffmpeg.cpp b/src/lib/src/ffmpeg.cpp
@@ -167,7 +167,10 @@ bool FFmpeg::execute(const QStringList &params, int msecs)
 	}
 
 	// Wait for FFmpeg to finish
-	bool ok = process.waitForFinished(msecs);
+	bool finishedOk = process.waitForFinished(msecs);
+	bool didntCrash = process.exitStatus() == QProcess::NormalExit;
+	bool exitCodeOk = process.exitCode() == 0;
+	bool ok = finishedOk && didntCrash && exitCodeOk;
 
 	// Print stdout and stderr to the log
 	const QString standardOutput = QString::fromLocal8Bit(process.readAllStandardOutput()).trimmed();