Remove unused seedsRefs from DuplicateListMerger.cc

[davidlt]

RecoTracker/FinalTrackSelectors/plugins/DuplicateListMerger.cc:196:49
was reported by UBSan 25 times during full matrix.

DuplicateListMerger.cc:196:49: runtime error: variable length array
bound evaluates to non-positive value 0

Example workflows:

• 33.0 / step3
• 1319.0 / step3
• 1001.0 / step2
• 1002.0 / step2

seedsRefs is not used and removing it does not trigger any side effects.
Otherwise this is a potential spot for out-of-bounds writes to stack if
a possible code path if found.

Signed-off-by: David Abdurachmanov David.Abdurachmanov@cern.ch

[cmsbuild]

A new Pull Request was created by @davidlt for CMSSW_7_2_X.

Remove unsued seedsRefs from DuplicateListMerger.cc

It involves the following packages:

RecoTracker/FinalTrackSelectors

@cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
@ghellwig, @GiacomoSguazzoni, @rovere, @VinInn, @gpetruc, @cerati, @venturia this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[cmsbuild]

+1
Tested at: 467b7fa
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5451/1034/summary.html

[VinInn]

+1

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5451/1034/summary.html

[StoyanStoynev]

+1
For 467b7fa . Based on jenkinsa - no diffs.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine). @nclopezo can you please take care of it?