From 8765798eb08d3dafc3f9362a12170ad0265ca6af Mon Sep 17 00:00:00 2001 From: mmagician Date: Mon, 11 Sep 2023 13:44:41 -0600 Subject: [PATCH] The scalar to be multiplied by should be at most `num_limbs` long (#171) Co-authored-by: Pratyush Mishra --- curve-constraint-tests/src/lib.rs | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/curve-constraint-tests/src/lib.rs b/curve-constraint-tests/src/lib.rs index 69547e9..d08397c 100644 --- a/curve-constraint-tests/src/lib.rs +++ b/curve-constraint-tests/src/lib.rs @@ -315,12 +315,14 @@ pub mod curves { } assert!(cs.is_satisfied().unwrap()); - let modulus = ::MODULUS.as_ref().to_vec(); + let modulus = C::ScalarField::MODULUS.as_ref().to_vec(); let mut max = modulus.clone(); for limb in &mut max { *limb = u64::MAX; } + let num_limbs = max.len(); + let modulus_num_bits_mod_64 = ::MODULUS_BIT_SIZE % 64; if modulus_num_bits_mod_64 != 0 { *max.last_mut().unwrap() >>= 64 - modulus_num_bits_mod_64; @@ -332,10 +334,10 @@ pub mod curves { .to_vec(), vec![u64::rand(&mut rng)], (-C::ScalarField::one()).into_bigint().as_ref().to_vec(), - ::MODULUS.as_ref().to_vec(), + modulus, max, - vec![0; 50], - vec![1000012341233u64; 36], + vec![0u64; num_limbs], + vec![1000012341233u64; num_limbs], ]; let mut input = vec![];