From bd9ff08715f762bbf60e559ff788b25aeec33fff Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Wed, 31 Oct 2012 21:34:20 +1030 Subject: [PATCH 1/1] crcsync: fix memory leak in tests. /home/rusty/devel/cvs/ccan/ccan/crcsync/test/run-crash.c:==6923== 112 bytes in 1 blocks are definitely lost in loss record 2 of 3 ==6923== at 0x402BA7A: malloc (vg_replace_malloc.c:261) ==6923== by 0x80492EE: crcblocks (run-crash.c:20) ==6923== by 0x8049448: main (run-crash.c:62) ==6923== 2,217 (2,212 direct, 5 indirect) bytes in 1 blocks are definitely lost in loss record 3 of 3 ==6923== at 0x402BA7A: malloc (vg_replace_malloc.c:261) ==6923== by 0x8048AA0: crc_context_new (crcsync.c:85) ==6923== by 0x8049476: main (run-crash.c:64) /home/rusty/devel/cvs/ccan/ccan/crcsync/test/run.c:==11793== 512 bytes in 1 blocks are definitely lost in loss record 1 of 2 ==11793== at 0x402A52B: calloc (vg_replace_malloc.c:462) ==11793== by 0x8049A3F: main (run.c:128) ==11793== 512 bytes in 1 blocks are definitely lost in loss record 2 of 2 ==11793== at 0x402A52B: calloc (vg_replace_malloc.c:462) ==11793== by 0x8049A57: main (run.c:129) Signed-off-by: Rusty Russell --- ccan/crcsync/test/run-crash.c | 3 ++- ccan/crcsync/test/run.c | 2 ++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/ccan/crcsync/test/run-crash.c b/ccan/crcsync/test/run-crash.c index 2be3b36d..3bf49f64 100644 --- a/ccan/crcsync/test/run-crash.c +++ b/ccan/crcsync/test/run-crash.c @@ -92,6 +92,7 @@ int main(int argc, char *argv[]) } } ok1(expected_i == ARRAY_SIZE(expected)); - + crc_context_free(crcctx); + free(crc_info1.crcs); return 0; } diff --git a/ccan/crcsync/test/run.c b/ccan/crcsync/test/run.c index 2c48b378..f51ac23f 100644 --- a/ccan/crcsync/test/run.c +++ b/ccan/crcsync/test/run.c @@ -285,5 +285,7 @@ int main(int argc, char *argv[]) BLOCK_SIZE, res, num_res); } + free(buffer1); + free(buffer2); return exit_status(); } -- 2.39.2