Skip to content

Commit c5498af

Browse files
committed
GPU TPC Merger: Fix out of bounds check
1 parent b8cacf6 commit c5498af

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

GPU/GPUTracking/Merger/GPUTPCGMMerger.cxx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1513,7 +1513,6 @@ GPUd() void GPUTPCGMMerger::CollectMergedTracks(int32_t nBlocks, int32_t nThread
15131513
GPUTPCGMSectorTrack* trackParts[kMaxParts];
15141514

15151515
for (int32_t itr = iBlock * nThreads + iThread; itr < SectorTrackInfoLocalTotal(); itr += nThreads * nBlocks) {
1516-
15171516
GPUTPCGMSectorTrack& track = mSectorTrackInfos[itr];
15181517

15191518
if (track.PrevSegmentNeighbour() >= 0) {
@@ -1712,7 +1711,7 @@ GPUd() void GPUTPCGMMerger::CollectMergedTracks(int32_t nBlocks, int32_t nThread
17121711
}
17131712

17141713
const uint32_t iMergedTrackFirstCluster = CAMath::AtomicAdd(&mMemory->nMergedTrackClusters, (uint32_t)nHits);
1715-
if (iMergedTrackFirstCluster >= mNMaxMergedTrackClusters) {
1714+
if (iMergedTrackFirstCluster + nHits > mNMaxMergedTrackClusters) {
17161715
raiseError(GPUErrors::ERROR_MERGER_HIT_OVERFLOW, iMergedTrackFirstCluster, mNMaxMergedTrackClusters);
17171716
CAMath::AtomicExch(&mMemory->nMergedTrackClusters, mNMaxMergedTrackClusters);
17181717
continue;

0 commit comments

Comments
 (0)