Dominique Quatravaux wrote:
Philippe M. Chiasson a écrit :

| 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?

Works! BTW, your patch was defective but Emacs' diff-mode fixed that
in no time. Therefore I'm re-attaching it for the record.

As is, it was leaking a SV. A better patch is now on the [EMAIL PROTECTED]

Index: src/modules/perl/modperl_cmd.c
===================================================================
--- src/modules/perl/modperl_cmd.c      (revision 156679)
+++ src/modules/perl/modperl_cmd.c      (working copy)
@@ -554,8 +554,8 @@
~         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

Attachment: signature.asc
Description: OpenPGP digital signature



Reply via email to