Skip to content
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

CSCDigiToRawModule deferences invalid edm::Handle #30916

Closed
Dr15Jones opened this issue Jul 25, 2020 · 6 comments
Closed

CSCDigiToRawModule deferences invalid edm::Handle #30916

Dr15Jones opened this issue Jul 25, 2020 · 6 comments

Comments

@Dr15Jones
Copy link
Contributor

The UBSAN build uncovered the following

/data/cmsbld/jenkins/workspace/build-any-ib/w/tmp/BUILDROOT/3d0af0c821cc86dc2e95a3561ce9bea6/opt/cmssw/slc7_amd64_gcc820/cms/cmssw/CMSSW_11_2_UBSAN_X_2020-07-24-2300/src/EventFilter/CSCRawToDigi/plugins/CSCDigiToRawModule.cc:172:28: runtime error: reference binding to null pointer of type 'const struct GEMPadDigiClusterCollection'

The problem is here with the variable padDigiClusters which is an edm::Handle
https://github.com/cms-sw/cmssw/blob/master/EventFilter/CSCRawToDigi/plugins/CSCDigiToRawModule.cc#L168-L179

The Handle is only set if useGEMs_ is true. If it is false, than an invalid edm::Handle is deferences and a nullptr is returned via the reference. [See bug report #30915 ]. Once the edm::Handle bug is fixed, this code will fail.

@cmsbuild
Copy link
Contributor

A new Issue was created by @Dr15Jones Chris Jones.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@Dr15Jones
Copy link
Contributor Author

assign reconstruction

@cmsbuild
Copy link
Contributor

New categories assigned: reconstruction

@slava77,@perrotta,@jpata you have been requested to review this Pull request/Issue and eventually sign? Thanks

@Dr15Jones
Copy link
Contributor Author

Fixed by #30925

@perrotta
Copy link
Contributor

+1
(see #30916 (comment), thank you @Dr15Jones )

@cmsbuild
Copy link
Contributor

This issue is fully signed and ready to be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants