diff options
| author | Brian Smith <brian@briansmith.org> | 2016-02-14 05:46:11 +0300 |
|---|---|---|
| committer | David Benjamin <davidben@google.com> | 2016-02-25 01:00:58 +0300 |
| commit | 76c6381c21ef07b13371120dc0b938da64f51ca7 (patch) | |
| tree | 22bc8a2c5f50bda8ec884dcb267b95c5e1fa4e9e /include/openssl/ec.h | |
| parent | 0bc2349375212f79cc4a5996f86389d6250fde4f (diff) | |
Return 0 on error in |EC_POINT_is_on_curve| instead of -1.
Callers of this function are not checking for the -1 result. Change
the semantics to match their expectations and to match the common
semantics of most other parts of BoringSSL.
Change-Id: I4ec537d7619e20e8ddfee80c72125e4c02cfaac1
Reviewed-on: https://boringssl-review.googlesource.com/7125
Reviewed-by: David Benjamin <davidben@google.com>
Diffstat (limited to 'include/openssl/ec.h')
| -rw-r--r-- | include/openssl/ec.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/include/openssl/ec.h b/include/openssl/ec.h index 4957066f..55ba194a 100644 --- a/include/openssl/ec.h +++ b/include/openssl/ec.h @@ -189,7 +189,8 @@ OPENSSL_EXPORT int EC_POINT_is_at_infinity(const EC_GROUP *group, const EC_POINT *point); /* EC_POINT_is_on_curve returns one if |point| is an element of |group| and - * zero otheriwse. If |ctx| is non-NULL, it may be used. */ + * and zero otherwise or when an error occurs. This is different from OpenSSL, + * which returns -1 on error. If |ctx| is non-NULL, it may be used. */ OPENSSL_EXPORT int EC_POINT_is_on_curve(const EC_GROUP *group, const EC_POINT *point, BN_CTX *ctx); |