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.

Sign up for GitHub

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

Jump to bottom

PROT_EXEC only for certain spaces #7

Closed
qinsoon opened this issue Mar 26, 2020 · 0 comments · Fixed by #1176
Closed

PROT_EXEC only for certain spaces #7

qinsoon opened this issue Mar 26, 2020 · 0 comments · Fixed by #1176
Labels
A-space Area: Space/PageResource C-bug Category: Bug F-project Call For Participation: Student projects. These issues are available student projects. G-safety Goal: Safety P-normal Priority: Normal.

Comments

@qinsoon
Copy link
Member

qinsoon commented Mar 26, 2020

In GitLab by @pzakopaylo01 on Dec 16, 2017, 19:40

e.g. large object space, code space etc.

This would mean we don't have to give the entire heap PROT_EXEC permissions, which is bad for security.
@qinsoon qinsoon added the C-enhancement Category: Enhancement label Mar 26, 2020
@qinsoon qinsoon added the A-space Area: Space/PageResource label Jun 5, 2020
@qinsoon qinsoon changed the title Add support for different allocators PROT_EXEC only for certain spaces Jun 5, 2020
@qinsoon qinsoon added C-bug Category: Bug and removed C-enhancement Category: Enhancement labels Jun 5, 2020
@qinsoon qinsoon mentioned this issue Jun 5, 2020
Open
@steveblackburn steveblackburn added F-project Call For Participation: Student projects. These issues are available student projects. G-safety Goal: Safety labels Feb 22, 2021
wenyuzhao referenced this issue in wenyuzhao/mmtk-core Sep 14, 2021
@wenyuzhao
Merge pull request #7 from mmtk/update-mmtk-core-pr-92
36bf7ae 
Fix API changes in mmtk-core PR#91
@qinsoon qinsoon added the P-normal Priority: Normal. label Jan 17, 2024
@qinsoon qinsoon mentioned this issue Jul 25, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Jul 26, 2024
@k-sareen @qinsoon
Only map with executable permissions when using code space (#1176)
9a49d6a 
This PR closes #7. This PR
* refactors `MmapStrategy` to include protection flags, and allows each
space to pass `Mmapstrategy` to the mmapper.
* removes exec permission from most spaces. Only code spaces will have
exec permission.
* adds a feature `exec_permission_on_all_spaces` for bindings that may
allocate code into normal spaces.

---------

Co-authored-by: Yi Lin <qinsoon@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-space Area: Space/PageResource C-bug Category: Bug F-project Call For Participation: Student projects. These issues are available student projects. G-safety Goal: Safety P-normal Priority: Normal.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Only map with executable permissions when using code space k-sareen/mmtk-core
2 participants
@qinsoon @steveblackburn