You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by jo...@apache.org on 2011/02/01 03:41:46 UTC
svn commit: r1065898 -
/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/HttpRequestHandler.java
Author: johnh
Date: Tue Feb 1 02:41:46 2011
New Revision: 1065898
URL: http://svn.apache.org/viewvc?rev=1065898&view=rev
Log:
Simple check to avoid StringIndexOutOfBounds on charAt for empty body.
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/HttpRequestHandler.java
Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/HttpRequestHandler.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/HttpRequestHandler.java?rev=1065898&r1=1065897&r2=1065898&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/HttpRequestHandler.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/HttpRequestHandler.java Tue Feb 1 02:41:46 2011
@@ -296,7 +296,7 @@ public class HttpRequestHandler {
} else if ("json".equalsIgnoreCase(request.format)) {
try {
body = body.trim();
- if(body.charAt(0) == '[') {
+ if(body.length() > 0 && body.charAt(0) == '[') {
return new JSONArray(body);
} else {
return new JSONObject(body);