Fix: GenerateNonce(ECPubKey) wasn't random as expected (#1228)

MetacoSA · Sep 22, 2024 · 46af199 · 46af199
1 parent 4623e56
commit 46af199
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/NBitcoin.Tests/Secp256k1Tests.cs b/NBitcoin.Tests/Secp256k1Tests.cs
@@ -3985,6 +3985,13 @@ public void musig_tweaked_test()
 			// Add the scripts there
 			var treeInfo = builder.Finalize(new TaprootInternalPubKey(aggregatedKey.ToXOnlyPubKey().ToBytes()));
 			musig = new MusigContext(ecPubKeys, msg32);
+
+			// Sanity check that GenerateNonce do not reuse nonces
+			var n1 = musig.GenerateNonce(ecPubKeys[0]);
+			var n2 = musig.GenerateNonce(ecPubKeys[0]);
+			Assert.NotEqual(Encoders.Hex.EncodeData(n1.CreatePubNonce().ToBytes()), Encoders.Hex.EncodeData(n2.CreatePubNonce().ToBytes()));
+			//
+
 			nonces = ecPubKeys.Select(c => musig.GenerateNonce(c)).ToArray();
 			musig.Tweak(treeInfo.OutputPubKey.Tweak.Span);
 			musig.ProcessNonces(nonces.Select(n => n.CreatePubNonce()).ToArray());