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

increase vm.max_map_count #170

Closed
wants to merge 1 commit into from

Conversation

monsieuremre
Copy link
Contributor

No description provided.

@adrelanos
Copy link
Member

For hardened-malloc this is already here:
https://github.com/Kicksecure/hardened_malloc/blob/master/debian/hardened_malloc.conf

Is this security improving outside of hardened malloc? If so, please add some references.

@monsieuremre
Copy link
Contributor Author

My understanding is, probably. Because there already is ASLR. We randomize the address space layout. If there is more space, there are more possibilities for random layouts, so the layout is in fact more random and more secure. So if I'm not misinterpreting something, this should be a net positive.

@adrelanos adrelanos changed the title New line 990-security-misc.conf increase vm.max_map_count Nov 20, 2023
@therealmate
Copy link

@monsieuremre
Copy link
Contributor Author

So the consensus is larger=better but nor for security reasons, and rather for compatibility reasons. I still think having a larger max value is better for security too. Because when you think very simple: larger area=more random=more secure. Unfortunately there does not seems to be a lot of resources on if this would actually increase ASLR effectiveness. But, I don't see why that would not be the case.

@adrelanos
Copy link
Member

@monsieuremre
Copy link
Contributor Author

Seeing this brings apparently very little benefit, closing.

@monsieuremre monsieuremre deleted the patch-5 branch December 3, 2023 19:36
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.

None yet

3 participants