QUIC QSM: Minor bugfixes

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

diff --git a/ssl/quic/quic_stream_map.c b/ssl/quic/quic_stream_map.c
index 9fc5ca9f8769a8c007c938d338e09d1280dc755c..680fe48ad9945a8892a329bb4188ef8c6f253604 100644 (file)
--- a/ssl/quic/quic_stream_map.c
+++ b/ssl/quic/quic_stream_map.c
@@ -21,7 +21,8 @@ static void list_insert_tail(QUIC_STREAM_LIST_NODE *l,
                              QUIC_STREAM_LIST_NODE *n)
 {
     /* Must not be in list. */
-    assert(n->prev == NULL && n->next == NULL);
+    assert(n->prev == NULL && n->next == NULL
+           && l->prev != NULL && l->next != NULL);
 
     n->prev = l->prev;
     n->prev->next = n;
@@ -43,6 +44,10 @@ static void list_remove(QUIC_STREAM_LIST_NODE *l,
 static QUIC_STREAM *list_next(QUIC_STREAM_LIST_NODE *l, QUIC_STREAM_LIST_NODE *n,
                               size_t off)
 {
+    assert(n->prev != NULL && n->next != NULL
+           && (n == l || (n->prev != n && n->next != n))
+           && l->prev != NULL && l->next != NULL);
+
     n = n->next;
 
     if (n == l)
@@ -50,6 +55,8 @@ static QUIC_STREAM *list_next(QUIC_STREAM_LIST_NODE *l, QUIC_STREAM_LIST_NODE *n
     if (n == l)
         return NULL;
 
+    assert(n != NULL);
+
     return (QUIC_STREAM *)(((char *)n) - off);
 }
 
@@ -196,13 +203,13 @@ static void stream_map_mark_inactive(QUIC_STREAM_MAP *qsm, QUIC_STREAM *s)
     if (!s->active)
         return;
 
-    list_remove(&qsm->active_list, &s->active_node);
-
     if (qsm->rr_cur == s)
         qsm->rr_cur = active_next(&qsm->active_list, s);
     if (qsm->rr_cur == s)
         qsm->rr_cur = NULL;
 
+    list_remove(&qsm->active_list, &s->active_node);
+
     s->active = 0;
 }
 
@@ -252,7 +259,7 @@ static int qsm_ready_for_gc(QUIC_STREAM_MAP *qsm, QUIC_STREAM *qs)
     assert(!qs->deleted
            || qs->sstream == NULL
            || qs->reset_stream
-           || ossl_quic_sstream_get_final_size(qs, NULL));
+           || ossl_quic_sstream_get_final_size(qs->sstream, NULL));
 
     return
         qs->deleted
@@ -372,9 +379,12 @@ size_t ossl_quic_stream_map_get_accept_queue_len(QUIC_STREAM_MAP *qsm)
 
 void ossl_quic_stream_map_gc(QUIC_STREAM_MAP *qsm)
 {
-    QUIC_STREAM *qs, *qsn;
+    QUIC_STREAM *qs, *qs_head, *qsn = NULL;
 
-    for (qs = ready_for_gc_head(&qsm->ready_for_gc_list); qs != NULL; qs = qsn) {
+    for (qs = qs_head = ready_for_gc_head(&qsm->ready_for_gc_list);
+         qs != NULL && qs != qs_head;
+         qs = qsn)
+    {
          qsn = ready_for_gc_next(&qsm->ready_for_gc_list, qs);
 
          ossl_quic_stream_map_release(qsm, qs);