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 2018/08/07 20:09:54 UTC

[trafficserver] branch 8.0.x updated: Cached object should not be removed when write is aborted

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

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


The following commit(s) were added to refs/heads/8.0.x by this push:
     new dbbd725  Cached object should not be removed when write is aborted
dbbd725 is described below

commit dbbd725b79ab23aa94ab38fd6e0d3342066c2363
Author: Vijay Mamidi <vi...@yahoo.com>
AuthorDate: Tue Jul 31 14:31:44 2018 -0700

    Cached object should not be removed when write is aborted
    
    (cherry picked from commit c99d010ac0d227f0c018e45f886c2ce6f0d24517)
---
 proxy/http/HttpCacheSM.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/proxy/http/HttpCacheSM.h b/proxy/http/HttpCacheSM.h
index f1d8872..435279f 100644
--- a/proxy/http/HttpCacheSM.h
+++ b/proxy/http/HttpCacheSM.h
@@ -142,7 +142,7 @@ public:
   {
     if (cache_read_vc) {
       HTTP_DECREMENT_DYN_STAT(http_current_cache_connections_stat);
-      cache_read_vc->do_io_close(); // abort
+      cache_read_vc->do_io_close(0); // passing zero as aborting read is not an error
       cache_read_vc = nullptr;
     }
   }
@@ -151,7 +151,7 @@ public:
   {
     if (cache_write_vc) {
       HTTP_DECREMENT_DYN_STAT(http_current_cache_connections_stat);
-      cache_write_vc->do_io_close(); // abort
+      cache_write_vc->do_io_close(0); // passing zero as aborting write is not an error
       cache_write_vc = nullptr;
     }
   }