Skip to content

Commit

Permalink
Clean up some stream refs on uninitialize
Browse files Browse the repository at this point in the history
  • Loading branch information
nibanks committed Oct 23, 2023
1 parent f8ad5d1 commit 2034f03
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/core/connection.c
Original file line number Diff line number Diff line change
Expand Up @@ -503,6 +503,8 @@ QuicConnUninitialize(
QuicCryptoUninitialize(&Connection->Crypto);
QuicTimerWheelRemoveConnection(&Connection->Worker->TimerWheel, Connection);
QuicOperationQueueClear(Connection->Worker, &Connection->OperQ);
QuicLossDetectionUninitialize(&Connection->LossDetection);
QuicSendUninitialize(&Connection->Send);

if (Connection->CloseReasonPhrase != NULL) {
CXPLAT_FREE(Connection->CloseReasonPhrase, QUIC_POOL_CLOSE_REASON);
Expand Down Expand Up @@ -5318,7 +5320,7 @@ QuicConnRecvFrames(
case QUIC_FRAME_IMMEDIATE_ACK: // Always accept the frame, because we always enable support.
AckImmediately = TRUE;
break;

case QUIC_FRAME_TIMESTAMP: { // Always accept the frame, because we always enable support.
if (!Connection->State.TimestampRecvNegotiated) {
QuicTraceEvent(
Expand All @@ -5345,7 +5347,7 @@ QuicConnRecvFrames(
Packet->SendTimestamp = Frame.Timestamp;
break;
}

default:
//
// No default case necessary, as we have already validated the frame
Expand Down

0 comments on commit 2034f03

Please sign in to comment.