summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2013-10-09 04:04:04 +0200
committerLennart Poettering <lennart@poettering.net>2013-10-09 04:08:00 +0200
commit510b857f7d1e7e8d38912890536342dd5dd647dd (patch)
tree6899d6b0f32eba46f2deefa848a96805b9966be1 /src
parent2ee0591d12b9e725c4585502285fd91cde682d9b (diff)
tests: fix some memory leaks in tests
Diffstat (limited to 'src')
-rw-r--r--src/journal/test-journal-interleaving.c29
-rw-r--r--src/journal/test-journal.c5
-rw-r--r--src/test/test-util.c2
3 files changed, 18 insertions, 18 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;
diff --git a/src/journal/test-journal.c b/src/journal/test-journal.c
index 190c426eba..189fe07b48 100644
--- a/src/journal/test-journal.c
+++ b/src/journal/test-journal.c
@@ -169,6 +169,11 @@ static void test_empty(void) {
assert_se(rm_rf_dangerous(t, false, true, false) >= 0);
}
+
+ journal_file_close(f1);
+ journal_file_close(f2);
+ journal_file_close(f3);
+ journal_file_close(f4);
}
int main(int argc, char *argv[]) {
diff --git a/src/test/test-util.c b/src/test/test-util.c
index c5762ede4b..7fd0572bcb 100644
--- a/src/test/test-util.c
+++ b/src/test/test-util.c
@@ -587,7 +587,7 @@ static void test_get_files_in_directory(void) {
_cleanup_strv_free_ char **l = NULL, **t = NULL;
assert_se(get_files_in_directory("/tmp", &l) >= 0);
- assert_se(get_files_in_directory(".", &l) >= 0);
+ assert_se(get_files_in_directory(".", &t) >= 0);
assert_se(get_files_in_directory(".", NULL) >= 0);
}