diff options
| author | Roman Arutyunyan <arut@nginx.com> | 2017-05-24 13:17:08 +0300 |
|---|---|---|
| committer | Roman Arutyunyan <arut@nginx.com> | 2017-05-24 13:17:08 +0300 |
| commit | c83922b18ddc83f654c1d0df48a6ca1ee9938078 (patch) | |
| tree | 19dda4ce5c6028dd11770e812eb1dfa1e2ac5758 /src | |
| parent | cce39344610ce83956379d369cf42a514d5bc258 (diff) | |
| download | nginx-c83922b18ddc83f654c1d0df48a6ca1ee9938078.tar.gz nginx-c83922b18ddc83f654c1d0df48a6ca1ee9938078.zip | |
Fixed deferred accept with EPOLLRDHUP enabled (ticket #1278).
Previously, the read event of the accepted connection was marked ready, but not
available. This made EPOLLRDHUP-related code (for example, in ngx_unix_recv())
expect more data from the socket, leading to unexpected behavior.
For example, if SSL, PROXY protocol and deferred accept were enabled on a listen
socket, the client connection was aborted due to unexpected return value of
c->recv().
Diffstat (limited to 'src')
| -rw-r--r-- | src/event/ngx_event_accept.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/event/ngx_event_accept.c b/src/event/ngx_event_accept.c index 1fce2e89b..87447d089 100644 --- a/src/event/ngx_event_accept.c +++ b/src/event/ngx_event_accept.c @@ -238,7 +238,7 @@ ngx_event_accept(ngx_event_t *ev) if (ev->deferred_accept) { rev->ready = 1; -#if (NGX_HAVE_KQUEUE) +#if (NGX_HAVE_KQUEUE || NGX_HAVE_EPOLLRDHUP) rev->available = 1; #endif } |