slava77 / cmssw

CMS Offline Software
0 stars 0 forks source link

70X memory leak in ElectronTagger and MuonTagger discriminator #20

Closed slava77 closed 10 years ago

slava77 commented 10 years ago

Visible in CMSSW_7_0_X_2014-01-17-0200 (should be in 700pre12) in TTBar 40PU25ns test run

Concise report in https://slava77sk.web.cern.ch/slava77sk/reco/valgrind_70X-201412170200.txt Full report in https://slava77sk.web.cern.ch/slava77sk/reco/valgrind_70X-201412170200.xml

    960,680 (937,320 direct, 23,360 indirect) bytes in 365 blocks are definitely lost in loss record 151,560 of 151,598
      operator new(unsigned long)
      TStorage::ObjectAlloc(unsigned long)
      ElectronTagger::discriminator(JetTagComputer::TagInfoHelper const&) const
      JetTagProducer::produce(edm::Event&, edm::EventSetup const&)

    726,432 (708,768 direct, 17,664 indirect) bytes in 276 blocks are definitely lost in loss record 151,535 of 151,598
      operator new(unsigned long)
      TStorage::ObjectAlloc(unsigned long)
      MuonTagger::discriminator(JetTagComputer::TagInfoHelper const&) const
      JetTagProducer::produce(edm::Event&, edm::EventSetup const&)

Both have the same problems:

                        TRandom3 *r = new TRandom3(0);
slava77 commented 10 years ago

fixed in https://github.com/cms-sw/cmssw/pull/2209 bb0765c