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

Import from Aegis with custom icons causes crash #371

Closed
cinimodev opened this issue Jul 16, 2024 · 4 comments
Closed

Import from Aegis with custom icons causes crash #371

cinimodev opened this issue Jul 16, 2024 · 4 comments
Assignees
Milestone

Comments

@cinimodev
Copy link

Aegis allows users to select custom images from their camera roll as the icon for for service. I was playing around with the feature as a gag. When importing an Aegis encrypted export, OTP Client crashes. I tested with and without custom icons and the custom icon set causes a crash each time. The imports without custom images (or the stock icon) does not crash.

This is not a major issue. Just wanted to pass along the info in case anyone else has this issue.

@elliotwutingfeng
Copy link

The Aegis android app itself is known to crash with an Out of Memory error when importing vaults with very large custom icons, perhaps this might be related.

@paolostivanin
Copy link
Owner

paolostivanin commented Jul 18, 2024

Thanks for reporting this. I'll have a look at it.

If you could provide a simple reproducer, that would be fantastic though 😄

@paolostivanin
Copy link
Owner

paolostivanin commented Jul 18, 2024

The crash is likely to happen because the icons are consuming all the secure memory. Can you please tell me whether the custom icons are using a dedicated key (e.g. custom_icon) or just the usual icon key?

@paolostivanin
Copy link
Owner

Could you please try the fix in this branch and let me know? Thanks!

paolostivanin added a commit that referenced this issue Jul 18, 2024
- handle null and eof in json files (fixes #369)
- load json into standard memory, so that we don't consume too much secure memory (fixes #371)
paolostivanin added a commit that referenced this issue Jul 24, 2024
- handle null and eof in json files (fixes #369)
- load json into standard memory, so that we don't consume too much secure memory (fixes #371)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants