Stas Bekman wrote:

Does this look good?

Index: modules/filters/mod_include.c
===================================================================
RCS file: /home/cvs/httpd-2.0/modules/filters/mod_include.c,v
retrieving revision 1.291
diff -u -r1.291 mod_include.c
--- modules/filters/mod_include.c 9 Dec 2003 01:58:21 -0000 1.291
+++ modules/filters/mod_include.c 9 Dec 2003 18:30:22 -0000
@@ -3603,7 +3603,8 @@
if (!(ap_allow_options(r) & OPT_INCLUDES)) {
ap_log_rerror(APLOG_MARK, APLOG_WARNING, 0, r,
"mod_include: Options +Includes (or IncludesNoExec) "
- "wasn't set, passing data unmodified");
+ "wasn't set, passing data unmodified, removing myself");
+ ap_remove_output_filter(f);
return ap_pass_brigade(f->next, b);

That looks good to me. I would have done it myself but I didn't anticipate having time in the short term, hence the nag.



Reply via email to