paul-stinchcombe / google-gson

Automatically exported from code.google.com/p/google-gson
0 stars 0 forks source link

JsonReader skipValue() can cause an infinite loop #605

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
2. run reader.beginObject() to start reading the stream

3. run reader.skipValue() repeatedly until you try and skip over the "}" to end 
the json stream.  the last skip will cause an infinite loop.

What is the expected output? What do you see instead?
I would expect an exception to be thrown indicating EOF was hit, or that you 
cannot skip over an object/array closing.  Even silently ignoring would be 
better then an infinite loop.

What version of the product are you using? On what operating system?
2.3 w/ Java 1.7.0_67

Please provide any additional information below.

JsonReader line 1249 decrements the count to -1 because PEEKED_END_OBJECT is 
encountered on the last skip, but since the next item is EOF, it forever 
infinite loops waiting for the counter to get incremented back to 0, which 
never occurs.

I put in a quick hack for my problem by adding an extra if condition

} else if (p == PEEKED_EOF) {
    count = 0;
}

Probably a better fix would be to check for count>0 instead of !=0 (assuming 
that doesn't cause a problem somewhere else, but I am not familiar enough with 
the code base to try this).

This code snippet can reproduce the problem:

   public void testEarlyEOF() throws Exception {
        String json = "{ \"errors\":[ { \"field\":null,\"code\":\"loanId-not-found\" } ] }";

        final InputStream is = new ByteArrayInputStream(json.getBytes());

        Thread t = new Thread() {
            @Override
            public void run() {
                JsonReader reader = new JsonReader(new InputStreamReader(is));

                try {
                    reader.beginObject();     // start of overall json object
                    reader.skipValue();       // skip "errors" label
                    reader.skipValue();       // skip array value for "errors"
                    reader.skipValue();       // try and skip the closing "}"
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        };

        t.setDaemon(true);
        t.start();

        t.join(10000);
        boolean alive = t.isAlive();
        if (alive) t.interrupt();
        assertFalse("should have finished by now, we must be hung", alive);
    }

Original issue reported on code.google.com by rocco.ga...@gmail.com on 3 Nov 2014 at 2:15