# HG changeset patch # User Vladimir Homutov # Date 1602054842 -10800 # Node ID 2b11bd0298150abae982dd0a924c53335dde93d4 # Parent 1d4417e4f2d0b367c705bce057ec8f54d30a2b49 QUIC: fixed memory leak in ngx_quic_send_frames(). The function did not free passed frames in case of error. diff -r 1d4417e4f2d0 -r 2b11bd029815 src/event/ngx_event_quic.c --- a/src/event/ngx_event_quic.c Tue Oct 06 18:08:55 2020 +0100 +++ b/src/event/ngx_event_quic.c Wed Oct 07 10:14:02 2020 +0300 @@ -3839,6 +3839,7 @@ len = ngx_quic_create_frame(p, f); if (len == -1) { + ngx_quic_free_frames(c, frames); return NGX_ERROR; } @@ -3897,11 +3898,13 @@ pkt.num_len, pkt.trunc); if (ngx_quic_encrypt(&pkt, ssl_conn, &res) != NGX_OK) { + ngx_quic_free_frames(c, frames); return NGX_ERROR; } len = c->send(c, res.data, res.len); if (len == NGX_ERROR || (size_t) len != res.len) { + ngx_quic_free_frames(c, frames); return NGX_ERROR; }