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

Paul B Mahol onemda at gmail.com
Fri Jan 11 20:57:13 CET 2013


On 1/11/13, James Almer <jamrial at gmail.com> wrote:
> On 08/01/13 3:14 PM, James Almer wrote:
>> Write the packet unaltered if found.
>>
>> Fixes ticket #1917
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavformat/latmenc.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/libavformat/latmenc.c b/libavformat/latmenc.c
>> index 233eab8..b882382 100644
>> --- a/libavformat/latmenc.c
>> +++ b/libavformat/latmenc.c
>> @@ -149,7 +149,7 @@ static int latm_write_packet(AVFormatContext *s,
>> AVPacket *pkt)
>>      int i, len;
>>      uint8_t loas_header[] = "\x56\xe0\x00";
>>
>> -    if (s->streams[0]->codec->codec_id == AV_CODEC_ID_AAC_LATM)
>> +    if (pkt->size > 2 && pkt->data[0] == 0x56 && (pkt->data[1] >> 4) ==
>> 0xe)
>>          return ff_raw_write_packet(s, pkt);
>>
>>      if (pkt->size > 2 && pkt->data[0] == 0xff && (pkt->data[1] >> 4) ==
>> 0xf) {

Shouldn't  you check exact header and not just part of it?

How possible is that valid packet than need header will have first 3 bytes same
as loas header?


More information about the ffmpeg-devel mailing list