[PATCH] Re: Seg fault: race conditions in mod_mem_cache.c

[Jean-Jacques Clar]

Are there any reasons why that part of decrement_refcount() is not needed anymore?   @@ -267,31 +271,9 @@   {       cache_object_t *obj = (cache_object_t *) arg; -    /* If obj->complete is not set, the cache update failed and the -     * object needs to be removed from the cache then cleaned up. -     */ -    if (!obj->complete) { -        if (sconf->lock) { -            apr_thread_mutex_lock(sconf->lock); -        } -        /* Remember, objects marked for cleanup are, by design, already -         * removed from the cache. remove_url() could have already -         * removed the object from the cache (and set obj->cleanup) -         */ -        if (!obj->cleanup) { -            cache_remove(sconf->cache_cache, obj); -            obj->cleanup = 1; -        } -        if (sconf->lock) { -            apr_thread_mutex_unlock(sconf->lock); -        } -    } -