diff options
| author | Nick Mathewson <nickm@torproject.org> | 2018-04-12 12:24:36 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2018-04-12 12:30:36 -0400 |
| commit | 46795a7be63b9a1b90a59fcf9efda4f4f1eacc37 (patch) | |
| tree | c19614937bdc61d65da6f46dc2e6fd1a99316148 /src/test/test_config.c | |
| parent | 4aaa4215e7e11f318c5a50124e29dc0b50ce21e1 (diff) | |
| download | tor-46795a7be63b9a1b90a59fcf9efda4f4f1eacc37.tar.gz tor-46795a7be63b9a1b90a59fcf9efda4f4f1eacc37.zip | |
Attempt to fix 32-bit clang builds, which broke with 31508a0abccfee1cd
When size_t is 32 bits, the unit tests can't fit anything more than
4GB-1 into a size_t.
Additionally, tt_int_op() uses "long" -- we need tt_u64_op() to
safely test uint64_t values for equality.
Bug caused by tests for #24782 fix; not in any released Tor.
Diffstat (limited to 'src/test/test_config.c')
| -rw-r--r-- | src/test/test_config.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/src/test/test_config.c b/src/test/test_config.c index d82f6d21da..e214a82771 100644 --- a/src/test/test_config.c +++ b/src/test/test_config.c @@ -5626,10 +5626,10 @@ test_config_compute_max_mem_in_queues(void *data) #if SIZEOF_VOID_P >= 8 /* We are on a 64-bit system. */ - tt_int_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, GIGABYTE(8)); + tt_u64_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, GIGABYTE(8)); #else /* We are on a 32-bit system. */ - tt_int_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, GIGABYTE(1)); + tt_u64_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, GIGABYTE(1)); #endif /* We are able to detect the amount of RAM on the system. */ @@ -5639,7 +5639,7 @@ test_config_compute_max_mem_in_queues(void *data) total_system_memory_output = GIGABYTE(1); /* We have 0.75 * RAM available. */ - tt_int_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, + tt_u64_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, 3 * (GIGABYTE(1) / 4)); /* We are running on a tiny machine with 256 MB of RAM. */ @@ -5648,28 +5648,30 @@ test_config_compute_max_mem_in_queues(void *data) /* We will now enforce a minimum of 256 MB of RAM available for the * MaxMemInQueues here, even though we should only have had 0.75 * 256 = 192 * MB available. */ - tt_int_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, MEGABYTE(256)); + tt_u64_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, MEGABYTE(256)); +#if SIZEOF_SIZE_T > 4 /* We are running on a machine with 8 GB of RAM. */ total_system_memory_output = GIGABYTE(8); /* We will have 0.4 * RAM available. */ - tt_int_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, + tt_u64_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, 2 * (GIGABYTE(8) / 5)); /* We are running on a machine with 16 GB of RAM. */ total_system_memory_output = GIGABYTE(16); /* We will have 0.4 * RAM available. */ - tt_int_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, + tt_u64_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, 2 * (GIGABYTE(16) / 5)); /* We are running on a machine with 32 GB of RAM. */ total_system_memory_output = GIGABYTE(32); /* We will at maximum get MAX_DEFAULT_MEMORY_QUEUE_SIZE here. */ - tt_int_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, + tt_u64_op(compute_real_max_mem_in_queues(0, 0), OP_EQ, MAX_DEFAULT_MEMORY_QUEUE_SIZE); +#endif done: UNMOCK(get_total_system_memory); |