[prev in list] [next in list] [prev in thread] [next in thread]
List: openssl-users
Subject: Re: Typos in man pages
From: Matt Caswell <matt () openssl ! org>
Date: 2020-06-18 8:13:57
Message-ID: e459b2ad-b40a-0a84-3651-80aa08cdcbed () openssl ! org
[Download RAW message or body]
On 17/06/2020 12:39, Hal Murray wrote:
>
> There are 3 cases of "structure of NULL" where the "of" should be "or".
> The "NULL" is actually "B<NULL>" in the pod file.
>
> doc/man3/EVP_PKEY_CTX_new.pod
> doc/man3/X509_NAME_add_entry_by_txt.pod
> doc/man3/X509V3_get_d2i.pod
>
> ----------
>
> There are several bugs/typos in the example code at the end of
> doc/man3/EVP_MAC.pod
>
> These lines have an extra NULL parameter.
> OSSL_PARAM_construct_utf8_string("cipher", cipher, 0, NULL);
> OSSL_PARAM_construct_utf8_string("digest", digest, 0, NULL);
> OSSL_PARAM_construct_octet_string("key", key, strlen(key), NULL);
>
> ----------
I created a PR to fix the above issues:
https://github.com/openssl/openssl/pull/12185
>
> That man page describes EVP_MAC_set_ctx_params and uses it in the example:
> || EVP_MAC_set_ctx_params(ctx, params) <= 0)
>
> evp.h has:
> int EVP_MAC_CTX_set_params(EVP_MAC_CTX *ctx, const OSSL_PARAM params[]);
>
> That's "set_ctx" vs "CTX_set" in case your eyes doesn't instantly lock on to
> the difference.
>
> There are similar problems with many other ctx routines.
> EVP_MAC_new_ctx => EVP_MAC_CTX_new
> EVP_MAC_free_ctx => EVP_MAC_CTX_free
> EVP_MAC_get_ctx_params ...
These names have recently changed and so there is probably a discrepancy
between whatever version you have locally and what you are reading on
the website. I don't think I agree with the name changes but see:
https://github.com/openssl/openssl/pull/11997
Matt
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic