maskit opened a new pull request #6861:
URL: https://github.com/apache/trafficserver/pull/6861
A possible fix for the crash on TLS resumption.
The crash seems to be caused by a small change made on ##6736. On the PR, I
added `this->_setSSLCurveNID(SSLGetCurveNID(ssl));` after a call for
`_setSSLSessionCacheHit` for consistency, but I realized it doesn't make sense
in `_getSessionInformation`.
So I removed the line and added a code similar to `fetch_ssl_curve`, which
was removed on #6736.
I couldn't reproduce the crash by myself and CI tests don't catch the crash
somehow, so please ran this locally if you can reproduce the crash.
----------------------------------------------------------------
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]
