Merge pull request alibaba#90 from stepinto/master

Fix issue alibaba#89
zdyz203 · Oct 14, 2013 · 78e2651 · 78e2651
2 parents 0761e0a + c38c67c
commit 78e2651
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/src/main/java/com/alibaba/fastjson/parser/JSONLexerBase.java b/src/main/java/com/alibaba/fastjson/parser/JSONLexerBase.java
@@ -2539,7 +2539,7 @@ public final boolean isBlankInput() {
 
     public final void skipWhitespace() {
         for (;;) {
-            if (whitespaceFlags[ch]) {
+            if (ch < whitespaceFlags.length && whitespaceFlags[ch]) {
                 next();
                 continue;
             } else {

diff --git a/src/test/java/com/alibaba/json/bvt/Bug89.java b/src/test/java/com/alibaba/json/bvt/Bug89.java
@@ -0,0 +1,20 @@
+package com.alibaba.json.bvt;
+
+import com.alibaba.fastjson.JSON;
+import com.alibaba.fastjson.JSONException;
+import org.junit.Test;
+
+import static org.junit.Assert.fail;
+
+public class Bug89 {
+    @Test
+    public void testBug89() {
+        try {
+            String s = "{\"a\":з」∠)_,\"}";
+            JSON.parseObject(s);
+            fail("Expect JSONException");
+        } catch (JSONException e) {
+            // good
+        }
+    }
+}