len is a size_t, which is an unsigned integer. Therefore, some

compilers will complain against the check for less than zero.

diff --git a/crypto/ec/ecp_smpl.c b/crypto/ec/ecp_smpl.c
index 30178f7fcd065db3d153ed7325a492b73acacce9..4666a052bfabf0859d465221c0fd2a679aba897a 100644 (file)
--- a/crypto/ec/ecp_smpl.c
+++ b/crypto/ec/ecp_smpl.c
@@ -889,7 +889,7 @@ int ec_GFp_simple_oct2point(const EC_GROUP *group, EC_POINT *point,
        size_t field_len, enc_len;
        int ret = 0;
 
-       if (len <= 0)
+       if (len == 0)
                {
                ECerr(EC_F_EC_GFP_SIMPLE_OCT2POINT, EC_R_BUFFER_TOO_SMALL);
                return 0;