[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