The official NGINX Open Source repository.
Go to file
Maxim Dounin 2c8dd1c33e Changed ngx_chain_update_chains() to test tag first (ticket #2248).
Without this change, aio used with HTTP/2 can result in connection hang,
as observed with "aio threads; aio_write on;" and proxying (ticket #2248).

The problem is that HTTP/2 updates buffers outside of the output filters
(notably, marks them as sent), and then posts a write event to call
output filters.  If a filter does not call the next one for some reason
(for example, because of an AIO operation in progress), this might
result in a state when the owner of a buffer already called
ngx_chain_update_chains() and can reuse the buffer, while the same buffer
is still sitting in the busy chain of some other filter.

In the particular case a buffer was sitting in output chain's ctx->busy,
and was reused by event pipe.  Output chain's ctx->busy was permanently
blocked by it, and this resulted in connection hang.

Fix is to change ngx_chain_update_chains() to skip buffers from other
modules unconditionally, without trying to wait for these buffers to
become empty.
2021-10-30 02:39:19 +03:00
auto Configure: fixed --test-build-epoll on FreeBSD 13. 2021-04-05 20:14:16 +03:00
conf MIME: added image/avif type. 2021-10-25 20:49:15 +03:00
contrib Contrib: vim syntax, update core and 3rd party module directives. 2021-05-30 12:26:00 +03:00
docs nginx-1.21.3-RELEASE 2021-09-07 18:21:02 +03:00
misc Updated OpenSSL used for win32 builds. 2021-08-31 17:54:54 +03:00
src Changed ngx_chain_update_chains() to test tag first (ticket #2248). 2021-10-30 02:39:19 +03:00
.hgtags release-1.21.3 tag 2021-09-07 18:21:03 +03:00