Fijxu/nginx-quic
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

QUIC: added missing check for backup path existence.

Browse source
This commit is contained in:
Vladimir Homutov 2021-12-16 11:42:28 +03:00
parent da7d48ca9f
commit 09e77a9751
1 changed files with 15 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
src/event/quic/ngx_event_quic_migration.c
View file

@ -81,6 +81,7 @@ ngx_int_t
ngx_quic_handle_path_response_frame(ngx_connection_t *c, ngx_quic_handle_path_response_frame(ngx_connection_t *c,
ngx_quic_path_challenge_frame_t *f) ngx_quic_path_challenge_frame_t *f)
{ {
ngx_uint_t rst;
ngx_queue_t *q; ngx_queue_t *q;
ngx_quic_path_t *path, *prev; ngx_quic_path_t *path, *prev;
ngx_quic_connection_t *qc; ngx_quic_connection_t *qc;
@ -127,13 +128,21 @@ valid:
* unless the only change in the peer's address is its port number. * unless the only change in the peer's address is its port number.
*/ */
prev = qc->backup->path; rst = 1;
if (ngx_cmp_sockaddr(prev->sockaddr, prev->socklen, if (qc->backup) {
path->sockaddr, path->socklen, 0) prev = qc->backup->path;
!= NGX_OK)
{ if (ngx_cmp_sockaddr(prev->sockaddr, prev->socklen,
/* address has changed */ path->sockaddr, path->socklen, 0)
== NGX_OK)
{
/* address did not change */
rst = 0;
}
}
if (rst) {
ngx_memzero(&qc->congestion, sizeof(ngx_quic_congestion_t)); ngx_memzero(&qc->congestion, sizeof(ngx_quic_congestion_t));
qc->congestion.window = ngx_min(10 * qc->tp.max_udp_payload_size, qc->congestion.window = ngx_min(10 * qc->tp.max_udp_payload_size,