Merge pull request #367 from scottaiton/fix_memleak_in_options

Fix memory leak in fclaw_options_register
ForestClaw · Nov 13, 2024 · fea33fc · fea33fc
2 parents 1e6eba4 + 9c1b240
commit fea33fc
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/fclaw_options.c b/src/fclaw_options.c
@@ -590,7 +590,7 @@ fclaw_options_t* fclaw_options_register (fclaw_app_t * a,
         fclaw_opt_sections = sc_keyvalue_new();
         fclaw_app_set_attribute(a, "fclaw_opt_sections", fclaw_opt_sections);
     }
-    char *key = strdup(section == NULL ? "Options" : section);
+    char *key = FCLAW_STRDUP(section == NULL ? "Options" : section);
     char* curr_char = key;
     while(*curr_char != '\0')
     {
@@ -602,6 +602,7 @@ fclaw_options_t* fclaw_options_register (fclaw_app_t * a,
     sc_keyvalue_set_int(fclaw_opt_sections, 
                         key,
                         1);
+    FCLAW_FREE(key);
 
     return fclaw_opt;
 }