-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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] stack-buffer-overflow in HGCalWaferMask::waferXY #41184
Comments
assign geometry |
New categories assigned: geometry @mdhildreth,@Dr15Jones,@makortel,@bsunanda,@civanch you have been requested to review this Pull request/Issue and eventually sign? Thanks |
A new Issue was created by @iarspider . @Dr15Jones, @perrotta, @dpiparo, @rappoccio, @makortel, @smuzaffar can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
FYI @cms-sw/hgcal-dpg-l2 |
Likely culprit: the local
but this loop cmssw/Geometry/HGCalCommonData/src/HGCalWaferMask.cc Lines 1345 to 1354 in bc670eb
has a potential to access the arrays with index 48 cmssw/Geometry/HGCalCommonData/src/HGCalWaferMask.cc Lines 1345 to 1354 in bc670eb
leading to the read of one-past-end element. |
UBSAN has additional info confirming @makortel's likely suspect:
|
Several unit tests (e.g. [1], [2], [3]) failed in 2023_03_25_2300 ASAN IB:
[1] https://cmssdt.cern.ch/SDT/cgi-bin/logreader/el8_amd64_gcc11/CMSSW_13_1_ASAN_X_2023-03-24-2300/pyRelValMatrixLogs/run/20834.0_TTbar_14TeV+2026D88/step2_TTbar_14TeV+2026D88.log#/56
[2] https://cmssdt.cern.ch/SDT/cgi-bin/logreader/el8_amd64_gcc11/CMSSW_13_1_ASAN_X_2023-03-24-2300/pyRelValMatrixLogs/run/20834.103_TTbar_14TeV+2026D88Aging3000/step2_TTbar_14TeV+2026D88Aging3000.log#/57
[3] https://cmssdt.cern.ch/SDT/cgi-bin/logreader/el8_amd64_gcc11/CMSSW_13_1_ASAN_X_2023-03-24-2300/pyRelValMatrixLogs/run/20834.21_TTbar_14TeV+2026D88_ProdLike/step2_TTbar_14TeV+2026D88_ProdLike.log#/24
The text was updated successfully, but these errors were encountered: