summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorDavid Oberhollenzer <david.oberhollenzer@sigma-star.at>2021-09-12 23:52:40 +0200
committerDavid Oberhollenzer <david.oberhollenzer@sigma-star.at>2021-09-13 00:03:39 +0200
commite5540b5a3a545ed4e3f3df97be131ec81a155032 (patch)
treeee8725f594ae602dd8043abf2f960e387d5a12d0 /tests
parent3ef7c3bb37e40de2653daf306e8bcb2a87446271 (diff)
Fix struct offset testing in ABI test case
The intention of the (severely incomplete) ABI test case is to detect changes to the ABI of libsquashfs. Currently it tries to blurt out if the layout of some structure is changed unintentionally. Unfortunately, the test uses some unportable assumptions. Among other things, it was assumed that a 64 bit field will always require 64 bit alignment. This is apparently no the case on 32 bit x86. This patch makes the check work on 32 bit and 64 bit x86, by adding an additional runtime check that relies on the __alignof__ extension offered by gcc and clang (the only 2 compilers that are really supported at the moment). Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'tests')
-rw-r--r--tests/libsqfs/abi.c71
1 files changed, 52 insertions, 19 deletions
diff --git a/tests/libsqfs/abi.c b/tests/libsqfs/abi.c
index 04bbc30..d409f67 100644
--- a/tests/libsqfs/abi.c
+++ b/tests/libsqfs/abi.c
@@ -35,8 +35,14 @@ static void test_compressor_opt_struct(void)
sizeof(sqfs_u32));
TEST_EQUAL_UI(offsetof(sqfs_compressor_config_t, level),
(2 * sizeof(sqfs_u32)));
- TEST_EQUAL_UI(offsetof(sqfs_compressor_config_t, opt),
- (4 * sizeof(sqfs_u32)));
+
+ if (__alignof__(sqfs_compressor_config_t) == __alignof__(sqfs_u32)) {
+ TEST_EQUAL_UI(offsetof(sqfs_compressor_config_t, opt),
+ (3 * sizeof(sqfs_u32)));
+ } else if (__alignof__(sqfs_compressor_config_t) == __alignof__(sqfs_u64)) {
+ TEST_EQUAL_UI(offsetof(sqfs_compressor_config_t, opt),
+ (4 * sizeof(sqfs_u32)));
+ }
}
static const char *names[] = {
@@ -65,33 +71,60 @@ static void test_compressor_names(void)
static void test_blockproc_stats(void)
{
sqfs_block_processor_stats_t stats;
+ size_t off;
+
+ TEST_EQUAL_UI(sizeof(stats.size), sizeof(size_t));
+ TEST_EQUAL_UI(sizeof(stats.input_bytes_read), sizeof(sqfs_u64));
+ TEST_EQUAL_UI(sizeof(stats.output_bytes_generated), sizeof(sqfs_u64));
+ TEST_EQUAL_UI(sizeof(stats.data_block_count), sizeof(sqfs_u64));
+ TEST_EQUAL_UI(sizeof(stats.frag_block_count), sizeof(sqfs_u64));
+ TEST_EQUAL_UI(sizeof(stats.sparse_block_count), sizeof(sqfs_u64));
+ TEST_EQUAL_UI(sizeof(stats.total_frag_count), sizeof(sqfs_u64));
+ TEST_EQUAL_UI(sizeof(stats.actual_frag_count), sizeof(sqfs_u64));
- TEST_ASSERT(sizeof(stats) >= (8 * sizeof(sqfs_u64)));
+ if (__alignof__(stats) == __alignof__(sqfs_u32)) {
+ TEST_ASSERT(sizeof(stats) >=
+ (sizeof(sqfs_u32) + 7 * sizeof(sqfs_u64)));
+ } else if (__alignof__(stats) == __alignof__(sqfs_u64)) {
+ TEST_ASSERT(sizeof(stats) >= (8 * sizeof(sqfs_u64)));
+ }
TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t, size), 0);
+
+ if (sizeof(size_t) < sizeof(sqfs_u64) &&
+ (__alignof__(sqfs_block_processor_stats_t) ==
+ __alignof__(sqfs_u64))) {
+ off = sizeof(sqfs_u64);
+ } else {
+ off = sizeof(stats.size);
+ }
+
TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t,
- input_bytes_read), sizeof(sqfs_u64));
- TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t,
- output_bytes_generated), 2 * sizeof(sqfs_u64));
+ input_bytes_read), off);
+ off += sizeof(sqfs_u64);
+
TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t,
- data_block_count), 3 * sizeof(sqfs_u64));
+ output_bytes_generated), off);
+ off += sizeof(sqfs_u64);
+
TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t,
- frag_block_count), 4 * sizeof(sqfs_u64));
+ data_block_count), off);
+ off += sizeof(sqfs_u64);
+
TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t,
- sparse_block_count), 5 * sizeof(sqfs_u64));
+ frag_block_count), off);
+ off += sizeof(sqfs_u64);
+
TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t,
- total_frag_count), 6 * sizeof(sqfs_u64));
+ sparse_block_count), off);
+ off += sizeof(sqfs_u64);
+
TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t,
- actual_frag_count), 7 * sizeof(sqfs_u64));
+ total_frag_count), off);
+ off += sizeof(sqfs_u64);
- TEST_EQUAL_UI(sizeof(stats.size), sizeof(size_t));
- TEST_EQUAL_UI(sizeof(stats.input_bytes_read), sizeof(sqfs_u64));
- TEST_EQUAL_UI(sizeof(stats.output_bytes_generated), sizeof(sqfs_u64));
- TEST_EQUAL_UI(sizeof(stats.data_block_count), sizeof(sqfs_u64));
- TEST_EQUAL_UI(sizeof(stats.frag_block_count), sizeof(sqfs_u64));
- TEST_EQUAL_UI(sizeof(stats.sparse_block_count), sizeof(sqfs_u64));
- TEST_EQUAL_UI(sizeof(stats.total_frag_count), sizeof(sqfs_u64));
- TEST_EQUAL_UI(sizeof(stats.actual_frag_count), sizeof(sqfs_u64));
+ TEST_EQUAL_UI(offsetof(sqfs_block_processor_stats_t,
+ actual_frag_count), off);
}
static void test_blockproc_desc(void)