X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=src%2Fjournal%2Ftest-compress.c;h=97577e827c8efa5c88df1bb344bd862aa4134135;hb=ab31f6b8713d43354c3716ee4c37fe4460db9254;hp=f5f5f8df39022008a0129f204f7e4d10390a3a8d;hpb=fa1c4b518ec7d8ec2d647213ee651cde4d6c9d7e;p=elogind.git diff --git a/src/journal/test-compress.c b/src/journal/test-compress.c index f5f5f8df3..97577e827 100644 --- a/src/journal/test-compress.c +++ b/src/journal/test-compress.c @@ -48,67 +48,82 @@ typedef int (decompress_stream_t)(int fdf, int fdt, off_t max_size); static void test_compress_decompress(int compression, compress_blob_t compress, - decompress_blob_t decompress) { - char text[] = "foofoofoofoo AAAA aaaaaaaaa ghost busters barbarbar FFF" - "foofoofoofoo AAAA aaaaaaaaa ghost busters barbarbar FFF"; + decompress_blob_t decompress, + const char *data, + size_t data_len, + bool may_fail) { char compressed[512]; size_t csize = 512; size_t usize = 0; _cleanup_free_ char *decompressed = NULL; int r; - log_info("/* testing %s blob compression/decompression */", - object_compressed_to_string(compression)); - - r = compress(text, sizeof(text), compressed, &csize); - assert(r == 0); - r = decompress(compressed, csize, - (void **) &decompressed, &usize, &csize, 0); - assert(r == 0); - assert_se(decompressed); - assert_se(streq(decompressed, text)); + log_info("/* testing %s %s blob compression/decompression */", + object_compressed_to_string(compression), data); + + r = compress(data, data_len, compressed, &csize); + if (r == -ENOBUFS) { + log_info_errno(r, "compression failed: %m"); + assert_se(may_fail); + } else { + assert_se(r == 0); + r = decompress(compressed, csize, + (void **) &decompressed, &usize, &csize, 0); + assert_se(r == 0); + assert_se(decompressed); + assert_se(memcmp(decompressed, data, data_len) == 0); + } r = decompress("garbage", 7, (void **) &decompressed, &usize, &csize, 0); - assert(r < 0); + assert_se(r < 0); /* make sure to have the minimal lz4 compressed size */ r = decompress("00000000\1g", 9, (void **) &decompressed, &usize, &csize, 0); - assert(r < 0); + assert_se(r < 0); r = decompress("\100000000g", 9, (void **) &decompressed, &usize, &csize, 0); - assert(r < 0); + assert_se(r < 0); memzero(decompressed, usize); } static void test_decompress_startswith(int compression, compress_blob_t compress, - decompress_sw_t decompress_sw) { + decompress_sw_t decompress_sw, + const char *data, + size_t data_len, + bool may_fail) { - char text[] = "foofoofoofoo AAAA aaaaaaaaa ghost busters barbarbar FFF" - "foofoofoofoo AAAA aaaaaaaaa ghost busters barbarbar FFF"; char compressed[512]; size_t csize = 512; size_t usize = 0; _cleanup_free_ char *decompressed = NULL; + int r; - log_info("/* testing decompress_startswith with %s */", - object_compressed_to_string(compression)); + log_info("/* testing decompress_startswith with %s on %s text*/", + object_compressed_to_string(compression), data); + + r = compress(data, data_len, compressed, &csize); + if (r == -ENOBUFS) { + log_info_errno(r, "compression failed: %m"); + assert_se(may_fail); + return; + } + assert_se(r == 0); - assert_se(compress(text, sizeof(text), compressed, &csize) == 0); assert_se(decompress_sw(compressed, csize, (void **) &decompressed, &usize, - "foofoofoofoo", 12, ' ') > 0); + data, strlen(data), '\0') > 0); assert_se(decompress_sw(compressed, csize, (void **) &decompressed, &usize, - "foofoofoofoo", 12, 'w') == 0); + data, strlen(data), 'w') == 0); assert_se(decompress_sw(compressed, csize, (void **) &decompressed, @@ -118,7 +133,7 @@ static void test_decompress_startswith(int compression, csize, (void **) &decompressed, &usize, - "foofoofoofoo", 12, ' ') > 0); + data, strlen(data), '\0') > 0); } static void test_compress_stream(int compression, @@ -145,11 +160,11 @@ static void test_compress_stream(int compression, assert_se((dst = mkostemp_safe(pattern, O_RDWR|O_CLOEXEC)) >= 0); - assert(compress(src, dst, -1) == 0); + assert_se(compress(src, dst, -1) == 0); if (cat) { assert_se(asprintf(&cmd, "%s %s | diff %s -", cat, pattern, srcfile) > 0); - assert(system(cmd) == 0); + assert_se(system(cmd) == 0); } log_debug("/* test decompression */"); @@ -169,7 +184,7 @@ static void test_compress_stream(int compression, assert_se(lseek(dst, 1, SEEK_SET) == 1); r = decompress(dst, dst2, st.st_size); - assert(r == -EBADMSG); + assert_se(r == -EBADMSG); assert_se(lseek(dst, 0, SEEK_SET) == 0); assert_se(lseek(dst2, 0, SEEK_SET) == 0); @@ -181,20 +196,44 @@ static void test_compress_stream(int compression, } int main(int argc, char *argv[]) { + const char text[] = + "text\0foofoofoofoo AAAA aaaaaaaaa ghost busters barbarbar FFF" + "foofoofoofoo AAAA aaaaaaaaa ghost busters barbarbar FFF"; + + char data[512] = "random\0"; log_set_max_level(LOG_DEBUG); + random_bytes(data + 7, sizeof(data) - 7); + #ifdef HAVE_XZ - test_compress_decompress(OBJECT_COMPRESSED_XZ, compress_blob_xz, decompress_blob_xz); - test_decompress_startswith(OBJECT_COMPRESSED_XZ, compress_blob_xz, decompress_startswith_xz); + test_compress_decompress(OBJECT_COMPRESSED_XZ, compress_blob_xz, decompress_blob_xz, + text, sizeof(text), false); + test_compress_decompress(OBJECT_COMPRESSED_XZ, compress_blob_xz, decompress_blob_xz, + data, sizeof(data), true); + test_decompress_startswith(OBJECT_COMPRESSED_XZ, + compress_blob_xz, decompress_startswith_xz, + text, sizeof(text), false); + test_decompress_startswith(OBJECT_COMPRESSED_XZ, + compress_blob_xz, decompress_startswith_xz, + data, sizeof(data), true); test_compress_stream(OBJECT_COMPRESSED_XZ, "xzcat", compress_stream_xz, decompress_stream_xz, argv[0]); #else log_info("/* XZ test skipped */"); #endif + #ifdef HAVE_LZ4 - test_compress_decompress(OBJECT_COMPRESSED_LZ4, compress_blob_lz4, decompress_blob_lz4); - test_decompress_startswith(OBJECT_COMPRESSED_LZ4, compress_blob_lz4, decompress_startswith_lz4); + test_compress_decompress(OBJECT_COMPRESSED_LZ4, compress_blob_lz4, decompress_blob_lz4, + text, sizeof(text), false); + test_compress_decompress(OBJECT_COMPRESSED_LZ4, compress_blob_lz4, decompress_blob_lz4, + data, sizeof(data), true); + test_decompress_startswith(OBJECT_COMPRESSED_LZ4, + compress_blob_lz4, decompress_startswith_lz4, + text, sizeof(text), false); + test_decompress_startswith(OBJECT_COMPRESSED_LZ4, + compress_blob_lz4, decompress_startswith_lz4, + data, sizeof(data), true); /* Produced stream is not compatible with lz4 binary, skip lz4cat check. */ test_compress_stream(OBJECT_COMPRESSED_LZ4, NULL,