]> git.ipfire.org Git - thirdparty/openssl.git/commitdiff
doc: clarify SSL_CIPHER_description allocation
authorDaniel McCarney <daniel@binaryparadox.net>
Thu, 21 Mar 2024 19:41:11 +0000 (15:41 -0400)
committerTomas Mraz <tomas@openssl.org>
Mon, 29 Apr 2024 09:07:38 +0000 (11:07 +0200)
Previously the documentation for `SSL_CIPHER_description` said:
> If buf is provided, it must be at least 128 bytes, otherwise a buffer
> will be allocated using OPENSSL_malloc().

In reality, `OPENSSL_malloc` is only invoked if the provided `buf`
argument is `NULL`. If the `buf` arg is not `NULL`, but smaller than
128 bytes, the function returns `NULL` without attempting to allocate
a new buffer for the description.

This commit adjusts the documentation to better describe the implemented
behaviour.

CLA: trivial

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/23921)

(cherry picked from commit 6a4a714045415be6720f4165c4d70a0ff229a26a)

doc/man3/SSL_CIPHER_get_name.pod

index e22a85a063706d2e7fdd3b8ca476711f3427ed27..c3109279a71c456f94032afb2b5e5919a808fcc8 100644 (file)
@@ -109,7 +109,7 @@ cipher B<c>.
 
 SSL_CIPHER_description() returns a textual description of the cipher used
 into the buffer B<buf> of length B<len> provided.  If B<buf> is provided, it
-must be at least 128 bytes, otherwise a buffer will be allocated using
+must be at least 128 bytes. If B<buf> is NULL it will be allocated using
 OPENSSL_malloc().  If the provided buffer is too small, or the allocation fails,
 B<NULL> is returned.