-
Notifications
You must be signed in to change notification settings - Fork 22
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
OEM crash when scrolling through wallet list #70
Comments
What version of flutter are you running? |
Would be weird the reason is the list of wallets, we are talking about only ~350 wallets and their images are small and cached by |
Have you looked at the logs?
My flutter version: |
Yes, I was able to repro by repeatedly scrolling all the way up and down. After the fourth of fifth scroll I got the crash. I may need to cache images on our side rather than leaving CachedNetworkImage do it 😓 |
Yes. Unless it's a specific image problem, like someone forgot to resize and uploaded a huge one... you'd know better though :) |
Could be that as well as apparently it always fail with the same image id. I'll be checking, thanks for reporting! |
It is BitFrost's image that is uploaded as SVG and of course it fails to be decoded by the library. We are going to convert the file but in the meantime you can hide it by using:
To be clear, this is not a crash, just a rendering issue. However, the SIGSEGV part of your logs is not showing up on my side. Could you confirm that it is anyway related to the same image rendering issue by adding the |
I still get the SIGSEGV native crash with this fix. I open wallet list, scroll up and down a while, then just wait for some time without closing the list bottom shit, like 5 seconds, and the crash happens. |
|
Interesting, I'll try to reproduce ASAP. I'm sorry to bother you with more test but would you be willing to create a brand new empty project and just add reown_appkit as dependency and try to reproduce this? I'm trying to directly link this issue to our SDK. |
Right now I have my hands full unfortunately, not much free time for this, but if you have trouble reproducing I will try to make time in the near future. |
Pretty consistent native crash when scrolling through wallet list on android. Looks like a out of memory error, probably related to loading so many images fast.
Attaching relevant logs from pixel 8 and pixel 8 pro.
Pixel 8 logs seemed to have more details.
error_logs.zip
The text was updated successfully, but these errors were encountered: