ryantimwilson / rpms / systemd

Forked from rpms/systemd 2 months ago
Clone
Zbigniew Jędrzejewski-Szmek 62fe94
From eee846339d2f76f568c62b3725bf75bcee728115 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: David Herrmann <dh.herrmann@gmail.com>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Mon, 22 Sep 2014 11:27:22 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] test-util: make valgrind happy
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
Properly free all temporary resources to make valgrind not complain about
Zbigniew Jędrzejewski-Szmek 62fe94
lost records.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/test/test-util.c | 28 +++++++++++++++++++++++-----
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 23 insertions(+), 5 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/test/test-util.c b/src/test/test-util.c
Zbigniew Jędrzejewski-Szmek 62fe94
index 80425ca61a..f8e42f3a55 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/test/test-util.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/test/test-util.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -843,11 +843,27 @@ static void test_is_valid_documentation_url(void) {
Zbigniew Jędrzejewski-Szmek 62fe94
 }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
 static void test_file_in_same_dir(void) {
Zbigniew Jędrzejewski-Szmek 62fe94
-        assert_se(streq(file_in_same_dir("/", "a"), "/a"));
Zbigniew Jędrzejewski-Szmek 62fe94
-        assert_se(streq(file_in_same_dir("/", "/a"), "/a"));
Zbigniew Jędrzejewski-Szmek 62fe94
-        assert_se(streq(file_in_same_dir("", "a"), "a"));
Zbigniew Jędrzejewski-Szmek 62fe94
-        assert_se(streq(file_in_same_dir("a/", "a"), "a/a"));
Zbigniew Jędrzejewski-Szmek 62fe94
-        assert_se(streq(file_in_same_dir("bar/foo", "bar"), "bar/bar"));
Zbigniew Jędrzejewski-Szmek 62fe94
+        char *t;
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        t = file_in_same_dir("/", "a");
Zbigniew Jędrzejewski-Szmek 62fe94
+        assert_se(streq(t, "/a"));
Zbigniew Jędrzejewski-Szmek 62fe94
+        free(t);
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        t = file_in_same_dir("/", "/a");
Zbigniew Jędrzejewski-Szmek 62fe94
+        assert_se(streq(t, "/a"));
Zbigniew Jędrzejewski-Szmek 62fe94
+        free(t);
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        t = file_in_same_dir("", "a");
Zbigniew Jędrzejewski-Szmek 62fe94
+        assert_se(streq(t, "a"));
Zbigniew Jędrzejewski-Szmek 62fe94
+        free(t);
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        t = file_in_same_dir("a/", "a");
Zbigniew Jędrzejewski-Szmek 62fe94
+        assert_se(streq(t, "a/a"));
Zbigniew Jędrzejewski-Szmek 62fe94
+        free(t);
Zbigniew Jędrzejewski-Szmek 62fe94
+
Zbigniew Jędrzejewski-Szmek 62fe94
+        t = file_in_same_dir("bar/foo", "bar");
Zbigniew Jędrzejewski-Szmek 62fe94
+        assert_se(streq(t, "bar/bar"));
Zbigniew Jędrzejewski-Szmek 62fe94
+        free(t);
Zbigniew Jędrzejewski-Szmek 62fe94
 }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
 static void test_endswith(void) {
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -1239,11 +1255,13 @@ static void test_unquote_many_words(void) {
Zbigniew Jędrzejewski-Szmek 62fe94
         assert_se(unquote_many_words(&p, &a, NULL) == 1);
Zbigniew Jędrzejewski-Szmek 62fe94
         assert_se(p == original+7);
Zbigniew Jędrzejewski-Szmek 62fe94
         assert_se(streq_ptr(a, "foobar"));
Zbigniew Jędrzejewski-Szmek 62fe94
+        free(a);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         p = original = "     foobar    ";
Zbigniew Jędrzejewski-Szmek 62fe94
         assert_se(unquote_many_words(&p, &a, NULL) == 1);
Zbigniew Jędrzejewski-Szmek 62fe94
         assert_se(p == original+15);
Zbigniew Jędrzejewski-Szmek 62fe94
         assert_se(streq_ptr(a, "foobar"));
Zbigniew Jędrzejewski-Szmek 62fe94
+        free(a);
Zbigniew Jędrzejewski-Szmek 62fe94
 }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
 int main(int argc, char *argv[]) {