bneradt opened a new pull request, #13336: URL: https://github.com/apache/trafficserver/pull/13336
Cherry-picked from https://github.com/apache/trafficserver/pull/13309 Redirected transactions can retain CACHE_WL_SUCCESS after the concrete cache write VC has been cleared. When a later response reaches cache write setup, we crash dereferencing the missing write VC. This only reuses a redirected cache write when the prepared write VC is still available. Otherwise, this resets the write state so ATS prepares a fresh cache write. The added test simply adds some coverage around the scenario but is not a true regression test. That is, the test still passes without the src/ code change. This addresses a crash in the cache write setup path: ``` #0 HttpSM::setup_cache_write_transfer(...) #1 HttpSM::perform_cache_write_action() #2 HttpSM::handle_api_return() #3 HttpSM::state_api_callout() #4 HttpSM::state_api_callback() #5 TSHttpTxnReenable() #6 EscalateResponse() ``` -- 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]
