Mercurial > hg > nginx
changeset 7788:9ca8fb98ef1c
Events: fixed eventport handling in ngx_handle_read_event().
The "!rev->ready" test seems to be a typo, introduced in the original
commit (719:f30b1a75fd3b). The ngx_handle_write_event() code properly
tests for "rev->ready" instead.
Due to this typo, read events might be unexpectedly removed during
proxying after an event on the other part of the proxied connection.
Catched by mail proxying tests.
author | Maxim Dounin <mdounin@mdounin.ru> |
---|---|
date | Fri, 05 Mar 2021 17:16:15 +0300 |
parents | 7ce28b4cc57e |
children | ab6257dac2a8 |
files | src/event/ngx_event.c |
diffstat | 1 files changed, 1 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/src/event/ngx_event.c Fri Mar 05 17:16:13 2021 +0300 +++ b/src/event/ngx_event.c Fri Mar 05 17:16:15 2021 +0300 @@ -318,7 +318,7 @@ return NGX_OK; } - if (rev->oneshot && !rev->ready) { + if (rev->oneshot && rev->ready) { if (ngx_del_event(rev, NGX_READ_EVENT, 0) == NGX_ERROR) { return NGX_ERROR; }