[FFmpeg-devel] [PATCH v1] libavfilter/af_channelsplit.c:fix memory leak

[Zhao Zhili]

> On Apr 12, 2024, at 17:49, Zhao Zhili <quinkblack at foxmail.com> wrote:
> 
> 
>> On Apr 12, 2024, at 17:19, LuMingYin <lumingyindetect at 163.com> wrote:
>> 
>> Signed-off-by: LuMingYin <lumingyindetect at 163.com>
>> ---
>> libavfilter/af_channelsplit.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>> 
>> diff --git a/libavfilter/af_channelsplit.c b/libavfilter/af_channelsplit.c
>> index d18d91dcb6..2cfac19cd3 100644
>> --- a/libavfilter/af_channelsplit.c
>> +++ b/libavfilter/af_channelsplit.c
>> @@ -163,8 +163,10 @@ static int filter_frame(AVFilterLink *outlink, AVFrame *buf)
>> 
>>    buf_out->data[0] = buf_out->extended_data[0] = buf_out->extended_data[s->map[i]];
>>    ret = av_channel_layout_from_mask(&buf_out->ch_layout, 1ULL << channel);
>> -    if (ret < 0)
>> +    if (ret < 0){
>> +        av_frame_free(&buf_out);
>>        return ret;
>> +    }
> 
> LGTM. Please pay attention to the coding style. I can fix it manually this time.

Applied as 5e380bcdb13dd47ce9c358a4edb281f05fde3f24.

> 
>> 
>>    return ff_filter_frame(ctx->outputs[i], buf_out);
>> }
>> -- 
>> 2.25.1
>> 
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> 
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>