On 2021-11-03 17:14+0100, Stefan Sperling wrote: > On Wed, Nov 03, 2021 at 04:47:39PM +0100, Thomas Weißschuh wrote: > > The svn_atomic__init_once() inside svn_repos_authz_initialize() seems to > > work > > correctly. synchronized_authz_initialize() is only executed once. > > > > I am arguing that it should *not* be executed only once because the pools > > that > > it gets called with the first time will be invalid when it is called the > > second > > time. So it should rebuild authz_pool with the new parent pool. > > > > Replacing the call through svn_atomic__init_once() with an unconditional > > call > > makes the issue go away. > > I see. So for some reason, under httpd's control, the SVN module's > post_config hook is invoked several times, each time with a different > pool? Do we know why this is happening?
I'll investigate that a bit more. > If this is valid httpd behaviour then the code in authz.c is making > an invalid assumption. And there could similar problems elsewhere, > in svn_fs_initialize() for example. Should we keep this discussion on the ML or should I open a ticket for it?