Re: [BUG MP1/MP2] Apache should bail out in case of syntax errors in section

[Philippe M. Chiasson]

I've identified the source of the problem I believe, so can you try this patch
and let me know if it works for you? If so, I'll be working it up a bit more and
get it in.

Index: src/modules/perl/modperl_cmd.c
===================================================================
--- src/modules/perl/modperl_cmd.c      (revision 156679)
+++ src/modules/perl/modperl_cmd.c      (working copy)
@@ -554,14 +554,17 @@
         ENTER;SAVETMPS;
         save_scalar(gv); /* local $0 */
         sv_setpv_mg(GvSV(gv), directive->filename);
-        eval_pv(arg, FALSE);
+        eval_sv(newSVpv(arg, 0), G_KEEPERR);
         modperl_env_sync_srv_env_hash2table(aTHX_ p, scfg);
         modperl_env_sync_dir_env_hash2table(aTHX_ p, dcfg);
         FREETMPS;LEAVE;
     }

--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

signature.asc
Description: OpenPGP digital signature