Reimar Döffinger <[email protected]> added the comment:

On Mon, Jul 26, 2010 at 03:11:10AM +0000, Pavel wrote:
> free allocated context on failure

That not really enough at all.
See av_close_input_stream.
I think everything except s->iformat->read_close(s);
should be done.
Otherwise every demuxer must manually free chapters,
metadata, index etc. if read_header() fails after
reading them.

________________________________________________
FFmpeg issue tracker <[email protected]>
<https://roundup.ffmpeg.org/issue2126>
________________________________________________

Reply via email to