diff options
| author | Jiasheng Jiang <jiasheng@iscas.ac.cn> | 2022-06-24 15:35:05 +0800 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2022-06-28 17:10:45 +0200 |
| commit | daa014b2061b94832415b1177ff2db6a17fc7274 (patch) | |
| tree | 6a37d790a8ca54bdcfcc2e656ab5ccec847280c0 | |
| parent | a99b372157561560da526e212a5c6a74e1cd2d82 (diff) | |
fuzz/asn1.c: Add check for ASN1_item_i2d
As the potential failure of the ASN1_item_i2d,
it should be better to check the return value.
Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/18647)
(cherry picked from commit 1cb35ce06a968dc82e7cd9502ecce8e89eca9580)
| -rw-r--r-- | fuzz/asn1.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fuzz/asn1.c b/fuzz/asn1.c index 69978cd054..ee602a08a3 100644 --- a/fuzz/asn1.c +++ b/fuzz/asn1.c @@ -317,8 +317,9 @@ int FuzzerTestOneInput(const uint8_t *buf, size_t len) ASN1_item_print(bio, o, 4, i, pctx); BIO_free(bio); } - ASN1_item_i2d(o, &der, i); - OPENSSL_free(der); + if (ASN1_item_i2d(o, &der, i) > 0) { + OPENSSL_free(der); + } ASN1_item_free(o, i); } } |