tests: Fix some minor leaks in the unit tests
authorPhilip Withnall <philip.withnall@collabora.co.uk>
Fri, 26 Sep 2014 15:18:46 +0000 (16:18 +0100)
committerPhilip Withnall <philip@tecnocode.co.uk>
Sat, 27 Sep 2014 09:30:39 +0000 (10:30 +0100)
https://bugzilla.gnome.org/show_bug.cgi?id=737446

glib/tests/bookmarkfile.c
glib/tests/fileutils.c
glib/tests/option-context.c

index 00605ed..7c2f8d7 100644 (file)
@@ -31,6 +31,7 @@ test_load_from_data_dirs (void)
   g_assert (!res);
   g_assert_error (error, G_FILE_ERROR, G_FILE_ERROR_NOENT);
   g_assert_null (path);
+  g_error_free (error);
 
   g_bookmark_file_free (bookmark);  
 }
index 1f66392..fff4217 100644 (file)
@@ -46,7 +46,7 @@
 #define S G_DIR_SEPARATOR_S
 
 static void
-check_string (gchar *str, gchar *expected)
+check_string (gchar *str, const gchar *expected)
 {
   g_assert (str != NULL);
   g_assert_cmpstr (str, ==, expected);
@@ -825,6 +825,7 @@ test_read_link (void)
   path = g_file_read_link (oldpath, &error);
   g_assert_error (error, G_FILE_ERROR, G_FILE_ERROR_INVAL);
   g_assert_null (path);
+  g_error_free (error);
 
   g_free (cwd);
   g_free (newpath);
index d56eb53..8cf77a6 100644 (file)
@@ -2384,12 +2384,13 @@ short_remaining (void)
     { NULL }
   };
   GOptionContext* context;
-  gchar **argv;
+  gchar **argv, **argv_copy;
   gint argc;
 
   g_test_bug ("729563");
 
   argv = split_string ("program -ri -n 4 -t hello file1 file2", &argc);
+  argv_copy = copy_stringv (argv, argc);
 
   context = g_option_context_new (NULL);
 
@@ -2407,7 +2408,10 @@ short_remaining (void)
   g_assert_cmpstr (files[1], ==, "file2"); 
   g_assert (files[2] == NULL); 
 
-  g_strfreev (argv);
+  g_free (text);
+  g_strfreev (files);
+  g_strfreev (argv_copy);
+  g_free (argv);
   g_option_context_free (context);
 }