musig: remove unnecessary constant time normalize in combine

jonasnick · jonasnick · commit fc26ca8ddef0 · 2021-07-14T19:59:38.000Z
diff --git a/src/modules/musig/main_impl.h b/src/modules/musig/main_impl.h
@@ -136,7 +136,7 @@ int secp256k1_musig_pubkey_combine(const secp256k1_context* ctx, secp256k1_scrat
         return 0;
     }
     secp256k1_ge_set_gej(&pkp, &pkj);
-    secp256k1_fe_normalize(&pkp.y);
+    secp256k1_fe_normalize_var(&pkp.y);
     pk_parity = secp256k1_extrakeys_ge_even_y(&pkp);
     secp256k1_xonly_pubkey_save(combined_pk, &pkp);
 

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ int secp256k1_musig_pubkey_combine(const secp256k1_context* ctx, secp256k1_scrat`
`136`	`136`	`return 0;`
`137`	`137`	`}`
`138`	`138`	`secp256k1_ge_set_gej(&pkp, &pkj);`
`139`		`- secp256k1_fe_normalize(&pkp.y);`
	`139`	`+ secp256k1_fe_normalize_var(&pkp.y);`
`140`	`140`	`pk_parity = secp256k1_extrakeys_ge_even_y(&pkp);`
`141`	`141`	`secp256k1_xonly_pubkey_save(combined_pk, &pkp);`
`142`	`142`