BitstreamConverter: Add option to set level 5 metadata to zero offsets

In the past, some users have had issues with media containing
invalid metadata for level 5 active area offsets.

Adds a workaround to override the offsets so that the
displayed image is not cropped incorrectly on some TVs.

Author: quietvoid

xbmc/utils/BitstreamConverter.cpp

@@ -329,6 +329,7 @@ CBitstreamConverter::CBitstreamConverter()
   m_convert_dovi = false;
   m_removeDovi = false;
   m_removeHdr10Plus = false;
+  m_setDoviZeroLevel5 = false;
 }
 
 CBitstreamConverter::~CBitstreamConverter()
@@ -1308,13 +1309,14 @@ bool CBitstreamConverter::mpeg2_sequence_header(const uint8_t *data, const uint3
 #ifdef HAVE_LIBDOVI
 // Processes Dolby Vision RPU
 //   - Converts to profile 8.1 if `m_convert_dovi` is enabled
+//   - Sets level 5 metadata to 0 offsets if `m_setDoviZeroLevel5` is enabled
 //
 // The returned data must be freed with `dovi_data_free`
 // May be NULL if no processing was done or if parsing errored
 const DoviData* CBitstreamConverter::processDoviRpu(uint8_t* buf, uint32_t nalSize)
 {
   // early exit if no processing option is enabled
-  if (!m_convert_dovi)
+  if (!m_convert_dovi && !m_setDoviZeroLevel5)
     return NULL;
 
   DoviRpuOpaque* rpu = dovi_parse_unspec62_nalu(buf, nalSize);
@@ -1330,12 +1332,18 @@ const DoviData* CBitstreamConverter::processDoviRpu(uint8_t* buf, uint32_t nalSi
     return rpuData;
   }
 
-  if (header->guessed_profile == 7)
+  if (m_convert_dovi && header->guessed_profile == 7)
   {
     ret = dovi_convert_rpu_with_mode(rpu, 2);
     processed = true;
   }
 
+  if (ret == 0 && m_setDoviZeroLevel5)
+  {
+    ret = dovi_rpu_set_active_area_offsets(rpu, 0, 0, 0, 0);
+    processed = true;
+  }
+
   if (ret == 0 && processed)
     rpuData = dovi_write_unspec62_nalu(rpu);
 

xbmc/utils/BitstreamConverter.h

@@ -157,4 +157,5 @@ class CBitstreamConverter
   bool m_convert_dovi;
   bool m_removeDovi;
   bool m_removeHdr10Plus;
+  bool m_setDoviZeroLevel5;
 };