Skip to content

Commit 02129a9

Browse files
author
Frantisek Tobias
committed
daemon/quic_conn: fix stream counting
1 parent 5ba2b49 commit 02129a9

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

daemon/quic_conn.c

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -897,19 +897,22 @@ static int pl_quic_conn_sess_deinit(struct session2 *session, void *sess_data)
897897
struct pl_quic_conn_sess_data *conn = sess_data;
898898
while (session2_tasklist_del_first(session, false) != NULL);
899899

900-
kr_log_info(DOQ, "Closing connection, %s useful, served %zu streams\n",
901-
conn->finished_streams ? "was" : "wasn't",
902-
conn->finished_streams);
903-
904900
struct pl_quic_stream_sess_data *s_node;
905901
WALK_LIST_FIRST(s_node, conn->streams) {
906902
struct pl_quic_stream_sess_data *s =
907903
container_of(s_node, struct pl_quic_stream_sess_data, list_node);
908904
rem_node(&s->list_node);
909905
session2_close(s->h.session);
906+
/* These streams die with the connection, stream_close_cb
907+
* will not be called so adjust counters here. */
910908
--conn->streams_count;
909+
++conn->finished_streams;
911910
}
912911

912+
kr_log_info(DOQ, "Closing connection, %s useful, served %zu streams\n",
913+
conn->finished_streams ? "was" : "wasn't",
914+
conn->finished_streams);
915+
913916
if (conn->priority) {
914917
gnutls_priority_deinit(conn->priority);
915918
}

0 commit comments

Comments
 (0)