GitHub user shinrich opened a pull request:
https://github.com/apache/trafficserver/pull/842
TS-4717: Http2 stack explosion.
Added a common state_process_frame_read method to loop over reading frames
while there is data available. The original state_start_frame_read and
state_complete_frame_read call into state_process_frame_read so the event
handling cases still work.
Have been running a version on of this code on two of our production boxes
for a day. We haven't had a load surge event, so I doubt we have seen a case
that would have caused the stack explosion. But the performance and error
stats seem similar to their peers, so I don't think I have messed up the normal
operating case.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/shinrich/trafficserver ts-4717
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/trafficserver/pull/842.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #842
----
commit a166cf0335672abd3514f43a081b4fce045725f2
Author: Susan Hinrichs <[email protected]>
Date: 2016-08-05T14:29:53Z
TS-4717: Http2 stack explosion.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
