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

Fix crash in HcalDigisClient for SLHC configurations #2483

Merged

Conversation

mark-grimes
Copy link

Moved the initialisation of doSLHC_ to before the histograms are booked.

An extra set of histograms are booked in the "booking(...)" method if doSLHC_ is true. If doSLHC_ is not initialised before these calls then the choice whether to book them or not is made on uninitialised memory.

…ed. Otherwise the histograms randomly may or may not be booked depending on the previous memory contents of doSLHC_, causing crashes later.
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mark-grimes (Mark Grimes) for CMSSW_6_2_X_SLHC.

Fix crash in HcalDigisClient for SLHC configurations

It involves the following packages:

Validation/HcalDigis

@ojeda, @danduggan, @rovere, @cmsbuild, @nclopezo, @deguio, @Degano can you please review it and eventually sign? Thanks.
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.
@andersonjacob, @mark-grimes you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@mark-grimes
Copy link
Author

merge

Is there a protocol for accepting my own pull requests? Anyway, this is needed because otherwise all the other other pull requests I'm testing will randomly pass or fail some of the tests. Tested with #2405 on top which was previously causing crashes. All these tests now pass:
3400 with combinedCustoms.cust_2019WithGem
Extended pixel GEN-SIM test
3300, 3400, 4100, 4400, 40001, 50002, 60002, 4502, 4500, 5001 and 15001

cmsbuild added a commit that referenced this pull request Feb 15, 2014
Fix crash in HcalDigisClient for SLHC configurations
@cmsbuild cmsbuild merged commit 3130640 into cms-sw:CMSSW_6_2_X_SLHC Feb 15, 2014
@mark-grimes mark-grimes deleted the fixCrashInHcalDigisClient branch March 13, 2014 18:51
This was referenced Mar 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants