lib: test-file-cache - Assert map is not null before memcmp

diff --git a/src/lib/test-file-cache.c b/src/lib/test-file-cache.c
index 0efb345dd60b6160d2b4d30175a8ba736a92cf88..6bac9abfe7ecdf851cad7660afa535e37ea15d4d 100644 (file)
--- a/src/lib/test-file-cache.c
+++ b/src/lib/test-file-cache.c
@@ -37,7 +37,7 @@ static void test_file_cache_read(void)
 
        test_assert(file_cache_read(cache, 0, 13) == 13);
        map = file_cache_get_map(cache, &size);
-       test_assert(size == 13 && memcmp(map, "initial data\n", 13) == 0);
+       test_assert(map != NULL && size == 13 && memcmp(map, "initial data\n", 13) == 0);
 
        file_cache_free(&cache);
        i_close_fd(&fd);
@@ -69,14 +69,14 @@ static void test_file_cache_write_read(void)
        test_assert(file_cache_read(cache, 0, 13) == 13);
        file_cache_write(cache, "updated data\n", 13, 0);
        map = file_cache_get_map(cache, &size);
-       test_assert(size == 13 && memcmp(map, "updated data\n", 13) == 0);
+       test_assert(map != NULL && size == 13 && memcmp(map, "updated data\n", 13) == 0);
        file_cache_free(&cache);
        i_close_fd(&fd);
 
        struct istream *is = i_stream_create_file(TEST_FILENAME, SIZE_MAX);
        const unsigned char *data;
        test_assert(i_stream_read_more(is, &data, &size) > 0 && size == 13);
-       test_assert(size == 13 && memcmp(data, "initial data\n", 13) == 0);
+       test_assert(map != NULL && size == 13 && memcmp(data, "initial data\n", 13) == 0);
        i_stream_destroy(&is);
        i_unlink(TEST_FILENAME);
 
@@ -101,7 +101,7 @@ static void test_file_cache_read_invalidate(void)
        size_t size;
        test_assert(file_cache_read(cache, 0, 13) == 13);
        const unsigned char *map = file_cache_get_map(cache, &size);
-       test_assert(size == 13 && memcmp(map, "initial data\n", 13) == 0);
+       test_assert(map != NULL && size == 13 && memcmp(map, "initial data\n", 13) == 0);
 
        /* update file */
        os = o_stream_create_file(TEST_FILENAME, 0, 0600, 0);
@@ -110,14 +110,14 @@ static void test_file_cache_read_invalidate(void)
        o_stream_destroy(&os);
 
        map = file_cache_get_map(cache, &size);
-       test_assert(size == 13 && memcmp(map, "initial data\n", 13) == 0);
+       test_assert(map != NULL && size == 13 && memcmp(map, "initial data\n", 13) == 0);
 
        /* invalidate cache */
        file_cache_invalidate(cache, 0, size);
        test_assert(file_cache_read(cache, 0, 13) == 13);
        map = file_cache_get_map(cache, &size);
        test_assert(size == 13);
-       test_assert(size == 13 && memcmp(map, "updated data\n", 13) == 0);
+       test_assert(map != NULL && size == 13 && memcmp(map, "updated data\n", 13) == 0);
        file_cache_free(&cache);
        i_close_fd(&fd);
        i_unlink(TEST_FILENAME);
@@ -194,8 +194,7 @@ static void test_file_cache_anon(void)
 
        size_t size;
        const unsigned char *map = file_cache_get_map(cache, &size);
-       test_assert(map != NULL);
-       test_assert(size == 12 && memcmp(map, "initial data", 12) == 0);
+       test_assert(map != NULL && size == 12 && memcmp(map, "initial data", 12) == 0);
 
        file_cache_free(&cache);
        i_unlink_if_exists(TEST_FILENAME);
@@ -225,8 +224,7 @@ static void test_file_cache_switch_fd(void)
        test_assert(file_cache_read(cache, 0, 13) == 13);
        size_t size;
        const unsigned char *map = file_cache_get_map(cache, &size);
-       test_assert(map != NULL);
-       test_assert(size == 13 && memcmp(map, "updated data\n", 13) == 0);
+       test_assert(map != NULL && size == 13 && memcmp(map, "updated data\n", 13) == 0);
 
        file_cache_free(&cache);
        i_close_fd(&fd);