On 10/9/19, Limin Wang <lance.lmw...@gmail.com> wrote: > On Wed, Oct 09, 2019 at 03:35:27PM +0800, Steven Liu wrote: >> Signed-off-by: Steven Liu <l...@chinaffmpeg.org> >> --- >> libavformat/rtmpproto.c | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c >> index eb08d4d424..ae5eef8bca 100644 >> --- a/libavformat/rtmpproto.c >> +++ b/libavformat/rtmpproto.c >> @@ -2880,6 +2880,12 @@ reconnect: >> return 0; >> >> fail: >> + if (rt->playpath) >> + av_free(rt->playpath); >> + if (rt->tcurl) >> + av_free(rt->tcurl); >> + if (rt->flashver) >> + av_free(rt->flashver); > > it's better to use av_freep
also ifs are not needed. > >> av_dict_free(opts); >> rtmp_close(s); >> return ret; >> -- >> 2.15.1 >> >> >> >> _______________________________________________ >> 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". > _______________________________________________ > 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". _______________________________________________ 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".