Skip to content

Commit

Permalink
QUIC CHANNEL: Set reason string for missing tparams extension
Browse files Browse the repository at this point in the history
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #22523)

(cherry picked from commit f94cacb)
  • Loading branch information
hlandau authored and t8m committed Nov 2, 2023
1 parent b45d053 commit 3c7c486
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion ssl/quic/quic_channel.c
Original file line number Diff line number Diff line change
Expand Up @@ -1283,8 +1283,10 @@ static int ch_on_transport_params(const unsigned char *params,
QUIC_CONN_ID cid;
const char *reason = "bad transport parameter";

if (ch->got_remote_transport_params)
if (ch->got_remote_transport_params) {
reason = "multiple transport parameter extensions";
goto malformed;
}

if (!PACKET_buf_init(&pkt, params, params_len)) {
ossl_quic_channel_raise_protocol_error(ch, QUIC_ERR_INTERNAL_ERROR, 0,
Expand Down

0 comments on commit 3c7c486

Please sign in to comment.