virtualenv/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-01-29 22:00:58 +02:00
Code Issues Releases Activity

lavc/tak: do not store invalid values in stream info

Browse Source 
When tak_get_nb_samples() fails, it will currently write
AVERROR_INVALIDDATA as TAKStreamInfo.frame_samples. The parser will then
use this negative value as a frame duration, which leads to various
breakage.

Avoid this by returning the error code from tak_parse_streaminfo()
directly; never store negative values in the parsed header.
This commit is contained in:
Anton Khirnov 2023-05-04 20:45:03 +02:00
parent ecdf1ac267
commit a59b4ac713
1 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
libavcodec/tak.c
View File

@ -92,10 +92,10 @@ int ff_tak_check_crc(const uint8_t *buf, unsigned int buf_size)
return 0; return 0;
} }
static void tak_parse_streaminfo(TAKStreamInfo *s, GetBitContext *gb) static int tak_parse_streaminfo(TAKStreamInfo *s, GetBitContext *gb)
{ {
uint64_t channel_mask = 0; uint64_t channel_mask = 0;
int frame_type, i; int frame_type, i, ret;
s->codec = get_bits(gb, TAK_ENCODER_CODEC_BITS); s->codec = get_bits(gb, TAK_ENCODER_CODEC_BITS);
skip_bits(gb, TAK_ENCODER_PROFILE_BITS); skip_bits(gb, TAK_ENCODER_PROFILE_BITS);
@ -124,7 +124,13 @@ static void tak_parse_streaminfo(TAKStreamInfo *s, GetBitContext *gb)
} }
s->ch_layout = channel_mask; s->ch_layout = channel_mask;
s->frame_samples = tak_get_nb_samples(s->sample_rate, frame_type);
ret = tak_get_nb_samples(s->sample_rate, frame_type);
if (ret < 0)
return ret;
s->frame_samples = ret;
return 0;
} }
int avpriv_tak_parse_streaminfo(TAKStreamInfo *s, const uint8_t *buf, int size) int avpriv_tak_parse_streaminfo(TAKStreamInfo *s, const uint8_t *buf, int size)
@ -135,9 +141,7 @@ int avpriv_tak_parse_streaminfo(TAKStreamInfo *s, const uint8_t *buf, int size)
if (ret < 0) if (ret < 0)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
tak_parse_streaminfo(s, &gb); return tak_parse_streaminfo(s, &gb);
return 0;
} }
int ff_tak_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb, int ff_tak_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb,
@ -159,7 +163,9 @@ int ff_tak_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb,
} }
if (ti->flags & TAK_FRAME_FLAG_HAS_INFO) { if (ti->flags & TAK_FRAME_FLAG_HAS_INFO) {
tak_parse_streaminfo(ti, gb); int ret = tak_parse_streaminfo(ti, gb);
if (ret < 0)
return ret;
if (get_bits(gb, 6)) if (get_bits(gb, 6))
skip_bits(gb, 25); skip_bits(gb, 25);