On Sat, Jun 27, 2020 at 01:02:52PM -0300, James Almer wrote:
> 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?
The old code use mallocz to memeset the allocate data. When switch to
av_buffer_realloc,
then memset is missing. Or the data is uninitialized if the following
get_bits_left(&gb)
checking is true. I didn't notice it for real testing is OK without trigger it.
>
> 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?
I think the fate testing have tested the data. Also I have tested with 2 CC
mpeg2 sample
case.
>
> > 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".
--
Thanks,
Limin Wang
_______________________________________________
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".
