On 6/27/2020 12:46 PM, lance.lmw...@gmail.com wrote:
> From: Limin Wang <lance.lmw...@gmail.com>
>
> The issue is introduced from a705bcd763e344fa, please tested with below
> command line:
> make V=1 fate-sub-cc-scte20 TARGET_EXEC="valgrind --error-exitcode=1"
>
> Reported-by: Martin Storsjö <mar...@martin.st>
> Signed-off-by: Limin Wang <lance.lmw...@gmail.com>
> ---
> libavcodec/mpeg12dec.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c
> index f0f92ac..2562027 100644
> --- a/libavcodec/mpeg12dec.c
> +++ b/libavcodec/mpeg12dec.c
> @@ -2276,6 +2276,8 @@ static int mpeg_decode_a53_cc(AVCodecContext *avctx,
> if (ret >= 0) {
> uint8_t field, cc1, cc2;
> uint8_t *cap = s1->a53_buf_ref->data;
> +
> + memset(s1->a53_buf_ref->data + old_size, 0, cc_count *
> UINT64_C(3));
Why is zeroing needed now to prevent use of uninitialized values but not
before this patch? Wouldn't it hint at some issue in your port to
AVBufferRef?
Did you for example make sure to read and write in the correct place in
the reallocated buffer when you're appending new captions to it?
> for (i = 0; i < cc_count && get_bits_left(&gb) >= 26; i++) {
> skip_bits(&gb, 2); // priority
> field = get_bits(&gb, 2);
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
