Skip to content

Commit

Permalink
Fix modular square root test with LibreSSL >= 3.8
Browse files Browse the repository at this point in the history
[ This is a backport to the 3.1 branch. ]

If x is a modular square root of a (mod p) then so is (p - x). Both
answers are valid. In particular, both 2 and 3 are valid square roots
of 4 (mod 5). Do not assume that a particular square root is chosen by
the algorithm. Indeed, the algorithm in OpenSSL and LibreSSL <= 3.7
returns a non-deterministic answer in many cases. LibreSSL 3.8 and
later will always return the smaller of the two possible answers. This
breaks the current test case.

Instead of checking for a particular square root, check that the square
of the claimed square root is the given value. This is always true. Add
the simplest test case where the answer is indeed non-deterministic.

(cherry picked from commit 93548ae)
  • Loading branch information
botovq authored and rhenium committed May 2, 2024
1 parent 09122c5 commit 3cd3c27
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion test/openssl/test_bn.rb
Expand Up @@ -175,7 +175,9 @@ def test_mod_sqr
end

def test_mod_sqrt
assert_equal(3, 4.to_bn.mod_sqrt(5))
assert_equal(4, 4.to_bn.mod_sqrt(5).mod_sqr(5))
# One of 189484 or 326277 is returned as a square root of 2 (mod 515761).
assert_equal(2, 2.to_bn.mod_sqrt(515761).mod_sqr(515761))
assert_equal(0, 5.to_bn.mod_sqrt(5))
assert_raise(OpenSSL::BNError) { 3.to_bn.mod_sqrt(5) }
end
Expand Down

0 comments on commit 3cd3c27

Please sign in to comment.