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);