diff options
| author | David Goulet <dgoulet@torproject.org> | 2018-11-29 10:02:50 -0500 |
|---|---|---|
| committer | David Goulet <dgoulet@torproject.org> | 2019-02-12 09:15:41 -0500 |
| commit | 46b6df7122f1fd152df92f1c6aa75c776d5a83b7 (patch) | |
| tree | 4053c8dce09149d51f51b38c2f020461a5c41748 | |
| parent | ab6534781996404ca57e9f5b0cbdda998579c329 (diff) | |
| download | tor-46b6df7122f1fd152df92f1c6aa75c776d5a83b7.tar.gz tor-46b6df7122f1fd152df92f1c6aa75c776d5a83b7.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); |