Merge pull request #737 from Stumble/yumin/fix-alias

fix: incorrect alias check brought from snarkjs. (possible security vulnerability)
semaphore-protocol · Apr 12, 2024 · a4a6837 · a4a6837
2 parents 2aee16e + 8e246c4
commit a4a6837
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/packages/contracts/contracts/base/SemaphoreVerifier.sol b/packages/contracts/contracts/base/SemaphoreVerifier.sol
@@ -241,7 +241,7 @@ contract SemaphoreVerifier {
 
         assembly {
             function checkField(v) {
-                if iszero(lt(v, q)) {
+                if iszero(lt(v, r)) {
                     mstore(0, 0)
                     return(0, 0x20)
                 }