Andreas Rheinhardt: > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com> > --- > How has this slipped through?
Answer: AVIOContext starts with an AVClass* that is unset when using ffio_init_context(). Therefore the av_freep() leads to freeing of a NULL pointer which does not segfault. > Btw: This patchset is based upon code inspection, not on reading > actual files. > > libavformat/dvdvideodec.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavformat/dvdvideodec.c b/libavformat/dvdvideodec.c > index ef2d4e6df4..f2f23affb2 100644 > --- a/libavformat/dvdvideodec.c > +++ b/libavformat/dvdvideodec.c > @@ -1202,7 +1202,6 @@ static void dvdvideo_subdemux_close(AVFormatContext *s) > DVDVideoDemuxContext *c = s->priv_data; > > av_freep(&c->mpeg_pb.pub.buffer); > - av_freep(&c->mpeg_pb); > avformat_close_input(&c->mpeg_ctx); > } > _______________________________________________ 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".
