JBossWeb SVN: r1487 - in trunk: webapps/docs and 1 other directory. - jbossweb-commits

Friday, 11 June 2010

Author: remy.maucherat(a)jboss.com
Date: 2010-06-11 09:07:46 -0400 (Fri, 11 Jun 2010)
New Revision: 1487

Modified:
   trunk/java/org/apache/catalina/connector/Request.java
   trunk/webapps/docs/changelog.xml
Log:
- NPE fix in chunk with no data edge case.

Modified: trunk/java/org/apache/catalina/connector/Request.java
===================================================================

--- trunk/java/org/apache/catalina/connector/Request.java	2010-06-10 21:52:05 UTC (rev
1486)
+++ trunk/java/org/apache/catalina/connector/Request.java	2010-06-11 13:07:46 UTC (rev
1487)
@@ -2807,7 +2807,9 @@
                 }
                 return;
             }
-            parameters.processParameters(formData, 0, formData.length);
+            if (formData != null) {
+                parameters.processParameters(formData, 0, formData.length);
+            }
         }
 
     }
@@ -2834,6 +2836,9 @@
                 body.append(buffer, 0, len);
             }
         }
+        if (body.getLength() == 0) {
+            return null;
+        }
         if (body.getLength() < body.getBuffer().length) {
             int length = body.getLength();
             byte[] result = new byte[length];

Modified: trunk/webapps/docs/changelog.xml
===================================================================
--- trunk/webapps/docs/changelog.xml	2010-06-10 21:52:05 UTC (rev 1486)
+++ trunk/webapps/docs/changelog.xml	2010-06-11 13:07:46 UTC (rev 1487)
@@ -16,6 +16,16 @@
 
 <body>
 
+<section name="JBoss Web 3.0.0.Beta7 (remm)">
+  <subsection name="Catalina">
+    <changelog>
+      <fix>
+        Fix NPE processing some POST. (markt)
+      </fix>
+    </changelog>
+  </subsection>
+</section>
+
 <section name="JBoss Web 3.0.0.Beta6 (remm)">
   <subsection name="General">
     <changelog>


    

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

JBossWeb SVN: r1487 - in trunk: webapps/docs and 1 other directory.