* double fastcgi_end_request on max_children limit

[dmitry-saprykin]

Php sends 2 fastcgi FCGI_END_REQUEST records instead of one when fastcgi_finish_request(); called and fpm param max_requests exceeded.

• one is sent on fastcgi_finish_request(); call
• other is sent in block
  fpm_main.c:1951 requests++; if (max_requests && (requests == max_requests)) { fcgi_finish_request(&request, 1); break; }

It confuses fastcgi client (i.e. nginx) receiving 2 FCGI_END_REQUEST records one after another.
So I have set request->closed flag to prevent second FastCGI record sending.

[Berbe]

1. Long-lasting bug (already there in 5.3.x branch (even before?)
2. 1-liner solution

Those facts should help pushing that modification into the main trunk...

[Berbe]

It would be better to close this one and reopen #709 requesting a merge to the master branch, rather than to the now outdated 5.5.14 one

[laruence]

close as we don't accept PR against TAG version. please reopen #709 instead.

[Berbe]

@laruence Could not you then reopen #709? Taking action to close is good, but not constructive without the counterpart.

[laruence]

@Berbe it truned out I can not reopen it.. because the origin branch seems deleted by the author.

[Berbe]

The branch seemes to have changed name.
I opened #1169 to take over #709 and this pull request