diff options
| author | Bob Beck <[email protected]> | 2026-08-11 20:26:43 +0200 |
|---|---|---|
| committer | Nikola Pajkovsky <[email protected]> | 2026-08-14 08:29:24 +0200 |
| commit | dad6ff265ab7923e7d59341507ef9db627a97008 (patch) | |
| tree | eaa7aaa4edee17de77728557d399e6d6656a497b /apps | |
| parent | Test EVP_PKEY_get_size for zero return (diff) | |
| download | openssl-dad6ff265ab7923e7d59341507ef9db627a97008.tar.gz openssl-dad6ff265ab7923e7d59341507ef9db627a97008.zip | |
Use an int for the return value of i2a_ASN1_OBJECT
And as a result in this loop. I changed this to a size_t
which was needed where the variable was used later on and
missed this case at the start.
While harmless, coverity notices. so let's do it right
Reviewed-by: Neil Horman <[email protected]>
Reviewed-by: Andrew Dinh <[email protected]>
Merge-date: Fri Aug 14 06:29:29 2026
Merged-from: https://github.com/openssl/openssl/pull/32310
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/ca.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/apps/ca.c b/apps/ca.c index a25661956b3..ff7d1cdf792 100644 --- a/apps/ca.c +++ b/apps/ca.c @@ -2497,11 +2497,12 @@ static int old_entry_print(const ASN1_OBJECT *obj, const ASN1_STRING *str) { char buf[25], *pbuf; const char *p; + int i; size_t j; - j = i2a_ASN1_OBJECT(bio_err, obj); + i = i2a_ASN1_OBJECT(bio_err, obj); pbuf = buf; - for (j = 22 - j; j > 0; j--) + for (i = 22 - i; i > 0; i--) *(pbuf++) = ' '; *(pbuf++) = ':'; *(pbuf++) = '\0'; |