summaryrefslogtreecommitdiff
path: root/src/lib
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2021-02-21 20:27:58 +0100
committerCarl Hetherington <cth@carlh.net>2021-02-23 20:09:39 +0100
commitd756e14c0bb2c2264248b26d59c363b6bc33a7fc (patch)
tree9b4033271e04e13db24674b48597556b20c89f27 /src/lib
parentf5fd225decd41e5716fcc5aaf972c3b05d114e0d (diff)
Don't write active picture area with zero width or height as it makes verification fail.
Diffstat (limited to 'src/lib')
-rw-r--r--src/lib/writer.cc7
1 files changed, 6 insertions, 1 deletions
diff --git a/src/lib/writer.cc b/src/lib/writer.cc
index ddb95151c..9ebdd92a3 100644
--- a/src/lib/writer.cc
+++ b/src/lib/writer.cc
@@ -642,7 +642,12 @@ Writer::finish (boost::filesystem::path output_dcp)
cpl->set_main_sound_configuration (msc.to_string());
cpl->set_main_sound_sample_rate (film()->audio_frame_rate());
cpl->set_main_picture_stored_area (film()->frame_size());
- cpl->set_main_picture_active_area (film()->active_area());
+
+ auto active_area = film()->active_area();
+ if (active_area.width > 0 && active_area.height > 0) {
+ /* It's not allowed to have a zero active area width or height */
+ cpl->set_main_picture_active_area (active_area);
+ }
vector<dcp::LanguageTag> sl = film()->subtitle_languages();
if (sl.size() > 1) {