diff options
| author | Niels Dossche <niels.dossche@ugent.be> | 2023-01-17 20:27:49 +0100 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2023-01-19 14:15:19 +0100 |
| commit | 53b5d6c30f3b8eaf7a582da2265c0d1cfe14d54f (patch) | |
| tree | 9d1294cfb569c1906a4421d588495f4a9d39943d /crypto | |
| parent | f225fbf9521395aff86e85883db9bcb083eab154 (diff) | |
Fix incorrect error return value in i2r_ADMISSION_SYNTAX()
The other implementations of i2r return 0 in case of an error, but
i2r_ADMISSION_SYNTAX() returns -1. That means the check on i2r does not
catch the error. Change it to return 0 like the others do.
Fixes: #20066
CLA: trivial
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/20068)
Diffstat (limited to 'crypto')
| -rw-r--r-- | crypto/x509/v3_admis.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/x509/v3_admis.c b/crypto/x509/v3_admis.c index cd875e9dd1..7aa8a4a420 100644 --- a/crypto/x509/v3_admis.c +++ b/crypto/x509/v3_admis.c @@ -199,7 +199,7 @@ static int i2r_ADMISSION_SYNTAX(const struct v3_ext_method *method, void *in, return 1; err: - return -1; + return 0; } const ASN1_OBJECT *NAMING_AUTHORITY_get0_authorityId(const NAMING_AUTHORITY *n) |