#864 Fix serialization of Kotlin lambdas

EsotericSoftware · Apr 5, 2023 · f6f6570 · f6f6570
1 parent 96259c7
commit f6f6570
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/src/com/esotericsoftware/kryo/util/DefaultGenerics.java b/src/com/esotericsoftware/kryo/util/DefaultGenerics.java
@@ -93,8 +93,9 @@ public Class nextGenericClass () {
 
 	@Override
 	public int pushTypeVariables (GenericsHierarchy hierarchy, GenericType[] args) {
-		// Do not store type variables if hierarchy is empty or we do not have arguments for all root parameters.
-		if (hierarchy.total == 0 || hierarchy.rootTotal > args.length) return 0;
+		// Do not store type variables if hierarchy is empty, or we do not have arguments for all root parameters, 
+		// or we have more arguments than the hierarchy has parameters.
+		if (hierarchy.total == 0 || hierarchy.rootTotal > args.length || args.length > hierarchy.counts.length) return 0;
 
 		int startSize = this.argumentsSize;
 

diff --git a/...n/com.esotericsoftware.kryo/LambdaTest.kt → ...sotericsoftware.kryo/SerializationTest.kt b/...n/com.esotericsoftware.kryo/LambdaTest.kt → ...sotericsoftware.kryo/SerializationTest.kt
@@ -4,16 +4,12 @@ import com.esotericsoftware.kryo.io.Input
 import com.esotericsoftware.kryo.io.Output
 import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Assertions.assertNotSame
-import org.junit.jupiter.api.Disabled
 import org.junit.jupiter.api.Test
 
-class LambdaTest {
-    class Example(private val p: (Long) -> String) {
-        constructor() : this({ it.toString() })
-    }
+class SerializationTest {
 
+    // https://github.com/EsotericSoftware/kryo/issues/864
     @Test
-    @Disabled("Expected to fail")
     fun testLambda() {
         val kryo = Kryo().apply {
             isRegistrationRequired = false
@@ -33,5 +29,9 @@ class LambdaTest {
         assertEquals(example::class.java, deserialized::class.java)
         assertNotSame(example, deserialized)
     }
+
+    class Example(private val p: (Long) -> String) {
+        constructor() : this({ it.toString() })
+    }
 }