fix: Memory leak in the bootstrap daemon

nurupo · nurupo · commit 5344d7f84d06 · 2024-04-03T23:31:27.000-04:00
diff --git a/other/bootstrap_daemon/src/config.c b/other/bootstrap_daemon/src/config.c
@@ -200,6 +200,8 @@ bool get_general_config(const char *cfg_file_path, char **pid_file_path, char **
     *keys_file_path = (char *)malloc(keys_file_path_len);
     if (*keys_file_path == nullptr) {
         log_write(LOG_LEVEL_ERROR, "Allocation failure.\n");
+        free(*pid_file_path);
+        *pid_file_path = nullptr;
         return false;
     }
     memcpy(*keys_file_path, tmp_keys_file, keys_file_path_len);

Original file line number	Diff line number	Diff line change
`@@ -200,6 +200,8 @@ bool get_general_config(const char cfg_file_path, char pid_file_path, char *`
`200`	`200`	`keys_file_path = (char )malloc(keys_file_path_len);`
`201`	`201`	`if (*keys_file_path == nullptr) {`
`202`	`202`	`log_write(LOG_LEVEL_ERROR, "Allocation failure.\n");`
	`203`	`+ free(*pid_file_path);`
	`204`	`+ *pid_file_path = nullptr;`
`203`	`205`	`return false;`
`204`	`206`	`}`
`205`	`207`	`memcpy(*keys_file_path, tmp_keys_file, keys_file_path_len);`