diff options
| author | Nick Brereton <nick@nbrereton.net> | 2010-07-27 21:44:09 +0000 |
|---|---|---|
| committer | Benjamin Larsson <banan@ludd.ltu.se> | 2010-07-27 21:44:09 +0000 |
| commit | 08634e7bda2c428d1403e07bd02d5c98c48d2981 (patch) | |
| tree | 78e1b85e621947ed90d75bcb7f350ca90fdd348b /libavcodec | |
| parent | af0554e5306ba163fcaa15e148cdad5f9781fd50 (diff) | |
| download | ffmpeg-08634e7bda2c428d1403e07bd02d5c98c48d2981.tar.gz | |
Setup correct channel value when downmixing is required.
Patch by Nick Brereton
Originally committed as revision 24554 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'libavcodec')
| -rw-r--r-- | libavcodec/dca.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/libavcodec/dca.c b/libavcodec/dca.c index 77299dfcb3..ff44ec78dd 100644 --- a/libavcodec/dca.c +++ b/libavcodec/dca.c @@ -1348,7 +1348,7 @@ static int dca_decode_frame(AVCodecContext * avctx, avctx->channel_layout = dca_core_channel_layout[s->amode]; if (s->xch_present && (!avctx->request_channels || - avctx->request_channels > num_core_channels)) { + avctx->request_channels > num_core_channels + !!s->lfe)) { avctx->channel_layout |= CH_BACK_CENTER; if (s->lfe) { avctx->channel_layout |= CH_LOW_FREQUENCY; @@ -1357,6 +1357,8 @@ static int dca_decode_frame(AVCodecContext * avctx, s->channel_order_tab = dca_channel_reorder_nolfe_xch[s->amode]; } } else { + channels = num_core_channels + !!s->lfe; + s->xch_present = 0; /* disable further xch processing */ if (s->lfe) { avctx->channel_layout |= CH_LOW_FREQUENCY; s->channel_order_tab = dca_channel_reorder_lfe[s->amode]; @@ -1364,8 +1366,8 @@ static int dca_decode_frame(AVCodecContext * avctx, s->channel_order_tab = dca_channel_reorder_nolfe[s->amode]; } - if (s->prim_channels > 0 && - s->channel_order_tab[s->prim_channels - 1] < 0) + if (channels > !!s->lfe && + s->channel_order_tab[channels - 1 - !!s->lfe] < 0) return -1; if (avctx->request_channels == 2 && s->prim_channels > 2) { @@ -1384,8 +1386,7 @@ static int dca_decode_frame(AVCodecContext * avctx, unset. Ideally during the first probe for channels the crc should be checked and only set avctx->channels when the crc is ok. Right now the decoder could set the channels based on a broken first frame.*/ - if (!avctx->channels) - avctx->channels = channels; + avctx->channels = channels; if (*data_size < (s->sample_blocks / 8) * 256 * sizeof(int16_t) * channels) return -1; |