On 6/27/2020 12:46 PM, [email protected] wrote:
> From: Limin Wang <[email protected]>
>
> 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ö <[email protected]>
> Signed-off-by: Limin Wang <[email protected]>
> ---
> 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
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".
