Author: timw
Date: Mon Apr 11 09:13:47 2011
New Revision: 1090984

URL: http://svn.apache.org/viewvc?rev=1090984&view=rev
Log:
Fixing debug logging for forwarded headers to not crash, and to include 
explicit Content-Length: 0 added for AJP13.

Modified:
    tomcat/jk/trunk/native/iis/jk_isapi_plugin.c

Modified: tomcat/jk/trunk/native/iis/jk_isapi_plugin.c
URL: 
http://svn.apache.org/viewvc/tomcat/jk/trunk/native/iis/jk_isapi_plugin.c?rev=1090984&r1=1090983&r2=1090984&view=diff
==============================================================================
--- tomcat/jk/trunk/native/iis/jk_isapi_plugin.c (original)
+++ tomcat/jk/trunk/native/iis/jk_isapi_plugin.c Mon Apr 11 09:13:47 2011
@@ -3276,11 +3276,11 @@ static int init_ws_service(isapi_private
                 }
 
                 if (real_header) {
-                    i++;
                     if (JK_IS_DEBUG_LEVEL(logger)) {
                         jk_log(logger, JK_LOG_DEBUG, "Forwarding request 
header %s : %s",
                                s->headers_names[i], s->headers_values[i]);
                     }
+                    i++;
                 }
             }
             /* Add a content-length = 0 header if needed.
@@ -3288,6 +3288,9 @@ static int init_ws_service(isapi_private
              * but non-zero length body.
              */
             if (need_content_length_header) {
+                if (JK_IS_DEBUG_LEVEL(logger)) {
+                    jk_log(logger, JK_LOG_DEBUG, "Incoming request needs 
explicit Content-Length: 0 in AJP13");
+                }
                 s->headers_names[cnt] = "Content-Length";
                 s->headers_values[cnt] = "0";
                 cnt++;



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to