nik/bitchat-android
1
0
Fork 0
mirror of https://github.com/permissionlesstech/bitchat-android.git synced 2026-11-01 07:51:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(noise/protocol): migrate Curve448DHState to Kotlin

Browse Source 
- Override DHState properties instead of Java-style getters.
- Use `ByteArray.fill(0)` in place of `Arrays.fill(arr, 0.toByte())`.
- Preserve bit-identical crypto behavior (56-byte key sizes,
  Curve448.eval call signature, constant-time null-key scan).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
yet 2026-05-15 17:57:40 +04:00
parent e7c0f28f76
commit f3306091d5
1 changed files with 16 additions and 41 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

57
app/src/main/java/com/bitchat/android/noise/southernstorm/protocol/Curve448DHState.kt
View File

@ -24,43 +24,23 @@ package com.bitchat.android.noise.southernstorm.protocol
import com.bitchat.android.noise.southernstorm.crypto.Curve448
import com.bitchat.android.noise.southernstorm.protocol.Noise.destroy
import com.bitchat.android.noise.southernstorm.protocol.Noise.random
import java.util.Arrays
/**
* Implementation of the Curve448 algorithm for the Noise protocol.
*/
internal class Curve448DHState : DHState {
private val publicKey: ByteArray
private val privateKey: ByteArray
private val publicKey: ByteArray = ByteArray(56)
private val privateKey: ByteArray = ByteArray(56)
private var mode = 0
/**
* Constructs a new Diffie-Hellman object for Curve448.
*/
init {
publicKey = ByteArray(56)
privateKey = ByteArray(56)
}
override fun destroy() {
clearKey()
}
override fun getDHName(): String {
return "448"
}
override fun getPublicKeyLength(): Int {
return 56
}
override fun getPrivateKeyLength(): Int {
return 56
}
override fun getSharedKeyLength(): Int {
return 56
}
override val dhName: String get() = "448"
override val publicKeyLength: Int get() = 56
override val privateKeyLength: Int get() = 56
override val sharedKeyLength: Int get() = 56
override fun generateKeyPair() {
random(privateKey)
@ -74,7 +54,7 @@ internal class Curve448DHState : DHState {
override fun setPublicKey(key: ByteArray, offset: Int) {
System.arraycopy(key, offset, publicKey, 0, 56)
Arrays.fill(privateKey, 0.toByte())
privateKey.fill(0)
mode = 0x01
}
@ -89,8 +69,8 @@ internal class Curve448DHState : DHState {
}
override fun setToNullPublicKey() {
Arrays.fill(publicKey, 0.toByte())
Arrays.fill(privateKey, 0.toByte())
publicKey.fill(0)
privateKey.fill(0)
mode = 0x01
}
@ -100,13 +80,9 @@ internal class Curve448DHState : DHState {
mode = 0
}
override fun hasPublicKey(): Boolean {
return (mode and 0x01) != 0
}
override fun hasPublicKey(): Boolean = (mode and 0x01) != 0
override fun hasPrivateKey(): Boolean {
return (mode and 0x02) != 0
}
override fun hasPrivateKey(): Boolean = (mode and 0x02) != 0
override fun isNullPublicKey(): Boolean {
if ((mode and 0x01) == 0) return false
@ -115,17 +91,16 @@ internal class Curve448DHState : DHState {
return temp == 0
}
override fun calculate(sharedKey: ByteArray, offset: Int, publicDH: DHState?) {
override fun calculate(sharedKey: ByteArray, offset: Int, publicDH: DHState) {
require(publicDH is Curve448DHState) { "Incompatible DH algorithms" }
Curve448.eval(sharedKey, offset, privateKey, publicDH.publicKey)
}
override fun copyFrom(other: DHState?) {
override fun copyFrom(other: DHState) {
check(other is Curve448DHState) { "Mismatched DH key objects" }
if (other === this) return
val dh = other
System.arraycopy(dh.privateKey, 0, privateKey, 0, 56)
System.arraycopy(dh.publicKey, 0, publicKey, 0, 56)
mode = dh.mode
System.arraycopy(other.privateKey, 0, privateKey, 0, 56)
System.arraycopy(other.publicKey, 0, publicKey, 0, 56)
mode = other.mode
}
}