diff options
| author | David Goulet <dgoulet@torproject.org> | 2018-11-29 10:02:50 -0500 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2019-01-23 13:11:49 -0500 |
| commit | 2d74da3d0efd7a67f9bdb73145c38fe047faf1f8 (patch) | |
| tree | 13cc319ad6f29fa3ffc5e5043a179befdfc2dd0c | |
| parent | 6144cf99adae5ffebbc549a4e0f3c6e05e51ad16 (diff) | |
| download | tor-2d74da3d0efd7a67f9bdb73145c38fe047faf1f8.tar.gz tor-2d74da3d0efd7a67f9bdb73145c38fe047faf1f8.zip | |
test: Fix a warning underflow in rend_cache/clean
Because the test is adding entries to the "rend_cache" directly, the
rend_cache_increment_allocation() was never called which made the
rend_cache_clean() call trigger that underflow warning:
rend_cache/clean: [forking] Nov 29 09:55:04.024 [warn] rend_cache_decrement_allocation(): Bug: Underflow in rend_cache_decrement_allocation (on Tor 0.4.0.0-alpha-dev 2240fe63feb9a8cf)
The test is still good and valid.
Fixes #28660
Signed-off-by: David Goulet <dgoulet@torproject.org>
| -rw-r--r-- | src/test/test_rendcache.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/test/test_rendcache.c b/src/test/test_rendcache.c index 7ec4771b6e..4f544cf21c 100644 --- a/src/test/test_rendcache.c +++ b/src/test/test_rendcache.c @@ -788,7 +788,9 @@ test_rend_cache_clean(void *data) desc_two->pk = pk_generate(1); strmap_set_lc(rend_cache, "foo1", one); + rend_cache_increment_allocation(rend_cache_entry_allocation(one)); strmap_set_lc(rend_cache, "foo2", two); + rend_cache_increment_allocation(rend_cache_entry_allocation(two)); rend_cache_clean(time(NULL), REND_CACHE_TYPE_CLIENT); tt_int_op(strmap_size(rend_cache), OP_EQ, 0); @@ -806,7 +808,9 @@ test_rend_cache_clean(void *data) desc_one->pk = pk_generate(0); desc_two->pk = pk_generate(1); + rend_cache_increment_allocation(rend_cache_entry_allocation(one)); strmap_set_lc(rend_cache, "foo1", one); + rend_cache_increment_allocation(rend_cache_entry_allocation(two)); strmap_set_lc(rend_cache, "foo2", two); rend_cache_clean(time(NULL), REND_CACHE_TYPE_CLIENT); |