nginx: src/event/ngx_event_quic_transport.c comparison

QUIC: fixed handling of incorrect packets. Instead of ignoring, connection was closed. This was broken in d0d3fc0697a0.

author	Vladimir Homutov <vl@nginx.com>
date	Thu, 01 Oct 2020 22:20:51 +0300
parents	b31c02454539
children	b95aa1cb8f98

comparison

equal deleted inserted replaced

-:0875101c08f7
+:a6784cf32c13
 ngx_quic_parse_packet(ngx_quic_header_t *pkt)
 {
 if (!ngx_quic_long_pkt(pkt->flags)) {
 pkt->level = ssl_encryption_application;
-return ngx_quic_parse_short_header(pkt, NGX_QUIC_SERVER_CID_LEN);
+if (ngx_quic_parse_short_header(pkt, NGX_QUIC_SERVER_CID_LEN) != NGX_OK)
+{
+return NGX_DECLINED;
+}
+return NGX_OK;
 }
 if (ngx_quic_parse_long_header(pkt) != NGX_OK) {
 return NGX_DECLINED;
 }
 return NGX_DECLINED;
 }
 pkt->level = ssl_encryption_initial;
-return ngx_quic_parse_initial_header(pkt);
+if (ngx_quic_parse_initial_header(pkt) != NGX_OK) {
+return NGX_DECLINED;
+}
+return NGX_OK;
 }
 if (ngx_quic_pkt_hs(pkt->flags)) {
 pkt->level = ssl_encryption_handshake;
 ngx_log_error(NGX_LOG_INFO, pkt->log, 0,
 "quic unknown long packet type");
 return NGX_DECLINED;
 }
-return ngx_quic_parse_handshake_header(pkt);
+if (ngx_quic_parse_handshake_header(pkt) != NGX_OK) {
+return NGX_DECLINED;
+}
+return NGX_OK;
 }
 static ngx_int_t
 ngx_quic_parse_long_header(ngx_quic_header_t *pkt)

Mercurial > hg > nginx