mirrors/bitcoin-s
1
0
Fork 0
mirror of https://github.com/bitcoin-s/bitcoin-s.git synced 2025-02-23 22:56:52 +01:00
Code Issues Projects Releases Packages Wiki Activity

Optimized sigPoint computation to use non-custom secp functions (#2665)

Browse source
This commit is contained in:
Nadav Kohen 2021-02-16 14:12:38 -06:00 • committed by GitHub
parent bcd2df6051
commit 74a30fe9b8
1 changed files with 20 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
crypto/src/main/scala/org/bitcoins/crypto/SchnorrPublicKey.scala
View file

@ -44,12 +44,27 @@ case class SchnorrPublicKey(bytes: ByteVector) extends NetworkElement {
def computeSigPoint(
bytesToHash: Vector[ByteVector],
nonces: Vector[SchnorrNonce]): ECPublicKey = {
bytesToHash
.zip(nonces)
.map { case (bytes, nonce) =>
computeSigPoint(CryptoUtil.sha256(bytes), nonce)
// TODO: when combine function is ported from secp, use that instead for nonces
val bytesAndNonces = bytesToHash.zip(nonces)
val hashesAndNoncePoints = bytesAndNonces.map { case (bytes, nonce) =>
val eBytes = CryptoUtil
.sha256SchnorrChallenge(
nonce.bytes ++ this.bytes ++ CryptoUtil
.sha256DLCAttestation(bytes)
.bytes)
.bytes
val e = ECPrivateKey(eBytes)
(e, nonce.publicKey)
}
val (aggHashes, aggNonces) =
hashesAndNoncePoints.reduce[(ECPrivateKey, ECPublicKey)] {
case ((aggHash, aggPoint), (hash, nonce)) =>
(aggHash.add(hash), aggPoint.add(nonce))
}
.reduce(_.add(_))
this.publicKey.tweakMultiply(aggHashes.fieldElement).add(aggNonces)
}
// TODO: match on CryptoContext once secp version is added