From 557ff072ceb45e0a328a4efc73d50524ba164512 Mon Sep 17 00:00:00 2001 From: Dan Riley Date: Tue, 3 Dec 2019 12:42:38 -0500 Subject: [PATCH] fix loop bounds off by one errors --- .../plugins/TrackerAlignment_PayloadInspector.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CondCore/AlignmentPlugins/plugins/TrackerAlignment_PayloadInspector.cc b/CondCore/AlignmentPlugins/plugins/TrackerAlignment_PayloadInspector.cc index 89ca32df40066..518cb6009a6fe 100644 --- a/CondCore/AlignmentPlugins/plugins/TrackerAlignment_PayloadInspector.cc +++ b/CondCore/AlignmentPlugins/plugins/TrackerAlignment_PayloadInspector.cc @@ -132,7 +132,7 @@ namespace { std::vector boundaries; AlignmentPI::partitions currentPart = AlignmentPI::BPix; - for (unsigned int i = 0; i <= ref_ali.size(); i++) { + for (unsigned int i = 0; i < ref_ali.size(); i++) { if (ref_ali[i].rawId() == target_ali[i].rawId()) { counter++; int subid = DetId(ref_ali[i].rawId()).subdetId(); @@ -359,7 +359,7 @@ namespace { } int loopedComponents(0); - for (unsigned int i = 0; i <= ref_ali.size(); i++) { + for (unsigned int i = 0; i < ref_ali.size(); i++) { if (ref_ali[i].rawId() == target_ali[i].rawId()) { loopedComponents++; int subid = DetId(ref_ali[i].rawId()).subdetId();