[FFmpeg-devel] [PATCH 1/2] movtextdec.c: Improve upon dynarrays and text_to_ass

Niklesh Lalwani niklesh.lalwani at iitb.ac.in
Thu Jun 18 18:56:52 CEST 2015


Updated patch attached.

On 18-Jun-2015 2:52 PM, "Clément Bœsch" <u at pkh.me> wrote:
av_dynarray_add(&style_flags, &index, flag);
> > +                    s_temp->style_flag = AV_RB8(tsmb);
> > +                    count = i;
>
> > +                    av_dynarray_add(&s, &count, s_temp);
>
> missing check
>
> (make sure not to leak anything if you return in the middle)
>

Is this the right way to implement it?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: movtextdec.c-Improve-upon-dynarrays-and-text_to_ass.patch
Type: application/octet-stream
Size: 6435 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150618/756c7684/attachment.obj>


More information about the ffmpeg-devel mailing list