You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by da...@apache.org on 2024/04/17 13:28:05 UTC

(doris) 08/15: [fix](cloud)(review) map end check added (#33612) (#33613)

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

dataroaring pushed a commit to branch branch-4.0-preview
in repository https://gitbox.apache.org/repos/asf/doris.git

commit acf4b4b07fa999420d26cace161f55c03420a5e7
Author: Vallish Pai <va...@gmail.com>
AuthorDate: Wed Apr 17 08:21:11 2024 +0530

    [fix](cloud)(review) map end check added (#33612) (#33613)
    
    close #33612,add validation for iterator
---
 be/src/cloud/cloud_txn_delete_bitmap_cache.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/be/src/cloud/cloud_txn_delete_bitmap_cache.cpp b/be/src/cloud/cloud_txn_delete_bitmap_cache.cpp
index 6f0d90b37bb..284587eb756 100644
--- a/be/src/cloud/cloud_txn_delete_bitmap_cache.cpp
+++ b/be/src/cloud/cloud_txn_delete_bitmap_cache.cpp
@@ -159,7 +159,7 @@ void CloudTxnDeleteBitmapCache::remove_expired_tablet_txn_info() {
             break;
         }
         auto txn_iter = _txn_map.find(iter->second);
-        if (iter->first == txn_iter->second.txn_expiration) {
+        if ((txn_iter != _txn_map.end()) && (iter->first == txn_iter->second.txn_expiration)) {
             LOG_INFO("clean expired delete bitmap")
                     .tag("txn_id", txn_iter->first.txn_id)
                     .tag("expiration", txn_iter->second.txn_expiration)


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