[FFmpeg-cvslog] aac: Wait to know the channels before allocating frame
Luca Barbato
git at videolan.org
Mon Jun 8 02:42:18 CEST 2015
ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Thu Jun 4 10:19:50 2015 +0200| [bc76c46943272515805d7ac48ca39f14826d1fed] | committer: Luca Barbato
aac: Wait to know the channels before allocating frame
The channel configuration can be delivered only by the PCE,
try to parse it first and not try to decode until a channel
configuration is set.
CC: libav-stable at libav.org
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=bc76c46943272515805d7ac48ca39f14826d1fed
---
libavcodec/aacdec.c | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c
index 9d14357..683aee2 100644
--- a/libavcodec/aacdec.c
+++ b/libavcodec/aacdec.c
@@ -2821,8 +2821,9 @@ static int aac_decode_frame_int(AVCodecContext *avctx, void *data,
}
}
- if ((err = frame_configure_elements(avctx)) < 0)
- goto fail;
+ if (avctx->channels)
+ if ((err = frame_configure_elements(avctx)) < 0)
+ goto fail;
// The FF_PROFILE_AAC_* defines are all object_type - 1
// This may lead to an undefined profile being signaled
@@ -2833,6 +2834,9 @@ static int aac_decode_frame_int(AVCodecContext *avctx, void *data,
while ((elem_type = get_bits(gb, 3)) != TYPE_END) {
elem_id = get_bits(gb, 4);
+ if (!avctx->channels && elem_type != TYPE_PCE)
+ goto fail;
+
if (elem_type < TYPE_DSE) {
if (!(che=get_che(ac, elem_type, elem_id))) {
av_log(ac->avctx, AV_LOG_ERROR, "channel element %d.%d is not allocated\n",
@@ -2919,6 +2923,11 @@ static int aac_decode_frame_int(AVCodecContext *avctx, void *data,
}
}
+ if (!avctx->channels) {
+ *got_frame_ptr = 0;
+ return 0;
+ }
+
spectral_to_sample(ac);
multiplier = (ac->oc[1].m4ac.sbr == 1) ? ac->oc[1].m4ac.ext_sample_rate > ac->oc[1].m4ac.sample_rate : 0;
More information about the ffmpeg-cvslog
mailing list