httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe Jr." <wr...@rowe-clan.net>
Subject Re: svn commit: r912666 - in /httpd/httpd/trunk: CHANGES include/util_mutex.h server/core.c server/util_mutex.c
Date Wed, 24 Feb 2010 04:07:05 GMT
On 2/22/2010 11:16 AM, trawick@apache.org wrote:
> Author: trawick
> Date: Mon Feb 22 17:16:29 2010
> New Revision: 912666
> 
> URL: http://svn.apache.org/viewvc?rev=912666&view=rev
> Log:
> Fix startup segfault when the Mutex directive is used but no loaded 
> modules use httpd mutexes.
> 
> Add an init call from core's pre-config hook to ensure init is
> performed before the config is parsed.

This logic seems broken.

The problem is that anonymous mutexes against a persistant data store can't be
closed and reopened between server generations.  This use of pre/post config
persists the broken design.

This layer of init really should happen against process pool, within the scope
of the register hooks.  No?

There's no hassle for a named mutex object that corresponds to the persistent
object, e.g. a named file lock against a named database.

Mime
View raw message