shinrich opened a new pull request #7807: URL: https://github.com/apache/trafficserver/pull/7807
This addresses the ink_assert described in #7705 two ways. First, it always clears the cancelled bit in the captive_action before returning the captive action. Second, the HttpSM handlers that are storing references to the HttpCacheSM captive_action, use the new try_clear() action to clear the pending_action member without cancelling it if it matches the action passed into the handler. Based on the stack trace in the issue, the handler in frame #27 with this change will not mark the captive action cancelled when clearing it. I'll try to spend some time to exercise this case, but I haven't run into this yet. So verification from those who can reproduce the case would be appreciated. This closes #7705 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
