omxaacdec: Do not accept unknown layouts
authorThiago Santos <thiagoss@osg.samsung.com>
Tue, 1 Sep 2015 19:08:11 +0000 (16:08 -0300)
committerThiago Santos <thiagoss@osg.samsung.com>
Tue, 1 Sep 2015 19:13:00 +0000 (16:13 -0300)
It was defaulting to RAW when an unknown layout was received but
the caps template would actually forbid that on the caps query
or accept-caps anyway.

omx/gstomxaacdec.c

index 6ecf9d5..66d881d 100644 (file)
@@ -148,8 +148,10 @@ gst_omx_aac_dec_set_format (GstOMXAudioDec * dec, GstOMXPort * port,
     aac_param.eAACStreamFormat = OMX_AUDIO_AACStreamFormatADIF;
   else if (strcmp (stream_format, "raw") == 0)
     aac_param.eAACStreamFormat = OMX_AUDIO_AACStreamFormatRAW;
-  else                          /* fallback instead of failing */
-    aac_param.eAACStreamFormat = OMX_AUDIO_AACStreamFormatRAW;
+  else {
+    GST_ERROR_OBJECT (self, "Unexpected format: %s", stream_format);
+    return FALSE;
+  }
 
   err =
       gst_omx_component_set_parameter (dec->dec, OMX_IndexParamAudioAac,