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:59 +0100 |
| commit | fe89566149fca339e4213b172c9928a5bf837506 (patch) | |
| tree | b75dbce286a4e322fc02667e0a35a997c7324b46 /crypto | |
| parent | 05040e9697104ad65a1f111ac29c9d8cf4ead705 (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)
(cherry picked from commit 53b5d6c30f3b8eaf7a582da2265c0d1cfe14d54f)
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 bee9cc9931..8ea8237d72 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) |