]> git.kaiwu.me - nginx.git/commit
HTTP/2: enforce writing the sync request body buffer to file.
authorValentin Bartenev <vbart@nginx.com>
Wed, 4 Oct 2017 18:15:15 +0000 (21:15 +0300)
committerValentin Bartenev <vbart@nginx.com>
Wed, 4 Oct 2017 18:15:15 +0000 (21:15 +0300)
commit9df7bd3439c172a5acb8a0af8bf57302d1dde56a
treef7366674c2bc47ebe09dbc04f3ab832d2f8e4000
parent1602ae60e4294a8be7c5677c6026fdf1c9ceb880
HTTP/2: enforce writing the sync request body buffer to file.

The sync flag of HTTP/2 request body buffer is used when the size of request
body is unknown or bigger than configured "client_body_buffer_size".  In this
case the buffer points to body data inside the global receive buffer that is
used for reading all HTTP/2 connections in the worker process.  Thus, when the
sync flag is set, the buffer must be flushed to a temporary file, otherwise
the request body data can be overwritten.

Previously, the sync buffer wasn't flushed to a temporary file if the whole
body was received in one DATA frame with the END_STREAM flag and wasn't
copied into the HTTP/2 body preread buffer.  As a result, the request body
might be corrupted (ticket #1384).

Now, setting r->request_body_in_file_only enforces writing the sync buffer
to a temporary file in all cases.
src/http/v2/ngx_http_v2.c