[FFmpeg-devel] [PATCH] aac: fix alignment

Michael Niedermayer michaelni at gmx.at
Thu Mar 21 05:16:16 CET 2013


On Wed, Mar 20, 2013 at 11:16:03PM -0300, Nicolás Dato wrote:
> Hello,
> I was having the problem described in ticket #1694. The ticket is
> fixed, but the problem decoding certain frames is still there (channel
> element #.# is not allocated...)
> So I found the problem, and made a patch, now it can decode all the
> audio frames. The problem was when the decode_pce() needed to align
> the bits, it was calculating the alignment from a wrong position; and
> a second problem was that it was changing always the configuration
> when decoding the PCE but it only has to do it when the chan_config is
> 0.
> 
> I wish I was in time to include this patch on the next 1.0.6 release
> 
> --
> 
> diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c
> index 37c7de5..133d6fb 100644
> --- a/libavcodec/aacdec.c
> +++ b/libavcodec/aacdec.c
> @@ -2506,6 +2506,7 @@ static int aac_decode_frame_int(AVCodecContext
> *avctx, void *data,
>          goto fail;

this patch is mangled/has newlines in the middle and cant be applied
with git
also it breaks playback of "Music\ Station\ Super\ Live\ -\ 2011.12.23.mkv"

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

DNS cache poisoning attacks, popular search engine, Google internet authority
dont be evil, please
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130321/b0650efe/attachment.asc>


More information about the ffmpeg-devel mailing list