Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: ModPerl: Dev

[httpd24] [PATCH] Make sure authn and authz providers are cleaned up in time

 

 

ModPerl dev RSS feed   Index | Next | Previous | View Threaded


ntyni at debian

Jun 23, 2013, 12:55 PM

Post #1 of 2 (87 views)
Permalink
[httpd24] [PATCH] Make sure authn and authz providers are cleaned up in time

As discussed in <http://bugs.debian.org/711213>, we encountered somewhat
non-deterministic SIGSEGVs at the end of the mod_perl test suite when
building our packages based on the httpd24 branch (merged with the
2.0.8 release.)

It turns out the 'pre_cleanup' function cleanup_perl_global_providers()
was getting called after mod_perl had been unloaded, because the cleanup
was registered against the server process pool. This pool has a longer
lifetime than that of the mod_perl shared object.

As suggested by Stefan Fritsch, registering the cleanup against the
configuration pool seems to work better. This fixes the problem completely
according to my tests. Proposed patch attached.
--
Niko Tyni ntyni [at] debian
Attachments: 0001-Make-sure-authn-and-authz-providers-are-cleaned-up-i.patch (1.39 KB)


jkaluza at redhat

Jun 24, 2013, 3:33 AM

Post #2 of 2 (75 views)
Permalink
Re: [httpd24] [PATCH] Make sure authn and authz providers are cleaned up in time [In reply to]

On 06/23/2013 09:55 PM, Niko Tyni wrote:
> As discussed in <http://bugs.debian.org/711213>, we encountered somewhat
> non-deterministic SIGSEGVs at the end of the mod_perl test suite when
> building our packages based on the httpd24 branch (merged with the
> 2.0.8 release.)
>
> It turns out the 'pre_cleanup' function cleanup_perl_global_providers()
> was getting called after mod_perl had been unloaded, because the cleanup
> was registered against the server process pool. This pool has a longer
> lifetime than that of the mod_perl shared object.
>
> As suggested by Stefan Fritsch, registering the cleanup against the
> configuration pool seems to work better. This fixes the problem completely
> according to my tests. Proposed patch attached.
>

Applied in 1495982.

Thanks,
Jan Kaluza

>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe [at] perl
> For additional commands, e-mail: dev-help [at] perl
>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] perl
For additional commands, e-mail: dev-help [at] perl

ModPerl dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.