Quoting Vittorio Giovara (2014-10-14 17:46:47) > CC: [email protected] > Bug-Id: CID 1087081 > --- > avconv.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/avconv.c b/avconv.c > index 48c095d..cd3f6f4 100644 > --- a/avconv.c > +++ b/avconv.c > @@ -169,7 +169,8 @@ static void avconv_cleanup(int ret) > AVFormatContext *s = of->ctx; > if (s && s->oformat && !(s->oformat->flags & AVFMT_NOFILE) && s->pb) > avio_close(s->pb); > - avformat_free_context(s); > + if (s) > + avformat_free_context(s); > av_dict_free(&of->opts); > > av_freep(&output_files[i]); > -- > 1.9.3 (Apple Git-50) >
Hmm, why doesn't avformat_free_context handle NULLs? Most other destructors do. That sound like the more correct thing to do here. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
