diff options
| -rw-r--r-- | changes/bug17724 | 4 | ||||
| -rw-r--r-- | src/test/test_rendcache.c | 8 |
2 files changed, 7 insertions, 5 deletions
diff --git a/changes/bug17724 b/changes/bug17724 new file mode 100644 index 0000000000..7ace99eece --- /dev/null +++ b/changes/bug17724 @@ -0,0 +1,4 @@ + o Minor bug fixes (unit tests, hidden services): + - Avoid relying on malloc internals in test_rend_cache_purge. + Closes ticket 17724. Bug fix on ade5005853c1 and 5e9f2384cf0f, + not in any released version of Tor. Patch by "teor". diff --git a/src/test/test_rendcache.c b/src/test/test_rendcache.c index 11f11147cb..92adf01273 100644 --- a/src/test/test_rendcache.c +++ b/src/test/test_rendcache.c @@ -1024,24 +1024,22 @@ test_rend_cache_entry_free(void *data) static void test_rend_cache_purge(void *data) { - strmap_t *our_rend_cache; - (void)data; // Deals with a NULL rend_cache rend_cache_purge(); tt_assert(rend_cache); - tt_int_op(strmap_size(rend_cache), OP_EQ, 0); + tt_assert(strmap_size(rend_cache) == 0); // Deals with existing rend_cache rend_cache_free_all(); rend_cache_init(); + tt_assert(rend_cache); + tt_assert(strmap_size(rend_cache) == 0); - our_rend_cache = rend_cache; rend_cache_purge(); tt_assert(rend_cache); tt_assert(strmap_size(rend_cache) == 0); - tt_assert(rend_cache == our_rend_cache); done: rend_cache_free_all(); |