Fix #126

Author: cowtowncoder

protobuf/src/main/java/com/fasterxml/jackson/dataformat/protobuf/ProtobufParser.java

@@ -960,10 +960,12 @@ private JsonToken _skipUnknownField(int tag, int wireType) throws IOException
         }
         while (true) {
             _skipUnknownValue(wireType);
+            // 05-Dec-2017, tatu: as per [#126] seems like we need to check this not just for
+            //    STATE_NESTED_KEY but for arrays too at least?
+            if (_checkEnd()) {
+                return (_currToken = JsonToken.END_OBJECT);
+            }
             if (_state == STATE_NESTED_KEY) {
-                if (_checkEnd()) {
-                    return (_currToken = JsonToken.END_OBJECT);
-                }
                 if (_inputPtr >= _inputEnd) {
                     loadMoreGuaranteed();
                 }

release-notes/CREDITS

@@ -29,3 +29,5 @@ marsqing@github
  (2.8.10)
 * Reported #106, contributed fix for: calling _skipUnknownValue() twice
  (2.8.11 / 2.9.1)
+* Reported #126, contributed fix for: always call checkEnd() when skip unknown field
+ (2.8.11 / 2.9.3)

release-notes/VERSION

@@ -18,6 +18,7 @@ Modules:
 #106: (protobuf) fix calling _skipUnknownValue() twice
  (reported, contributed fix for by marsqing@github@github)
 #108: (protobuf) fix NPE in skip unknown nested key
+#126: (protobuf) always call checkEnd() when skip unknown field
 
 2.8.10 (24-Aug-2017)