diff options
author | Jia Tan <jiat0218@gmail.com> | 2023-06-28 20:46:31 +0800 |
---|---|---|
committer | Jia Tan <jiat0218@gmail.com> | 2023-06-29 00:07:16 +0800 |
commit | 66bdcfa85fef2911cc80f5f30fed3f9610faccb4 (patch) | |
tree | 22d671a1a9c74df9a00d9f6c237e988b3c51ec67 /tests | |
parent | Tests: Fix memory leaks in test_block_header. (diff) | |
download | xz-66bdcfa85fef2911cc80f5f30fed3f9610faccb4.tar.xz |
Tests: Fix memory leaks in test_index.
Several tests were missing calls to lzma_index_end() to clean up the
lzma_index structs. The memory leaks were discovered by using
-fsanitize=address with GCC.
Diffstat (limited to '')
-rw-r--r-- | tests/test_index.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/tests/test_index.c b/tests/test_index.c index b8efebcb..168e919f 100644 --- a/tests/test_index.c +++ b/tests/test_index.c @@ -648,6 +648,8 @@ test_lzma_index_iter_init(void) assert_false(lzma_index_iter_next(&iter, LZMA_INDEX_ITER_STREAM)); assert_false(lzma_index_iter_next(&iter, LZMA_INDEX_ITER_STREAM)); assert_uint_eq(iter.stream.number, 3); + + lzma_index_end(first, NULL); } @@ -1157,6 +1159,9 @@ test_lzma_index_cat(void) assert_lzma_ret(lzma_index_cat(dest, src, NULL), LZMA_DATA_ERROR); // Check for compressed size overflow + lzma_index_end(src, NULL); + lzma_index_end(dest, NULL); + dest = lzma_index_init(NULL); assert_true(dest != NULL); @@ -1297,6 +1302,7 @@ test_lzma_index_dup(void) assert_true(copy != NULL); assert_true(index_is_equal(idx, copy)); + lzma_index_end(copy, NULL); lzma_index_end(idx, NULL); } @@ -1428,6 +1434,7 @@ test_lzma_index_encoder(void) verify_index_buffer(idx, buffer, buffer_size); + lzma_index_end(idx, NULL); lzma_end(&strm); #endif } @@ -1589,6 +1596,8 @@ test_lzma_index_buffer_encode(void) // Validate results verify_index_buffer(idx, buffer, buffer_size); + + lzma_index_end(idx, NULL); #endif } @@ -1639,6 +1648,8 @@ test_lzma_index_buffer_decode(void) assert_true(index_is_equal(decode_test_index, idx)); + lzma_index_end(idx, NULL); + // Test too small memlimit in_pos = 0; memlimit = 1; |