[FFmpeg-devel] [PATCH] latmenc: Check for LOAS sync word

[Michael Niedermayer]

On Wed, Jan 16, 2013 at 05:43:03AM -0300, James Almer wrote:
> On 15/01/13 10:34 PM, Michael Niedermayer wrote:
> > you can check extradata
> > as a sideeffect it would fix the crash
> > with the right (well wrong) data the current patch not fix the crash
> 
> How about the attached patch? (It still needs reindentation)
> If there's no extradata then the packet is either LOAS or ADTS, and it should be
> used if it's the former or discarded if it's anything else.
> 

> This also prevents potential misdetection of raw packets as ADTS packets, since 
> the check was on the first 3 bytes as well and without checking if there was any
> extradata available.

raw aac cant start with a 0xF? or 0xE? these would be end tags

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- 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/20130116/8bfbbe8c/attachment.asc>