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

[Bug]: Fix memory leak #1300

Open
tmolitor-stud-tu opened this issue Nov 20, 2024 · 0 comments
Open

[Bug]: Fix memory leak #1300

tmolitor-stud-tu opened this issue Nov 20, 2024 · 0 comments
Labels
🦸 Help wanted If you are motivated, this is a good start!

Comments

@tmolitor-stud-tu
Copy link
Member

tmolitor-stud-tu commented Nov 20, 2024

WE seem to have a smaller memory leak in our core.
That doesn't impact the main app process, but the push app extension process is heavily memory constrained ~40MiB and having a memory leak there can result in crashes (kills by iOS) of the push app extension.

I'm not sure, where this comes from. Currently my best guess would be the NSXMLParser by Apple, used by Monal to parse the incoming XML stream into MLXMLNode instances.

Since that NSXMLParser crashes occasionally, too, maybe we could replace it by a rust-based parser.
Replacing it does not guarantee to fix the memory leak, though.

@tmolitor-stud-tu tmolitor-stud-tu added the 🦸 Help wanted If you are motivated, this is a good start! label Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🦸 Help wanted If you are motivated, this is a good start!
Projects
Status: No status
Development

No branches or pull requests

1 participant