[PR] 10.1.x: Fix redirected cache write without write VC [trafficserver]

["bneradt (via GitHub)" <gi...@apache.org>]

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: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] 10.1.x: Fix redirected cache write without write VC [trafficserver]

["cmcfarlen (via GitHub)" <gi...@apache.org>]

cmcfarlen merged PR #13336:
URL: https://github.com/apache/trafficserver/pull/13336


-- 
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: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org