Author: torsten
Date: Wed Apr  7 11:39:02 2010
New Revision: 931506

URL: http://svn.apache.org/viewvc?rev=931506&view=rev
Log:
Fix the check for the FilterInitHandler attribute in 
modperl_filter_resolve_init_handler()

Modified:
    perl/modperl/trunk/Changes
    perl/modperl/trunk/src/modules/perl/modperl_filter.c

Modified: perl/modperl/trunk/Changes
URL: 
http://svn.apache.org/viewvc/perl/modperl/trunk/Changes?rev=931506&r1=931505&r2=931506&view=diff
==============================================================================
--- perl/modperl/trunk/Changes (original)
+++ perl/modperl/trunk/Changes Wed Apr  7 11:39:02 2010
@@ -12,6 +12,9 @@ Also refer to the Apache::Test changes l
 
 =item 2.0.5-dev
 
+Fix the filter init handler attribute check in 
+modperl_filter_resolve_init_handler() [Torsten Foertsch]
+
 Make sure buffer is a valid SV in modperl_filter_read() [Torsten Foertsch]
 
 Move modperl_response_finish() out of modperl_response_handler_run in

Modified: perl/modperl/trunk/src/modules/perl/modperl_filter.c
URL: 
http://svn.apache.org/viewvc/perl/modperl/trunk/src/modules/perl/modperl_filter.c?rev=931506&r1=931505&r2=931506&view=diff
==============================================================================
--- perl/modperl/trunk/src/modules/perl/modperl_filter.c (original)
+++ perl/modperl/trunk/src/modules/perl/modperl_filter.c Wed Apr  7 11:39:02 
2010
@@ -404,10 +404,12 @@ int modperl_filter_resolve_init_handler(
         FREETMPS;LEAVE;
 
         if (init_handler) {
+            modperl_mgv_resolve(aTHX_ init_handler, p, init_handler->name, 1);
+
             MP_TRACE_h(MP_FUNC, "found init handler %s",
                        modperl_handler_name(init_handler));
 
-            if (!init_handler->attrs & MP_FILTER_INIT_HANDLER) {
+            if (!(init_handler->attrs & MP_FILTER_INIT_HANDLER)) {
                 Perl_croak(aTHX_ "handler %s doesn't have "
                            "the FilterInitHandler attribute set",
                            modperl_handler_name(init_handler));


Reply via email to