Consider all supertypes for serialization constructors

loicottet · Sep 30, 2024 · 03860a4 · 03860a4
1 parent 9cadc84
commit 03860a4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/...rc/com.oracle.svm.core/src/com/oracle/svm/core/configure/SerializationMetadataParser.java b/...rc/com.oracle.svm.core/src/com/oracle/svm/core/configure/SerializationMetadataParser.java
@@ -65,7 +65,7 @@ protected void parseSerializationDescriptorObject(EconomicMap<String, Object> da
 
         Object optionalCustomCtorValue = data.get(CUSTOM_TARGET_CONSTRUCTOR_CLASS_KEY);
         if (optionalCustomCtorValue != null) {
-            RuntimeReflectionSupport.increaseCount(false);
+            RuntimeReflectionSupport.increaseCount(true);
         }
         registerType(targetSerializationClass.get(), condition.get(), optionalCustomCtorValue);
     }

diff --git a/...m.oracle.svm.hosted/src/com/oracle/svm/hosted/reflect/serialize/SerializationFeature.java b/...m.oracle.svm.hosted/src/com/oracle/svm/hosted/reflect/serialize/SerializationFeature.java
@@ -553,9 +553,11 @@ public void registerWithTargetConstructorClass(ConfigurationCondition condition,
                         return;
                     }
                 }
-                Optional.ofNullable(addConstructorAccessor(cnd, serializationTargetClass, customTargetConstructorClass))
-                                .map(ReflectionUtil::lookupConstructor)
-                                .ifPresent(methods -> ImageSingletons.lookup(RuntimeReflectionSupport.class).register(ConfigurationCondition.alwaysTrue(), false, methods));
+                for (Class<?> superclass = serializationTargetClass.getSuperclass(); superclass != null; superclass = superclass.getSuperclass()) {
+                    Optional.ofNullable(addConstructorAccessor(cnd, serializationTargetClass, superclass))
+                                    .map(ReflectionUtil::lookupConstructor)
+                                    .ifPresent(methods -> ImageSingletons.lookup(RuntimeReflectionSupport.class).register(ConfigurationCondition.alwaysTrue(), false, methods));
+                }
 
                 Class<?> superclass = serializationTargetClass.getSuperclass();
                 if (superclass != null) {