Merge pull request #8 from solana-mobile/fix-signature-size

fix sig length
solana-mobile · Jan 10, 2024 · 78237e6 · 78237e6
2 parents 5a836d9 + 226da0f
commit 78237e6
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/solana/src/commonMain/kotlin/com/solana/transaction/Transaction.kt b/solana/src/commonMain/kotlin/com/solana/transaction/Transaction.kt
@@ -4,17 +4,19 @@ import com.solana.serialization.ByteStringSerializer
 import com.solana.serialization.TransactionFormat
 import kotlinx.serialization.*
 
-object SignatureSerializer : ByteStringSerializer(64)
+object SignatureSerializer : ByteStringSerializer(Transaction.SIGNATURE_LENGTH_BYTES)
 
 @Serializable
 data class Transaction(
     val signatures: List<@Serializable(with = SignatureSerializer::class) ByteArray>,
     @Serializable(with = MessageSerializer::class) val message: Message
 ) {
 
-    constructor(message: Message): this(buildList(message.signatureCount.toInt()) { ByteArray(size) }, message)
+    constructor(message: Message):
+            this(buildList(message.signatureCount.toInt()) { ByteArray(SIGNATURE_LENGTH_BYTES) }, message)
 
     companion object {
+        const val SIGNATURE_LENGTH_BYTES = 64
         fun from(bytes: ByteArray) = TransactionFormat.decodeFromByteArray(serializer(), bytes)
     }