You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by yi...@apache.org on 2022/07/11 11:04:23 UTC

[doris] branch master updated: [bug](be) be asan core doris::DiskIoMgr::~DiskIoMgr(#10759) (#10760)

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

yiguolei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 5eb38467ef [bug](be) be asan core doris::DiskIoMgr::~DiskIoMgr(#10759) (#10760)
5eb38467ef is described below

commit 5eb38467ef5e14146753681b5e9a668dfe13fc1a
Author: Lei Zhang <10...@qq.com>
AuthorDate: Mon Jul 11 19:04:16 2022 +0800

    [bug](be) be asan core doris::DiskIoMgr::~DiskIoMgr(#10759) (#10760)
---
 be/src/runtime/disk_io_mgr.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/be/src/runtime/disk_io_mgr.cc b/be/src/runtime/disk_io_mgr.cc
index 90a25b0d7a..c0bbea36e0 100644
--- a/be/src/runtime/disk_io_mgr.cc
+++ b/be/src/runtime/disk_io_mgr.cc
@@ -144,9 +144,9 @@ public:
         }
     }
 
-    // Validates that all readers are cleaned up and in the inactive state.  No locks
-    // are taken since this is only called from the disk IoMgr destructor.
+    // Validates that all readers are cleaned up and in the inactive state.
     bool validate_all_inactive() {
+        lock_guard<mutex> l(_lock);
         for (list<RequestContext*>::iterator it = _all_contexts.begin(); it != _all_contexts.end();
              ++it) {
             if ((*it)->_state != RequestContext::Inactive) {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org