diff options
| author | Roman Arutyunyan <arut@nginx.com> | 2017-06-14 20:13:41 +0300 |
|---|---|---|
| committer | Roman Arutyunyan <arut@nginx.com> | 2017-06-14 20:13:41 +0300 |
| commit | efa61f42c10355263fd883e53f3e690fe01770a0 (patch) | |
| tree | 4d1c6afd7a7216dd7b9ba3d89004ac1491222ce5 /src/os/unix/ngx_linux_init.c | |
| parent | 439e205255267662c04d13c3404937e3324e06a5 (diff) | |
| download | nginx-efa61f42c10355263fd883e53f3e690fe01770a0.tar.gz nginx-efa61f42c10355263fd883e53f3e690fe01770a0.tar.bz2 | |
Upstream: fixed running posted requests (ticket #788).
Previously, the upstream resolve handler always called
ngx_http_run_posted_requests() to run posted requests after processing the
resolver response. However, if the handler was called directly from the
ngx_resolve_name() function (for example, if the resolver response was cached),
running posted requests from the handler could lead to the following errors:
- If the request was scheduled for termination, it could actually be terminated
in the resolve handler. Upper stack frames could reference the freed request
object in this case.
- If a significant number of requests were posted, and for each of them the
resolve handler was called directly from the ngx_resolve_name() function,
posted requests could be run recursively and lead to stack overflow.
Now ngx_http_run_posted_requests() is only called from asynchronously invoked
resolve handlers.
Diffstat (limited to 'src/os/unix/ngx_linux_init.c')
0 files changed, 0 insertions, 0 deletions
