bryancall commented on issue #7871: URL: https://github.com/apache/trafficserver/issues/7871#issuecomment-4827448809
I am closing this as obsolete. This crash was reported against the 2021 HTTP/2 closing path, where a reentrant do_io_read(netvc->read_vio_cont(), ...) in HttpSM::state_add_to_list bound the state machine's read continuation to the Http2ClientSession and broke the session to stream relationship. That code path has since been reworked: on current master, state_add_to_list no longer uses read_vio_cont() (it calls do_io_read(this, 0, nullptr)), and the workaround proposed in this thread, calling do_io_close() in Http2ClientSession::destroy(), is now present in the tree (src/proxy/http2/Http2ClientSession.cc). The referenced pull requests are all defunct ([#7586](https://github.com/apache/trafficserver/pull/7586) merged but did not resolve this, and [#7595](https://github.com/apache/trafficserver/pull/7595) and [#7845](https://github.com/apache/trafficserver/pull/7845) were both closed without merging), and the HTTP/2 session closing path has had substantial rework since. If this crash can still be reproduced on a current release, please reopen with a fresh reproduction. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
