[FFmpeg-devel] [PATCH 1/4] lavc/libaribb24: add error handling to region handling

Jan Ekström jeebjp at gmail.com
Tue Feb 12 20:23:05 EET 2019


On Tue, Feb 12, 2019 at 2:16 AM Michael Niedermayer <michaelni at gmx.at> wrote:
>
> On Mon, Feb 11, 2019 at 03:06:51AM +0200, Jan Ekström wrote:
> > Fixes some rather embarrassing mistakes that somehow passed my
> > eyes.
> >
> > * Now catches if memory allocation has failed during bprint usage
> >   by checking av_bprint_is_complete().
> > * Now catches if adding an ASS rectangle into an AVSubtitle failed.
> > * Returns AVERROR_INVALIDDATA if we get an invalid region buffer
> >   length.
> > ---
> >  libavcodec/libaribb24.c | 25 ++++++++++++++++++-------
> >  1 file changed, 18 insertions(+), 7 deletions(-)
>
> probably ok
>
> thx
>

Thanks for the review, will do a final check-up and apply.

Jan


More information about the ffmpeg-devel mailing list