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
Fixed BUS error due to uninitialized MEs #29484
Fixed BUS error due to uninitialized MEs #29484
Conversation
The code-checks are being triggered in jenkins. |
please test |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29484/14691
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
645e321
to
cd134a1
Compare
The code-checks are being triggered in jenkins. |
please test |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29484/14696
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
Added initializers to the constructor
cd134a1
to
8abab4f
Compare
please test |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29484/14698
|
The tests are being triggered in jenkins. |
Comparison is ready Comparison Summary:
|
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Added
MonitorElement
pointer initializers to the constructor because booking logic is conditional and in some cases MEs are left uninitialized, pointing to random places in the memory. A BUS error because of this is observed by the new unit tests: #29372PR validation:
PR was validated locally.