diff options
author | Lennart Poettering <lennart@poettering.net> | 2013-10-09 04:04:04 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2013-10-09 04:08:00 +0200 |
commit | 510b857f7d1e7e8d38912890536342dd5dd647dd (patch) | |
tree | 6899d6b0f32eba46f2deefa848a96805b9966be1 /src/journal/test-journal-interleaving.c | |
parent | 2ee0591d12b9e725c4585502285fd91cde682d9b (diff) |
tests: fix some memory leaks in tests
Diffstat (limited to 'src/journal/test-journal-interleaving.c')
-rw-r--r-- | src/journal/test-journal-interleaving.c | 29 |
1 files changed, 12 insertions, 17 deletions
diff --git a/src/journal/test-journal-interleaving.c b/src/journal/test-journal-interleaving.c index 1a058eaedd..af0d43e39e 100644 --- a/src/journal/test-journal-interleaving.c +++ b/src/journal/test-journal-interleaving.c @@ -50,20 +50,17 @@ _noreturn_ static void log_assert_errno(const char *text, int eno, const char *f log_assert_errno(#expr, -_r_, __FILE__, __LINE__, __PRETTY_FUNCTION__); \ } while (false) -static JournalFile *test_open (const char *name) -{ +static JournalFile *test_open(const char *name) { JournalFile *f; assert_ret(journal_file_open(name, O_RDWR|O_CREAT, 0644, true, false, NULL, NULL, NULL, &f)); return f; } -static void test_close (JournalFile *f) -{ +static void test_close(JournalFile *f) { journal_file_close (f); } -static void append_number(JournalFile *f, int n, uint64_t *seqnum) -{ +static void append_number(JournalFile *f, int n, uint64_t *seqnum) { char *p; dual_timestamp ts; struct iovec iovec[1]; @@ -74,13 +71,12 @@ static void append_number(JournalFile *f, int n, uint64_t *seqnum) iovec[0].iov_base = p; iovec[0].iov_len = strlen(p); assert_ret(journal_file_append_entry(f, &ts, iovec, 1, seqnum, NULL, NULL)); - free (p); + free(p); } -static void test_check_number (sd_journal *j, int n) -{ +static void test_check_number (sd_journal *j, int n) { const void *d; - char *k; + _cleanup_free_ char *k; size_t l; int x; @@ -92,9 +88,10 @@ static void test_check_number (sd_journal *j, int n) assert_se(n == x); } -static void test_check_numbers_down (sd_journal *j, int count) -{ - for (int i = 1; i <= count; i++) { +static void test_check_numbers_down (sd_journal *j, int count) { + int i; + + for (i = 1; i <= count; i++) { int r; test_check_number(j, i); assert_ret(r = sd_journal_next(j)); @@ -106,8 +103,7 @@ static void test_check_numbers_down (sd_journal *j, int count) } -static void test_check_numbers_up (sd_journal *j, int count) -{ +static void test_check_numbers_up (sd_journal *j, int count) { for (int i = count; i >= 1; i--) { int r; test_check_number(j, i); @@ -144,8 +140,7 @@ static void setup_interleaved(void) { test_close(two); } -static void test_skip(void (*setup)(void)) -{ +static void test_skip(void (*setup)(void)) { char t[] = "/tmp/journal-skip-XXXXXX"; sd_journal *j; int r; |