diff mbox series

[FFmpeg-devel] avutil/channel_layout: Fix leak of string

Message ID AS1PR01MB9564460D5CC14C2BAD62B6428F109@AS1PR01MB9564.eurprd01.prod.exchangelabs.com
State Accepted
Commit 9e241bdffd310711c48799cb7a919df61488432c
Headers show
Series [FFmpeg-devel] avutil/channel_layout: Fix leak of string | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_aarch64_jetson success Make finished
andriy/make_fate_aarch64_jetson success Make fate finished
andriy/make_armv7_RPi4 success Make finished
andriy/make_fate_armv7_RPi4 success Make fate finished

Commit Message

Andreas Rheinhardt March 15, 2022, 2:55 p.m. UTC 
Fixes memleaks in the channel_layout FATE-test.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavutil/channel_layout.c | 1 +
 1 file changed, 1 insertion(+)

Comments

James Almer March 15, 2022, 3:11 p.m. UTC | #1 
On 3/15/2022 11:55 AM, Andreas Rheinhardt wrote:
> Fixes memleaks in the channel_layout FATE-test.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>   libavutil/channel_layout.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/libavutil/channel_layout.c b/libavutil/channel_layout.c
> index 05ed35c078..8cc4efe4cf 100644
> --- a/libavutil/channel_layout.c
> +++ b/libavutil/channel_layout.c
> @@ -587,6 +587,7 @@ int av_channel_layout_from_string(AVChannelLayout *channel_layout,
>               av_free(channel);
>               av_free(chname);
>           }
> +        av_free(chlist);
>   
>           return 0;
>       }

LGTM.
diff mbox series

Patch

diff --git a/libavutil/channel_layout.c b/libavutil/channel_layout.c
index 05ed35c078..8cc4efe4cf 100644
--- a/libavutil/channel_layout.c
+++ b/libavutil/channel_layout.c
@@ -587,6 +587,7 @@  int av_channel_layout_from_string(AVChannelLayout *channel_layout,
             av_free(channel);
             av_free(chname);
         }
+        av_free(chlist);
 
         return 0;
     }