I believe it is a kosher for a filter to return EOS, but return APR_SUCCESS

via ap_get_brigade.  So, we should treat this as end-of-input.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@91311 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/server/protocol.c b/server/protocol.c
index 3bdba03c5a1f0a51d8bba87d271b3c832194504e..9aebceb5cb7d24937a5a56e2290c5322e700b9db 100644 (file)
--- a/server/protocol.c
+++ b/server/protocol.c
@@ -227,6 +227,10 @@ AP_CORE_DECLARE(int) ap_getline(char *s, int n, request_rec *r, int fold)
             }
         }
         e = APR_BRIGADE_FIRST(b); 
+        if (APR_BUCKET_IS_EOS(e)) {
+            apr_brigade_destroy(b);
+            return -1;
+        }
         if (e->length == 0) {
             apr_bucket_delete(e);
             continue;