On Thu, Nov 28, 2019 at 10:37 PM Andreas Rheinhardt
<andreas.rheinha...@gmail.com> wrote:
>
> Jun Zhao:
> > From: Jun Zhao <barryjz...@tencent.com>
> >
> > check avformat_new_stream() return.
> >
> > Signed-off-by: Jun Zhao <barryjz...@tencent.com>
> > ---
> > libavformat/vividas.c | 4 ++++
> > 1 files changed, 4 insertions(+), 0 deletions(-)
> >
> > diff --git a/libavformat/vividas.c b/libavformat/vividas.c
> > index f20af3d..16642c0 100644
> > --- a/libavformat/vividas.c
> > +++ b/libavformat/vividas.c
> > @@ -319,6 +319,8 @@ static int track_header(VividasDemuxContext *viv,
> > AVFormatContext *s, uint8_t *
> >
> > for (i = 0; i < num_video; i++) {
> > AVStream *st = avformat_new_stream(s, NULL);
> > + if (!st)
> > + return AVERROR(ENOMEM);
> >
> > st->id = i;
> >
> > @@ -352,6 +354,8 @@ static int track_header(VividasDemuxContext *viv,
> > AVFormatContext *s, uint8_t *
> > for(i=0;i<viv->num_audio;i++) {
> > int q;
> > AVStream *st = avformat_new_stream(s, NULL);
> > + if (!st)
> > + return AVERROR(ENOMEM);
> >
> > st->id = num_video + i;
> >
> >
> Actually, the AVIOContext pb leaks when these errors happen. But don't
> bother updating your patch, because pb should actually not be
> allocated at all: I'll send a patch using ffio_init_context soon.
>
I think you comments the other patch :), and Michael have applied your patch
_______________________________________________
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".
