about summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorJames Booth <boothj5@gmail.com>2014-04-14 23:41:45 +0100
committerJames Booth <boothj5@gmail.com>2014-04-14 23:41:45 +0100
commit79088d01500ba5f6224a8d49a45ed1e1bc96ef07 (patch)
treecc32af365112c74b4593cec7704a43f52890fdb6
parent3e69d6b71ed57964df92f172d8f7a041e798a2e3 (diff)
downloadprofani-tty-79088d01500ba5f6224a8d49a45ed1e1bc96ef07.tar.gz
Clean up keys after using parse_options
-rw-r--r--src/command/commands.c2
-rw-r--r--tests/test_parser.c9
2 files changed, 11 insertions, 0 deletions
diff --git a/src/command/commands.c b/src/command/commands.c
index 51334934..63ecb48e 100644
--- a/src/command/commands.c
+++ b/src/command/commands.c
@@ -93,6 +93,7 @@ cmd_connect(gchar **args, struct cmd_help_t help)
         }
 
         options_destroy(options);
+        g_list_free(opt_keys);
 
         char *user = args[0];
         char *lower = g_utf8_strdown(user, -1);
@@ -1587,6 +1588,7 @@ cmd_join(gchar **args, struct cmd_help_t help)
     passwd = g_hash_table_lookup(options, "password");
 
     options_destroy(options);
+    g_list_free(opt_keys);
 
     // In the case that a nick wasn't provided by the optional args...
     if (nick == NULL) {
diff --git a/tests/test_parser.c b/tests/test_parser.c
index 6e1d4cde..df7836a0 100644
--- a/tests/test_parser.c
+++ b/tests/test_parser.c
@@ -521,6 +521,7 @@ parse_options_when_none_returns_empty_hasmap(void **state)
     assert_true(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
 
 void
@@ -539,6 +540,7 @@ parse_options_when_opt1_no_val_sets_error(void **state)
     assert_false(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
 
 void
@@ -559,6 +561,7 @@ parse_options_when_one_returns_map(void **state)
     assert_true(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
 
 void
@@ -578,6 +581,7 @@ parse_options_when_opt2_no_val_sets_error(void **state)
     assert_false(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
 
 void
@@ -601,6 +605,7 @@ parse_options_when_two_returns_map(void **state)
     assert_true(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
 
 void
@@ -621,6 +626,7 @@ parse_options_when_opt3_no_val_sets_error(void **state)
     assert_false(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
 
 void
@@ -647,6 +653,7 @@ parse_options_when_three_returns_map(void **state)
     assert_true(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
 
 void
@@ -667,6 +674,7 @@ parse_options_when_unknown_opt_sets_error(void **state)
     assert_false(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
 
 void
@@ -687,4 +695,5 @@ parse_options_with_duplicated_option_sets_error(void **state)
     assert_false(res);
 
     options_destroy(options);
+    g_list_free(keys);
 }
\ No newline at end of file