MINOR: cache: use pool_alloc(), not pool_alloc_dirty()

pool_alloc_dirty() is the version below pool_alloc() that never performs
the memory poisonning. It should only be called directly for very large
unstructured areas for which enabling memory poisonning would not bring
anything but could significantly hurt performance (e.g. buffers). Using
this function here will not provide any benefit and will hurt the ability
to debug.

It would be desirable to backport this, although it does not cause any
user-visible bug, it just complicates debugging.

diff --git a/src/cache.c b/src/cache.c
index 12609e1ac0d0fe88ae80928db3a541b882a7c1a0..cffd63b6f5c50bd08c57aad3d6cacd9db110bb12 100644 (file)
--- a/src/cache.c
+++ b/src/cache.c
@@ -493,7 +493,7 @@ cache_store_strm_init(struct stream *s, struct filter *filter)
 {
        struct cache_st *st;
 
-       st = pool_alloc_dirty(pool_head_cache_st);
+       st = pool_alloc(pool_head_cache_st);
        if (st == NULL)
                return -1;