From d69d951b11e80dbcc7f62938951832638119bdd9 Mon Sep 17 00:00:00 2001
From: Martin Pitt <martin.pitt@ubuntu.com>
Date: Mon, 29 Feb 2016 15:16:11 +0100
Subject: test-clock: fix assertions

Use assert_se() to ensure that they don't get optimized away.
---
 src/test/test-clock.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'src/test/test-clock.c')

diff --git a/src/test/test-clock.c b/src/test/test-clock.c
index b049d97ce7..0949793e5f 100644
--- a/src/test/test-clock.c
+++ b/src/test/test-clock.c
@@ -56,10 +56,10 @@ static void test_clock_is_localtime(void) {
         assert_se(clock_is_localtime("/nonexisting/adjtime") == 0);
 
         fd = mkostemp_safe(adjtime, O_WRONLY|O_CLOEXEC);
-        assert(fd > 0);
+        assert_se(fd >= 0);
         log_info("adjtime test file: %s", adjtime);
         f = fdopen(fd, "w");
-        assert(f);
+        assert_se(f);
 
         for (size_t i = 0; i < ELEMENTSOF(scenarios); ++i) {
                 log_info("scenario #%zu:, expected result %i", i, scenarios[i].expected_result);
@@ -82,10 +82,10 @@ static void test_clock_is_localtime_system(void) {
                 log_info("/etc/adjtime exists, clock_is_localtime() == %i", r);
                 /* if /etc/adjtime exists we expect some answer, no error or
                  * crash */
-                assert(r == 0 || r == 1);
+                assert_se(r == 0 || r == 1);
         } else
                 /* default is UTC if there is no /etc/adjtime */
-                assert(r == 0);
+                assert_se(r == 0);
 }
 
 int main(int argc, char *argv[]) {
-- 
cgit v1.2.3-54-g00ecf


From 07edd3b9e5855332a02ef7b91602c93aa12c1fd1 Mon Sep 17 00:00:00 2001
From: Martin Pitt <martin.pitt@ubuntu.com>
Date: Mon, 29 Feb 2016 17:40:08 +0100
Subject: test-clock: fix FILE* leak

Close the FILE* instead of the underlying fd.
---
 src/test/test-clock.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/test/test-clock.c')

diff --git a/src/test/test-clock.c b/src/test/test-clock.c
index 0949793e5f..84f775e5bc 100644
--- a/src/test/test-clock.c
+++ b/src/test/test-clock.c
@@ -28,8 +28,8 @@
 
 static void test_clock_is_localtime(void) {
         char adjtime[] = "/tmp/test-adjtime.XXXXXX";
-        _cleanup_close_ int fd = -1;
-        FILE* f;
+        int fd = -1;
+        _cleanup_fclose_ FILE* f = NULL;
 
         static const struct scenario {
                 const char* contents;
-- 
cgit v1.2.3-54-g00ecf