Add ‘ignore’ for UTF8 decoding to avoid error on non-UTF8 characters (#…

…959)
Zulko · Aug 6, 2020 · 953357d · 953357d
1 parent b65d293
commit 953357d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -25,6 +25,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed <!-- for any bug fixes -->
 - `OSError: MoviePy error: failed to read the first frame of video file...` would occasionally occur for no reason [#1220]
 - Warnings are no longer being supressed by MoviePy [#1191]
+- Fixed `UnicodeDecodeError` crash when file metadata contained non-UTF8 characters [#959]
 
 
 ## [v2.0.0.dev1](https://github.com/zulko/moviepy/tree/v2.0.0.dev1) (2020-06-04)

diff --git a/moviepy/video/io/ffmpeg_reader.py b/moviepy/video/io/ffmpeg_reader.py
@@ -285,7 +285,7 @@ def ffmpeg_parse_infos(
 
     proc = sp.Popen(cmd, **popen_params)
     (output, error) = proc.communicate()
-    infos = error.decode("utf8")
+    infos = error.decode("utf8", errors="ignore")
 
     proc.terminate()
     del proc