Skip to content

Commit a2e9307

Browse files
author
Felipe Zimmerle
committed
Revert "Proposed fix for msc_process_request_body call too early"
This reverts commit 96fbca4.
1 parent 61f2ff1 commit a2e9307

File tree

3 files changed

+3
-7
lines changed

3 files changed

+3
-7
lines changed

CHANGES

-2
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@ v3.0.x - YYYY-MMM-DD (To be released)
33

44
- Fix: avoids crash if UNIQUE_ID is not set
55
[Issue #46 - @Goron1606, @victorhora, @zimmerle]
6-
- Proposed fix for msc_process_request_body call too early
7-
[Issue #22 - @victorhora]
86
- Adds make install to the build system
97
[Issue #36 - @zimmerle, @ROBERT-MCDOWELL]
108
- Uses mod_unique if available

src/mod_security3.c

+3-2
Original file line numberDiff line numberDiff line change
@@ -398,8 +398,9 @@ static int hook_request_late(request_rec *r)
398398
return it;
399399
}
400400
#endif
401-
// FIXME: memsc_append_request_body wasn't called yet. Too early?
402-
// msc_process_request_body(msr->t);
401+
402+
403+
msc_process_request_body(msr->t);
403404
it = process_intervention(msr->t, r);
404405
if (it != N_INTERVENTION_STATUS)
405406
{

src/msc_filters.c

-3
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,6 @@ apr_status_t input_filter(ap_filter_t *f, apr_bucket_brigade *pbbOut,
6262
return send_error_bucket(msr, f, it);
6363
}
6464

65-
// FIXME: Now we should have the body. Is this sane?
66-
msc_process_request_body(msr->t);
67-
6865
pbktOut = apr_bucket_heap_create(data, len, 0, c->bucket_alloc);
6966
APR_BRIGADE_INSERT_TAIL(pbbOut, pbktOut);
7067
apr_bucket_delete(pbktIn);

0 commit comments

Comments
 (0)