maskit opened a new pull request, #13113: URL: https://github.com/apache/trafficserver/pull/13113
The plugin crashes if when it calls `TSHttpTxnEffectiveUrlStringGet` because `txnp` can be freed and invalid. It happens because the `txnp` (=`HttpSM`) may be destroyed while the plugin processes the upstream data, if the client closes the connection early. The `txnp` is only used for getting an URL on the race condition. This PR change the `Data` structure to keep the URL (string) instead of a pointer for `txnp`. No need to worry about the lifetime. The `txnp` in `Data` structure and the race have been there since the beginning, but the issues was introduced by https://github.com/apache/trafficserver/pull/11618. -- 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]
