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
build: re-enable partition alloc on mac #33981
Conversation
Failing test would be a side-effect of #33832, needs further investigation. |
aed872b
to
3b25115
Compare
@VerteDinde With #40128 I think this PR should be ready to go, can you rebase it on main? |
Also changing memory allocator is a bit dangerous and can potentially break things, I think we should at least not backport this change to stable branches. |
Release Notes Persisted
|
I have automatically backported this PR to "28-x-y", please check out #40230 |
There are several cases we've encountered where allocs and frees don't match the correct allocator when going from Electron to LibreOfficeKit, causing crashes. Usually an allocaiton is preformed in LOK with the system default allocator but then Electron tries to free it with the partition allocator. This is already disabled on macOS by default so disabling it for Windows and Linux doesn't really change much. They only just re-enabled partition alloc on Mac last week anyway: electron/electron#33981
Description of Change
#33704 fixed a partition alloc-related crash in MacOS versions <= 10.15. With that crash fixed, we should be able to re-enable partition alloc!
This PR re-enables partition alloc on MacOS by removing the Chromium patch.
Checklist
npm test
passesRelease Notes
Notes: Re-enabled partition alloc on Mac