QUIC: Minor comment and editorial fixes

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19703)

diff --git a/doc/designs/quic-design/congestion-control.md b/doc/designs/quic-design/congestion-control.md
index 7881ca9871692a7328b3545f0dba7a107a3eebf3..d5bb4a1d7957b0e2494126b89c753e04493b496d 100644 (file)
--- a/doc/designs/quic-design/congestion-control.md
+++ b/doc/designs/quic-design/congestion-control.md
@@ -178,8 +178,8 @@ struct ossl_cc_method_st {
     size_t (*get_bytes_in_flight_max)(OSSL_CC_DATA *ccdata);
 
     /*
-     * Returns the time at which the CC will next release more budget
-     * for sending, or ossl_time_infinite().
+     * Returns the next time at which the CC will release more budget for
+     * sending, or ossl_time_infinite().
      */
     OSSL_TIME (*get_next_credit_time)(OSSL_CC_DATA *ccdata);
 

diff --git a/include/internal/quic_cc.h b/include/internal/quic_cc.h
index 6cf78913f1a4e44c0c257d9c28715efb5c12ca4e..8e318e33ff45160f5c771d9ea254081a6f33d92e 100644 (file)
--- a/include/internal/quic_cc.h
+++ b/include/internal/quic_cc.h
@@ -84,8 +84,8 @@ typedef struct ossl_cc_method_st {
     uint64_t (*get_bytes_in_flight_max)(OSSL_CC_DATA *ccdata);
 
     /*
-     * Returns the time at which the CC will next release more budget
-     * for sending, or ossl_time_infinite().
+     * Returns the next time at which the CC will release more budget for
+     * sending, or ossl_time_infinite().
      */
     OSSL_TIME (*get_next_credit_time)(OSSL_CC_DATA *ccdata);
 

diff --git a/include/internal/quic_channel.h b/include/internal/quic_channel.h
index 9c1adfd667a4594ecac45f2a71cecc59d01378be..31e218cb2fb9038a01c4a848dd849c95a768fa0a 100644 (file)
--- a/include/internal/quic_channel.h
+++ b/include/internal/quic_channel.h
@@ -81,7 +81,6 @@ void ossl_quic_channel_free(QUIC_CHANNEL *ch);
  * code (e.g. ossl_quic_channel_raise_protocol_error), others are for very
  * specific use by particular components only (e.g.
  * ossl_quic_channel_on_handshake_confirmed).
- *
  */
 
 /*

diff --git a/include/internal/quic_record_rx.h b/include/internal/quic_record_rx.h
index 2a8a5bca102516fe66acaf4615a7b9efea855fd4..a72fb249728bb49c83df4e224fa113d6f561fda1 100644 (file)
--- a/include/internal/quic_record_rx.h
+++ b/include/internal/quic_record_rx.h
@@ -247,8 +247,8 @@ typedef struct ossl_qrx_pkt_st {
  *
  * On success, *pkt points to a OSSL_QRX_PKT structure. The structure should be
  * freed when no longer needed by calling ossl_qrx_pkt_release(). The structure
- * is refcounted; to gain extra references, call ossl_qrx_pkt_ref(). This will
- * cause a corresponding number of calls to ossl_qrx_pkt_release() to be
+ * is refcounted; to gain extra references, call ossl_qrx_pkt_up_ref(). This
+ * will cause a corresponding number of calls to ossl_qrx_pkt_release() to be
  * ignored.
  *
  * The resources referenced by (*pkt)->hdr, (*pkt)->hdr->data and (*pkt)->peer

diff --git a/include/internal/quic_record_tx.h b/include/internal/quic_record_tx.h
index 4a011d535b3a1687b464a6e152c088ee098becbc..d010a2f64a367b4234bbb84aae63a2f4881cf1f7 100644 (file)
--- a/include/internal/quic_record_tx.h
+++ b/include/internal/quic_record_tx.h
@@ -244,8 +244,8 @@ size_t ossl_qtx_get_unflushed_pkt_count(OSSL_QTX *qtx);
 
 /*
  * Change the BIO being used by the QTX. May be NULL if actual transmission is
- * not currently required. Does not ref the BIO; the caller is responsible for
- * ensuring the lifetime of the BIO exceeds the lifetime of the QTX.
+ * not currently required. Does not up-ref the BIO; the caller is responsible
+ * for ensuring the lifetime of the BIO exceeds the lifetime of the QTX.
  */
 void ossl_qtx_set_bio(OSSL_QTX *qtx, BIO *bio);
 

diff --git a/ssl/quic/quic_impl.c b/ssl/quic/quic_impl.c
index e59e6d85373d4e8bd2d4d31d3f77ef3a1e2c8c16..d94dacfe7e17044eb7c5877825ebb59ab5cbc4d0 100644 (file)
--- a/ssl/quic/quic_impl.c
+++ b/ssl/quic/quic_impl.c
@@ -1077,7 +1077,7 @@ static int quic_read_again(void *arg)
         /* got at least one byte, the SSL_read op can finish now */
         return 1;
 
-    return 0; /* did not write anything, keep trying */
+    return 0; /* did not read anything, keep trying */
 }
 
 static int quic_read(SSL *s, void *buf, size_t len, size_t *bytes_read, int peek)