-
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_X] heap-buffer-overflow in l1t::L1GTSingleCollectionCut::checkObject (checkEtaDependentCuts) #47194
Comments
assign L1Trigger/Phase2L1GT |
New categories assigned: l1,upgrade @aloeliger,@epalencia,@Moanwar,@srimanob,@subirsarkar you have been requested to review this Pull request/Issue and eventually sign? Thanks |
cms-bot internal usage |
A new Issue was created by @iarspider. @Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
Log examples: RelVal 24834.0, RelVal 29634.0, RelVal 24834.911. |
Thanks for pinging! FYI @HaarigerHarald @qvyz Is this a concrete workflow where this appears? How can one reproduce this? |
To precise: I see the logs of the failed workflows but aren’t these also run for the PR tests? What is the difference? |
The ASAN builds (address sanitizer) link to a special library which tracks memory behavior. This library is rather slow so we do not include it in PR tests. This problem was only uncovered by the ASAN library. To be more accurate, ASAN builds our code in a different way in order to utilize the library. |
cmsrel CMSSW_15_0_ASAN_X_2025-01-27-2300
cd CMSSW_15_0_ASAN_X_2025-01-27-2300/src
cmsenv
runTheMatrix.py -l 24834.0, 29634.0, 24834.911 this should reproduce. |
The issue appeared when the number of eta regions (bounds) did not match e.g. the number of pt region thresholds. Fixed in #47204 |
In CMSSW_15_0_ASAN_X_2025-01-27-2300, several RelVals failed with
heap-buffer-overflow in l1t::L1GTSingleCollectionCut::checkObject
:Running locally after building with
-g
indicates that the failure happens on line 211, whenindex == 2
andregionsMinPt_.size() == 2
The text was updated successfully, but these errors were encountered: