matroska-demux: make up an OpusHead block if possible when missing
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Thu, 4 Feb 2016 15:59:04 +0000 (15:59 +0000)
committerSebastian Dröge <sebastian@centricular.com>
Tue, 23 Feb 2016 08:47:43 +0000 (10:47 +0200)
https://bugzilla.gnome.org/show_bug.cgi?id=761489

gst/matroska/matroska-demux.c

index 96700494a64307bda6f5d28fea1eac65e0cd0074..f176ce0c66497e6da48b1a41110063f0452af1df 100644 (file)
@@ -5614,8 +5614,37 @@ gst_matroska_demux_audio_caps (GstMatroskaTrackAudioContext *
       caps = gst_codec_utils_opus_create_caps_from_header (tmp, NULL);
       gst_buffer_unref (tmp);
       *codec_name = g_strdup ("Opus");
+    } else if (context->codec_priv_size == 0) {
+      GST_WARNING ("No Opus codec data found, trying to create one");
+      if (audiocontext->channels >= 0 && audiocontext->channels <= 2) {
+        guint8 streams, coupled, channels;
+        guint32 samplerate;
+
+        samplerate =
+            audiocontext->samplerate == 0 ? 48000 : audiocontext->samplerate;
+        channels = audiocontext->channels == 0 ? 2 : audiocontext->channels;
+        if (channels == 1) {
+          streams = 1;
+          coupled = 0;
+        } else {
+          streams = 1;
+          coupled = 1;
+        }
+
+        caps =
+            gst_codec_utils_opus_create_caps (samplerate, channels, 0, streams,
+            coupled, NULL);
+        if (caps) {
+          *codec_name = g_strdup ("Opus");
+        } else {
+          GST_WARNING ("Failed to create Opus caps from audio context");
+        }
+      } else {
+        GST_WARNING ("No Opus codec data, and not enough info to create one");
+      }
     } else {
-      GST_WARNING ("Invalid Opus codec data size");
+      GST_WARNING ("Invalid Opus codec data size (got %" G_GSIZE_FORMAT
+          ", expected 19)", context->codec_priv_size);
     }
   } else if (!strcmp (codec_id, GST_MATROSKA_CODEC_ID_AUDIO_ACM)) {
     gst_riff_strf_auds auds;