From a48dd9d481579017252596a66a3cde22e801cda6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=A9bastien=20Helleu?= Date: Mon, 30 Oct 2023 12:25:28 +0100 Subject: [PATCH] core: fix memory leak when config version is invalid or not supported --- ChangeLog.adoc | 1 + src/core/wee-config-file.c | 8 ++++++++ 2 files changed, 9 insertions(+) diff --git a/ChangeLog.adoc b/ChangeLog.adoc index 7b00b62c6..85d2f8c5a 100644 --- a/ChangeLog.adoc +++ b/ChangeLog.adoc @@ -15,6 +15,7 @@ For a list of important changes that require manual actions, please look at rele Bug fixes:: + * core: fix memory leak when config version is invalid or not supported * core: fix crash when "config_version" is present in a configuration file without a value * core: display an error on startup if environment variable "HOME" is not set * ruby: fix use of NULL variable when displaying exception diff --git a/src/core/wee-config-file.c b/src/core/wee-config-file.c index 3f5a72c7c..f11dd36b2 100644 --- a/src/core/wee-config-file.c +++ b/src/core/wee-config-file.c @@ -3569,6 +3569,10 @@ config_file_read_internal (struct t_config_file *config_file, int reload) filename, line_number, line); config_file_backup (filename); + if (option) + free (option); + if (value) + free (value); goto end_file; } else @@ -3586,6 +3590,10 @@ config_file_read_internal (struct t_config_file *config_file, int reload) config_file->version_read, config_file->version); config_file_backup (filename); + if (option) + free (option); + if (value) + free (value); goto end_file; } }