Thank you for your reminding, George.
I really ignore av_freep() above.
Please ignore this patch.
// xufuji456
----
??:
"zhilizhao(赵志立)" (12023-03-03):
> > av_freep(_files);
> > av_freep(_files);
> > +input_files = NULL;
> > +output_files = NULL;
> Until we decided to make ffmpeg cmd works as a library,
> it doesn’t matter.
It is worse than that: the patch completely ignores the semantic
> On Mar 3, 2023, at 14:51, xufuji456 <839789...@qq.com> wrote:
>
> ---
> fftools/ffmpeg.c | 7 +++
> 1 file changed, 7 insertions(+)
>
> diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
> index 743bc0c6b6..0ce9531235 100644
> --- a/fftools/ffmpeg.c
> +++ b/fftools/ffmpeg.c
> @@ -569,6
---
fftools/ffmpeg.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index 743bc0c6b6..0ce9531235 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/ffmpeg.c
@@ -569,6 +569,13 @@ static void ffmpeg_cleanup(int ret)
av_freep(_files);