-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ASAN][heap-buffer-overflow] DataFormats/Common/interface/DataFrameContainer.h:181 #11982
Comments
@davidlt @Dr15Jones I will see if I can reproduce this. |
@davidlt @Dr15Jones The problem is reproducible. It did indeed fail on event 78. Debugging starts tomorrow. |
@davidlt @Martin-Grunewald @perrotta @fwyzard @Dr15Jones The problem is in HLTrigger/special/src/HLTRechitsToDigis.cc. outputEBDigiCollection->push_back( (_digisEB->find(hit.id())).id(), (_digisEB->find(hit.id())).begin() ); outputEEDigiCollection->push_back( (_digisEE->find(hit.id())).id(), (_digisEE->find(hit.id())).begin() ); It is up to HLT to decide what to do if find() returns end(). |
This seems to be a common issue (where something returns end() and code never checks for that). |
The only issue could be that vector (or different container) might not be exposed (e.g. it's private). In that case you cannot verify that integrator is In this particular case, is it possible to check that |
Yes. It is easy. It works. |
I contacted the author of the HLTRechitsToDigis.cc code, Joshua Hardenbrook [email protected] @hardenbr |
I think, this one is closed by #12041 . Closing. |
slc6_amd64_gcc493 and CMSSW_7_6_X_2015-10-19-1100
Originally the report pointed to
pluginHLTriggerspecial.so
(HLTrigger/special
)HLTRechitsToDigis::produce(edm::Event&, edm::EventSetup const&)
Recompiled with
-O0 -g3
and gotDataFormats/Common/interface/DataFrameContainer.h:181
Assert
Assert patch
Basically you also need to recompile
HLTrigger/special
because it use this header.Fails on 134.805, step2, 78th event.
The text was updated successfully, but these errors were encountered: