[FFmpeg-devel] [PATCH] lavf/tta: Check header and seek table CRC

Paul B Mahol onemda at gmail.com
Fri May 31 19:43:22 CEST 2013


On 5/31/13, James Almer <jamrial at gmail.com> wrote:
> On 31/05/13 12:41 AM, James Almer wrote:
>> +    if (crc != avio_rl32(s->pb)) {
>> +        av_log(s, AV_LOG_WARNING, "Header CRC error\n");
>> +        return AVERROR_INVALIDDATA;
>> +    }
> [...]
>> +    crc = ffio_get_checksum(s->pb) ^ UINT32_MAX;
>> +    if (crc != avio_rl32(s->pb)) {
>> +        av_log(s, AV_LOG_WARNING, "Seek table CRC error\n");
>> +        return AVERROR_INVALIDDATA;
>> +    }
>
> This should have been AV_LOG_ERROR, sorry.
> I'll send the patch again after it gets reviewed.

Please do, patch looks perfect.


More information about the ffmpeg-devel mailing list