From c34e5ece6ac1d8ad8a2036172461680700990733 Mon Sep 17 00:00:00 2001 From: uzhastik Date: Mon, 5 Aug 2024 20:50:24 +0300 Subject: [PATCH 1/3] check vector access --- ydb/services/metadata/ds_table/accessor_snapshot_base.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp b/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp index 662c7bbdd8c5..8aaeafec1e10 100644 --- a/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp +++ b/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp @@ -34,6 +34,7 @@ void TDSAccessorBase::Handle(NRequest::TEvRequestResultsecond); if (it->second == 1) { + Y_ABORT_UNLESS(replyIdx < qResult.result_sets().size()); *qResultFull.add_result_sets() = std::move(qResult.result_sets()[replyIdx]); ++replyIdx; } else { From 7be92b38ed6e8304306ca99c4338bff5f830f157 Mon Sep 17 00:00:00 2001 From: uzhastik Date: Mon, 5 Aug 2024 21:08:44 +0300 Subject: [PATCH 2/3] fix build --- ydb/services/metadata/ds_table/accessor_snapshot_base.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp b/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp index 8aaeafec1e10..687165cd802d 100644 --- a/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp +++ b/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp @@ -34,7 +34,7 @@ void TDSAccessorBase::Handle(NRequest::TEvRequestResultsecond); if (it->second == 1) { - Y_ABORT_UNLESS(replyIdx < qResult.result_sets().size()); + Y_ABORT_UNLESS((size_t)replyIdx < qResult.result_sets().size()); *qResultFull.add_result_sets() = std::move(qResult.result_sets()[replyIdx]); ++replyIdx; } else { From a542e614e5854654e2773a3367b4e0bc5e290cd5 Mon Sep 17 00:00:00 2001 From: uzhastik Date: Mon, 5 Aug 2024 21:21:57 +0300 Subject: [PATCH 3/3] fix build again --- ydb/services/metadata/ds_table/accessor_snapshot_base.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp b/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp index 687165cd802d..6e90bbe904ff 100644 --- a/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp +++ b/ydb/services/metadata/ds_table/accessor_snapshot_base.cpp @@ -34,7 +34,7 @@ void TDSAccessorBase::Handle(NRequest::TEvRequestResultsecond); if (it->second == 1) { - Y_ABORT_UNLESS((size_t)replyIdx < qResult.result_sets().size()); + Y_ABORT_UNLESS((int)replyIdx < qResult.result_sets().size()); *qResultFull.add_result_sets() = std::move(qResult.result_sets()[replyIdx]); ++replyIdx; } else {