PR: 1612

Submitted by: Robert Jackson <robert@rjsweb.net>
Reviewed by: steve

Fix EC_POINT_cmp function for case where b but not a is the point at infinity.

diff --git a/crypto/ec/ec2_smpl.c b/crypto/ec/ec2_smpl.c
index cf357b462a27bcda50ff8f600bdfcf9635dab1e6..af94458ca7bab35d41cf137d317cd6f3fdb8dc47 100644 (file)
--- a/crypto/ec/ec2_smpl.c
+++ b/crypto/ec/ec2_smpl.c
@@ -937,6 +937,9 @@ int ec_GF2m_simple_cmp(const EC_GROUP *group, const EC_POINT *a, const EC_POINT
                {
                return EC_POINT_is_at_infinity(group, b) ? 0 : 1;
                }
                {
                return EC_POINT_is_at_infinity(group, b) ? 0 : 1;
                }

+
+       if (EC_POINT_is_at_infinity(group, b))
+               return 1;

        
        if (a->Z_is_one && b->Z_is_one)
                {
        
        if (a->Z_is_one && b->Z_is_one)
                {

diff --git a/crypto/ec/ecp_smpl.c b/crypto/ec/ecp_smpl.c
index 4d26f8bdf6921f4ae9a9e5be06fbde3f54b55628..66a92e2a9005c51688cbfcfd86ed048845ae6c14 100644 (file)
--- a/crypto/ec/ecp_smpl.c
+++ b/crypto/ec/ecp_smpl.c
@@ -1406,6 +1406,9 @@ int ec_GFp_simple_cmp(const EC_GROUP *group, const EC_POINT *a, const EC_POINT *
                {
                return EC_POINT_is_at_infinity(group, b) ? 0 : 1;
                }
                {
                return EC_POINT_is_at_infinity(group, b) ? 0 : 1;
                }

+
+       if (EC_POINT_is_at_infinity(group, b))
+               return 1;

        
        if (a->Z_is_one && b->Z_is_one)
                {
        
        if (a->Z_is_one && b->Z_is_one)
                {