QUIC: Revise and add some TODO lines

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/ssl/quic/quic_impl.c b/ssl/quic/quic_impl.c
index 6d78990cecf28782712b048b4be9d41f73cb20fa..aac427bc4925ad49f95c29023c406a32db18fedf 100644 (file)
--- a/ssl/quic/quic_impl.c
+++ b/ssl/quic/quic_impl.c
@@ -184,7 +184,7 @@ int ossl_quic_reset(SSL *s)
     if (!expect_quic_conn(qc))
         return 0;
 
-    /* Currently a no-op. */
+    /* TODO(QUIC); Currently a no-op. */
     return 1;
 }
 
@@ -196,7 +196,7 @@ int ossl_quic_clear(SSL *s)
     if (!expect_quic_conn(qc))
         return 0;
 
-    /* Currently a no-op. */
+    /* TODO(QUIC): Currently a no-op. */
     return 1;
 }
 

diff --git a/ssl/quic/quic_rx_depack.c b/ssl/quic/quic_rx_depack.c
index 8652a6ed6a0774be5b039816ab7cd6ce91150a4b..da941dc10393488041e29b40c6b68d17b5d3fe6a 100644 (file)
--- a/ssl/quic/quic_rx_depack.c
+++ b/ssl/quic/quic_rx_depack.c
@@ -356,7 +356,6 @@ static int depack_do_frame_max_streams(PACKET *pkt,
     ackm_data->is_ack_eliciting = 1;
 
     if (max_streams > (((uint64_t)1) << 60)) {
-        /* TODO: Protocol violation (FRAME_ENCODING_ERROR) */
         ossl_quic_channel_raise_protocol_error(ch,
                                                QUIC_ERR_FRAME_ENCODING_ERROR,
                                                frame_type,

diff --git a/ssl/quic/quic_wire_pkt.c b/ssl/quic/quic_wire_pkt.c
index de3466daa2129890fddd696039cd5da4176ee091..e0180ec0604f7db9602616dd79e16ad68fc017df 100644 (file)
--- a/ssl/quic/quic_wire_pkt.c
+++ b/ssl/quic/quic_wire_pkt.c
@@ -836,6 +836,7 @@ int ossl_quic_calculate_retry_integrity_tag(OSSL_LIB_CTX *libctx,
         return 0;
 
     /* Create and initialise cipher context. */
+    /* TODO(QUIC): Cipher fetch caching. */
     if ((cipher = EVP_CIPHER_fetch(libctx, "AES-128-GCM", propq)) == NULL)
         goto err;