
rpluem at apache
Jul 4, 2009, 12:54 AM
Views: 135
Permalink
|
|
svn commit: r791078 - /httpd/httpd/branches/2.2.x/STATUS
|
|
Author: rpluem Date: Sat Jul 4 07:54:07 2009 New Revision: 791078 URL: http://svn.apache.org/viewvc?rev=791078&view=rev Log: * Add comment and vote. Modified: httpd/httpd/branches/2.2.x/STATUS Modified: httpd/httpd/branches/2.2.x/STATUS URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=791078&r1=791077&r2=791078&view=diff ============================================================================== --- httpd/httpd/branches/2.2.x/STATUS (original) +++ httpd/httpd/branches/2.2.x/STATUS Sat Jul 4 07:54:07 2009 @@ -92,13 +92,16 @@ Trunk version of patch: #1 folded in during core output filter refactoring #2 http://svn.apache.org/viewvc?view=rev&revision=521681 - +1: jorton, jim + +1: jorton, jim, rpluem rpluem asks: Are we sure that b is never NULL? Otherwise we would need to add http://svn.apache.org/viewvc?view=rev&revision=568202 as on trunk to avoid segfaults. trawick responds: if b were NULL, we would have segfaulted earlier when ap_pass_brigade "calls" APR_BRIGADE_LAST(bb) + rpluem: Ahh good point. Meanwhile I had a look at trunk and the + event MPM is calling the core output filter directly without + ap_pass_brigade. So I am +1. PATCHES ACCEPTED TO BACKPORT FROM TRUNK: [ start all new proposals below, under PATCHES PROPOSED. ]
|