fix FasterXML#141: Avoid attempting to optimize varargs methods

carterkozak · Jul 7, 2021 · d2c000e · d2c000e
1 parent 1a4517e
commit d2c000e
Show file tree

Hide file tree

Showing 3 changed files with 56 additions and 3 deletions.
diff --git a/blackbird/src/main/java/com/fasterxml/jackson/module/blackbird/deser/CreatorOptimizer.java b/blackbird/src/main/java/com/fasterxml/jackson/module/blackbird/deser/CreatorOptimizer.java
@@ -107,7 +107,10 @@ private MethodHandle directHandle(AnnotatedElement element) {
             Stream.of(element)
                 .filter(Constructor.class::isInstance)
                 .map(Constructor.class::cast)
-                .filter(c -> !Modifier.isPrivate(c.getModifiers()))
+                .filter(c -> !Modifier.isPrivate(c.getModifiers())
+                        // 07-Jul-2021, ckozak: modules-base#141Avoid attempting to optimize varargs
+                        // methods due to class-cast edge cases
+                        && !c.isVarArgs())
                 .flatMap(t -> {
                     try {
                         return Stream.of(_lookup.unreflectConstructor(t));
@@ -120,7 +123,10 @@ private MethodHandle directHandle(AnnotatedElement element) {
                 .map(Method.class::cast)
                 .filter(m -> {
                     int mods = m.getModifiers();
-                    return Modifier.isStatic(mods) && !Modifier.isPrivate(mods);
+                    return Modifier.isStatic(mods) && !Modifier.isPrivate(mods)
+                            // 07-Jul-2021, ckozak: modules-base#141Avoid attempting to optimize varargs
+                            // methods due to class-cast edge cases
+                            && !m.isVarArgs();
                 })
                 .flatMap(t -> {
                     try {

diff --git a/...bird/failing/DoubleArrayDeser141Test.java → ...ckbird/deser/DoubleArrayDeser141Test.java b/...bird/failing/DoubleArrayDeser141Test.java → ...ckbird/deser/DoubleArrayDeser141Test.java
@@ -1,4 +1,4 @@
-package com.fasterxml.jackson.module.blackbird.failing;
+package com.fasterxml.jackson.module.blackbird.deser;
 
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;

diff --git a/...d/src/test/java/com/fasterxml/jackson/module/blackbird/deser/ObjectArrayDeser141Test.java b/...d/src/test/java/com/fasterxml/jackson/module/blackbird/deser/ObjectArrayDeser141Test.java
@@ -0,0 +1,47 @@
+package com.fasterxml.jackson.module.blackbird.deser;
+
+import com.fasterxml.jackson.annotation.JsonCreator;
+import com.fasterxml.jackson.annotation.JsonProperty;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.module.blackbird.BlackbirdTestBase;
+
+import java.util.Arrays;
+import java.util.List;
+
+// [modules-base#141]
+public class ObjectArrayDeser141Test extends BlackbirdTestBase
+{
+    static class Bar {
+        @JsonProperty("baz")
+        String baz;
+
+        @JsonCreator
+        public Bar(@JsonProperty("baz") String baz) {
+            this.baz = baz;
+        }
+    }
+    // [modules-base#141]
+    static class Foo141 {
+        @JsonProperty("bar")
+        List<Bar> bar;
+
+        @JsonCreator
+        public Foo141(@JsonProperty("bar") Bar... bar) {
+          this.bar = Arrays.asList(bar);
+        }
+    }
+
+    private final ObjectMapper MAPPER = newObjectMapper();
+
+    // [modules-base#141]
+    public void testDoubleArrayViaCreator() throws Exception
+    {
+        Foo141 foo = new Foo141(new Bar("a"), new Bar("b"));
+        String serialized = MAPPER.writeValueAsString(foo);
+        Foo141 foo2 = MAPPER.readValue(serialized, Foo141.class);
+
+        assertEquals(2, foo2.bar.size());
+        assertEquals("a", foo2.bar.get(0).baz);
+        assertEquals("b", foo2.bar.get(1).baz);
+    }
+}