You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2022/08/10 17:11:15 UTC

[trafficserver] branch 9.2.x updated: Add RangeTransform::m_write_vio state checks (#8980)

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.2.x by this push:
     new ffd53059c Add RangeTransform::m_write_vio state checks (#8980)
ffd53059c is described below

commit ffd53059ca7f6daafa05dcd7cb3084f2b43b61c5
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Wed Aug 3 07:37:09 2022 +0900

    Add RangeTransform::m_write_vio state checks (#8980)
    
    (cherry picked from commit e912ecef7065b6c04183e10e29b15f1056a7f281)
---
 proxy/Transform.cc | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/proxy/Transform.cc b/proxy/Transform.cc
index 664e310df..5013d3192 100644
--- a/proxy/Transform.cc
+++ b/proxy/Transform.cc
@@ -791,7 +791,9 @@ RangeTransform::handle_event(int event, void *edata)
   } else {
     switch (event) {
     case VC_EVENT_ERROR:
-      m_write_vio.cont->handleEvent(VC_EVENT_ERROR, &m_write_vio);
+      if (m_write_vio.cont) {
+        m_write_vio.cont->handleEvent(VC_EVENT_ERROR, &m_write_vio);
+      }
       break;
     case VC_EVENT_WRITE_COMPLETE:
       ink_assert(m_output_vio == (VIO *)edata);
@@ -814,6 +816,10 @@ RangeTransform::handle_event(int event, void *edata)
         }
       }
 
+      if (!m_write_vio.mutex) {
+        return 0;
+      }
+
       MUTEX_TRY_LOCK(trylock, m_write_vio.mutex, this_ethread());
       if (!trylock.is_locked()) {
         retry(10);