Backport fix for #76

Author: cowtowncoder

protobuf/src/main/java/com/fasterxml/jackson/dataformat/protobuf/schema/ProtobufField.java

@@ -191,7 +191,9 @@ public final boolean isArray() {
     }
 
     public final boolean isValidFor(int typeTag) {
-        return (typeTag == type.getWireType());
+        return (typeTag == type.getWireType()
+                // 13-Apr-2017, tatu: to fix [dataformats-binary#76]
+                || packed && repeated && typeTag == WireType.LENGTH_PREFIXED);
     }
 
     @Override

protobuf/src/test/java/com/fasterxml/jackson/dataformat/protobuf/ReadPackedRepeatedTest.java

@@ -0,0 +1,28 @@
+package com.fasterxml.jackson.dataformat.protobuf;
+
+import com.fasterxml.jackson.databind.JsonNode;
+import com.fasterxml.jackson.dataformat.protobuf.schema.ProtobufSchema;
+
+import java.io.StringReader;
+
+public class ReadPackedRepeatedTest extends ProtobufTestBase
+{
+    final ProtobufMapper MAPPER = new ProtobufMapper();
+
+    public void testPacked() throws Exception
+    {
+        final String SCHEMA_STR =
+            "package mypackage;\n"
+            + "message t {\n"
+            + "        repeated uint32 f = 1 [packed=true];\n"
+            + "}";
+        final byte[] pb = {0xa, 0x3, 0x64, (byte)0xc8, 0x1}; // f = [100, 200]
+
+        ProtobufSchema schema = MAPPER.schemaLoader().load(new StringReader(SCHEMA_STR));
+        JsonNode t = MAPPER.readerFor(JsonNode.class).with(schema).readValue(pb);
+
+        assertEquals(2, t.get("f").size());
+        assertEquals(100, t.get("f").get(0).asInt());
+        assertEquals(200, t.get("f").get(1).asInt());
+    }
+}

release-notes/CREDITS

@@ -15,3 +15,8 @@ Michael Zeng (shotbythought@github)
 
 * Reported #58 (avro): Regression due to changed namespace of inner enum types
  (2.8.8)
+
+Kenji Noguchi (knoguchi@github)
+
+* Reported #70 (protobuf), contributed fix: Can't deserialize packed repeated field
+ (2.8.9)

release-notes/VERSION

@@ -18,6 +18,8 @@ Modules:
  (reported by Peter R)
 #62: (cbor) `java.lang.ArrayIndexOutOfBoundsException` at `CBORGenerator.java`:548
 #67: (proto) Serialization of multiple nesting levels has issues
+#70: (proto) Can't deserialize packed repeated field
+ (reported by Kenji N)
 
 2.8.7 (21-Feb-2017)