]> git.ipfire.org Git - thirdparty/openssl.git/commitdiff
The comment should refer `ossl_quic_stream_has_recv_buffer()`
authorsashan <anedvedicky@gmail.com>
Fri, 18 Apr 2025 17:44:51 +0000 (19:44 +0200)
committerMatt Caswell <matt@openssl.org>
Fri, 20 Jun 2025 14:52:59 +0000 (15:52 +0100)
and `ossl_quic_stream_has_send_buffer()` explicitly.

This small tweak allows to find those functions more easily.

Reviewed-by: Neil Horman <nhorman@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/27438)

include/internal/quic_stream_map.h

index 76834e7434385a4faa943efcc7d21cf9d8b2bb51..d98f813261f3c2e8e1b9a434f093c1f28dd037e2 100644 (file)
@@ -137,16 +137,17 @@ struct quic_stream_st {
      *
      * DO NOT test these pointers (sstream, rstream) for NULL. Determine the
      * state of the send or receive stream part first using the appropriate
-     * function; then the invariant of that state guarantees that sstream or
-     * rstream either is or is not NULL respectively, therefore there is no
-     * valid use case for testing these pointers for NULL. In particular, a
-     * stream with a send part can still have sstream as NULL, and a stream with
-     * a receive part can still have rstream as NULL. QUIC_SSTREAM and
-     * QUIC_RSTREAM are stream buffer resource management objects which exist
-     * only when they need to for buffer management purposes. The existence or
-     * non-existence of a QUIC_SSTREAM or QUIC_RSTREAM object does not
-     * correspond with whether a stream's respective send or receive part
-     * logically exists or not.
+     * function (ossl_quic_stream_has_send_buffer() resp.
+     * ossl_quic_stream_has_recv_buffer()  ; then the invariant of that state
+     * guarantees that sstream or rstream either is or is not NULL respectively,
+     * therefore there is no valid use case for testing these pointers for NULL.
+     * In particular, stream with a send part can still have sstream as NULL,
+     * and a stream with a receive part can still have rstream as NULL.
+     * QUIC_SSTREAM and QUIC_RSTREAM are stream buffer resource management
+     * objects which exist only when they need to for buffer management
+     * purposes. The existence or non-existence of a QUIC_SSTREAM or
+     * QUIC_RSTREAM object does not correspond with whether a stream's
+     * respective send or receive part logically exists or not.
      */
     QUIC_SSTREAM    *sstream;   /* NULL if RX-only */
     QUIC_RSTREAM    *rstream;   /* NULL if TX only */