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 the address and thread sanitizer builds on newer kernels #1304

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

joka921
Copy link
Member

@joka921 joka921 commented Mar 14, 2024

Recent versions of the linux kernel and also recent versions of the Ubuntu distributions have increased the number of bits that are used for virtual address layout randomization. The sanitizer tools have to be aware of this randomization but have not yet picked up that change. Our current fix is to manually reduce the number of randomization bits back to 28 (the previous value) for our GitHub actions builds.

Copy link

codecov bot commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.10%. Comparing base (28366a1) to head (3d3ea93).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1304      +/-   ##
==========================================
+ Coverage   88.09%   88.10%   +0.01%     
==========================================
  Files         310      310              
  Lines       28436    28436              
  Branches     3148     3148              
==========================================
+ Hits        25050    25053       +3     
  Misses       2231     2231              
+ Partials     1155     1152       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@joka921 joka921 changed the title First playing around to see if this is indeed the problem. Fix the address and thread sanitizer builds on newer kernels Mar 14, 2024
@joka921 joka921 marked this pull request as ready for review March 14, 2024 11:13
@joka921
Copy link
Member Author

joka921 commented Mar 14, 2024

Recent versions of the linux kernel and also recent versions of the Ubuntu distributions have increased the number of bits that are used for virtual address layout randomization. The sanitizer tools have to be aware of this randomization but have not yet picked up that change. Our current fix is to manually reduce the number of randomization bits back to 28 (the previous value) for our GitHub actions builds.

@joka921 joka921 merged commit bcc05cc into ad-freiburg:master Mar 14, 2024
18 checks passed
@joka921 joka921 deleted the fix-sanitizers-random-layout branch March 14, 2024 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant