Fix segfault in PixelTrackReconstruction

[makortel]

Fixes the segfault reported by Giulio in https://hypernews.cern.ch/HyperNews/CMS/get/swReleases/4012.html. The problem was a spurious delete of theRegionProducer in halt() (called from PixelTrackProducer::endRun()), while it is constructed now in the PixelTrackReconstruction constructor. The bug was introduced in 4f569be (probably by mistake in rebase or something).

The theMerger_ would have suffered from the same problem, so I changed it too to std:unique_ptr.

[cmsbuild]

A new Pull Request was created by @makortel (Matti Kortelainen) for CMSSW_7_1_X.

Fix segfault in PixelTrackReconstruction

It involves the following packages:

RecoPixelVertexing/PixelTrackFitting

@nclopezo, @cmsbuild, @anton-a, @thspeer, @slava77, @Degano can you please review it and eventually sign? Thanks.
@cerati, @GiacomoSguazzoni, @rovere 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, @ktf 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
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2571/251/summary.html

[ktf]

This is crashing right now. I'll bypass signature to have it in the IB, @slava77 @anton-a complain if not ok.

[slava77]

Giulio, did you mean not crashing anymore?

[davidlange6]

@slava77 - this PR fixes a crash in the recent IBs, yes.

On Feb 21, 2014, at 2:12 PM, slava77 notifications@github.com
wrote:

Giulio, did you mean not crashing anymore?

—
Reply to this email directly or view it on GitHub.